Как уже на собеседовании понять, какое в компании качество кода?

Question

[Alline]

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

Каким образом можно выяснить еще на собеседовании, какое на проекте качество кода? Хотя бы примерно? Пока ориентируюсь на косвенные признаки: стек, сколько лет проекту, сколько разработчиков в команде, как настроен процесс деплоя, как с тестированием...

Не будешь же спрашивать: "Скажите честно, какой у вас процент говнокода?" :)

Comments

[Максим Федоров]

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

Вас берут для чего-то, за что платят деньги -- учитывайте этот момент

[Lord_Dantes]

Не знаю как везде, а вообще на собеседованиях обычно дают задать всякие интересующиеся вопросы.

[Антон Р.]

Максим Федоров, а что значит "экстремальное программирование"?

[Антон Р.]

"Скажите честно, какой у вас процент говнокода?" - это хамство, а не вопрос. Напоминает дебильные вопросы HR-ов "Назовите ваши слабые стороны".

[Alline]

Максим Федоров, спасибо. Возьму на вооружение)

[Alline]

Антон Р., В вас бага. Скачайте чю и установите его себе куда-нибудь, чтоли...
Имхо, "назовите слабые стороны" - это не хамство, а просто некомпетентность.

[Антон Р.]

Alline, ЧЮ установлено по-умолчанию, еще с релиза. А из-за вопросов про говнокод отношения с тимлидом могут быть так себе, оно тебе надо?

[ArgosX]

Антон Р., а вы счастливый человек если не сталкивались с экстремальным программированием=)))

[Антон Р.]

ArgosX, так что это такое? "Макет нужен еще вчера"?

[ArgosX]

Антон Р., макет и сидишь кодишь до тех пор пока макет не сделаешь 10-20 часов не важно. пока не будет

[Антон Р.]

ArgosX, я так каталог верстал в Corel перед очередной выставкой, до 22-00 пару дней, а потом приезжал домой в пригород к часу ночи. И ничё, живой.

[Максим Федоров]

Антон Р., Экстремальное программирование

[Алекс Глебов]

Качество кода вещь субъективная, пока не увидишь, никак не поймешь

[CityCat4]

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

Answer 1

[g_s_e]

Ну вообще стоит начать с того, что Вы подразумеваете под говонокодом. Зачастую говнокод - это не только криво написаный метод/класс, это еще плохо оптимизированная бд, кривая архитектура приложения и неверно выбранные инструменты для решения задач поставленных перед проектом. В любом случае проект делают люди, которые меняться на проекте, посему нужно выяснить детали самого подхода разработки проектов в самой компании:
- спросить сколько проект находится в разработке, степень покрытия тестами, если прокт древний и не покрыт, то плохо;
- если проект древний, то узнать обновляются ли на нем версии инструментов/фреймворков на более новые, выяснить что с обратной совместимостью;
- спросить как происходит процесс мерджа новых фич, если если через мерджреквесты на тимлида, которые он аппрувит или отклоняет, то уже не плохо, и Вам подскажут, проконтролируют, что не плохо. Eсли пуш в ветку девелопмент, то плохо;
- если используют мерджреквесты на тимлида, то спросить используется ли git-flow, если да, то хорошо, это минимизирует варинт того, что Вы получите кучу крешей от того, что было запушино коллегами, если не используется, то плохо;
- узнать что проиходит с CI на проекте, если есть тесты, то запускаются ли они при деплойменте, если да то хорошо, нет - плохо;
- если это web проект, на котором используется бекенд и фронтенд фреймворки, то стоит спросить развернуты ли они на разных серверах, если да то хорошо, если нет то плохо;
Выяснить это все иначе, чем по косвенным признакам, является маловероятным.

Answer 2

[Developer]

Хочется сказать - скажи.
Только учти, что тогда никакой работы не найдёшь, везде есть говнокод

Comments

[Максим Федоров]

Alline,

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

Тостер — это просто место скопления людей :) А вот люди всегда одинаковые и разные :), просто обращать внимание не стоит на часть

[Alline]

Максим Федоров, Просто видимо, когда чужие вопросы читаешь, такое игнорится автоматом. И создается впечатление, что на тостере очень интеллигентная и дружелюбная тусовка)
А потом как свой вопрос задашь, как наприлетают уведомления с такими вот ответами... У меня каждый раз пригорает :D

[FanatPHP]

А что не так с этим ответом?
Мне кажется волне правда жизни.
Правда, с выводом я тоже не согласен, но почему должно пригорать-то? Деже если человек пишет о своих проблемах, то это ведь его проблемы, не ваши?

Answer 3

[devalone]

Как вариант, собеседоваться в компании, которые занимаются open-source продуктами

Answer 4

[Stanislav]

Просто спросить. Прямо и четко. Врать вам не будут (им же с вами еще работать), разве что приукрасить могут, но это не так страшно.

Answer 5

[Егор Казанцев]

Говнокод ... mvp делается из тяп-ляп и в продакшен а потом бесконечный рефакторинг. Вообще рефакторинг - он бесконечный.

Answer 6

[Пума Тайланд]

Так попроси, у меня когда просят я всегда показываю

Answer 7

[HellWalk]

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

Так и сказать - "ваша компания мне нравится, но можно взглянуть на код проекта?"
Обычно готовы показать.

Answer 8

[yuopi]

Чем больше вопросов по программированию вам зададут на собеседовании, тем больше внимания они уделяют качеству кода

Comments

[index0h]

Это ни как не связано

[yuopi]

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

[index0h]

yuopi

он не с потолка вопросы берет, а спрашивает то, что применяется на практике у них.

Это очень зависит от компетентности интервьюера.

