альтернатива физзбаззу

[avva]

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

Интесная закономерность выявляется. Мы начинаем интервью с того, что просим кандидата прочитать вот такой код, и сказать, что он делает. Как бы он назвал эту функцию?

private static int ok(int a, int b) {
   while (a >= b) a -= b;
   return a;
}

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

Comments

[tybr.livejournal.com]

Штирлиц выдал себя написав ok(0,0); :)

[vasja-iz-aa.livejournal.com]

а как ее цензурно можно назвать? она при нулевом б повисает навсегда

[oryx_and_crake]

За такой код надо выгонять с работы с волчьим билетом.
Какой смысл предъявлять на интервью код, с которым _в нормальной компании_ человек никогда не столкнется?

[vasja-iz-aa.livejournal.com]

и при отрицательных. кофе надо пойти выпить

[dzz.livejournal.com]

Пожалуй, это самый "прямолинейный" способ найти остаток от деления положительного a на положительное b, который мне встречался.

Но, IMHO, это не гарантирует знания других важных концепций. Например, контроля области определения входных параметров функции ;)

[summerdream-t.livejournal.com]

Agree

[dzz.livejournal.com]

Смысл - посмотреть, поймёт ли разработчик, что в этом коде не так.

[dmarck.livejournal.com]

вот тут немножко неоднозначно.

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

в реальной библиотеке, кмк, надо чётко описать как ею пользоваться, а корректность внешних данных (если это не общающийся с внешним потоком интерфейс, конечно) оставить на долю вызывающего.

Но, быть может, я в чём-то ключевом неправ (некий червяк меня, как профессионального параноика, ест, да ;-P)

[archaicos.livejournal.com]

Мне думается, что в любой компании, где вообще есть код, найдутся похожие «произведения искусства».

[xxqs.livejournal.com]

poor man's modulo :)

[ichthuss]

Вот как раз насчёт контроля входных значений в ответе тоже можно услышать. Если, конечно, речь об устном собеседовании, а не письменном тесте с одним полем "впишите название".

[mea-sententia.livejournal.com]

ok(a, b) = a mod b, where a >= b > 0
so ok == modPos

[dzz.livejournal.com]

По опыту, если функция допускает возможность впихнуть неправильное значение параметра, всегда найдётся умник, который это радостно сделает.

Ассерты не сильно перегружают код библиотеки, но очень здорово помогают искать ошибки в использующем её приложении.

[0x8.livejournal.com]

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

[lazy-clown.livejournal.com]

Вроде, должно работать и для отрицательных a и b...

[morfizm.livejournal.com]

Удивлён, что в комментариях ещё никто не обратил внимание на самое главное. Название функции подсказывает, что её результат будут интерпретироваться как bool, соответственно, что она делает - это проверяет, что положительное a не делится на положительное b. То, что в случае true возвращаемое число является остатком от деления a на b, это всего лишь любопытный побочный эффект.

[dmarck.livejournal.com]

Ну, да, закон Чизхолма (IIRC) ;)

Ок, проверки суём под #ifdef WITNESS ;P

[lazy-clown.livejournal.com]

Если a отрицательное а b положительное, функция вообще не рработает..

[dzz.livejournal.com]

Я вообще не вижу смысла в письменных тестах такого рода без собеседования при приёме на работу.

[lazy-clown.livejournal.com]

+1

[egorfine.livejournal.com]

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

[dmarck.livejournal.com]

стоп-стоп, откуда там обработка результата как логического?!

[dzz.livejournal.com]

написано int, значит - никаких bool-ов ;)

[dmarck.livejournal.com]

ну, точнее -- оставляем на откуп коллеру ;-P

[morfizm.livejournal.com]

Она будет в вызывающем коде:
if (ok(a, b)) {
...

Вы когда-нибудь видели целочисленное выражение, в котором один из членов вычислялся бы функцией с названием "ok"? :)