itanev/hackbulgaria-java

hackbulgaria-java's Issues

За задачите

                  for (.......)
                              if (......)
                                result = false;
                break;
            }
        }

        return result;

е много по-трудно за четене от

             for(........)
                       if (........)
                         return false;
                        }
               }
              return true;

Ползвай втория начин.

• Нямаше нужда да пишеш сортировка. Можеше спокойно да ползваш Collections.sort.
• В задачата беше казано да използвате List, а не Collection за първата.
• Това ти затруднява и по-лесната имплементация на isMonotonic, и затова ги прехвърляш в масив.
• Не викаш правилно static методи. Викаш ги през инстанция, която е напълно излишна. Това го говорихме доста пъти в залата, явно не си присъствал.
• За да провериш дали 2 неща не са една и съща инстанция, и че sort-а реално ти е върнал различна инстанция, може да ползваш assertNotSame, вместо да сравняваш с == или !=.
  http://junit.sourceforge.net/javadoc/org/junit/Assert.html#assertNotSame%28java.lang.Object,%20java.lang.Object%29
  (То прави същото, ама е по-четимо)
• Имена на класове в Java почват с главна буква!!
• default level-a трябваше да е 3 винаги. А не каквото се подаде в началото. Освен това, защо не е final при теб ? :) Ти не го променяш след създаването.
• хубаво си изкарал проверката за level-а във validateLevel.
• IllegalArgumentException е добър избор на Exception в случая.
• можеше далеч по-доре да използваш да напишеш log-а в DateLogger - можеше просто да prepend-неш датата и да викенш super.log.