Jawa / Говнокод #2774 Ссылка на оригинал

0

  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 maxt, (Updated )

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

  • Это просто для отладки. e.printStackTrace();
    Не говно.
    Ответить
    • КО намекает, что конструкция try-catch здесь явно лишняя. Хотя, возможно, это просто заглушка.
      Ответить
      • КО. Кончай тупить.
        Как ты собрался e.printStackTrace(); без try-catch вызвать?
        Ответить
        • Может, я идиот, но все же:
          Exception exception = new NullPointerException();
          exception.printStackTrace();
          throw exception;
          Ответить
          • всё равно говнокод, выводить трейс нужно там где ловим, а не там где кидаем
            Ответить
            • Согласен.
              Только printStackTrace - зло.
              Лучше юзать логгеры для трасировки.
              Ответить
  • IAE сам по бы вывалился и напечатал бы stack trace, если конечно не хватит ума отлавливать такие exception-ы
    Ответить

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?


    8