Код ревью (code review): лучшие практики, как проводить.

2:23 - начало рекомендаций

Code review - это практика мануальной проверки, написанного кода, который, как правило, еще не ушел в production среду. То есть того кода, который пока содержится в некой ветке задачи.
Так как проверка мануальная, то осуществляет ее другой человек, не тот же самый, что данный код написал, а как правило - некий коллега, специалист, со схожим уровнем знаний или повыше.
Проверка кода - это отдельный шаг в процессе continuous integration (непрерывной интеграции).

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

[zerg]

У Вас преступно мало просмотров и подписчиков! Я как разработчик со стажем и опытом управления командой с интересом смотрю ваш канал, узнаю что-то новое и систематизирую уже накопленное. Спасибо, что делитесь своим опытом!

[ДенисДенисов-уо]

Здравствуйте, Андрей! Очень рад, что вы возобновили записывать новые видео! У вас очень хорошо получается передавать сложные темы простыми и понятными словами. Буду ждать ваших новых выпусков! С новогодними Вас праздниками:)

[meteysh]

Сразу видно, что Андрей шарит прилично) приятно слушать. 🎉

[romanzenia]

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

[suleymanaliev]

Удивительно, суперполезный контент, но мало просмотров. Думаю это дело времени)

[tomclancys]

Здравствуйте Андрей. Спасибо за видео!

[hruser]

ко мне как то типа гениальный разработчик привязывался без причинно на код ревью, на что я ему ответил: у меня хотя бы есть жена. Больше он ко мне не лез

[neznaika]

Андрей, можете рассказать о хорошем написании логики? Постоянно мучают сомнения, что делаю не так.
Конкретно, как организовать котроллеры? Со всеми нюансами, а не обрезанные примеры "на коленке". Допустим CRUD. Принять ID, из модели выбрать объект, проверить всё это, выдавать юзеру уведомление, а дальше работа с сервисами (классами), валидация... Было бы интересно узнать

[ВладВлад-ьфд]

Добрый день. А подскажите пожалуйста, последнее время очень интересует тема асинхронности в php, если быть точнее заинтересовался темой swoole, но почему то в сети хоть и много инфы и документаций (openswoole) но все как то очень мутно, разбросано и не понятно. Могли бы вы прояснить эту тему

[mushnikov]

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