重构好还是坏？

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

你在简洁中获得的东西会在可读性上失去。在当前示例中，我可以很容易地分辨出什么在哪里。在新的代码中，我必须更加努力地思考。

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