重构长的 try-except 链答案

【问题标题】：Refactor long try-except chain重构长的 try-except 链
【发布时间】：2015-03-05 11:27:39
【问题描述】：

我感觉这个序列可能会写得更短：

dim = Dimension.objects.get(pk=rows['pk'])
try:
    dim.name = rows['name']
except KeyError:
    pass
try:
    dim.external_flg = rows['external_flg']
except:
    pass
try:
    dim.ext_owner = rows['ext_owner']
except KeyError:
    pass
try:
    dim.ext_table_name = rows['ext_table_name']
except KeyError:
    pass
try:
    dim.ext_start_date_column_name = rows['ext_start_date_column_name']
except KeyError:
    pass
try:
    dim.ext_end_date_column_name = rows['ext_end_date_column_name']
except KeyError:
    pass

我在 Python 代码优化和处理异常方面从未有过任何经验，但我很高兴有一个替代示例如何缩短它。

谢谢！

【问题讨论】：

rows 是你的字典对象吗？
也许在codereview.stackexchange.com 上提问会更合适。
它的类型（行）.__name__可以是'dict'或'list'。
bvidal，StackExchange 是否具有将问题移至更合适部分的功能？

标签： python django exception try-catch

【解决方案1】：

使用dict.get() 方法：

dim.name = rows.get('name', dim.name)

或者，对于更多 DRY 解决方案，将所有分配放入循环中：

for field_name in ('name', 'external_flg', 'ext_owner', 'ext_table_name',
                   'ext_start_date_column_name', 'ext_end_date_column_name'):
    if field_name in rows:
        setattr(dim, field_name, rows[field_name])

要处理list 数据，您可以进行类似的循环：

for i, field_name in enumerate(('name', 'external_flg', 'ext_owner',
                                'ext_table_name', 'ext_start_date_column_name',
                                'ext_end_date_column_name')):
    if i < len[rows]:
        setattr(dim, field_name, rows[i])

【讨论】：

就我而言，它也可以是一个列表。
您可以对列表执行相同的操作。查看更新的答案，

【解决方案2】：

这样的事情应该可以工作 - 这个答案包括 try/catch 块来捕获 KeyError :-)

attributes = ['name', 'external_flg', 'ext_owner', 'ext_table_name',
               'ext_start_date_column_name', 'ext_end_date_column_name']

dim = Dimension.objects.get(pk=rows['pk'])

for attr in attributes:
    try:
        setattr(dim, attr, rows[attr])
    except KeyError:
        pass

【讨论】：