Collappebilifstatements
-
11-11-2019 - |
Domanda
Di recente ho inciampato sul seguente avvertimento usando PMD (incorporato in Hudson), il mio codice sembra soffrire Collappebilifstatements, che non capisco completamente. Il codice sembra così
// list to be filled with unique Somethingness
List list = new ArrayList();
// fill list
for (SomeObject obj : getSomeObjects()) { // interating
if (!obj.getSomething().isEmpty()) { // check if "Something" is empty *
if (!list.contains(obj.getSomething())) { // check if "Something" is already in my list **
list.add(obj.getSomething()); // add "Something" to my list
}
}
}
A mio avviso, questo codice non è più "pieghevole" (altrimenti sarebbe ancora più illeggibile per il prossimo ragazzo che legge il codice). D'altra parte voglio risolvere questo avvertimento (senza disattivare PMD;).
Nessuna soluzione corretta
Autorizzato sotto: CC-BY-SA insieme a attribuzione
Non affiliato a StackOverflow