我使用switch语句13个案例,每个案例只有一行返回值.
麦卡贝用红色涂抹.有没有更简单的方法来写一个大的switch语句？看起来似乎不复杂,但我不喜欢默认设置变红.如果其他人在我的代码上使用相同的工具,看到红色的东西,他们可能认为我很愚蠢:-)
编辑：我将不同的sql类型映射到我自己的更抽象类型,从而减少总数量的类型.
case Types.TIME:
return AbstractDataType.TIME;
case Types.TIMESTAMP:
return AbstractDataType.TIME;
case Types.DATE:
return AbstractDataType.TIME;
case Types.BIGINT:
return AbstractDataType.NUMERIC;
case Types.DECIMAL:
return AbstractDataType.NUMERIC;
等等…
解决方法
我不太了解McCabe工具.环形复杂性所考虑的事情之一是多个出口点.
我喜欢EnumMap的想法.
如果要使用开关,您可以使用一个结果变量,并取消所有的返回语句.您还可以折叠具有相同结果类型的所有源值：
result = null; case Types.TIME: case Types.DATE: case Types.TIMESTAMP: result = AbstractDataType.TIME // etc. return result;
我认为这样会降低圈圈的复杂性,不管任何人认为它是风格.而且这个写法是一种不同的方式,尽管你应该判断更容易.
