60 антипаттернов для С++ программиста

Мне кажется я что то уже такое видел, не Ваше ли? Полистал бегло, понравилась толерантность сдержанность - вместо «категорически нельзя делать вот так вот» пояснение что бывают случаи когда вот так вот все таки можно;-)

Спасибо!

Спасибо.

Было. Но это расширенная версия, как по количеству «советов» так и по полноте пояснений.

Спасибо!

ЗЫ

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

Ключевым словом должно было быть Андрей-спамер. Сорян, на ЛОРе вы сами заработали себе эту репутацию.

В pdf версии не хватает оглавления.

Да, пожалуйста. И за такое скажу книгу выдавать :)

P.S. Эх… Побольше бы «спама» такого типа и качества :)

Вредный совет N31. Всё в h-файлах

пардон, а как же шаблоны? О них там ни слова.

Прочитал по диагонали до середины, остальное в оглавлении. Вроде все по делу. Буду своим даунам показывать.

UPD найс у меня телефон «опечатку» исправил с джунов на даунов.

N45 «Ноль, один, два, Фредди заберёт тебя»

Раз и другой, Фредди придет за тобой! :)

А причём здесь шаблоны? Весь реальный большой проект из шаблонов? Не верю.

Из сплошных шаблонов будут вспомогательные библиотеки. Это другое.

WTF большой проект? У нас стандартная история - main в сишнике, остальное шаблоны в h файлах

Насчет везде int (№8?) - есть нюанс, int это самый быстрый целочисленный тип. На лоре есть люди для которых это важно, и которые вообще живут на 32х битных платформах и им эти size_t не уперлись ни разу.

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

WTF большой проект?

В моём понимании это проекты более 5 MLOC.

Буду своим даунам показывать.

Может не надо даунов мучить? Им и так несладко живется.

на винфак

главный антипаттерн: всё пиши на C++

В контексте loop’чиков по вектору самый быстрый выглядит примерно так:

for (int i = 0, n = v.size(); i < n; ++i) {
  // something touching v[i]
}

И да - там где вектор «длинным» быть не может из физических / бизнес соображений - size_t не впился, от слова совсем. Даже (тем более) на 64-ёх битах.

Но даже в 32-ух битах есть нюансы связанные с тем что signed int «не переполняется», а size_t - unsigned.

На хабре наверное… Там постоянно постят это

Андрей, очень классный текст, большое спасибо!

интересная граница.

Видимо, где-то на 1-2 MLOC точно заканчивается возможность одного человека удержать все исходники в голове и для того чтобы дорасти до 5 MLOC нужны уже методы управления выше уровнем?

Город: Berlin

рот бы захлопнул, воняет оттуда

Спасибочки, думаю, весьма полезно почитать.

Что не так с С++?

for (int i = 0, n = v.size(); i < n; ++i)

Почему не просто

for (int i = 0, i < v.size(); ++i)

?

2 нюанса:

1. v.size() при прочих равных может привести к лишнему memfetch на каждой итерации.

2. type of v.size() гарантировано не int.

type of v.size() гарантировано не int.

Почему?

size_t - unsigned.

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

Что правда - то правда ;)

Разве здесь не напрашивается size_t,раз уж у нас есть v.size?

Там и это есть, причем в самом начале.

потому что он будет size_t, по крайней мере для вектора и списка cppreference

рот бы захлопнул, воняет оттуда

Проблема на вашей стороне экрана.

Вредный совет N1. Только C++

Нас невозможно сбить с пути!

Вредный совет N24. Прося помощь на форуме, делай так, чтобы из тебя вытягивали информацию. Так всем будет интереснее.

Ещё скажите что нет. :)

Вредный совет N26. Напишу всё сам

Та шож ты делаеш, афтар?!

Не пользуйтесь стандартной библиотекой языка. Что может быть интереснее, чем написать свои строки и списки с уникальным синтаксисом и семантикой?

Не, ну не до такой же степени, конечно. Хотя в stdlib кроме строк да списков всякого говна хватает.

Вредный совет N27. Удалите stdafx.h

Чо?

как в среде Visual Studio

Чо?! А, ну да, афтар даже что-то про бумажную книгу говорил. Люди! Не ведитесь! Вас там покусают, и вы станете виндузятниками!

Все знают,

Да щас.

Придайте коду оригинальность, используя конструкции вида 1[array] = 0.

Блин, всё время забываю. Спасибо, что напомнили.

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

Во-первых, https://rsdn.org/forum/humour/1600906.1 (см. цитируемое зелёненьким). Впрочем, афтар тоже пишет:

Естественно, виноват не сам по себе оператор goto, а его необдуманное использование.

Но дальше он же пишет:

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

