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

    +79.1

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    if (at == null) {
	try {
	throw new NullPointerException();
	}catch (Exception e) {
	e.printStackTrace();
	throw new NullPointerException();
	}
}

    проверка корректности полученного аргумента в функции.
    хорошо придумано, что бы не вызывать
    throw new IllegalArgumentException();

    Запостил: maxt, 13 Марта 2010

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

    • ava guest 13.03.2010 14:37 # −0.2

      Это просто для отладки. e.printStackTrace();
      Не говно.
      Ответить

      • ava guest 13.03.2010 17:05 # +1

        КО намекает, что конструкция try-catch здесь явно лишняя. Хотя, возможно, это просто заглушка.
        Ответить

        • ava guest 13.03.2010 18:27 # −2

          КО. Кончай тупить.
          Как ты собрался e.printStackTrace(); без try-catch вызвать?
          Ответить

          • ava guest 13.03.2010 19:31 # +3

            Может, я идиот, но все же:
            Exception exception = new NullPointerException();
exception.printStackTrace();
throw exception;
            Ответить

            • ava guest 13.03.2010 20:38 # 0

              Забавно.
              Ответить

            • ava Suor 15.03.2010 07:38 # −1.2

              всё равно говнокод, выводить трейс нужно там где ловим, а не там где кидаем
              Ответить

              • ava Amok 15.03.2010 20:23 # −1

                Согласен.
                Только printStackTrace - зло.
                Лучше юзать логгеры для трасировки.
                Ответить

    • ava guest 15.03.2010 20:51 # −2.4

      0
      Где здесь С++?
      Ответить

    • ava guest 17.03.2010 10:31 # +1.2

      IAE сам по бы вывалился и напечатал бы stack trace, если конечно не хватит ума отлавливать такие exception-ы
      Ответить

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