All streams
Search
Write a publication
Pull to refresh

Comments 4

Совет №6. Полная власть

Это жиза.

В старой компании переписывал парсер почты так, чтобы для каждой почты был свой объект обработчик.

Сделано, пришло время код ревью и...

  • Ну... тут метод непонятно называется, могу это интерпретировать как ... (parseMessage, ладно -_-)

  • Надо было алиасы для импортов сделать

  • Слишком много методов

  • Тут слишком сложно сделай проще

Т.е. концепция "разделяй и властвуй" оказалась для ревьювера слишком сложной, поэтому надо сократить все в один, убив основную концепцию кода, гениально! С таким подходом, пора вернутся с мидла на джуна

Я время от времени прошу переименовать методы, т.к. реально по имени метода думаешь одно, а делает он совсем другое, вполне себе повод докопаться )))

Тот момент когда сатира = реальные кейсы.. Жизнь бывает разной

Увы... Основано на реальных событиях

Sign up to leave a comment.

Articles