java - 代码优化建议

Question

public Object getValue()
{
    ValueItem valueItem = null;
    Object returnValue = null;

    if(this.value instanceof StringValueImpl)
    {
        valueItem = (StringValueImpl) this.value;
    }
    else if(this.value instanceof ListValueImpl)
    {
        valueItem = (ListValueImpl) this.value;
    }
    else if(this.value instanceof MapValueImpl)
    {
        valueItem = (MapValueImpl) this.value;
    }

    if(valueItem!=null)
        returnValue = valueItem.getValue();

    return returnValue;
}

ValueItem 是一个接口(interface)，由ListValueImpl、MapValueImpl 等实现。我想要返回值是一个对象。代码工作正常，但我想知道是否可以通过任何方式改进它？

Answer 1

this.value 的类型是什么？如果它是 ValueItem 那么你不需要做任何这些并且可以用这个替换方法:

public Object getValue()
{
    Object returnValue = null;
    if(this.value!=null)
        returnValue = this.value.getValue();
    return returnValue;
}

或者更短:

public Object getValue()
{
    return this.value!=null ? this.value.getValue() : null;
}

如果 this.value 不是 ValueItem 类型但是它必须 包含一个 ValueItem，那么您手头就有一个设计问题。