e3ky / javacourse Goto Github PK

 return ((list.equals(ListManipulator.reverse(list)) || list.equals(ListManipulator.sort(list))) ? true : false);

е излишно дълго.
Можеш да реферираш статични методи в същия клас без да пишеш името на класа.
Освен това, return expression ? true: false е леко излишно. return expression е със същата семантика и е достатъчно четимо.

• При тестване на utility методи няма нужда да имаме каквото и да било в setUp метода. In fact, той не ни трябва въобще. Иначе тестовете са ОК.
  -Изкарал си 3-ката в константа за Logger-а, хубаво.
• Защитаваш потребителя от неправилен usage, като му пренабиваш logLevel-a. Тук идеята ми беше да се сетите да ползвате Exception, за да му кажете, че тъй не бива. Това, което ти правиш, е опасно, защото тихомълком класа ти взима решение вместо клиента на класа, което не е ок.
• Преизползване на конструктурите, хубаво.
• Преизползваш методите на parent-a в DateLogger-a, супер.

С изключение на exception-а, напълно ОК решение на задачите.