如何判断函数在Python中返回的值？

Question

我正在尝试调试我编写的程序。 如何判断给定的word ， hand和word_list是否返回True或False？ 我尝试初始化变量失败 ，然后修改它并打印它的值。 它不是在打印，所以我不知道它的行为是否如预期的那样。 任何帮助表示赞赏。

我有一个函数load_words（）返回一个单词列表。 我知道这个词是在WORD_LIST（我查了），所以只是想看看字完全是从字典中的手 ，在这种情况下，它是不是按键的字母组成的，所以它应该返回False。

此外，.keys（）和.iterrkeys（）之间有什么区别，还有没有更好的方法来遍历手 （可能带有hand.iteritems（）的字母）值 ？

word = 'axel'

hand2 = {'b':1, 'x':2, 'l':3, 'e':1}

def is_valid_word(word, hand, word_list):
    """
    Returns True if word is in the word_list and is entirely
    composed of letters in the hand. Otherwise, returns False.
    Does not mutate hand or word_list.

    word: string
    hand: dictionary (string -> int)
    word_list: list of lowercase strings
    """
    failure = False
    if word in word_list:
        print hand
        print [list(i) for i in word.split('\n')][0]
        for letter in [list(i) for i in word.split('\n')][0]:
            print letter
            if letter in hand.keys():
                print letter
                return True
                failure = True
                print failure
            else:
                return False
                failure = False
                print failure
    else:
        return False
        failure = False
    print failure

is_valid_word(word,hand2,load_words())

UPDATE我希望在我的函数中使用此函数，但是它会给出一个关键错误，即使它本身可以正常工作也是如此。

def update_hand(hand, word):
    """
    Assumes that 'hand' has all the letters in word.
    In other words, this assumes that however many times
    a letter appears in 'word', 'hand' has at least as
    many of that letter in it. 

    Updates the hand: uses up the letters in the given word
    and returns the new hand, without those letters in it.

    Has no side effects: does not modify hand.

    word: string
    hand: dictionary (string -> int)    
    returns: dictionary (string -> int)
    """
    for letter in [list(i) for i in word.split('\n')][0]:
        if letter in hand.keys():
            hand[letter] = hand[letter]-1
        if hand[letter] <= 0:
            del hand[letter]
    display_hand(hand)
    return hand

Answer 1

它不打印出来的原因是因为您要在print s之前返回该函数。 这意味着程序在到达print语句之前就停止了。 例如：

def foo(x):
    return x
    print x

foo("asdf")

在以下情况下将不返回任何内容：

def foo(x):
    print x
    return x

foo("asdf")

将print ：

asdf

因此，您在return之前的所有语句。 如果没有，它将不会执行。

为了进一步澄清，这篇文章已经有了您的答案https://stackoverflow.com/a/3617008 ：

在Python 2中， iter(d.keys())和d.iterkeys()并不完全等效，尽管它们的行为相同。 在第一个中， keys()将返回字典的键列表的副本，然后iter将在此列表上返回一个迭代器对象，第二个，则永远不会构建完整的键列表的副本。

请注意，Python 3也没有.iterkeys() 。 Python 3使用先前的.iterkeys()作为新的.keys() 。

最后 ，我将回顾您的代码普遍存在的问题以及您要实现的严重性从高到低的顺序。

1. 您的代码仅检查一个字母
2. [list(i) for i in word.split('\\n')][0]并不是从单词中获取所有字母的方式。
3. 您应该首先返回短代码，这样就不会有大的缩进块。

您的代码仅检查一个字母

在for循环中，检查第一个单词后立即return True 。 您应在循环完成后return True 。

for letter in word:
    if letter not in hand.keys():
        return False
return True

清单理解

您不需要列表理解（稍后会告诉您原因），并且不必太复杂，仅从单词中获取字母即可。 例如

[list(i) for i in word.split('\n')][0]

实际上只这样做：

list(word)

实际上，您应该直接对单词进行直接迭代（就像我之前所做的那样），它将一一返回字母：

for letter in word:
    # code...

先让短代码返回

通常我不喜欢大块高度缩进的代码。 您可以做的是先让短代码返回。 例如：

if word in word_list:
    for letter in word:
        if letter in hand.keys():
            return True
        else:
            return False
else:
    return False

可以简单地写成：

if word not in word_list:
    return False

for letter in word:
    if letter in hand.keys():
        return True
    else:
        return False

但是，这只是我的意见。 其他一些人可能更喜欢else语句，以便他们知道何时执行代码。

您的最终代码如下所示：

def is_valid_word(word, hand, word_list):
    if word not in word_list:
        return False

    for letter in word:
        if letter not in hand.keys():
            return False
    return True

干净吧？ 但是，我假设您正在制作类似拼字游戏的游戏，因此您可以算出您hand的单词是否能代表您选择的单词。 如果单词中的字母数小于或等于您手中的字母数，您可以添加一些要计数的东西：

def is_valid_word(word, hand, word_list):
    if word not in word_list:
        return False
    # This makes the word into a "unique list"
    letters = set(word)
    for letter in letters:
        if hand[letter] < word.count(letter):
            return False

    return True

编辑代码有问题。 它不检查，如果letter是在hand的if语句： if hand[letter] < word.count(letter): 。

def is_valid_word(word, hand, word_list):
    if word not in word_list and word not in hand.keys():
        return False
    letters = set(word)
    for letter in letters:
        # Add this extra clause
        if letter in hand.keys() or hand[letter] < word.count(letter):
            return False

    return True

Answer 2

您可以直接print is_valid_word(word,hand2,load_words())结果print is_valid_word(word,hand2,load_words())

Answer 3

您有一些缩进问题， 在 return语句之后做一些事是徒劳的。

您不需要使用keys或iterkeys ， in运算符将为您检查，并且可以使用列表，集合，字典（键），元组，字符串等。

in运算符会调用__contains__ ，这是大多数python集合所支持的。

也可以查看https://docs.python.org/2/reference/expressions.html#membership-test-details 。

他是您要进行3个测试的最小化示例。

def is_valid_word(word, hand, word_list):
    """
    Returns True if word is in the word_list and is entirely composed
    of letters in the hand. Otherwise, returns False.  Does not mutate
    hand or word_list.

    word: string
    hand: dictionary (string -> int)
    word_list: list of lowercase strings

    """

    if word not in word_list:
        return False

    for letter in word:
        if letter not in hand:
            return False

    return True

print(is_valid_word('bxel',
                    {'b': 1, 'x': 2, 'l': 3, 'e': 1},
                    ['foo', 'bar', 'bxel']))
print(is_valid_word('axel',
                    {'b': 1, 'x': 2, 'l': 3, 'e': 1},
                    ['foo', 'bar', 'axel']))
print(is_valid_word('axel',
                    {'a': 1, 'x': 2, 'l': 3, 'e': 1},
                    ['foo', 'bar', 'axel']))