- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
for (int i=0;i<16;i++)
{
if ((rbuf[i+1]>>16)!=(1<<i))
{
break;
res=SOME_ERROR;
}
};
/* in another function */
DWORD * rbuf =new DWORD[Size];
...
if (rbuf!=NULL) free(rbuf);-
Си / Говнокод #9434
+132
Вот такими библиотеками пользуются клиенты некоторых фирм ;)
Запостил:
klaks,
15 Февраля 2012
Follow us!
Ух ты. Какие люди!
у нас на проекте этого дохера - но на куче систем работает. я давлю где вижу - ну да я ж один. против легионов.
ЗЫ а в чем именно ссуть загадочного цикла, я честно говорю не улавливаю.
А для free() проверка на не нулл не нужна.
мысль запостить на говнокод пришла не сразу, поэтому море срача сюда не попало.
и самое главное - все работало :)
?
если программист не понимает рефлекторно, что {} тут уже самостоятельный stmt и дописывание ; делает второй stmt, значит написанную неверно элементарнейшую ситуацию if () {}; else {}; он будет исправлять только после ругательств компилятора
т.к. программист не может написать базовые вещи верно с первой попытки, значит ему надо дальше читать книжки и набираться опыта, а не доказывать, что дескать он заранее понимал, что компилятор проигнорирует лишний пустой stmt, ведь он гуру
А может, он знает, просто такая привычка?
Вкусовщина чистая это всё. И судить о профессиональном уровне по таким мелочам стиля кода - это тупо.
Вот прикинь, чувак пришёл на собеседование и за полчаса написал суперкрутой быстрый алгоритм, а ему в ответ - "вы не приняты, вы ставите ; после }". А он в ответ нахуй пошлёт, очевидно.
бывают конечно уебаны:
первые, кто имеют такие дурные привычки как "; после}", но "суперкруты",
вторые, кто говорит первым: "вы не приняты, вы ставите ; после }".
если он на собеседовании сделает }; это будет всего лишь повод спросить его зачем он так сделал - может переволновался
когда человек в коде, который он написал в комфортной обстановке, ставит ; после } - это 100% означает, что у него нет нормального опыта программирования на С++ - он в своей жизни банально МАЛО писал таких конструкций на С++, и хоть конкретно этот недочет очень быстро исправить, 100% всплывут миллион других недочетов
и да, обычно программиста берут на работу, чтобы он реализовывал задачи по ТЗ, разрабатывая корректный, поддерживаемый и приемлемый по быстродействию код, а не писал write-only код, итерации жизненного цикла которого "всё удаляем, пишем заново", поэтому суперкрутые алгоритмы на собеседованиях не задают (кроме узкоспециализированных областей, где кандидат должен знать эти алгоритмы для соответствия вакансии)
Edit->Replace("};", "}")
а запятая в "точке с запятой" - это оселедец.