И хрен ты, афтар, угадал: например, при кодогенерации не помню уже каких парсеров у меня в генерируемом коде были массивы меток. Ну и например bison-овский таблично-управляемый автомат ЕМНИП тоже кучу меток содержал (во вручную написанном движке, а не в скармливаемых движку сгенерированных таблицах); полагаю, это вопрос эффективности.

А ещё лучше и короче старый добрый C-style cast: (Type)(expr).

Не могу не согласиться. :) Общего правила для себя так и не выработал, но как минимум ради narrowing (int)long ломать себе пальцы об эту плюсовую дичь точно не буду.

Вредный совет N43. Не инициализируй

Я бы иной раз и рад, например когда int c; if (b) init c; ...; if (b) use c;, да дурной компилятор ругается.

Модификаторы доступа private и protected для параноиков.

Гы. :)

Вредный совет N56. Больше классов!

Гораздо вреднее и распространённее идиотизм каждому классу добавлять парный интерфейс. Впрочем это среди жаверов.

Вредный совет N61.

Везде обман. :(

Полистал, выглядит интересно, почитаю вечером.

PS:

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

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

Никогда не читай стандарт, ищи ответы по синтаксису языка на форумах.

Если очень нужно изменить const, то измени ее через ссылку!

Экономь память, не заводи новые переменные!

… много их

Вредный совет N27. Удалите stdafx.h

Действительно странно. Первым делом его удаляю, ведь у меня его нет. Его я видел исключительно в проектах для Visual Studio C++.

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

Легко: выход из всех вложенных циклов наружу без нагромождения дополнительных «переменных для проверки нужно ли выйти наружу».

Но за книжку спасибо.

Постоянно писать «как делать правильный код» надоедает

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

Вот читать core guidelines почему-то не надодедает. А от вашего стиля слегка блевотное ощущение.

Легко: выход из всех вложенных циклов наружу без нагромождения дополнительных «переменных для проверки нужно ли выйти наружу».

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

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

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

ИМХО, и да, и нет.

Я бы предпочел читать в формате «best practices», «code guideline», «clean code» и аналогичных. Но! Разные люди по разному открыты к восприятию информации, особенно в «трудном» возрасте. Возможно эта форма изложения найдет своего читателя. Чем больше разных форм – тем, в целом, лучше.

for (int i = 0, n = v.size(); i < n; ++i) {
  // something touching v[i]
}

Если речь только о прохождении по списку, и позиция не нужна в теле цикла, такое уже давно (с C++11) моветон, а рекомендуется использовать вариант без явного использования переменной-счетчика:

for (auto &x : v) {
  // something touching x
}

Спасибо, классный текст :) Уложил в голове пару вещей для себя.

Кампилятор ворнингами ругацца будет аднака:-(

А заказчик тебе и говорит - у нас дефолтный компайлер VS2008. И ты такой упс… и переписываешь все эти красивые циклы без счётчика на циклы со счётчиком.

А уж во что обходы с итератором превращаются…

Настоящий программист имеет красные глаза, толстую задницу и не боится условных переходов(с)

Еще когда-то был свод правил настоящего программиста:

0. Настоящий программист никогда не пишет комментариев, комментарии для ламеров, не способных читать код.

1. Настоящий программист никогда не пишет документацию, пользователь должен брать от программы только то, на что он способен.

2. Настоящий программист никогда не пишет тестов, т.к. тесты пишут неуверенные в себе люди, а настоящий программист всегда уверен в своем коде.

3. Настоящий программист не боится использовать оператор goto, если в коде настоящего программиста стоит оператор goto, значит так надо.

…

Я же написал: после С++11. Для старья, конечно остаются циклы. А ещё есть BOOST_FOREACH.

А, ок.

Буст я бы не стал поминать всуе, а вот насчет сравнения скорости

for(auto &x: L) ...

и варианта bugfixer конечно интересно. Насколько компайлер может развернуть первый вариант для массива небольшой статической длины например? Мне то первый вариант самому больше нравится, есть в нем что то питонье;-)

А заказчик тебе и говорит - у нас дефолтный компайлер VS2008

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

Visual Studio поддерживается 10 лет с момента выхода.

Visual Studio 2022 End of Support Jan 13, 2032
Visual Studio 2019 End of Support April 2029
Visual Studio 2017 End of Support April 2027
Visual Studio 2015 End of Support October 2025
Visual Studio 2013 End of Support April 2024
Visual Studio 2012 and earlier Out of Support

Если речь только о прохождении по списку, и позиция не нужна в теле цикла, такое уже давно (с C++11) моветон

Вам «шашечки», или ехать?

Есть, есть такие заказчики. У них свои резонные резоны.