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
scroll top