java - 键值映射而不是很长的 ELSE-IF 结构？

Question

我目前正在研究一种将一些字符串映射到其他字符串的方法。

它有很多值，方法开始看起来像这样:

ValueHelper.isEqual() 方法正在寻找完全匹配。

private IValue1 mapValue(IValue2 value2) {
    if (ValueHelper.isEqual(value2.getName(), StatusValues.ACTIVE)) {
            return ValueHelper.getName(StatusValues2.WORKING);
    } else if (ValueHelper.isEqual(value2.getName(), StatusValues.INACTIVE)) {
            return ValueHelper.getName(StatusValues2.NOT_WORKING);
    } else if (ValueHelper.isEqual(value2.getName(), StatusValues.IN_SERVICE)) {
            return ValueHelper.getName(StatusValues2.SERVICE);
    }
}

目前我有 10 个 else-if 代码块。

让这个方法更简单、更短的最佳方法是什么？将值提取到键值映射？还是另一种选择？

Answer 1

我想一张 map 就可以了。您还可以考虑 switch-case 以使其不那么冗长:

private IValue1 mapValue(IValue2 value2) {
 switch(value2.getName()) {
     case StatusValues.ACTIVE:     return StatusValues2.WORKING;
     case StatusValues.INACTIVE:   return StatusValues2.NOT_WORKING;
     case StatusValues.IN_SERVICE: return StatusValues2.SERVICE;
     default: throw new RuntimeException();
 }