一个字母游戏问题?

8

最近在一次面试中,我被给了以下问题:

  1. 编写一个可在命令行上作为Python运行的脚本

  2. 它应该在命令行上接收两个单词(或者如果您愿意,可以通过控制台查询用户来提供这两个单词)。

  3. 给定这两个单词: a. 确保它们长度相等 b. 确保它们都是英语词汇表中有效单词 (您下载的英语词汇表)中存在的单词。

  4. 如果是这样,请计算是否可以通过以下一系列步骤从第一个单词到达第二个单词 a. 您可以每次更改一个字母 b. 每次更改字母时,所得到的单词必须也存在于词典中 c. 您不能添加或删除字母

  5. 如果两个单词是可达的,则脚本应打印出作为单个、最短路径从一个单词到另一个单词的路径。

  6. 您可以使用 /usr/share/dict/words 作为您的单词词典。

我的解决方案是使用广度优先搜索来查找两个单词之间的最短路径。但显然这还不足以得到这份工作 :(

您们知道我可能做错了什么吗?非常感谢。

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和Matt Anderson指出的那样使用倒排索引。 A*算法肯定也有帮助。非常感谢,现在我知道我做错了什么。

以下是使用倒排索引的相同代码:

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


7
我没看过你的代码,但是你没有得到这份工作并不意味着错误在你,他们有告诉你吗? - MJB
类似问题:https://dev59.com/lUzSa4cB1Zd3GeqPqNkI - FogleBird
2
我同意MJB的观点。可能有更有效率的解决方案,但你的代码看起来很好。如果他们含糊不清、不善于沟通,我无法想象那会是一个愉快的工作场所。 - Cerin
@Alex K:我没有深入研究过,但这不是一个可以通过使用记忆化大大加速的问题吗? - SyntaxT3rr0r
3
这个问题作为一道面试题似乎有些不可思议。除非这是一个在线面试或类似的情况,否则我不会期望求职者能提供完美的解决方案。但是不要沮丧,正如其他人所说,这并不意味着你不能得到这份工作。有时候面试可能会出现不理想的结果。 - The Jug
显示剩余2条评论
5个回答

10

我不会说你的解决方案是错误的,但它有点慢,原因如下:

  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

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'))

1
也许他们期望使用编辑距离作为估算来进行A*搜索?

1
也许你一开始就不想在那样的公司工作。我个人不相信代码审查。我认为如果你能够仔细检查作品集和过去的参考资料,就没有必要进行现场代码测试了。像这样有严格政策的公司最终都无法成功,因为他们只雇用一些沉迷于编码、24小时思考代码的人。这只是我的个人看法。

5
这是一个讽刺性的帖子吗?老实说我分不清。 - Andrew Barrett
你为什么觉得这是讽刺的? - Strong Like Bull
1
因为“一心只想写代码的极客”是使项目成功的人...而且很大程度上是访问这样网站的人。 - BlueRaja - Danny Pflughoeft

0

也许你忘记添加shebang了? >-|

或者他们只是不喜欢你的编码风格...例如,对于这样一个简单的问题,我不会创建一个类,这是过度设计解决方案(当然,我不会仅仅基于这个来做招聘决策)。


网页内容由stack overflow 提供, 点击上面的
可以查看英文原文,
原文链接