代码之家 › 专栏 › 技术社区 › FacundoGFlores

发现类似代码的改进代码

code-climate optimization python

FacundoGFlores · 技术社区 · 8 年前

我将codeclimate传递给我的代码,得到了以下结果:

在其他1个位置发现类似代码

这是我的代码:

stradd = 'iterable_item_added'
if stradd in ddiff:
    added = ddiff[stradd]
    npos_added = parseRoots(added)
    dics_added = makeAddDicts(localTable, pk, npos_added)
else:
    dics_added = []
strchanged = 'values_changed'
if strchanged in ddiff:
    updated = ddiff[strchanged]
    npos_updated = parseRoots(updated)
    dics_updated = makeUpdatedDicts(localTable, pk, npos_updated)
else:
    dics_updated = []

哪里 iterable_item_added 和 values_changed 重复。如何改变它?

1 回复 | 直到 8 年前

Jean-François Fabre 8 年前

只需提取参数并创建一个helper方法:

def testmethod(name,localTable,m,ddiff,pk):
    if name in ddiff:
        npos = parseRoots(ddiff[name])
        rval = m(localTable, pk, npos)
    else:
        rval = []

    return rval

称之为:

dics_added = testmethod('iterable_item_added',localTable,makeAddDicts,ddiff,pk)
dics_updated = testmethod('values_changed',localTable,makeUpdatedDicts,ddiff,pk)

注意:分解代码时要小心,您可以引入bug(并使代码更具可读性:)。

另外:该helper方法强制传递许多局部变量。也许创建一个对象和成员变量会更简单。

在这种情况下,为了关闭审查工具而这样做似乎有点“过分”。

推荐文章

July · 如何定义数字间隔,然后四舍五入

1 年前

Community wiki · 对象名称前的单下划线和双下划线的含义是什么?

1 年前

Brian Johnson · 为什么在Python中列出字典列表会引发TypeError?[已关闭]

1 年前

user026 · 如何根据特定窗口的平均值(行数)创建新列?

1 年前

Ashok Shrestha · 需要追踪特定的颜色线并获取坐标

1 年前

Nicote Ool · 在FastApi和Vue3中获得422

1 年前

NeoExceptCodeBad · 如果我有很多垂直线,我如何找到它们的边缘?

1 年前

Abdulaziz · 如何对集合内的列表进行排序[重复]

1 年前

user2743931 · 带有src目录的Python setup.py

1 年前

asmgx · 为什么合并数据帧不能按照python中的预期方式工作

1 年前