Ахтунг, барабанщик!
Небольшая предыстория...
Вторую неделю плююсь от кода на работе.
Началось все с того как коллега спросил меня почему одна из страниц не работает.
После просмотра свн-логов стало понятно, что сломал ее один недавно взятый программер.
Его код меня поразил: копипаст на копипасте, почти полное отсутствие тестирования, изменение кусков кода которые он не понял и, в результате, поломка всего и вся, особая любовь к ненужным переделкам и переименованиям.
Реопнули ему с коллегой баг, написали комментов, пришел - все объяснили... наступем на второй недотест/недофикс - совершенно случайно. Опять реопнули, написали, объяснили...
Сегодня смотрю - портянка фиксов приехала. Поменяли форму ввода паролей. Дай, думаю, гляну по-быстрому. Интересно же чего добавили. Сходу нахожу кучу копипаста, поломку валидации на странице и баг: коммит пустого пароля. Реопаю. Фиксит. Реопаю опять. Опять фиксит. Опять реопаю... ДОКОЛЕ?!!! Голова ведь дана чтобы ей думать, а не еду в нее есть!
Похоже, объяснения не помогают, надо вводить дедовщину code-review. Делать этого очень не хочется - задач куча, времени нет.
Так вот, про барабанщика. Полезла я смотреть в "МойКруг" что же это за чудо такое и где оно работало. Оказалось, бывший тестер (почему плохо проверяет свой код?), потом junior java developer (почему такое качество кода?). Из увлечений - барабаны! Так вот откуда копипаст взялся! Барабанщик, он же долбит и долбит - как дятел. :)
Вывод: не берите барабанщика в программеры, ни к чему хорошему это не приведет.
Вторую неделю плююсь от кода на работе.
Началось все с того как коллега спросил меня почему одна из страниц не работает.
После просмотра свн-логов стало понятно, что сломал ее один недавно взятый программер.
Его код меня поразил: копипаст на копипасте, почти полное отсутствие тестирования, изменение кусков кода которые он не понял и, в результате, поломка всего и вся, особая любовь к ненужным переделкам и переименованиям.
Реопнули ему с коллегой баг, написали комментов, пришел - все объяснили... наступем на второй недотест/недофикс - совершенно случайно. Опять реопнули, написали, объяснили...
Сегодня смотрю - портянка фиксов приехала. Поменяли форму ввода паролей. Дай, думаю, гляну по-быстрому. Интересно же чего добавили. Сходу нахожу кучу копипаста, поломку валидации на странице и баг: коммит пустого пароля. Реопаю. Фиксит. Реопаю опять. Опять фиксит. Опять реопаю... ДОКОЛЕ?!!! Голова ведь дана чтобы ей думать, а не еду в нее есть!
Похоже, объяснения не помогают, надо вводить дедовщину code-review. Делать этого очень не хочется - задач куча, времени нет.
Так вот, про барабанщика. Полезла я смотреть в "МойКруг" что же это за чудо такое и где оно работало. Оказалось, бывший тестер (почему плохо проверяет свой код?), потом junior java developer (почему такое качество кода?). Из увлечений - барабаны! Так вот откуда копипаст взялся! Барабанщик, он же долбит и долбит - как дятел. :)
Вывод: не берите барабанщика в программеры, ни к чему хорошему это не приведет.