我很想看到常规if语句的替代方法,例如

if(x)
     do a;
if(y)
     do b;
if(z)
    do c;

因此,您可以看到所有if语句是单独的,没有其他条件。请注意,X Y Z是完全独立的条件,因此切换不适合。

有帮助吗?

解决方案

一个“真正面向对象”答案是为“规则”定义一个界面。 (使用condition()和action()方法),创建3个实现,将它们填充到一个集合中,然后一般地迭代它们,如下所示:

List<Rule> rules = .... ; // your 3 rules initialized here somehow
for(Rule r : rules) {
  if(r.condition()) {
    r.action();
  }
}

如果您有300条规则/条件而不仅仅是3条,那么这就更有意义了。

在Java8中,如果规则是CPU密集型的,您可能希望这样做:

rules.parallelStream().filter(Rule::condition).forEach(Rule::action);

其他提示

简短的回答是肯定的。

有一些时间你可以避免使用if for conditional evluation和branching all。 他们有适当的时刻。

  1. 多态性,当行为取决于初始值时
  2. 参考分配,当您知道可能的初始值并且它们与返回值具有1对1的相关性时。列表比数组要好,但是......
       

    // Example:  
    if (a==1) { b=2;  }  
    if (a==2) { b=17; }  
    
    // Becomes  
    int fx(2);  // our array of answers  
    fx[0] = 2;   
    fx[1] = 17;  
    b = fx[ a - 1 ];
    
  3. 参考分支,当您知道可能的初始值并且它们与要使用的函数/分支具有1对1的相关性时。 (例如,不是Java)

    // Example:
    if (a==1) { doSomething1();  }  
    if (a==2) { doSomething2(); }  
    
    // Becomes
    function * fx(2);  // our array or better still, list of functions  
    fx[0] = &doSomething1;   
    fx[1] = &doSomething2;  
    `fx[ a - 1 ](); `
    
  4. 直接布尔赋值。

    我们讨厌:

    if (thisCondition == true) {  
      b = true;  
    } else {  
      b = false;  
    }
    

    应该是:

      

    b = thisCondition;

Java中 if-else 的替代方法是switch语句和条件三元组?:)运算符,它们都没有完全符合您的要求(只处理 if 没有 else )。在我看来,你发布的代码是最好的方法。

使用多态。

interface SomethingDoer {
    public void doSomething();
}

class ADoer implements SomethingDoer { ... }
class BDoer implements SomethingDoer { ... }
class CDoer implements SomethingDoer { ... }

public class Main {
     public static void main (String[] args) {
          SomethingDoer doer = new SomethingDoerFactory(args).getDoer();
          doer.doSomething();
     }
}

if没有完全消除,但它被移动到SomethingDoerFactory。此解决方案并非适用于所有情况,但在其中一些情况下,它是多个ifs的非常好的替代方案。

这是一个很好的谈论:
http://misko.hevery.com / 2008/12/08 /清洁代码会谈继承-多态性测试/

这是最简单,可读但有效的解决方案。我会惊讶地发现这里有效的选择。

修改

您可以尝试多次应用提取方法

doAIfX();
doBIfY();
doCifZ();

其中方法由:

定义
void doAIfX() {
    if (!X) {
        return;
    }

    // do 'a'
}

它实际上取决于x,y,z和a,b,c是什么。有时,如果陈述更合适。有时多态性更合适。

您需要在某处使用这些if语句。

正如其他人所建议的那样,它们可以重构为其他方法以保持本方法的清洁。如果您需要在多个地方重复使用相同的设置 if语句,则可能可以使用装饰模式

这听起来像是使用闭包的完美案例。但为此你需要 groovy 或类似的东西。

怎么样...... myAnimator(thing,pos,speed | computeSpeed());

如果速度=== 未定义那么它将是computeSpeed ......我想。

这实际上取决于上下文...但是,如果 - else只是java中 control flow 语句之一: https://docs.oracle.com/javase/tutorial/java/ nutsandbolts / flow.html

许可以下: CC-BY-SA归因
不隶属于 StackOverflow
scroll top