Есть такое понятие "ближайшая зона развития" вот у взрослого человека эта зона достаточно большая и если сначала решать задачу и только потом ее записывать, то обучение происходит нормально и эффективно, в другом случае обучение происходит на неправильных решениях. От того, что человек сделал "неправильно", а вы потом ему показали как "правильно", то у него скорее отложиться как раз первый (неправильный) вариант, а не второй.

Ну вот для этого и надо учить людей, но сначала думать, а потом делать. Ну если не все, то пусть учатся это делать, но опять же, сначала придумываете решение, а потом его делаете.

Ну блин, у вас есть человек, вы ему говорите: сделай табурет. Он делает, вы смотрите и такой: плохой табурет переделывай

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

Хотя есть конечно и мудаки которые по 5 раз заворачивают код-ревью, при этом сами не соблюдают своих же рекомендаций, а когда их тыкаешь ссаной тряпкой в свои же косяки истерят и применяют аргумент "потому что я здесь техлид, делай так как я сказал". Я рад что я за 15 лет в IT встретил такого мудака лишь однажды и что проработал я с ним не долго.

Может лучше сначала сделать с ним чертеж и модель табуретки, чтобы он понял принцип и приступать к изготовлению, только когда все +- понятно?

Ниндзя кодеры, которые проектируют по ходу написания это же адская жесть

С первой частью согласен, а со второй близко не встречался

Лучше не встречаться... 🙂 Я долго восстанавливался потом.

Оооо, вторая часть это прямо классека

Я вообще не понимаю вот этого задрачивания на кодревью. Компания вам специально платит деньги, чтобы вы вместе работали и могли думать и потом делать хорошо с первого раза, но нет, мы будем делать все наоборот. В опен сорсе совсем другая модель, там люди вообще не знают о том, что может кто-то, что-то закоммитить, опенсорс это совсем другое.

У меня классикой всегда было "LGTM" и падающий прод. Иногда еще висящие по 2 недели без ревью тикеты, т.к. все заняты другими делами.
А описаный выше случай худшим опытом с код-ревью.
Но не разу я не встретил реально выстроенного механизма ревью который бы помогал обучаться или избегать багов.
Исключение - OpenSource, но там ревью нужен для других целей. А именно: Чтобы делали именно так как мне (мэйнтейнеру/овнеру) нужно.

Так может быть вы просто нормального ревью не видели и дело только в этом

А есть нормальное?

На себя я натягиваю потому что я среднестатичтический IT-шник. 15 лет в индустрии, чуть меньше 10 поменянных компаний разного размера (от стартапов до компаний 1500+ человек). Среднее по рынку. Конечно я допускаю что есть где то исключения (Возможно в FAANG или аналогах (Yandex, Сбер, Mail), там я не работал). Но в остальном увы.

Я тоже когда-то таким был. Мне кажется люди со временем учатся сознательно проектировать

Вот поэтому я и топлю за то, чтобы была специальная фаза, где приходится думать и есть только пишущая палочка и поверхность для надписей и никакого кода

Если такая фаза есть, то обучение идёт в разы быстрее и эффективнее, потому что оно становится осознанным. Программирование это же не про написание большого количества кода, а как раз наоборот, стараться в идеале вообще кода не написать, но при этом результат получить.

Мне кажется круто когда человек пытается что-то сделать, пробует раз, другой, думает, задаёт вопросы, снова думает...

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

Круто, особенно круто когда его труд при этом уважают и не заставляют переделывать с позиции силы (опыта).
Я ненавижу код ревью, потому что уважаю работу коллег, даже если это джун и он накодил полное говно.
Я лучше сяду с ним и при нем отрефакторю его же код своими руками, чтобы показать как можно по другому, чем буду
вещать ему о том какую херню он накодил. Нет способа эффективнее демотивировать разработчика с горящими глазами чем код-ревью.