0

我将 PMD 添加到我的 Eclipse 工作区，并且正在清理代码。

我最复杂的方法说它的 NPath 复杂度为 804，没问题，它可能需要重构。

但是我有一个看起来相当简单的东西，它的 NPath 复杂度为 3125，这似乎是错误的：

public void toUpperParams() {
    valueParam1 = StringUtils.isNotEmpty(valueParam1) ? valueParam1.toUpperCase() : null;
    valueParam2 = StringUtils.isNotEmpty(valueParam2) ? valueParam2.toUpperCase() : null;
    valueParam3 = StringUtils.isNotEmpty(valueParam3) ? valueParam3.toUpperCase() : null;
    valueParam4 = StringUtils.isNotEmpty(valueParam4) ? valueParam4.toUpperCase() : null;
    valueParam5 = StringUtils.isNotEmpty(valueParam5) ? valueParam5.toUpperCase() : null;
}

我错过了什么吗？

4

2 回答 2

1

似乎三元增加了 NPath 因子。我可以找到以下两种解释：

关于 Guthub 的问题

关于stackoverflow的问题

于 2016-05-13T10:07:05.080 回答
0

我不认为你错过了什么。根据我的计算，根据下一篇文章中给出的定义，NPath 应该是 10：NPath 复杂度

于 2016-05-13T14:47:06.617 回答