1. Java / Говнокод #19951

    −24

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    @Test
    public void testDeleteMessage() {
        try {

            try {
                Thread.sleep(3000);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }

            ChatImpl.deleteMessage(testMessage.getId());
            assertTrue(true);
        } catch (ServerException e) {
            assertNull(e);
        }
    }

    ну из названия теста все понятно!
    больше всего прикалывает эта проверка:
    - вылетел тебе эксепшен
    - а ты такой в кетче его ловишь и думаешь - “А давай я его проверю, вдруг это даже не эксепшн, а так просто нулевая ссылка, и тогда все норм!!!” !!!!
    )))))))))))))))))))))))))))))))))))))))) )))))))) как тут в эдиторе разрывающийся смайлик поставить ???

    А если там метод прошел. давайте проверять что "true" - это все таки "true". А то знаем мы этих программистов: переопределят понятия,а потом дебаж сутки )))))))))))))))
    Не могу оторваться, АЖ за Душу берет. Оооооооооооо

    Запостил: vfite, 06 Мая 2016

    Комментарии (21) RSS

    • ava roman_gemini 06.05.2016 13:41 # −1

      скорее автор решил перехватить исключение и бросить AssertionError таким уникальным образом) хотя можно было просто ServerException обернуть AssertionError. 12 строчка определенно лишняя. так, больший интерес вызывает Thread.sleep в тесте)))
      Ответить

      • ava vfite 06.05.2016 13:44 # −1

        автор просто не знал что эксепшн можно не ловить, а в сигнатуру метода теста вставить. и тогда тест итак не пройдет. Нужно просто больше кода))))) много кода
        Ответить

        • ava ShortKedr 20.05.2016 22:48 # 0

          Ага и это исключение вылетит на верхний (уровень по стеку вызовов). Эх...
          Ответить

    • ava CHayT 06.05.2016 15:07 # +3

      въебал минуса за несбалансированные скобки
      Ответить

      • ava kerman 06.05.2016 18:32 # +1

        Обычные египетские скобки. Я сам такие не люблю, но у жабоводов это в порядке вещей.
        Ответить

        • ava 3.14159265 06.05.2016 18:35 # +3

          > но у жабоводов это в порядке вещей
          Ньюфаги-ньюфажики. K&R же. А за ними Линус и Сиплюструп.

          Кстати со временем так и стал писать, метод, класс, функция -
          {
          на следующей строке, операторные скобки без переноса.
          Ответить

          • ava bormand 06.05.2016 18:44 # +1

            > у метода, класса, функции { с новой строки
            > у операторов - рядом с ними
            +1. Мой любимый стиль.
            Ответить

          • ava Xom94ok 06.05.2016 19:06 # 0 

            switch(A_kak_bryak_v_svitche) {
case tak: {
    break;
}
case ili_tak/*?*/: {
} break;
}
            Ответить

            • ava bormand 06.05.2016 19:14 # 0

              Внутри блока конечно
              Ответить

              • ava Xom94ok 06.05.2016 19:39 # 0

                Так всегда и писал, а недавно посмотрел и подумал:
                do {                  case val: {
} while(expr);        } break;
                Ответить

              • ava kegdan 06.05.2016 21:45 # 0

                а как вы относитесь к 

                оператор
    однострочное тело оператора


                ?
                Ответить

                • ava bormand 07.05.2016 03:00 # 0

                  Вполне нормально, но только на 1 уровень. Если оператор внутри оператора - у внешнего скобки напишу.
                  Ответить

            • ava 3.14159265 06.05.2016 20:06 # +1 


              switch(vo_1_nenavizhy_sishkoblyadskiy_svishch) {
    case tak: {
         break;
    }
              Ответить

      • ava 3.14159265 06.05.2016 18:38 # 0

        Это ты еще лиспо-/хачкель- блядков и гнусного форматирования не видел.
        Ответить

        • ava CHayT 06.05.2016 21:57 # 0

          видел, конечно
          там-то всё сбалансировано в конечном счёте
          Ответить

          • ava 3.14159265 06.05.2016 22:20 # +2 

            Вот{ 
   вот{
      А выглядит как полное говно. }}


            ЗЫ Хотя не совсем понимаю термин "сбалансировано". Можно его как-то вормализировать? Только у GNU закрывающая под открывающей.
            Ответить

            • ava kegdan 06.05.2016 22:24 # 0

              да, я бы долго искал глазами закрывающие скобки, мотая код вниз
              Ответить

    • ava guest 06.05.2016 16:18 # 0

      ))))))))))000000000))))))))))))))))))000 0000)))))))))))))))))))))))))))))))))))) ))))0)))))))))))))))))))))))))))))000))) )))))))))))))))))))))))))))))))))))))))0 00))))))))))))))))))))))))

      НУЖНО БОЛЬШЕ СКОБОЧЕК, БОЛЬШЕ!
      Ответить

    • ava gost 07.05.2016 22:48 # +3

      Говнокод и говноописание - идеальное сочетание.
      Ответить

    Добавить комментарий