Аааааааа!!!!
Код-ревью. Подпрограмма на 20 строк. Замечания по первой версии:
1. удалить ненужное тут
2. вставить пробел там
3. заменить ту длинную строку на эти более короткие и лучшие 5, проверенные мной
Версия 2: первые два дефекта исправлены, в исправлении последнего вдруг появляется разъ*б пробелов и логическая ошибка (копи-паста подгорела!). Тесты мы, конечно, не запускаем чтобы обнаружить ошибку чем-то ещё, кроме смотрящих и невидящих глаз. Это же ревью, а не тестирование, а тестирование уже было сделано в первой версии!
Версия 3: rebase.
Версия 4: идентична версии 1.
Версия 5: идентична версии 3.
Версия 6: частично исправлен разъ*б пробелов, прочие ужасы версии 2 сохранены.
Тут я не выдерживаю (до сих пор молча наблюдал этот фарс) и говорю, что в копи-пасте остались ещё две проблемы, и вообще шозанах творится-то.
Версия 7: то, что должно было быть в версии 2, rebase я могу и сам, спасибо, вот моя большая красная (или зелёная?) кнопка, нажимаю, все свободны.
Ватсон, шоэтобыло? Кто все эти люди? Где я?
1. удалить ненужное тут
2. вставить пробел там
3. заменить ту длинную строку на эти более короткие и лучшие 5, проверенные мной
Версия 2: первые два дефекта исправлены, в исправлении последнего вдруг появляется разъ*б пробелов и логическая ошибка (копи-паста подгорела!). Тесты мы, конечно, не запускаем чтобы обнаружить ошибку чем-то ещё, кроме смотрящих и невидящих глаз. Это же ревью, а не тестирование, а тестирование уже было сделано в первой версии!
Версия 3: rebase.
Версия 4: идентична версии 1.
Версия 5: идентична версии 3.
Версия 6: частично исправлен разъ*б пробелов, прочие ужасы версии 2 сохранены.
Тут я не выдерживаю (до сих пор молча наблюдал этот фарс) и говорю, что в копи-пасте остались ещё две проблемы, и вообще шозанах творится-то.
Версия 7: то, что должно было быть в версии 2, rebase я могу и сам, спасибо, вот моя большая красная (или зелёная?) кнопка, нажимаю, все свободны.
Ватсон, шоэтобыло? Кто все эти люди? Где я?