为何调用lexicon.scan('north')返回None？如何正确获取items列表？

首先先把你的代码整理成可读性更强的正确缩进格式（原代码的缩进混乱是引发问题的核心）：

class alexicon(object):
    def __init__(self):
        self.directions = ['north', 'south', 'west', 'east', 'up', 'down']
        self.items = []
    
    def scan(self, words):
        self.word = words.split()
        # 筛选方向词
        self.direction_word = [i for i in self.word if i in self.directions]
        self.direction_list = []
        for x in self.direction_word:
            self.direction_list.append(('direction', '%s' %(x)) )
        
        if self.direction_list != []:
            self.items.extend(self.direction_list)
        else:
            pass
            return self.items  # 这里的缩进完全错了！

lexicon = alexicon()
result = lexicon.scan('north')
print result

为什么print result输出None？ #

问题根源就是缩进错误：你把return self.items写进了else代码块里。这会导致两种情况：

• 当扫描到有效方向词（比如'north'）时，self.direction_list不为空，程序会执行if分支里的self.items.extend(self.direction_list)，之后函数没有遇到任何return语句，Python中函数如果没有显式返回值，默认会返回None，所以result拿到的就是None。
• 而self.items是实例lexicon的属性，已经被正确修改了，所以直接打印lexicon.items能看到预期的结果。

怎么正确获取items列表？ #

只需要把return self.items的缩进调整到和if语句同级，也就是整个scan方法的最后一行，确保不管if分支是否执行，函数最后都会返回self.items：

class alexicon(object):
    def __init__(self):
        self.directions = ['north', 'south', 'west', 'east', 'up', 'down']
        self.items = []
    
    def scan(self, words):
        self.word = words.split()
        # 筛选方向词
        self.direction_word = [i for i in self.word if i in self.directions]
        self.direction_list = []
        for x in self.direction_word:
            self.direction_list.append(('direction', '%s' %(x)) )
        
        if self.direction_list:  # 简化判断，空列表在布尔判断中为False
            self.items.extend(self.direction_list)
        
        return self.items  # 调整缩进，放在方法最后

lexicon = alexicon()
result = lexicon.scan('north')
print result  # 现在会输出[('direction', 'north')]

另外提个小优化：if self.direction_list != []可以简化成if self.direction_list，因为空列表在布尔判断里等价于False，非空列表等价于True，这样代码更简洁。

内容的提问来源于stack exchange，提问作者J.Zeng