Штрих-М: ошибку искусственно заложил уволенный программист.

mbr.livejournal.com — Штрих-М - Segmentation fault: Инсайд подтвердил, что ошибку искусственно заложил увольняющийся программист.
Новости, Компьютеры | rahs 08:48 21.12.2017
5 комментариев | 122 за, 0 против |
#1 | 09:15 21.12.2017 | Кому: Всем
т.е. QA либо отсутствует как таковой, либо отдел контроля качества пропустил критический баг в продукте. Но виноват программист, который уволился.
#2 | 10:00 21.12.2017 | Кому: Hamsterling
обычно код поступает на тестирование, после того как несколько программистов поглядят что нового внес их коллега. Если процесс построен правильно - такие вещи как "специальная закладка" практически исключены.
#3 | 10:31 21.12.2017 | Кому: Hamsterling
>
> В каких конторах вы работаете что у вас так?

Преимущественно в индийских. Из трех IT контор, которые я знаю в НН, ни одна не работает без code review. Я с трудом представляю как без этого вообще возможно работать в команде и написать что то работающее.
#4 | 14:14 21.12.2017 | Кому: TPEHEP
>
> Я ж о том же - практика явно неплохая, но как факт - жрущая некоторое время как исполнителя, так и смежников. Тот факт, что данные человекочасы могут сэкономить оные же в дальнейшем не ебет до первого жареного петуха. А после оного - поздно да и ваще, "авось пронесет

Совсем не согласен. Время занимает немного, а культуру поддерживает - все в команде стремятся писать понятный код. Желание давать переменным дурацкие имена уходит после нескольких возвращений кода на доработку. Также отсекаются ошибки копипасты и опечатки, а иногда и вовсе кому-нибудь приходится полностью переделать свою работу, ибо на этапе ревью придумали и обосновали решение получше.
Если в команде 3 и более человек, то нет ни одной достойной причины, чтобы отказаться от этого этапа разработки ПО.
#5 | 15:47 21.12.2017 | Кому: kobanchi
> ИМХО это легко обосновать когда есть куча дешевых погромистов на удаленке. Индусов или у наших из провинции.
> С московскими или упаси ТНБ европейскими ЗП програмеров код получается золотым, если его 3 человека ревьюят.

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