В назві статті фігурує код ревью, але приклади беруться з лайвкодінгу на інтервью. Вибачте, можливо я чогось не розумію, але лайв кодінг на інтервью та реальний код ревью після закінчення якоїсь таски це дві різні речі. Тим паче на задачах з літкоду, де код стайл взагалі не відіграє значну роль, як складність та пам’ять. Ви вказали лише на стилістичні помилки, але в різних командах стиль різний і відповідно правильна назва змінної буде різна. Як на мене то при код ревью в першу чергу потрібно звертати увагу на те чи правильно реалізований функціонал, чи дотримані NFR та чи підходить це рішення в плані архітектури нашої системи. На вузьких задачах це виявити досить складно. Краще вже тоді підготувати фрагмент коду, якщо є така можливість то з реального проекту, де будуть певні код смели та проблеми в реплізації та дивитись як кандидит будет рефакторити його та на що буде звертати увагу.
В назві статті фігурує код ревью, але приклади беруться з лайвкодінгу на інтервью. Вибачте, можливо я чогось не розумію, але лайв кодінг на інтервью та реальний код ревью після закінчення якоїсь таски це дві різні речі. Тим паче на задачах з літкоду, де код стайл взагалі не відіграє значну роль, як складність та пам’ять. Ви вказали лише на стилістичні помилки, але в різних командах стиль різний і відповідно правильна назва змінної буде різна. Як на мене то при код ревью в першу чергу потрібно звертати увагу на те чи правильно реалізований функціонал, чи дотримані NFR та чи підходить це рішення в плані архітектури нашої системи. На вузьких задачах це виявити досить складно. Краще вже тоді підготувати фрагмент коду, якщо є така можливість то з реального проекту, де будуть певні код смели та проблеми в реплізації та дивитись як кандидит будет рефакторити його та на що буде звертати увагу.
Ну і на тему код ревью я поки що не бачив кращого матеріалу ніж цей mtlynch.io/human-code-reviews-1