Question

我们如何以更好的方式处理此功能

或

我们如何才能提高效率

或

优化代码：

def is_interesting(tt):
    return tt['target_type'] == 670 and tt.get('script') is not None and tt.get('script') in ('abc','syz','mno')

Answer 1

tt.get('script') is not None部分是多余的，因为None也不在下一个元组中。

你的测试效率很高;如果第一个表达式已经是False，Python将不会评估第二个表达式。

def is_interesting(tt):
    return tt['target_type'] == 670 and tt.get('script') in ('abc','syz','mno')

演示：

>>> def is_interesting(tt):
...     return tt['target_type'] == 670 and tt.get('script') in ('abc','syz','mno')
... 
>>> is_interesting({'target_type': 42})
False
>>> is_interesting({'target_type': 670})
False
>>> is_interesting({'target_type': 670, 'script': ''})
False
>>> is_interesting({'target_type': 670, 'script': 'abc'})
True

Answer 2

我的回答与Martijin完全相同。

def is_interesting(tt):
    return tt['target_type'] == 670 and tt.get('script') in ('abc','syz','mno')

我只是想回答这个问题：

＆＃34;返回tt [＆＃39; target_type＆＃39;] == 670和tt.get（＆＃39;脚本＆＃39;）（＆＃39; abc＆＃39;，＆＃39; syz＆＃39;，＆＃39; mno＆＃39;）TypeError：＆＃39; in＆＃39;需要字符串作为左操作数＆＃34;因此我添加了这个语句以应对tt.get的空值（＆＃39; script＆＃39;）

示例：

>>> {'foo':'bar'}.get('notexist') in ('foo')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: 'in <string>' requires string as left operand, not NoneType

>>> {'foo':'bar'}.get('notexist') in ('foo',)
False

>>> {'foo':'bar'}.get('notexist') in ['foo']
False

说明：第一个例子，（）中只有一个元素，所以（＆＃39; foo＆＃39;）实际上会返回你的foo＆＃39; - 一个字符串;第二和第三可以确保你得到一个元组/列表。在括号内的单个元素后面放一个逗号是获取一个元素的方法。我想这就是你遇到的。

更好的方法来做到这一点

2 个答案: