2016-05-13 3 views
0

Я добавил PMD в рабочее пространство Eclipse, и я работаю над очисткой кода.Нужна информация о сложности NPath

Мой самый сложный метод говорит, что он имеет сложность 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; 
} 

я упускаю что-то?

ответ

1

Кажется, что тройной увеличивает коэффициент NPath. Ниже два объяснения я мог бы найти:

Question on Guthub

Question on stackoverflow

+0

Спасибо. Все еще звучит странно, если кто-нибудь знает, почему это решение было принято, я очень хочу его услышать. – TheBakker

0

Я не думаю, что вы пропустите что-нибудь. Согласно моим расчетам, NPath должно быть 10 на основании определения, которое было дано в следующей статье: NPath complexity

+0

Я попробовал это с помощью SourceMeter, и это дало 11 для сложности NPath для этого метода. – Csuki

Смежные вопросы