重构好还是不好？

Question

给出以下代码：

    status = row[COL_STATUS]

    if status == "idle":
        row[COL_EDITABLE] = True
        row[COL_FONTSTYLE] = pango.STYLE_NORMAL
        row[COL_WEIGHT] = pango.WEIGHT_NORMAL
    elif status == "DCed":
        row[COL_EDITABLE] = True
        row[COL_FONTSTYLE] = pango.STYLE_ITALIC
        row[COL_WEIGHT] = pango.WEIGHT_NORMAL
    else:
        row[COL_EDITABLE] = False
        row[COL_FONTSTYLE] = pango.STYLE_NORMAL
        row[COL_WEIGHT] = pango.WEIGHT_BOLD

您认为以下内容将是有益的净重构吗？

    d = {"idle": (True,  pango.STYLE_NORMAL, pango.WEIGHT_NORMAL),
         "DCed": (True,  pango.STYLE_ITALIC, pango.WEIGHT_NORMAL),
         None:   (False, pango.STYLE_NORMAL, pango.WEIGHT_BOLD)}
    e,f,w = d.get(status, d[None])
    row[COL_EDITABLE] = e
    row[COL_FONTSTYLE] = f
    row[COL_WEIGHT] = w

如果要编辑更多案例或更多行组件怎么办？

Answer 1

使用对象并执行类似于“用子类替换类型代码”的操作怎么样？ http://www.refactoring.com/catalog/replaceTypeCodeWithSubclasses.html

Answer 2

您获得的简洁性将失去可读性。 在当前示例中，我可以轻松判断出什么。 在新代码中，我不得不加倍努力。

将其乘以下一千次编辑，您将面临一些严重的可维护性问题。