java - 摆脱丑陋的 Actor 阵容

Question

Add.java

public class Add implements Value, Reducible{
Value left;
Value right;
public Add(Value left, Value right){
    this.left = left;
    this.right = right;
}

@Override
public String toString(){
    return (left + " + " + right);
}

@Override
public Value reduce(Machine environment) {
    Value val = new Add(left, right);
    if(left instanceof Reducible){
        ((Add)val).left = ((Reducible) ((Add)val).left).reduce(environment);
    }
    else if(right instanceof Reducible){
        ((Add)val).right = ((Reducible) ((Add)val).right).reduce(environment);
    }else{
        val = new VNumber(((VNumber)((Add)val).left).getValue() + ((VNumber)((Add)val).right).getValue());
    }
    return val;
}

}

Value.java

public interface Value {
    public abstract String toString();
}

Reducible.java

public interface Reducible {
public abstract Value reduce(Machine environment);
}

Add.reduce 真的很难看。这是符合规定的完美代码，但几乎不可读。有没有办法摆脱那些丑陋的 Actor ？

Answer 1

你真是让自己的生活变得艰难了。首先，您已经有了“左”和“右”:您不需要不断地将它们从“val”中取出。其次，当您在三种情况之一中不打算使用新的 Add() 且还没有正确的参数时，不应该预先创建它。

尝试这样:

public Value reduce(Machine environment) {
    if(left instanceof Reducible){
        return new Add(((Reducible)left).reduce(environment), right);
    }
    if(right instanceof Reducible){
        return new Add(left, ((Reducible)right).reduce(environment));
    }
    return new VNumber(((VNumber)left).getValue() + ((VNumber)right).getValue());
}

E&OE。但我会非常努力地在这里使用泛型。当然有一个缺失的情况，即“左”和“右”都是可约的？