Говнокод #27303 — C++ — Говнокод.ру

C++ / Говнокод #27303

0
1. 1
2. 2
3. 3
4. 4
5. 5
6. 6
7. 7
8. 8
9. 9
```
metrics_key::metrics_key(std::initializer_list<std::string> const& il) {
  TRI_ASSERT(il.size() > 0);
  TRI_ASSERT(il.size() < 3);
  name = *il.begin();
  if (il.size() == 2) {
    labels = *(il.begin()+1);
  }
  _hash = std::hash<std::string>{}(name + labels);
}
```
просто гавно

Запостил: gnusmas, 18 Марта 2021

Tweet
Комментарии (36) RSS
- 6apry3uHckuu_nemyx 19.03.2021 07:38 # 0
  
  Говно.
  Зачем аргументы передавать списком?
  Ответить
  - bormand 19.03.2021 07:54 # +1
    
    Надо вообще было списком из std::any.
    Ответить
- Soul_re@ver 20.03.2021 15:13 # +2
  Слишком мало Modern C++.
  
  metrics_key::metrics_key(std::string name_, std::optional<std::string> label = std::nullopt_t) : name(std::move(name_)), labels(std::move(label).value_or(std::string()) { _hash = std::hash<std::string>{}(name + labels); // На случай если порядок объявления запутанный }
  Ответить
  - bormand 20.03.2021 15:24 # +1
    
    Код не эквивалентен, в оригинале можно было больше аргументов передать.
    Ответить
    - guest6 20.03.2021 15:28 # 0
      
      но не больше двух?
      Ответить
      
      bormand 20.03.2021 15:29 # 0
      
      Да можно и больше 2, оно просто ассёртнется в рантайме. А теперь оно не конпелируется!
      Ответить
      
      booratihno 20.03.2021 15:30 # 0
      
      а что такое TRI_ASSERT, кстати?
      всмысле TRI это 3?
      Ответить
      
      bormand 20.03.2021 15:31 # +1
      
      Какое-то кастомное говно. Но точно не статик, т.к. size() в комплайтайме неизвестен.
      
      > всмысле TRI это 3
      
      Я думаю это "неймспейс" их либы.
      Ответить
      
      booratihno 20.03.2021 15:33 # 0
      
      я уже понял, что статик не может быть
      
      я просто подумал, что initializer_list ты можешь проверитьв компайл тайм, но видимо не всегда
      Ответить
      
      bormand 20.03.2021 15:34 # 0
      
      > initializer_list ты можешь проверитьв компайл тайм
      
      Не, для сложных проверок в комплайтайме придётся вариадик темплейт городить... Лист здесь бессилен. А простые можно оверлоадами или дефолтными аргументами разрулить.
      Ответить
      
      bormand 20.03.2021 15:40 # 0
      
      З.Ы. Я когда-то ваял функцию, которая в компайлтайме проверяла, что ей передали чётное количество аргументов правильного типа, а потом отдавала их в сишный вариадик.
      Ответить
      
      guest6 20.03.2021 15:43 # 0
      
      какой кошмар)
      
      но тут еще смешнее: аргументов может быть от 1 до 2 включительно. Нахуя вообще все это городить?
      Ответить
      
      bormand 20.03.2021 15:45 # 0
      
      > какой кошмар
      
      Просто часто промахивался при вызове сишной версии. Там были key/value пары и нулл в конце. А в крестах это таки получилось выразить красиво и типобезопасно.
      Ответить
      
      guest6 20.03.2021 15:50 # 0
      
      почему было не передать (указатель на) массив структур с key/value или как-то хуйнуть их вараргом?
      Ответить
      
      bormand 20.03.2021 15:51 # 0
      
      Неибу. Сишное апи не я проектировал. Видимо на пирфоманс и лаконичность дрочили, как обычно.
      Ответить
      
      guest6 20.03.2021 15:54 # 0
      
      ох сишка, моя сишка
      эти вечные
      ioctl(int fd, unsigned long request, ...);
      Ответить
      
      Soul_re@ver 20.03.2021 16:02 # +2
      
      Fail Fast это тоже один из принципов современного программирования. Раньше компайлтайма сломаться сложно.
      Ответить
      
      bormand 20.03.2021 16:09 # +1
      
      Ну вот взял и сломал программу! Всё же работало нормально в релизной сборке, а теперь она даже не собирается.
      Ответить
      
      Soul_re@ver 20.03.2021 16:16 # +2
      
      template <typename ...Ts> metrics_key::metrics_key(std::string, std::optional<std::string>, Ts...) : metrics_key() {} metrics_key::metrics_key() {TRI_ASSERT(false);}
      
      Починил, компилируется.
      Ответить
      
      bormand 20.03.2021 16:29 # +1
      
      Сообщения об ассёртах не эквивалентны.
      std::array<std::string, 0> il; TRI_ASSERT(il.size() > 0);
      Ответить
      
      bormand 20.03.2021 16:45 # +1
      
      > Раньше компайлтайма сломаться сложно.
      
      Можно и раньше, если приучить юзера, что после изменений всё ломается. Тогда у него рефлекс будет срабатывать сразу, когда он задумается о новой фиче: "опять эти мудаки что-нибудь сломают, лучше сделаю вручную".
      Ответить
      
      booratihno 20.03.2021 16:45 # +1
      
      >. Раньше компайлтайма сломаться сложно
      статический анализ в IDE?
      Ответить
      
      Soul_re@ver 20.03.2021 17:02 # +2
      
      Но он же просто варнингами сыплет и строчки подсвечивает, а кто на это обращает внимание? Вот если бы он разряд тока давал программисту, или просто закрывался без сохранения, тогда да, было бы эффективно.
      Ответить
      
      KoWe4Ka_l7porpaMMep 20.03.2021 17:27 # 0
      
      Он может уйти в астрал и перестать подсвечивать питушню и дополнять длинную пиииитуууушню, что принесет программисту большие страдания. В code::blocks, если открыть большую либу, то статический анализатор начнет готовить автоподстановки и сломается, а ещё будет фризить экран.
      Ответить
      
      bormand 20.03.2021 17:52 # +2
      
      Ага, я точку боялся нажимать после пары дней работы с code blocks.
      Ответить
      
      KoWe4Ka_l7porpaMMep 20.03.2021 17:55 # 0
      
      А ты же в QTCreator пишешь крестокод, да?
      
      Почему такая вселенская несправедливость имеет место быть: код блокс написан на wxWidgets и он страшный, а QTCreator написан на QT и он прикольный (((
      Ответить
      
      bormand 20.03.2021 17:56 # 0
      
      Х.з., в свежих code blocks поди тоже шланговский парсер привернули вместо цтагс? Или до сих пор лагает?
      Ответить
      
      KoWe4Ka_l7porpaMMep 20.03.2021 18:20 # 0
      
      Там какой-то кошмарный hand-made плагин:
      
      http://wiki.codeblocks.org/index.php/Code_Completion_Design
      
      И его хотят переписать:
      
      http://wiki.codeblocks.org/index.php/Code::Completion_Rewrite
      
      Ахуеть, но я нагуглил шланговский плагин:
      
      http://forums.codeblocks.org/index.php?topic=15521.0
      
      Note, this plugin will probably fail on advanced projects(at least it fails on this project), just use it on simple stuff for now
      
      Ну бля (((
      Ответить
      
      booratihno 20.03.2021 18:37 # 0
      
      зачем это говно нужно?
      
      с тагсами я и в виме писать могу
      Ответить
      
      KoWe4Ka_l7porpaMMep 20.03.2021 18:51 # 0
      
      Я не знаю, если честно. Оказывается, код::блокс хуевый (((
      
      Пора перейти на CodeLite. Там даже новейшая питушня для автодополнения кода есть. Как вам? «Language Server» называется...
      
      https://wiki.codelite.org/pmwiki.php/Main/LanguageServer
      Ответить
      
      guest6 20.03.2021 20:34 # +1
      
      с ленгвидж сервером можно и на VSCode сидеть
      Ответить
      
      guest6 20.03.2021 20:34 # 0
      
      можно гонять его на CI и запрещать коммитить с ошибкой прекомитхуком
      Ответить
- booratihno 20.03.2021 15:27 # 0
  
  видал у нас подобное говно на жабе. Там люди тоже делали API для измерения чего, и передавали туда сплошные Object.
  
  Имхо, такой код нужно генерить, а не писать вручную
  Ответить
- guest6 20.03.2021 17:16 # +1
  
  тем временем, человек изобрел компьютер из крабов!
  https://twitter.com/emollick/status/1371932428982493185
  Ответить
  - KoWe4Ka_l7porpaMMep 20.03.2021 17:27 # 0
    
    А можно туда пхпшников вместо крабов? И у тех, и у других клешни)
    Ответить
    - guest6 20.03.2021 18:38 # +3
      
      пхпшников я бы не допускал до компутера (в отличие от крабов)
      Ответить
Добавить комментарий
Ошибка компиляции комментария:

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

А не использовать ли нам bbcode?

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

Говнокод: по колено в коде.

C++ / Говнокод #27303

Комментарии (36) RSS

Добавить комментарий