- 1
- 2
- 3
- 4
- 5
try {
...
} catch (Exception ex) {
wdComponentAPI.getMessageManager().reportException("ERROR", true);
}-
Java / Говнокод #7523
+80
Информативно так
Запостил:
foGa,
12 Августа 2011
Говнокод: по колено в коде.
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
© 2008-2024 «Говнокод.ру»
Обратная связь | Лицензионное соглашение | 
Follow us!
Follow us!
В проекте бывает более одного человека???? =)
>которые считаю такую обработку ошибок неприемлемой.
А может это они ошибаются? Ведь может так случится, что эта ошибка показывается конечному пользователю. А конечному пользователю, что "ERROR", "NullPointerException", что стектрейс равнозначны.
Ты не поверишь...
>А конечному пользователю, что "ERROR", "NullPointerException", что стектрейс равнозначны.
Вот тут не согласен. Если обработка ошибки достаточна информативна, то и её проблему намного проще и быстрее вычислить.
это как?
Краткий конспект сказанного выше.
1. Если обработка ошибки достаточна информативна, то и её проблему намного проще и быстрее вычислить. Проект не только пишется, но и сопровождается. А чтобы было легче сопровождать, надо и на такие мелочи тоже обращать внимание.
2. Смотря что за проект. Если у приложения одна точка входа, то это особо и не надо. Для этого есть логи при надобности. Но иногда можно и только сообщением об ошибке обойтись.
Можете прокомментировать эти 2 пункта? Мне кажется, что мы опять как глухой со слепым разговариваем, т. е. говорим о разных вещах. Что-то мне еще 234 коммента фигачить не хочеццо =)
А может этот try/catch из той серии, когда что-то падает и сообщение для пользователя не имеет никакого значения, кроме того, что что-то упало.
Вдруг "ERROR" это лишь код ошибки, а по нему уже формируется полноценное сообщение. Или эта строка показывает, что в UI надо показать диалог с ошибкой (т. е. что надо рисовать картинку ошибки, а не информации)?
Я все это к тому, что здесь тоже отсутствует контекст. И тяжело что-то однозначно сказать.
Вот было бы в комменте написано: "в этой try/catch секции производится анализ ввода пользователя. В reportException первый параметр текст, который будет отображаться, а второй флаг, отвечающий за показ этого сообщения пользователю". Вот тут уже можно говорить, что это говно. А так...
Батенька, когда же вы перестанете делать предположения?
В разработке приложений должна быть дисциплина, почти армейская, и порядок, как в аптеке... Машина очень щепетильна в таких делах. А все баги - как раз недочеты людей, от невнимательности и безответственности.
Если исключению можно задать информативное описание, то оно должно быть, и быть именно информативным!
А еще (вам на будущее) то же касается всяческих проверок, как внешних данных, так и данных между более-менее независимыми частями; так же касается и контроля прав доступа - как людей, так и программ\процессов\объектов
А почему вы проигнорировали вариант?
Предположения не закончу делать. Нельзя смотреть на код однобоко. В #7549 вы видели одну сторону, а я (и не только я) показали вам другую. Надо стараться увидеть как можно больше сторон.
П. С. Мне так нравится, что меня тут все учат. Как с детем неразумным возятся. Мило =)
Урок геометрии в школе.
- Докажите, что это равнобедренный треугольник.
- Мамой клянусь!