Очень часто бывают ситуации, когда интервьюер задает вопрос в стиле "вы работали с технологией X, а с Y?", хотя ответ не важен, так как все равно в проекте это не будет использоваться. Особенно это касается unit тестирования: "имеете ли вы опыт написания unit тестов?", допустим вы ответили "да", дальше приходите на проект, а там покрытия тестами, либо просто нет, либо стремится к нулю.

[yuopi]

index0h, ну это как раз вопросы - пустышки, на которые ответ да/нет. Должны попросить пример реализации.
Как быть не должно:
-Знаешь паттерны?
-Да
-Молодец!
Как должно быть:
-Знаешь паттерны?
-Да.
-Какие?
-..
-Как применять такой то?.. зачем он, для чего применяется и тд

Так вот я к чему и писал первое сообщение - к тому, что больше интервью на поверхности, тем хреновее там код и работа. А если тебе прислали хренового интервьюера, вместо тимлида, то в компании либо нет тимлида или компетентного прогера, либо такое вот отношение к программированию и там точно говнокод

Answer 9

[Кирилл Горелов]

Тебе уже g_s_e подсказал, как правильно сделать.
Задавать косвенные вопросы.
К тому списку я бы добавил.
За проект отвечает один программист или его могут делать несколько разных.
Проводится ли код ревью.
Ведется ли документация.
Есть ли стандарт написания кода.
Бывает ли необходимость работать после работы, в выходной, к примеру.
Ну и что-то в этом дохе. Почему это важно? Так ты поймешь, если фирма уделяет этому внимание, то наверняка они и к коду относятся с почтением. А не делают спустя рукава.

Comments

[Армянское Радио]

Какая связь между сверхурочными и говнокодом?

[Кирилл Горелов]

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

Answer 10

[nozd up]

Answer 11

[index0h]

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

Можете задать такой вопрос. Программистов можно условно разделить на три категории по тому, что им нравится:
* Исполнитель - его радует результат, максимально быстро и чтоб работало.
* Гик - ему интересно попробовать всякие моднявые штуки, только вышла новая либа - он хочет ее заюзать.
* Головолом - ему интересны сложные крупные задачи, там где надо посидеть, по медитировать.
Собсно вопрос: какого человека по этим категориям вы ищите?
Чаще всего идет микс из двух, если говорят все три важны одинаково - интервьюеру глубоко фиолетово. Если при этом интервьюером является технический специалист - это очень плохой знак.
Так же можете спросить, кто по этой шкале есть в команде. Если команда состоит из одного типа программистов - это очень плохо.

Что касается качества кода: спросите про используемые соглашения, критерии на код ревью и покрытость тестами

Comments

[Alline]

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

Понимаю, поэтому и не спрашиваю) Да и странно это
Мне кажется, если спросить про гика, исполнителя и головолома, работодатель напряжется) Я бы напряглась

Что касается качества кода: спросите про используемые соглашения, критерии на код ревью и покрытость тестами

А вот тут согласна

[index0h]

Alline,

Мне кажется, если спросить про гика, исполнителя и головолома, работодатель напряжется) Я бы напряглась

Я задавал этот вопрос на множестве собеседований и как соискатель и как интервьюер, были ли удивления - самую малость, были ли напряжения - нет. Интервьюер должен знать какого человека ищет, если не знает - это плохой маячок. На самом деле это вопрос о характере задач, которые требует выполнять та, или иная должность. Чаще всего вы не найдете подобной информации в вакансии. А она важна, причем очень.
Примеры:
* Исполнитель - сайты визитки, где вообще похрен как.
* Исполнитель+Гик - скорее всего это небольшие проекты, до полу года, качество - ну такоэ, потому что на вчера надо... есть, немного свободы выбора тех. стека
* Исполнитель+Головолом - похоже на крупный/старый проект, не знаю как по качеству, но "головоломство" пойдет на разгребание чужого.
* Головолом+Гик - вероятно что-то более-менее новое и крупное, либо не большое, но сложное.
...
Грубо говоря суммируйте ту информацию, что вам рассказывают про проект с тем, какая команда уже есть с тем, что хотят от вас. Так вы поймете, надо ли оно вам.

Answer 12

[Северное Сияние]

Если вопросы "сложные", ООП к примеру, скорее всего там нет говнокода.
Если хотят выяснить, знаете ли вы какие-то банальные функции - значит собеседующий сам мало компетентен, а работники еще хуже и как следствие говнокод.

Comments

[index0h]

Если вопросы "сложные", ООП к примеру, скорее всего там нет говнокода.

Это ни как не связано

Answer 13

[Армянское Радио]

Вопрос эквивалентен следующему "как уже по фотке в Тиндере понять, что парень не мудак, 19 советов от Космо".

Никак. Названные здесь признаки будут лишь косвенно отвечать на вопрос о мудаковатости кода в организации.

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

Как многие уже тут заметили, мудаковатость, извините, говнокод, есть в любом более-менее живом проекте, иначе получается тоскливый налет полной предсказуемости и отсутствия рисков - что это за бизнес такой, где не нужны решения "вчера", где никто не проверяет концепцию быстрым тяп-ляпом по дендрофекальной технлогии? Дайте два, я тоже так хочу!

Для справки, в случае с мужиками, такое чудо называется manicorn, в природе, как следует из названия, не водится.

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

Answer 14

[Валерий Тихомиров]

Говнокод и легаси будут везде. Потому что бизнесу всегда нужно чтобы было готово уже вчера, что подталкивает сначала быстренько наговнокодить, а потом уже (когда будет время) исправить это (главное чтоб комменты оставляли в таких участках кода)...