René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"За задачите","articleBody":"``` java\n return ((list.equals(ListManipulator.reverse(list)) || list.equals(ListManipulator.sort(list))) ? true : false);\n```\n\nе излишно дълго.\nМожеш да реферираш статични методи в същия клас без да пишеш името на класа.\nОсвен това, `return expression ? true: false` е леко излишно. `return expression` е със същата семантика и е достатъчно четимо.\n- При тестване на utility методи няма нужда да имаме **каквото и да било** в setUp метода. In fact, той не ни трябва въобще. Иначе тестовете са ОК. \n  -Изкарал си 3-ката в константа за Logger-а, хубаво.\n- Защитаваш потребителя от неправилен usage, като му пренабиваш logLevel-a. Тук идеята ми беше да се сетите да ползвате Exception, за да му кажете, че тъй не бива. Това, което ти правиш, е опасно, защото тихомълком класа ти взима решение вместо клиента на класа, което не е ок. \n- Преизползване на конструктурите, хубаво.\n- Преизползваш методите на parent-a в DateLogger-a, супер. \n\nС изключение на exception-а, напълно ОК решение на задачите. \n","author":{"url":"https://github.com/GeorgiPachov","@type":"Person","name":"GeorgiPachov"},"datePublished":"2014-06-19T10:57:50.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/1/JavaCourse/issues/1"}