Я бы назначил пару ошибок с низким приоритетом в первый день, так, чтобы никто не кричал, если они не будут сделаны сразу же, давая новому разработчику некоторое время, чтобы ознакомиться с базой кода.
Самое важное, что нужно сделать - это просмотреть код всей своей работы в первые пару недель. Вы не хотите узнать, что парень идет в неправильном направлении или не соблюдает стандарты кодирования компании месяцами в вещах. Лучше убедиться, что он знает, чего ожидать с самого начала, и проверки кода гарантируют это. Конечно, я думаю, что проверки кода хороши для всех сотрудников (мы проверяем 100% нашего кода перед развертыванием), но они важны для новых сотрудников и должны проводиться лично, чтобы вы могли ответить на вопросы и отослать их к документации, которой у них может не быть. видел еще, если нужно.
Чего вы не хотите, так это того, что приходит новый парень и использует стиль, отличный от остальных. Люди часто пытаются продолжать использовать стиль кода своей предыдущей работы, даже если это противоречит стилю кода, используемому на новом месте, что может создать путаницу и раздражение со стороны других разработчиков.
Одна вещь, которую я заметил даже у опытных разработчиков, это то, что некоторые из них не так хороши, как казалось в интервью, проверка кода поможет вам быстро это выяснить, так что вы можете это исправить. Это также будет стимулировать их к тому, чтобы они действительно что-то сделали, я видел, как новые сотрудники, которые не прошли проверку кода, затягивают проект, не показывая никому, что они делали, а затем оставляют за неделю до крайнего срока, который, как они знали, не наступит, потому что они были над головой и фактически не завершили какую-либо часть проекта. Лучше проверять рано и часто с новыми людьми, пока вы действительно не уверены, что они работают.
Кроме того, это нормально, что новый парень потрясен состоянием вашего унаследованного проекта. Он создан не так, как он думает. Ожидайте этого, выслушайте его и не отклоняйте автоматически все, что он говорит. В частности, этот человек, кажется, имеет больше опыта, чем вы или другие разработчики, он может видеть вещи, которые вы не рассматривали. Однако, как менеджер, вы должны сбалансировать предлагаемые изменения с текущей рабочей нагрузкой и сроками. Вы все можете потратить некоторое время на изучение того, как реорганизовать существующий код, и потратить несколько часов на оценку своего времени, особенно если у нового парня есть какие-то веские основания. Вы, вероятно, не можете поддержать полное переписывание (многие люди, которые приходят с новостями, думают, что мы должны начать все сначала и сделать это лучше),
Если у вас есть какое-то время, когда он, как ожидается, не внесет свой вклад (и полностью учитывает свое время клиентом), это может быть также время, когда он может начать с некоторых из тех вещей рефакторинга, которые вы хотели сделать, но не сделали этого ». у меня было время сделать Иногда полезно использовать период обучения нового человека для решения некоторых вопросов, которые не включены в план проекта. Они могут изучить базу кода, и если то, что они хотят сделать, не работает, вы не повлияли на существующие расписания, потому что еще не включили их в существующее расписание. И если это сработает, у вас может получиться большой выигрыш, упростив будущее обслуживание или улучшив безопасность, или в чем бы то ни было проблема.