НК
Size: a a a
2019 June 05
АШ
спасибо 😊
АШ
Кстати Хуяк если что не матерное слово
PK
Всем привет.
Собираюсь на собеседование на тим-лида backend разработки. Дали предварительное тестовое задание провести code-review кода на php и оформить в виде документа. По коду в целом все понятно, проблема с правильным оформлением - раньше не сталкивался, все на словах было, правильность оформления оценивается отдельно. Поделитесь, пожалуйста, каким-нибудь шаблоном, если такой имеется. Ну и чеклист не помешает - вдруг что-то упущу. Большой респект и плюсик в карму гарантирую. Заранее спасибо.
Собираюсь на собеседование на тим-лида backend разработки. Дали предварительное тестовое задание провести code-review кода на php и оформить в виде документа. По коду в целом все понятно, проблема с правильным оформлением - раньше не сталкивался, все на словах было, правильность оформления оценивается отдельно. Поделитесь, пожалуйста, каким-нибудь шаблоном, если такой имеется. Ну и чеклист не помешает - вдруг что-то упущу. Большой респект и плюсик в карму гарантирую. Заранее спасибо.
T
ты имеешь в виду правильное оформление ревью или вопрос про codestyle похапе?
T
если первое, то я бы у чуваков спросил что они имеют в виду и какой формат им нужен
АШ
Да ладно
АШ
Код-ревью зачем нужен?
АШ
Чтоб автор сделал код лучше, наверное
АШ
Ну и надо оформлять так, как будто пишешь для другого разработчика, чтоб ему понятно было, удобно, и чтоб лучше стало
T
но в целом бы задался вопросом а чо это вообще от меня такое странное хотят - формат код-ревью какой-то
лучше ж просто накомментить - где что-как - ок-неок, какие варианты
лучше ж просто накомментить - где что-как - ок-неок, какие варианты
T
а чо-чо, кстати, реально у кого-то в компаниях есть шаблоны и чеклисты про код-ревью? С чеклистом ещё поверю может быть, но чтоб формат?
BZ
я не сантехник (я только учусь ;), но имхо ситуация такая - ни у кого таких форм нет, они не нужны. втч спрашивающий прекрасно всю жизнь без форм обходился
BZ
сейчас же он выполняет ТЗ, и его попросили оформить выхлоп этого ТЗ так чтобы в нём удобно разобраться было
BZ
а он с перепугу решиол что есть какие-то стнадртные формы
СА
Если бы я очень хотел работать в этой компании, то я бы оформил так, как это оформляется в GitHub/GitLab: форматированный monospace код с нумерованными строками со вставками между строками комментариев, форматированных Arial c фрагментами monospace, и небольшое резюме в конце. Если их это не устроит - не пошёл бы работать, даже если бы в начале очень хотел.
2019 June 06
AA
Если бы я очень хотел работать в этой компании, то я бы оформил так, как это оформляется в GitHub/GitLab: форматированный monospace код с нумерованными строками со вставками между строками комментариев, форматированных Arial c фрагментами monospace, и небольшое резюме в конце. Если их это не устроит - не пошёл бы работать, даже если бы в начале очень хотел.
Это если на лида верстальщиков собеседование? )
2019 June 13
AA
Может у них и спросить, по каким параметрам оформление код ревью будет оцениваться? Из ответов станет понятнее, к какому варианту стремиться
AA
Всем привет.
Собираюсь на собеседование на тим-лида backend разработки. Дали предварительное тестовое задание провести code-review кода на php и оформить в виде документа. По коду в целом все понятно, проблема с правильным оформлением - раньше не сталкивался, все на словах было, правильность оформления оценивается отдельно. Поделитесь, пожалуйста, каким-нибудь шаблоном, если такой имеется. Ну и чеклист не помешает - вдруг что-то упущу. Большой респект и плюсик в карму гарантирую. Заранее спасибо.
Собираюсь на собеседование на тим-лида backend разработки. Дали предварительное тестовое задание провести code-review кода на php и оформить в виде документа. По коду в целом все понятно, проблема с правильным оформлением - раньше не сталкивался, все на словах было, правильность оформления оценивается отдельно. Поделитесь, пожалуйста, каким-нибудь шаблоном, если такой имеется. Ну и чеклист не помешает - вдруг что-то упущу. Большой респект и плюсик в карму гарантирую. Заранее спасибо.
k
этому вопросу уже недели две)) да оформите уже в виде комментариев к дифу/коду