?

Log in

No account? Create an account
Преисполненный стыдом и яростью, хочу сказать .. - Вход - только для падающих звезд .. [entries|archive|friends|userinfo]
Anatoly

[ website | My Website ]
[ userinfo | livejournal userinfo ]
[ archive | journal archive ]

Преисполненный стыдом и яростью, хочу сказать .. [Feb. 16th, 2017|11:17 pm]
Anatoly
[Tags|]

НИКОГДА ! НИКОГДА ! НИКОГДА !

Не верьте чужому коду !!!


В программеже, как в разведке, нет "своих" - есть "еще не предавшие".

Особенно, если этот код является часть проекта, построенного без документации/регламентов/устава и итерационным ( "так исторически сложилось" (С) ) методом.

Впервые за 12, минимум, лет я не стал проверять один-единственный запрос
, наивно (в мои-то годы !) поверив на слово, что этот несчастный запрос из 5 вьюшек корректен - за что и поплатился .. Shame on me ! Mea culpa ! Хатану вэ пашану !



За 4 дня было перелопачено не менее 200К кода оракловых пакетов, сделаны 2 (два) новых общесистемных инструмента, обруганы 3 (трое) коллег - вскрытие показало, что весь этот буторкод загрузки данных работал корректно, пусть и не оптимально. И вот тут я, наконец-то, удосужился заглянуть в изначальный запрос, использовавший загруженные данные ..

Ошибка несистемная (требуется, чтоб документ формировался более 1 дня, а в этот день как раз была создана новая версия клиентской записи в реестре - и все это не более 3 дней до конца месяца).

Факин шииииит !!!

ЗЫ: в предыдущий раз, в 2005, причиной подобного шухера стала строка кода, ошибочно внесенная программером-стажером внутрь условного блока, а не после него.
Результатом стал недельный аврал всего отдела разработки в неслабом ИТ-интеграторе + 4 дня командировки всего отдела на площадку к крупному бюджетообразующему заказчику + неделя аврала половины ИТ-отдела заказчика + первая седина у начальников обоих ИТ-отделов и у меня ..
linkReply

Comments:
[User Picture]From: sandblaster
2017-02-16 09:01 pm (UTC)
A Code Inspection у вас не практикуют?
(Reply) (Thread)
[User Picture]From: raffal
2017-02-16 09:47 pm (UTC)
Если Вы про code review - увы, нет :-(
(Reply) (Parent) (Thread)
[User Picture]From: sandblaster
2017-02-16 09:52 pm (UTC)
Вещь полезная. Быстро становится видно ху из ху, чей код проверять, а чей можно пропустить, ну и время не так много берет, если тул хороший, типа Crucible.
(Reply) (Parent) (Thread)
[User Picture]From: raffal
2017-02-17 04:18 am (UTC)
Увы, дефицит ресурсов - страшная вещь :-(

А за рекомендацию спасибо.
(Reply) (Parent) (Thread)