CollapsibleIfStatements

Недавно я наткнулся на следующее предупреждение, используя PMD (встроенный в hudson), мой код, похоже, страдает CollapsibleIfStatements , что я не совсем понимаю . Код выглядит так

// 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
    }
  }
}

На мой взгляд, этот код не более "сворачиваемый" (иначе он был бы еще более нечитаемым для следующего парня, читающего код). С другой стороны, я хочу устранить это предупреждение (не отключая PMD;).

5
задан NPE 23 February 2012 в 10:29
поделиться