How do you like this concept, when code review is done not by experienced, but by beginners?

Question

Development Methodologies

Смотрите.
Новичок приходит работать в проект. Тот сложен и велик. Хорошо бы поскорее и побольше всего изучить, чтобы либо облегчить работу, либо выложиться на все 100146 (сон по 4 часа в сутки, кофе литрами до болей в сердце и все такое) и быстро заслужить одобрение и даже повышение.
И он берет репозиторий, выбирает более менее актуальную по текущему и следующему спринту часть проекта, и садится читать код. Внимательно читать.
А "внимательно читать код" - это автоматически дает побочный эффект - делается дополнительный code review. Как попутный газ при добыче нефти.
И этим "попутным газом" можно неплохо "топить". Как минимум можно подрядиться писать тесты на тот самый код, который ты изучаешь. И этими тестами "топить" отчеты на daily scrum'ах.
А если будут найдены какие-то баги, то еще и свое ЧСВ получится растопить, и самоутвердиться за счет коллег... которые более опытны, вплоть до лидов, но из-за сложности разработки им не хватает сил на банальную рутинную проверку кода, которая как раз под силу новичку, хотя опыт его весьма мал.

Я сейчас описал все это от лица самого новичка. Но и если посмотреть со стороны руководителя команды, то не видно в этом ничего такого уж ужасного. Возможно, тимлидам стоит намеренно такое организовывать.

P. S. Я изобретатель оружия, военный теоретик... и программист-джуниор.

Answer 1

PS I'm a weapons inventor, military theorist... and junior programmer.

Rather, a master of crooked analogies and bad style.
On the subject: think about whether the business will pay for the time of the green June (even these pennies), which will stare at someone else's code in vain. Wasted, because he does not understand anything.
And the thread will close automatically.

Answer 2

Why not? A bit like https://ru.wikipedia.org/wiki/%D0%AD%D0%BA%D1%81%D... extreme programming - when all project participants check each other's code.

Answer 3

(sleep for 4 hours a day, liters of coffee until heartaches and all that) and quickly earn approval and even promotion.

what a stupid worldview, ruining your youth / youth in order to quickly study something, well, not to the same extent, well, you yourself will unlearn at the institute and read books, and you will no longer be a junior but a middle with 1-3 years of experience, unless of course did term papers and diplomas and their projects for interest

And he takes the repository, selects the part of the project that is more or less relevant for the current and next sprint, and sits down to read the code. Read carefully.

for any task, any programmer in an existing project, imagine having to carefully study the existing code, and from the fact that a junior does it, this is not nonsense, everyone will have to
add something from the top of the new one after studying

And "carefully read the code" - this automatically gives a side effect - an additional code review is done.

as I wrote above - anyone is studying, and not just a junior,
what should everyone do a code review now? everyone does, but not to the extent that it is understood in the true sense
the result of the code review from the pros - there will be a detailed analysis of the jambs in the code and how to refactor / rewrite
the result of the code review from the junior - a lot of questions "what is it and how is it works", and certainly not searching for bugs
if the review is not done by experienced ones, then there are reasons for lack of time, and you were not hired for this because your effectively 0 will be in this review, and if there is a need, there is someone to conduct a quality review
and tests for the same reason - or they are not written, because there is no efficiency from anyone (time is short, business needs to be faster and tasks are not so good for unit testing everything), and even more so it will not be from June, and if plans suddenly change - there is an experienced one to write these tests