Comments | jakobz: Вопросы на собесах №3

jakobz

Вопросы на собесах №3

Feb 19, 2020 00:58

Что я понял со своего опыта - на собесе надо обязательно просить писать код. Причём не на бумажке - а в приближенных к боевым условиям: в IDE, в консоли хрома, где удобно. Очень часто люди либо подкупают хорошими soft skills, либо хорошо рубят в теории - и ты под впечатлением такой: "ну тут - джедай, тут даже как-то неудобно просить задачки решать ( Read more... )

Leave a comment

Back to all threads

rdia February 19 2020, 02:57:24 UTC

> И F5 ему религия не позволит нажать пока ни одного лишнего пробела не по conventions не будет.

Это онанизм.

jakobz February 19 2020, 08:44:44 UTC

Сколько видел нормальных, практикующих, дотнетчиков - оформлять код по гайдлайну у всех на автомате происходит. И от нарушения гайдлайна - всех прям передёргивает. И надо вот тебе человека получить, который будет комиттать код не в гайд, чтобы потом с ним команда сралась и учила?

rdia February 19 2020, 13:51:13 UTC

Ну не до компиляции же. Скомпилировать быстрее, чем форматировать, а сначала надо делать то, что быстрее вылавливает ошибки.

max630 February 19 2020, 17:07:33 UTC

Вот ещё. Студия сама форматирует, ещё и по рукам приходится бить, чтобы не срала в гит своими переформатированиями.

ratnos February 19 2020, 19:36:59 UTC

Нужно на уровне центрального репозитория запретить мержить в основную ветку неотформатированный код. Тогда весь код будет отформатирован сразу.

max630 February 19 2020, 20:46:28 UTC

Можно так, а ещё можно не насиловать мозг себе и людям: ревьювер код понимает значит всё в порядке.

ratnos February 19 2020, 20:48:37 UTC

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

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

max630 February 19 2020, 22:14:48 UTC

Не, я всё правильно написал.

> васин код будет числиться за авторством Пети

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

> читаемость кода будет постепенно деградировать

следование сумбурным и избыточно специфичным правилам больше относится к ЧСВ тех кто их енфорсит чем к читаемости.

ratnos February 19 2020, 22:21:26 UTC

Ебать

jakobz February 22 2020, 05:30:43 UTC

У меня был такой товарищ на проекте - автоформатил всё что трогает. Я на него все баги переассайнивал с каментом "блейм на тебя показал". Ибо дичайше задалбывало что он всю историю портит.

max630 February 22 2020, 08:25:48 UTC

Да ну реально ерунда же. У нас как-то концы строк нормализвали - вот теперь на них всё показывает (а не каждый гуйный blame умеет в -w). Команды есть которые по всему коду ходят и один внутренний тип строчек на другой меняют (ну это уже плюсы, в шарпе такого меньше, но тоже). Ничо, все живы

Когда это на самом деле неприятно - это на ревью.

metaclass February 23 2020, 07:05:32 UTC

Так мож лучше хуки для коммитов-пушей настроить и режектить их пока общий формат соблюден не будет?

daedmen February 23 2020, 16:20:41 UTC

А вот интересно, почему IDE научившись в Show History for Selection и diff без учета пробелов и даже форматирования, ещё не умеют в честный blame.

О семантическом diff-е я уже молу, хз уже сколько лет в youtrack топик висит. Хотя тот же semanticmerge dot com тоже ни как не windows версию сделать не могут. Хотя ладно, просто я не ощущаю жадный, что он стоит 60 баксов в год.

max630 February 19 2020, 17:24:05 UTC

> И надо вот тебе человека получить, который будет комиттать код не в гайд, чтобы потом с ним команда сралась и учила?

Это вообще это-то непонятное суждение, если я не делаю по какому-то гайду (который я ещё не видел) сам, что я не могу сделать по гайду когда надо?

nponeccop February 21 2020, 23:11:54 UTC

Ну вот есть люди которые ебашат по дефолтовому гайду.

Есть люди которые видели какой-то гайд и на собеседовании ебашат по нему

А есть люди у которых одна строчка так вторая сяк, а вот тут мы аккуратненько пробелами отобьём.

А есть люди которые вообще не слышали о гайдах т.к. реально на этом писали только ICFP да сортировку пузырьком.

Собственно если все работают по гайду - то нанимать человека, который вот прямо на обеседовании говорит "выбросить из схемы процессора реализацию 80 % редко используемых команд, оставив только 20 % используемых часто следование сумбурным и избыточно специфичным правилам больше относится к ЧСВ" то такого человека надо нанимать только, если он совсем уж бох.

У меня был такой случай, 1 коммит с именем коммит (ну не совсем так но почти) на 1 пуш. Но в остальном было классно.

max630 February 22 2020, 03:19:32 UTC

Я бы сказал что я видел уже 100500 этих гайдов, и я во-первых не помню как оно там именно в сишарповом гайде, во-вторых ну реально вот мне на собеседовании надо писать проперти с акксессорами там где публичьное поле пошло бы, может ещё и документацио xmlем не забыть?

> 1 коммит с именем коммит (ну не совсем так но почти) на 1 пуш

вот тут уж не пиздитека. Вообще коммиты к сожалению мало кто нормально комментирует, и "C# программисты - всё-по-гайту" к их числу точно не относятся.

Back to all threads