Code review без компромиссов
Первая неделя работы с Грейвзом показалась Нике адом. Вторая — адом, к которому она почти привыкла. К третьей она начала понимать, почему студенты уходили от него, не выдержав.
Он требовал отчёты. Не просто «я почитала книжки и написала сто строк кода» — детальные, с графиками, с анализом ошибок, с выводами. Он комментировал каждую строчку её кода на GitHub, и комментарии эти были... безжалостными.
«Здесь вы используете O(n²), хотя можно O(n log n). Переписать».
«Нейминг переменных. Вы серьёзно назвали переменную 'temp123'? Я не знаю, что она делает. Переименовать».
«Этот метод делает слишком много. Разделить на три».
«Вы уверены, что это вообще работает? Протестируйте на граничных значениях и пришлите результаты».
Ника сидела ночами, переписывая код, читая статьи из списка литературы, который всё рос и рос, и пила кофе литрами. Хлоя и Сэм видели её всё реже — она либо торчала в библиотеке, либо в своей комнате, обложившись распечатками и ноутбуком.
— Ты превращаешься в зомби, — заметила Сэм, заглянув к ней как-то вечером. — Глаза красные, под ними круги. Ты вообще спишь?
— Сплю, — буркнула Ника, не отрываясь от экрана. — По четыре часа. Этого достаточно.
— Для человека — нет. Для робота — возможно. Ты становишься роботом, Ника.
Ника хотела огрызнуться, но поняла, что сил нет даже на это.
Первая очная встреча с Грейвзом состоялась через две недели, как он и сказал. Она пришла с распечатанным отчётом на пятнадцать страниц и планом работы на семестр. Он прочитал всё при ней, делая пометки ручкой, потом поднял глаза и сказал:
— Неплохо для начала. Но вы копаете слишком мелко. Я хочу, чтобы вы глубже разобрались в байесовских сетях. И добавьте в план сравнительный анализ с другими подходами. Переделать к следующей встрече.
И всё. Ни «хорошо», ни «молодец». Просто — «переделать».
Ника вышла из кабинета, чувствуя, как горят щёки от усталости и разочарования. Она пахала как проклятая две недели, а он даже не заметил.
Следующие две недели были ещё хуже. Она спала по три часа. Забила на Mobile Legends — просто не было сил. Перестала отвечать Эдриану, который пару раз писал с предложением «расслабиться». Даже с девчонками виделась урывками, на бегу, в кафетерии.
На второй очной встрече Грейвз пролистал её новый отчёт, помолчал и сказал:
— Уже лучше. Вижу прогресс. Но вы всё ещё боитесь ошибаться. Ошибки — это нормально. Я не жду от вас идеального кода с первого раза. Я жду, что вы будете думать. Продолжайте.