|
|
|
| /* Русский Вариант */ |
| « Опасная ошибка | Саммит директоров » |
"У нас в компании Х", - пишет Jay Smith, - "есть старший программист, который на протяжении многих лет создавал ситуацию, когда весь контроль за кодом был завязан на него, и который заставлял клиентов находиться в зависимости от его авторитета, настаивая на том, что все его решения безапеляционно правильны (невзирая на реальные последствия)".
"Это программист, который ненавидит пользоваться бесплатными библиотеками, наподобие STL, boost и какими бы то ни было сторонними API, которые могли бы упростить жизнь ему и команде. Он переписывает структуры данных для каждого проекта, избегает обычного XML-парсера, которого другим хватило бы с головой, этот парень уже столько раз заново изобрёл колесо, что со спокойной совестью мог бы получить на него патент. Проблемы его самоуважения и контроля над кодом были такими сильными, что если ему предлагали решение, при котором не было завязки на его код, в ответ звучало только недовольное сопение.
Вразрез со своей компетенцией и опытом, он предпочитает придерживаться своих текущих принципов: если не я это написал, значит оно не вполне подходит. В купе с нынешней атмосферой срочности (у проекта как раз скоро демо) это означает, что релизация зачастую поспешная, без особых размышлений над последствиями своих действий".
"Всё вышесказанное", - продолжает Jay, - "позволяет больше не удивляться следующим находкам в коде его проекта":
int Utilities_CompareDouble(double P, double Q)
{
_asm
{
fld P
fld Q
FCOMIP ST, ST(1) ;23
fstp Q
jc LESS
and ah,20h
Jnz EQUAL
mov eax,0C00002h ;Greater
jmp EXIT
EQUAL:
mov eax,0C00001h
jmp EXIT
LESS:
mov eax,0C00003h
EXIT:
}
}
Jay продолжает: "Неужели же этот парень пытается переписать абсолютно всё? Ведь вот, что было изначально закоммичено в репозиторий":
int StringUtil_Compare(const char *left, const char *right)
{
int result = 0;
if(!strcmp(left,right))
result = 1;
return result;
}
"Нет нужды говорить," - добавляет Jay, - "что рецензирование его кода - это самый отличный способ быстро развеселить себя".
Оригинал: http://thedailywtf.com/Articles/Code-Ownership-Gone-Awry.aspx
Перевод: 21csm
Автор перевода выражает особую благодарность Дмитрию Гриценко за помощь в адаптации труднопереводимых фрагментов публикации.
| « Опасная ошибка | Саммит директоров » |