如何重构此代码并应用OO模式？

Question

我的RichTable中有四个RichTable实例，并且有一个current表实例的概念。 取决于标志resetAll我需要清除所有表或除当前表以外的所有表的选择 。 如果resetAll为true，则清除所有内容，否则忽略当前内容。 当前表的索引作为参数传递给执行清除操作的方法。

清除所有内容的调用看起来像这样：

clearSubTypeSettings(true,-1);

清除当前请求以外的所有请求的调用看起来像这样：

clearSubTypeSettings(true, col);

上面方法的实现是这样的：

private void clearSubTypeSettings(boolean resetAll, int exceptControl) {
    if (!resetAll) {
        clearAllExceptCurrent(exceptControl);
    } else {
        clearAll();
    }
 }

现在，这两种方法clearAllExceptCurrent(exceptControl)和clearAll()看起来几乎相同。 这里是实现：

  private void clearAll() {
        for (int i = 0; i < SUBTYPE_TABLES; i++)
            if (getSubTypeTable(i).getSelectedRowKeys() != null) {
                RichTable richTable = getSubTypeTable(i);
                RowKeySet rowkeySet = richTable.getSelectedRowKeys();
                rowkeySet.clear();
                AdfFacesContext.getCurrentInstance().addPartialTarget(richTable);
            }
    }

和

private void clearAllExceptCurrent(int exceptControl) {
    for (int i = 0; i < SUBTYPE_TABLES; i++)
        if (i != exceptControl && getSubTypeTable(i).getSelectedRowKeys() != null) {
            RichTable richTable = getSubTypeTable(i);
            RowKeySet rowkeySet = richTable.getSelectedRowKeys();
            rowkeySet.clear();
            AdfFacesContext.getCurrentInstance().addPartialTarget(richTable);
        }
}

我觉得我在这里编写重复的冗余代码，将来会使维护复杂化。 如何改善此代码并使它更面向对象？

Answer 1

您可以让clearAll() 委托 （=> OOP模式）到clearAllExceptCurrent() （=>通过删除重复的代码来改进代码，使其更易于维护）：

private void clearAll() {
   clearAllExceptCurrent(-1);
}

两种方法之间的唯一区别是clearAllExceptCurrent()的条件i != exceptControl 。 通过传递-1此条件始终为true ，因此实际上不存在。

Answer 2

重复代码的大部分是清除表的位。 那么如何：

private void clearTable(int id) {
    if (getSubTypeTable(i).getSelectedRowKeys() != null) {
        RichTable richTable = getSubTypeTable(i);
        RowKeySet rowkeySet = richTable.getSelectedRowKeys();
        rowkeySet.clear();
        AdfFacesContext.getCurrentInstance().addPartialTarget(richTable);
    }
}

然后：

 private void clearAll() {
    for (int i = 0; i < SUBTYPE_TABLES; i++) {
        clearTable(i);
    }
 }

private void clearAllExceptCurrent(int exceptControl) {
    for (int i = 0; i < SUBTYPE_TABLES; i++) {
        if (i != exceptControl) {
            clearTable(i)
        }
    }
}

编辑：感动if语句中clearTable