Про SAP и их подход к написанию документации

← →
И. Павел © (2011-06-14 13:41) [0]

В официальной документации к САП - порталу постоянно встречаю блоки кода вида:

try {
<тут какой-то код>
} catch (Exception e) {
}
Например:http://help.sap.com/saphelp_nw70/helpdata/en/89/8a185c148e4f6582560a8d809210b4/content.htm

У коллеги в коде уже встречал прятание исключений таким же образом. Все думал - зачем, а теперь дошло - бездумный copy/paste.

Вопрос: зачем они эти catch ставят? Причем не обхватывают функцию целиком, а только отдельные блоки: т.е. коннект не произошел, подавляем ошибку и начинаем работу с коннектом, который равен NULL...

← →
И. Павел © (2011-06-14 13:44) [1]

Кстати:

catch (Exception e) { response.write("Exception occurred"); }

- это почти то же самое: выводится сообщение об ошибке и работа продолжается как будто ошибки не было.

← →
Kerk © (2011-06-14 13:49) [2]

Вот они какие, буржуйские 1Сники... :))

← →
Ega23 © (2011-06-14 14:04) [3]

> Вот они какие, буржуйские 1Сники... :))

Там у них всё очень прикольно

← →
TUser © (2011-06-14 15:30) [4]

Главное манагер может не только в ppt показать, что "усе работает" ))

И имхо, не cccv, а "времени нет, сдавать вчера".

← →
sniknik © (2011-06-14 16:58) [5]

блин, больше всего времени потрачено именно из-за такой "обработки" ошибок...
ну и еще "русификаторов" - типа

catch (Exception e) { response.write("Случилась ошибка"); }

неважно что закрыли причину, зато по русски. (хотя вот в 10-ке дельфевой уже историю посмотреть можно, т.е. не перекрывает все, может и там так?)

← →
Slym © (2011-06-15 06:31) [6]

Главная задача программиста - убить в себе перфекциониста.
http://bash.org.ru/quote/411597

← →
Slym © (2011-06-15 07:22) [7]

"запостю" сюда так как целевая ветка http://delphimaster.net/view/2-1307976499/ закрыта

мастера! почему неоптимальность\корявость кода, заглушки исключений вы замечаете сразу и постите многочисленные исправления (прим. http://delphimaster.net/view/2-1307976499/)
а очевидные классические баги безопасности игнорируете, не замечаете и не исправляете? (http://delphimaster.net/view/2-1307976499/)
вдрух афтар поста скопипастит свой креатив в "боевой" продукт и будет вам
UserName="Администратор"
Password="1 or 1=1"

← →
Ega23 © (2011-06-15 07:41) [8]

> а очевидные классические баги безопасности игнорируете,
> не замечаете и не исправляете?

Тебе не кажется, что если человек не может понять, как проверить рекордсет на наличие в нём записей, то ему рассказывать что-то про SQL-инъекцию это тупо пустая трата времени?

← →
Anatoly Podgoretsky © (2011-06-15 07:44) [9]

> Ega23 (15.06.2011 07:41:08) [8]

Ему было сказано об этой дырке в другой ветке. Результат с негодованием
проигнорировано. А здесь то чего стараться, ветка то закрыта, но кой кому
неймется.

> [7] Slym © (15.06.11 07:22)

Одно дело - не использование параметров/экранирования начинающим программистом, который не знает, что такое IsEmpty или count(*), и другое - странно безразличное отношение огромной корпорации к своей документации.

К тому же, не отвечать же на вопрос об определении пустой выборки рассказом про параметры, преимущества ADODataSet, повестью про то, когда with может привести к проблемам (и еще добавить, что если используется Access то не стоит использовать пользователя "admin" и т.д. и т.п. - все это на практике узнается, а иначе автор вопроса решит, что к нему специально цепляются и обидится - так тут уже бывало).

Ну, хоть какая-то документация и то хлеб. У нашей буржуйской - вообще, не было. Когда-то чего-то было и даже комментарии в коде. Но потом поделку купила мелкософт и все потерли - голый код остался. И документации шиш.

Сейчас пытаются исправить, но медленно. В основном вдумчивое курение кода дает ясность - что там и как.

> зачем они эти catch ставят? Причем не обхватывают функцию
> целиком, а только отдельные блоки: т.е. коннект не произошел,
> подавляем ошибку и начинаем работу с коннектом, который
> равен NULL...

Так проще код поддерживать. ООП-то нет, а километровых if/then/else и так хватает.

Когда без try...catch обламывается чей-то новенький код в совокупности например с обработкой расчета себестоимости выпуска - пользователям системы это сильно не нравится.

On-the-fly разработка, она сильно отличается от привычной.

> И. Павел © (14.06.11 13:41)

Это же справочная документация, а не учебник. И посвящена она не обработке исключений, а способам соединения с SAP.

Про SAP и их подход к написанию документации Найти похожие ветки