对副作用使用条件表达式是否有效？

Question

我正在尝试找出最好的方法是：

resource['contents'][media_type] = []
resource['contents'][media_type].append(row[0].toPython()) if row[0] is not None else None
resource['contents'][media_type].append(row[2].toPython()) if row[2] is not None else None

我认为代码很简单；如果行有一个值，则将它们添加到列表中。这种方法被认为可以吗？还有其他更好的方法吗？ toPython 方法将返回包含对象的字符串描述。

Answer 1

对副作用使用“三元”conditional expression (x if C else y) 根本不是 Pythonic。以下是我的做法：

resource['contents'][media_type] = []
for index in (0, 2):
    item = row[i]
    if item is not None:
        resource['contents'][media_type].append(item.toPython())

或使用列表推导来减少冗长：

resource['contents'][media_type] = [row[i].toPython() for i in (0, 2) 
                                    if row[i] is not None]

这些方法更具可读性，并且减少了重复。

Answer 2

不，这不是条件表达式的有效使用。它会使任何试图阅读您的代码的人感到困惑。

使用if 语句；您可以通过创建对列表的另一个引用来节省一些空间：

lst = resource['contents'][media_type] = []
if row[0] is not None: lst.append(row[0].toPython()) 
if row[2] is not None: lst.append(row[2].toPython())

但为本地引用使用更好的名称（contents 或许？），或者使用列表推导：

resource['contents'][media_type] = [
    col.toPython() for col in (row[0], row[2]) if col is not None]

Answer 3

我不认为这样做是好的做法。你可以做的是：

resource['contents'][media_type] = []

for irow in [0, 2]:
    if row[irow] is not None:
        resource['contents'][media_type].append(row[irow].toPython())

这使您可以灵活地使用范围（range(5) 中的 irow），或者如果您可以直接访问它们，则使用行 (for row in rows:)。