减少 If 语句中的代码重复

Question

我有相当大的代码块，用于生成/返回由多个数据源构造的搜索结果的数组列表。我被要求包含一个日期限制方法，用户可以根据 2 个日期字段构建数组。如果填充了这些日期字段之一，假设其名为 dateFrom，则数组将返回指定日期之后的数据。类似地，如果填充了 dateTo，那么它将用前面的数据填充数组，如果两者都填充，那么 if 显然会根据该范围进行填充。

这一切都很好&使用几个简单的嵌套 if 语句或 switch case 很容易做到，但我的问题是，这也意味着我将不得不重复大量代码。有没有办法可以防止相同的基本数组列表构造代码重复 4 次？我知道这是可怕的伪代码，但它明白了我的观点：

if(condition1){
  setItem1(getMethod());
  setItem2(getMethod());
  setItem3(getMethod());
} else if (condition2) {
  setItem1(getMethod());
  setItem2(getMethod());
  setItem3(getMethod());
} else if (condition3) {
  setItem1(getMethod());
  setItem2(getMethod());
  setItem3(getMethod());
} else {
  setItem1(getMethod());
  setItem2(getMethod());
  setItem3(getMethod());
}

return ArrayList();

值得注意的是，虽然这更像是一个广义的 Java 问题，但我正在使用 JSF2 框架，并且此方法发生在 ViewScoped bean 和 ViewScoped bean 中。使用@PostContruct标签实现。

干杯

Answer 1

我建议将 if 语句移至另一个方法以减少耦合，然后将 setItems 移至另一个方法并将 getmethods 的结果作为参数发送。 （如果修改对象的字段会更好）
只是伪代码：

 return createArray();

 private List createArray()
 {
     if(condition1){
        setItems(getMethod(),getMethod(),getMethod());
        return ArrayList();
     }
     if (condition2) {
        setItems(getMethod(),getMethod(),getMethod());
        return ArrayList();
     } 
     ...
 }

 private void setItems(Object var1, Object var2, Object var3)
 {
     ...
 }

如果你想完全摆脱 if ，你可以使用多态性和依赖注入，但在这种情况下它可能有点矫枉过正......

Answer 2

我是不是

if(condition1 || condition2 || condition3 ||condition4){
    stuff
}

错过了什么？