- 1
- 2
- 3
- 4
- 5
try {
...
} catch (Exception ex) {
wdComponentAPI.getMessageManager().reportException("ERROR", true);
}
Jawa / Говнокод #7523
0
Информативно так
Запостил: 
foGa,
(Updated )
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
- "PHP" (6228)
- Куча говна (4362)
- Кресты (2940)
- ЙажаСценарий (2225)
- Си диез (2081)
- Jawa (1688)
- Няшная (1341)
- Змея (894)
- Школоло (781)
- mysql_real_escape_string (579)
- Бухгалтерия (547)
- ДействиеСценарий (419)
- Beri bash i ebash (300)
- Яuбy (294)
- Субъективный Си (290)
- =()= (267)
- Школоло (200)
- Сборный (158)
- Метатабличный (91)
- Хламидомонада (75)
- Ябло (Свежее) (49)
- Goвно (37)
- Koko (17)
- HTML (1)
В проекте бывает более одного человека???? =)
>которые считаю такую обработку ошибок неприемлемой.
А может это они ошибаются? Ведь может так случится, что эта ошибка показывается конечному пользователю. А конечному пользователю, что "ERROR", "NullPointerException", что стектрейс равнозначны.
Ты не поверишь...
>А конечному пользователю, что "ERROR", "NullPointerException", что стектрейс равнозначны.
Вот тут не согласен. Если обработка ошибки достаточна информативна, то и её проблему намного проще и быстрее вычислить.
это как?
Краткий конспект сказанного выше.
1. Если обработка ошибки достаточна информативна, то и её проблему намного проще и быстрее вычислить. Проект не только пишется, но и сопровождается. А чтобы было легче сопровождать, надо и на такие мелочи тоже обращать внимание.
2. Смотря что за проект. Если у приложения одна точка входа, то это особо и не надо. Для этого есть логи при надобности. Но иногда можно и только сообщением об ошибке обойтись.
Можете прокомментировать эти 2 пункта? Мне кажется, что мы опять как глухой со слепым разговариваем, т. е. говорим о разных вещах. Что-то мне еще 234 коммента фигачить не хочеццо =)
А может этот try/catch из той серии, когда что-то падает и сообщение для пользователя не имеет никакого значения, кроме того, что что-то упало.
Вдруг "ERROR" это лишь код ошибки, а по нему уже формируется полноценное сообщение. Или эта строка показывает, что в UI надо показать диалог с ошибкой (т. е. что надо рисовать картинку ошибки, а не информации)?
Я все это к тому, что здесь тоже отсутствует контекст. И тяжело что-то однозначно сказать.
Вот было бы в комменте написано: "в этой try/catch секции производится анализ ввода пользователя. В reportException первый параметр текст, который будет отображаться, а второй флаг, отвечающий за показ этого сообщения пользователю". Вот тут уже можно говорить, что это говно. А так...
Батенька, когда же вы перестанете делать предположения?
В разработке приложений должна быть дисциплина, почти армейская, и порядок, как в аптеке... Машина очень щепетильна в таких делах. А все баги - как раз недочеты людей, от невнимательности и безответственности.
Если исключению можно задать информативное описание, то оно должно быть, и быть именно информативным!
А еще (вам на будущее) то же касается всяческих проверок, как внешних данных, так и данных между более-менее независимыми частями; так же касается и контроля прав доступа - как людей, так и программ\процессов\объектов
А почему вы проигнорировали вариант?
Предположения не закончу делать. Нельзя смотреть на код однобоко. В #7549 вы видели одну сторону, а я (и не только я) показали вам другую. Надо стараться увидеть как можно больше сторон.
П. С. Мне так нравится, что меня тут все учат. Как с детем неразумным возятся. Мило =)
Урок геометрии в школе.
- Докажите, что это равнобедренный треугольник.
- Мамой клянусь!