Заставь дурака богу молиться - он лоб расшибет

[spamsink]

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

Сейчас - уже которую неделю - я занимаюсь расчисткой кода от каловых завалов, оставшихся от старой хардверной архитектуры, которую перестали продавать уже, пожалуй, года три как, но руки дошли сделать branch совсем недавно.

И вижу:

// desperation debugging -- not called
void check_for_some_property(...) {
...
}

Некая процедура, которая пригодна только для того, чтобы вызываться исключительно из-под отладчика, мало того что была checked in, так в ней еще и все отладочные печати снабжены уникальными идентификаторами!

Вот какие они, американские пиэйчди!

Comments

[sab123.livejournal.com]

Ну так если обычный stderr переключен в /dev/null, то единственный способ чего-то напечатать - через логгер.

[ivan-ghandhi.livejournal.com]

А всё из-за дефицита евреев!

[spamsink]

Так логгеру же все равно - можно было и нулями в качестве идентификаторов обойтись.

[relyef.livejournal.com]

Бывает хуже. Вот я недавно игрался с неким фреймворком, в котором был H-file следующего содержания:

template<typename T> class foo {
...
T value;
...
void toDebugStream(std::ostream & os) const
{
os << value;
}
...
}
/////////////
В тестах эти многомудрые граждане ничего сложнее foo<int> не использовали. А как только я написал foo<map<bar,vector<baz> > > - тут-то компиляция и рухнула.

[ygam.livejournal.com]

Сшарп 2+ такого не потерпит.

[sab123.livejournal.com]

Ну это смотря как он устроен. Если все сообщения должны быть в табличке, откуда они извлекаются по коду - то без кода не обойтись.

[cema.livejournal.com]

Вот какие они, типа, молодцы.

[spamsink]

В порядочных системах всегда есть - и в данном случае был - graceful degradation. Если в табличке не нашлось - печатается переданный в качестве параметра текст. Дело не столько в том, как именно это было реализовано и как именно это можно было с минимальными потерями обойти, а о подходе в целом.

[spamsink]

Да уж. Алгоритмист он, может, и хороший, но как дело до coding style доходит - мама дорогая.

[malaya-zemlya.livejournal.com]

В checked in виде все лучше, чем локально держать. Версии, бэкапы - очено полезно.

[spamsink]

Держать этот код вообще не надо. Отладил, тесты написал - всё, леса и опалубки можно разбирать.

[ivan-ghandhi.livejournal.com]

Дийкстра говорил так: самолёт испытали - можно снимать приборы.

[cema.livejournal.com]

Ох, не видел ты нашего кода.

[spamsink]

Смотря какие приборы. А вот что гири на крыльях, которые вешали для испытания статических нагрузок, можно снимать - эт точно.

[cema.livejournal.com]

"Сто раз вам говорил: не снимайте леса, пока не поклеили обои!"

[cema.livejournal.com]

Эти не то что можно, эти нужно.

А у нас без гирь не летает.