Question

我在Java中具有if-else结构，如下所示：

                    if (A || B || C){
                        if (A){
                            //Do something
                        }
                        if (B){
                            //Do something
                        }
                        if (C){
                            //Do something
                        }
                    } else {
                        //Do something
                    }

我想知道是否有更清洁，更简便的方法来替换它？

Answer 1

如果A，B和C是评估成本很高的条件，则可以使用附加标志来确保它们仅被评估一次：

boolean found = false;
if (A) {
    //Do something
    found = true;
}
if (B){
    //Do something
    found = true;
}
if (C){
    //Do something
    found = true;
}
if (!found) {
    //Do something
}

否则（例如，如果评估起来并不昂贵），我会保留您当前的状况。

Answer 2

是的

if (A) {
  // do A
}
if (B) {
  // do B
}
if (C) {
  // do C
}
if (! (A || B || C)) {
  // do "neither A or B or C"
}

Answer 3

虽然我认为您所拥有的没问题，但您可以这样做：

boolean d = false;
if (d |= A) { ... }
if (d |= B) { ... }
if (d |= C) { ... }
if (!d) {
  // What to do if A..C are all false.
}

如果满足任何条件（d用于“做某事”），这会将d设置为true。

Answer 4

仅是一种可能的解决方案：

public class ABC {

    private static final Logger log = LoggerFactory.getLogger(ABC.class);

    @Test
    public void abc() {
        boolean A = true;
        boolean B = false;
        boolean C = true;

        boolean abcProcessed = false;

        abcProcessed |= process(A, () -> {
            // Do something for A
            log.debug("A");
        });
        abcProcessed |= process(B, () -> {
            // Do something for B
            log.debug("B");
        });
        abcProcessed |= process(C, () -> {
            // Do something for B
            log.debug("C");
        });

        if (!abcProcessed) {
            // Do something for !(A||B||C)
            log.debug("!(A||B||C)");
        }
    }

    private boolean process(boolean flag, DoSomethingInterface doSomethingInterface) {
        // check condition
        if (flag) {
            // execute code specific for condition
            doSomethingInterface.doSomething();
            // return true (processed)
            return true;
        }

        // return false (not processed)
        return false;
    }

    public interface DoSomethingInterface {

        // code specific for condition
        void doSomething();
    }
}

一个想法是将条件检查和相关代码移至单独的方法。

Answer 5

我认为如果有条件，您应该使用else

if(A){

}else if(B){

}else if(C){

}else if(D){

}

但是，如果您有许多条件，例如A，B，C等，则感觉程序逻辑有问题

有没有更干净的方法在Java中编写多个if语句

5 个答案: