如何用Optionals替换我的旧Java代码？

Question

我有这个密码：

existMusic.isPresent(music -> {
if (music.getAuthor() == null && music.getType() == Type.POP){
methode1();
}else if (music.getAuthor() == Author.USA && music.getType() == Type.RAP){
methode2()
} else {
throw exception;

还有很多if语句。我想用可选的替换它，但我不知道如何替换，因为我只是在研究这个主题。你能帮我吗？

Answer 1

我不认为代码的可读性会因为不使用选项而受到影响，而是因为你实现了你的如果。我会分两步重构它：

第一：重新排列ifs并减少嵌套ifs

Optional<Class1> existingClass1 = class1Service.get(class2.getNumber());

existingClass1.ifPresent(class1 -> {
    if (Enum.PARAM1 == class1.getParam1() && Enum.PARAM3 == class1.getParam2()){
        class1.setParam1(Enum.PARAM2);
        class1.setParam2(null);
        class2.setClass1(class1);
    } else if (class1.getParam1() == null && Enum.PARAM2 == class1.getParam2() 
        && (class1.getList() == null || class1.getList().size() != 2)){
        class2.setClass1(class1);
    } else {
        throw new Exception();
    } 
});

第二：移动一些只涉及Class1到Class1源代码的条件检查

public Class1 {

    ...

    public boolean isConditionOne() {
        return Enum.PARAM1 == this.getParam1() && Enum.PARAM3 == this.getParam2()
    }
    
    public boolean isConditionTwo() {
        return this.getParam1() == null && Enum.PARAM2 == this.getParam2() 
        && (this.getList() == null || this.getList().size() != 2)
    }

}

所以你的另一个类看起来像这样：

Optional<Class1> existingClass1 = class1Service.get(class2.getNumber());

existingClass1.ifPresent(class1 -> {
    if (class1.isConditionOne()){
        class1.setParam1(Enum.PARAM2);
        class1.setParam2(null);
        class2.setClass1(class1);
    } else if (classOne.isConditionTwo()){
        class2.setClass1(class1);
    } else {
        throw new Exception();
    } 
});