1. 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

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

    • ava 6apry3uHckuu_nemyx 19.03.2021 07:38 # 0

      Говно.
      Зачем аргументы передавать списком?
      Ответить

    • ava 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); // На случай если порядок объявления запутанный
}
      Ответить

      • ava bormand 20.03.2021 15:24 # +1

        Код не эквивалентен, в оригинале можно было больше аргументов передать.
        Ответить

        • ava guest6 20.03.2021 15:28 # 0

          но не больше двух?
          Ответить

          • ava bormand 20.03.2021 15:29 # 0

            Да можно и больше 2, оно просто ассёртнется в рантайме. А теперь оно не конпелируется!
            Ответить

            • ava booratihno 20.03.2021 15:30 # 0

              а что такое TRI_ASSERT, кстати?
              всмысле TRI это 3?
              Ответить

              • ava bormand 20.03.2021 15:31 # +1

                Какое-то кастомное говно. Но точно не статик, т.к. size() в комплайтайме неизвестен.

                > всмысле TRI это 3

                Я думаю это "неймспейс" их либы.
                Ответить

                • ava booratihno 20.03.2021 15:33 # 0

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

                  я просто подумал, что initializer_list ты можешь проверитьв компайл тайм, но видимо не всегда
                  Ответить

                  • ava bormand 20.03.2021 15:34 # 0

                    > initializer_list ты можешь проверитьв компайл тайм

                    Не, для сложных проверок в комплайтайме придётся вариадик темплейт городить... Лист здесь бессилен. А простые можно оверлоадами или дефолтными аргументами разрулить.
                    Ответить

                  • ava bormand 20.03.2021 15:40 # 0

                    З.Ы. Я когда-то ваял функцию, которая в компайлтайме проверяла, что ей передали чётное количество аргументов правильного типа, а потом отдавала их в сишный вариадик.
                    Ответить

                    • ava guest6 20.03.2021 15:43 # 0

                      какой кошмар)

                      но тут еще смешнее: аргументов может быть от 1 до 2 включительно. Нахуя вообще все это городить?
                      Ответить

                      • ava bormand 20.03.2021 15:45 # 0

                        > какой кошмар

                        Просто часто промахивался при вызове сишной версии. Там были key/value пары и нулл в конце. А в крестах это таки получилось выразить красиво и типобезопасно.
                        Ответить

                        • ava guest6 20.03.2021 15:50 # 0

                          почему было не передать (указатель на) массив структур с key/value или как-то хуйнуть их вараргом?
                          Ответить

                          • ava bormand 20.03.2021 15:51 # 0

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

                            • ava guest6 20.03.2021 15:54 # 0

                              ох сишка, моя сишка
                              эти вечные
                              ioctl(int fd, unsigned long request, ...);
                              Ответить

            • ava Soul_re@ver 20.03.2021 16:02 # +2

              Fail Fast это тоже один из принципов современного программирования. Раньше компайлтайма сломаться сложно.
              Ответить

              • ava bormand 20.03.2021 16:09 # +1

                Ну вот взял и сломал программу! Всё же работало нормально в релизной сборке, а теперь она даже не собирается.
                Ответить

                • ava 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);}

                  Починил, компилируется.
                  Ответить

                  • ava bormand 20.03.2021 16:29 # +1

                    Сообщения об ассёртах не эквивалентны.
                    std::array<std::string, 0> il;
TRI_ASSERT(il.size() > 0);
                    Ответить

              • ava bormand 20.03.2021 16:45 # +1

                > Раньше компайлтайма сломаться сложно.

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

              • ava booratihno 20.03.2021 16:45 # +1

                >. Раньше компайлтайма сломаться сложно
                статический анализ в IDE?
                Ответить

                • ava Soul_re@ver 20.03.2021 17:02 # +2

                  Но он же просто варнингами сыплет и строчки подсвечивает, а кто на это обращает внимание? Вот если бы он разряд тока давал программисту, или просто закрывался без сохранения, тогда да, было бы эффективно.
                  Ответить

                  • ava KoWe4Ka_l7porpaMMep 20.03.2021 17:27 # 0

                    Он может уйти в астрал и перестать подсвечивать питушню и дополнять длинную пиииитуууушню, что принесет программисту большие страдания. В code::blocks, если открыть большую либу, то статический анализатор начнет готовить автоподстановки и сломается, а ещё будет фризить экран.
                    Ответить

                    • ava bormand 20.03.2021 17:52 # +2

                      Ага, я точку боялся нажимать после пары дней работы с code blocks.
                      Ответить

                      • ava KoWe4Ka_l7porpaMMep 20.03.2021 17:55 # 0

                        А ты же в QTCreator пишешь крестокод, да?

                        Почему такая вселенская несправедливость имеет место быть: код блокс написан на wxWidgets и он страшный, а QTCreator написан на QT и он прикольный (((
                        Ответить

                        • ava bormand 20.03.2021 17:56 # 0

                          Х.з., в свежих code blocks поди тоже шланговский парсер привернули вместо цтагс? Или до сих пор лагает?
                          Ответить

                          • ava 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

                            Ну бля (((
                            Ответить

                            • ava booratihno 20.03.2021 18:37 # 0

                              зачем это говно нужно?

                              с тагсами я и в виме писать могу
                              Ответить

                              • ava KoWe4Ka_l7porpaMMep 20.03.2021 18:51 # 0

                                Я не знаю, если честно. Оказывается, код::блокс хуевый (((

                                Пора перейти на CodeLite. Там даже новейшая питушня для автодополнения кода есть. Как вам? «Language Server» называется...

                                https://wiki.codelite.org/pmwiki.php/Main/LanguageServer
                                Ответить

                                • ava guest6 20.03.2021 20:34 # +1

                                  с ленгвидж сервером можно и на VSCode сидеть
                                  Ответить

                  • ava guest6 20.03.2021 20:34 # 0

                    можно гонять его на CI и запрещать коммитить с ошибкой прекомитхуком
                    Ответить

    • ava booratihno 20.03.2021 15:27 # 0

      видал у нас подобное говно на жабе. Там люди тоже делали API для измерения чего, и передавали туда сплошные Object.

      Имхо, такой код нужно генерить, а не писать вручную
      Ответить

    • ava guest6 20.03.2021 17:16 # +1

      тем временем, человек изобрел компьютер из крабов!
      https://twitter.com/emollick/status/1371932428982493185
      Ответить

      • ava KoWe4Ka_l7porpaMMep 20.03.2021 17:27 # 0

        А можно туда пхпшников вместо крабов? И у тех, и у других клешни)
        Ответить

        • ava guest6 20.03.2021 18:38 # +3

          пхпшников я бы не допускал до компутера (в отличие от крабов)
          Ответить

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