8

最近の就職の面接で、私は次の問題を与えられました:

  1. コマンドラインで実行できるスクリプトをPythonとして記述します

  2. コマンドラインで2語を使用する必要があります(または、必要に応じて、コンソールを介して2語を入力するようにユーザーに問い合わせることができます)。

  3. これらの2つの単語を考えると:a。それらが同じ長さであることを確認してくださいb。ダウンロードした英語の有効な単語の辞書に両方とも存在する単語であることを確認してください。

  4. その場合、次の一連の手順で最初の単語から2番目の単語に到達できるかどうかを計算します。一度に1文字ずつ変更できますb。文字を変更するたびに、結果の単語も辞書に存在する必要がありますc。文字を追加または削除することはできません

  5. 2つの単語が到達可能である場合、スクリプトは、1つの単語から別の単語への単一の最短パスとしてつながるパスを出力する必要があります。

  6. 単語の辞書に/usr/ share / dict/wordsを使用できます。

私の解決策は、幅優先探索を使用して2つの単語間の最短経路を見つけることでした。しかし、どうやらそれは仕事を得るのに十分ではありませんでした:(

君たちは私が間違ったことをしたかもしれないことを知っていますか?どうもありがとう。

import collections
import functools
import re

def time_func(func):
    import time

    def wrapper(*args, **kwargs):
        start = time.time()
        res = func(*args, **kwargs)
        timed = time.time() - start

        setattr(wrapper, 'time_taken', timed)
        return res

    functools.update_wrapper(wrapper, func)
    return wrapper

class OneLetterGame:
    def __init__(self, dict_path):
        self.dict_path = dict_path
        self.words = set()

    def run(self, start_word, end_word):
        '''Runs the one letter game with the given start and end words.
        '''
        assert len(start_word) == len(end_word), \
            'Start word and end word must of the same length.'

        self.read_dict(len(start_word))

        path = self.shortest_path(start_word, end_word)
        if not path:
            print 'There is no path between %s and %s (took %.2f sec.)' % (
                start_word, end_word, find_shortest_path.time_taken)
        else:
            print 'The shortest path (found in %.2f sec.) is:\n=> %s' % (
                self.shortest_path.time_taken, ' -- '.join(path))

    def _bfs(self, start):
        '''Implementation of breadth first search as a generator.

        The portion of the graph to explore is given on demand using get_neighboors.
        Care was taken so that a vertex / node is explored only once.
        '''
        queue = collections.deque([(None, start)])
        inqueue = set([start])

        while queue:
            parent, node = queue.popleft()
            yield parent, node

            new = set(self.get_neighbours(node)) - inqueue
            inqueue = inqueue | new
            queue.extend([(node, child) for child in new])

    @time_func
    def shortest_path(self, start, end):
        '''Returns the shortest path from start to end using bfs.
        '''
        assert start in self.words, 'Start word not in dictionnary.'
        assert end in self.words, 'End word not in dictionnary.'

        paths = {None: []}
        for parent, child in self._bfs(start):
            paths[child] = paths[parent] + [child]
            if child == end:
                return paths[child]
        return None

    def get_neighbours(self, word):
        '''Gets every word one letter away from the a given word.

        We do not keep these words in memory because bfs accesses 
        a given vertex only once.
        '''
        neighbours = []

        p_word = ['^' + word[0:i] + '\w' + word[i+1:] + '$' 
            for i, w in enumerate(word)]
        p_word = '|'.join(p_word)

        for w in self.words:
            if w != word and re.match(p_word, w, re.I|re.U):
                neighbours += [w]
        return neighbours

    def read_dict(self, size):
        '''Loads every word of a specific size from the dictionnary into memory.
        '''
        for l in open(self.dict_path):
            l = l.decode('latin-1').strip().lower()
            if len(l) == size:
                self.words.add(l)

if __name__ == '__main__':
    import sys
    if len(sys.argv) not in [3, 4]:
        print 'Usage: python one_letter_game.py start_word end_word'
    else:
        g = OneLetterGame(dict_path = '/usr/share/dict/words')
        try:
            g.run(*sys.argv[1:])
        except AssertionError, e:
            print e

すべての素晴らしい答えをありがとう。私が本当に得たのは、辞書内のすべての単語を毎回繰り返して、隣接する可能性のある単語を検討しているという事実だと思います。代わりに、DuncanとMattAndersonが指摘した転置インデックスを使用することもできました。*アプローチも間違いなく役に立ちました。どうもありがとう、今私は自分が間違ったことを知っています。

転置インデックスを使用した同じコードを次に示します。

import collections
import functools
import re

def time_func(func):
    import time

    def wrapper(*args, **kwargs):
        start = time.time()
        res = func(*args, **kwargs)
        timed = time.time() - start

        setattr(wrapper, 'time_taken', timed)
        return res

    functools.update_wrapper(wrapper, func)
    return wrapper

class OneLetterGame:
    def __init__(self, dict_path):
        self.dict_path = dict_path
        self.words = {}

    def run(self, start_word, end_word):
        '''Runs the one letter game with the given start and end words.
        '''
        assert len(start_word) == len(end_word), \
            'Start word and end word must of the same length.'

        self.read_dict(len(start_word))

        path = self.shortest_path(start_word, end_word)
        if not path:
            print 'There is no path between %s and %s (took %.2f sec.)' % (
                start_word, end_word, self.shortest_path.time_taken)
        else:
            print 'The shortest path (found in %.2f sec.) is:\n=> %s' % (
                self.shortest_path.time_taken, ' -- '.join(path))

    def _bfs(self, start):
        '''Implementation of breadth first search as a generator.

        The portion of the graph to explore is given on demand using get_neighboors.
        Care was taken so that a vertex / node is explored only once.
        '''
        queue = collections.deque([(None, start)])
        inqueue = set([start])

        while queue:
            parent, node = queue.popleft()
            yield parent, node

            new = set(self.get_neighbours(node)) - inqueue
            inqueue = inqueue | new
            queue.extend([(node, child) for child in new])

    @time_func
    def shortest_path(self, start, end):
        '''Returns the shortest path from start to end using bfs.
        '''
        assert self.in_dictionnary(start), 'Start word not in dictionnary.'
        assert self.in_dictionnary(end), 'End word not in dictionnary.'

        paths = {None: []}
        for parent, child in self._bfs(start):
            paths[child] = paths[parent] + [child]
            if child == end:
                return paths[child]
        return None

    def in_dictionnary(self, word):
        for s in self.get_steps(word):
            if s in self.words:
                return True
        return False

    def get_neighbours(self, word):
        '''Gets every word one letter away from the a given word.
        '''
        for step in self.get_steps(word):
            for neighbour in self.words[step]:
                yield neighbour

    def get_steps(self, word):
        return (word[0:i] + '*' + word[i+1:] 
            for i, w in enumerate(word))

    def read_dict(self, size):
        '''Loads every word of a specific size from the dictionnary into an inverted index.
        '''
        for w in open(self.dict_path):
            w = w.decode('latin-1').strip().lower()
            if len(w) != size:
                continue
            for step in self.get_steps(w):
                if step not in self.words:
                    self.words[step] = [] 
                self.words[step].append(w)

if __name__ == '__main__':
    import sys
    if len(sys.argv) not in [3, 4]:
        print 'Usage: python one_letter_game.py start_word end_word'
    else:
        g = OneLetterGame(dict_path = '/usr/share/dict/words')
        try:
            g.run(*sys.argv[1:])
        except AssertionError, e:
            print e

そしてタイミングの比較:

%python one_letter_game_old.py happy hello最短パス(91.57秒で見つかりました)は次のとおりです:
=> happy --harpy --harps --harts --halts --halls --hells --hello

%python one_letter_game.py happy hello最短パス(1.71秒で見つかりました)は次のとおりです:
=> happy --harpy --harps --harts --halts --halls --hells --hello

4

5 に答える 5

10

私はあなたの解決策が間違っているとは言いませんが、それは少し遅いです。2つの理由で。

  1. 幅優先探索では、必要な長さより1つ短いすべてのパスに加えて、必要な長さのパスの一部からすべてにアクセスしてから、回答を得ることができます。最良優先探索(A *)は、ほとんどの無関係なパスをスキップするのが理想的です。

  2. 隣人を探すたびに、辞書のすべての単語を隣人としての候補としてチェックしています。Duncanが示唆しているように、データ構造を構築して、ネイバーを検索するのではなく、基本的にネイバーを検索することができます。

これがあなたの問題に対する別の解決策です:

import collections
import heapq
import time

def distance(start, end):
    steps = 0
    for pos in range(len(start)):
        if start[pos] != end[pos]:
            steps += 1
    return steps


class SearchHeap(object):
    def __init__(self):
        self.on_heap = set()
        self.heap = []

    def push(self, distance, word, path):
        if word in self.on_heap:
            return
        self.on_heap.add(word)
        heapq.heappush(self.heap, ((distance, len(path)), word, path))

    def __len__(self):
        return len(self.heap)

    def pop(self):
        return heapq.heappop(self.heap)


class OneLetterGame(object):
    _word_data = None

    def __init__(self, dict_path):
        self.dict_path = dict_path

    def run(self, start_word, end_word):
        start_time = time.time()
        self._word_data = collections.defaultdict(list)
        if len(start_word) != len(end_word):
            print 'words of different length; no path'
            return

        found_start, found_end = self._load_words(start_word, end_word)
        if not found_start:
            print 'start word %r not found in dictionary' % start_word
            return
        if not found_end:
            print 'end word %r not found in dictionary' % end_word
            return

        search_start_time = time.time()
        path = self._shortest_path(start_word, end_word)
        search_time = time.time() - search_start_time
        print 'search time was %.4f seconds' % search_time

        if path:
            print path
        else:
            print 'no path found from %r to %r' % (start_word, end_word)

        run_time = time.time() - start_time
        print 'total run time was %.4f seconds' % run_time

    def _load_words(self, start_word, end_word):
        found_start, found_end = False, False
        length = len(start_word)
        with open(self.dict_path) as words:
            for word in words:
                word = word.strip()
                if len(word) == length:
                    if start_word == word: found_start = True
                    if end_word == word: found_end = True
                    for bucket in self._buckets_for(word):
                        self._word_data[bucket].append(word)
        return found_start, found_end

    def _shortest_path(self, start_word, end_word):
        heap = SearchHeap()
        heap.push(distance(start_word, end_word), start_word, (start_word,))
        while len(heap):
            dist, word, path = heap.pop()
            if word == end_word:
                return path
            for neighbor in self._neighbors_of(word):
                heap.push(
                    distance(neighbor, end_word), 
                    neighbor, 
                    path + (neighbor,))
        return ()

    def _buckets_for(self, word):
        buckets = []
        for pos in range(len(word)):
            front, back = word[:pos], word[pos+1:]
            buckets.append(front+'*'+back)
        return buckets

    def _neighbors_of(self, word):
        for bucket in self._buckets_for(word):
            for word in self._word_data[bucket]:
                yield word

if __name__ == '__main__':
    import sys
    if len(sys.argv) not in [3, 4]:
        print 'Usage: python one_letter_game.py start_word end_word'
    else:
        matt = OneLetterGame(dict_path = '/usr/share/dict/words')
        matt.run(*sys.argv[1:])

そして、タイミングの比較:

% python /tmp/one_letter_alex.py canoe happy
The shortest path (found in 51.98 sec.) is:
=> canoe -- canon -- caxon -- taxon -- taxor -- taxer -- taper -- paper -- papey -- pappy -- happy

% python /tmp/one_letter_matt.py canoe happy
search time was 0.0020 seconds
('canoe', 'canon', 'caxon', 'taxon', 'taxor', 'taxer', 'taper', 'paper', 'papey', 'pappy', 'happy')
total run time was 0.2416 seconds
于 2010-04-27T16:21:17.497 に答える
3

このプログラミングテストへのあなたの答えが彼らが他の誰かを選んだ唯一の理由であると期待するのは奇妙だと私は同意しますが、実際にはあなたのコードに問題があります。パスのすべてのステップ、またはすべての潜在的なパスについて、辞書を線形検索します。大規模な辞書と多くの潜在的なパスの場合、これには長い時間がかかる可能性があります。また、パスがない場合は失敗するため、徹底的にテストしなかったことは明らかです。

これをコーディングしている場合は、単語をロードするときに辞書を作成します。これにより、次の単語を直接選択できるようになり、線形検索が削除されます。このコードは完全な解決策ではありませんが、私が何を意味するかを示す必要があります。

words = {}

def get_keys(word):
    """Generate keys from a word by replacing each letter in turn by an asterisk"""
    for i in range(len(word)):
        yield word[:i]+'*'+word[i+1:]

def get_steps(word):
    """Find the set of all words that can link to the given word."""
    steps = []
    for key in get_keys(word):
        steps.extend(words[key])
    steps = set(steps) - set([word])
    return steps

# Load the dictionary
for word in ('start', 'stare', 'spare', 'spore'):
    for key in get_keys(word):
        if key not in words:
            words[key] = []
        words[key].append(word)

print(words)
print(get_steps('stare'))
于 2010-04-27T14:27:52.547 に答える
1

そもそもそんなジャーク会社で働きたくなかったのかもしれません。私は個人的にコードレビューを信じていません。ポートフォリオや過去の参考文献をチェックして十分な仕事をしていれば、スポットコードテストではその必要はないと思います。このような厳格なポリシーを持つ企業は、24時間年中無休でコードを考えている1つのトラックコードオタクだけを採用しているため、最終的には決して成功しない企業です。ちょうど私の2セント。

于 2010-04-27T14:08:34.950 に答える
1

おそらく彼らは、推定として編集距離を使用したA *検索を期待していましたか?

于 2010-04-27T13:40:35.060 に答える
0

たぶんあなたはシェバンを追加するのを忘れましたか?>-|

あるいは、彼らはあなたのコーディングスタイルが気に入らなかったのかもしれません...たとえば、私はそのような単純な問題のクラスを作成しなかったでしょう、それはソリューションを過剰に設計しています(私は採用決定の基礎となるのはそれほど好きではありませんがもちろん、その上でのみ)。

于 2010-04-27T13:54:10.960 に答える