1. C++ / Говнокод #20076

    +6

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    for(int i = 0; i < codes.size(); ++i) {
    switch(i) {
        case 0: ret.code0 = codes[i]; break;
        case 1: ret.code1 = codes[i]; break;
        case 2: ret.code2 = codes[i]; break;
        case 3: ret.code3 = codes[i]; break;
        case 4: ret.code4 = codes[i]; break;
        case 5: ret.code5 = codes[i]; break;
    }
}

    А всё потому, что ret.code[0-5] - битовые поля. Эх.

    Запостил: Xom94ok, 25 Мая 2016

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

    • ava Bobik 25.05.2016 21:58 # +5 

      #define LINE(x) ret.code # x = codes[x]; 
LINE(0); LINE(1); LINE(2); LINE(3); LINE(4); LINE(5);
#undef LINE
      Ответить

      • ava bormand 25.05.2016 21:59 # +4

        Въебал плюс за отборное говнецо.

        З.Ы. Я бы не пропустил на ревью ;)
        Ответить

        • ava Bobik 25.05.2016 22:00 # +1

          Потому что точки с запятой дублируются? :-(
          Ответить

          • ava bormand 25.05.2016 22:02 # +1

            Потому что break проебал, как минимум.

            Про # вместо ## промолчу, сам часто путаю, т.к. очень редко пользуюсь.
            Ответить

            • ava inkanus-gray 25.05.2016 22:04 # 0

              Стоп! Тут же говнопаттерн: свитч в цикле.
              Ответить

              • ava bormand 25.05.2016 22:04 # 0

                Ой, да. Цикл то я и не заметил.
                Ответить

                • ava inkanus-gray 25.05.2016 22:06 # +2

                  У меня всё-таки остаётся надежда, что codes.size() не всегда равно шести и в некоторых случаях нужно выполнять меньше строк. Иначе это полное говно.
                  Ответить

                  • ava Xom94ok 25.05.2016 22:32 # +3

                    > не всегда равно шести
                    Афёрматив. Иначе цикл-свитч не нужны.
                    Ну или много писанины.
                    Ответить

                    • ava bormand 25.05.2016 22:41 # +6

                      > Афёрматив
                      > Негатив
                      Зэ бомб хэз бин плантэд.
                      Ответить

                      • ava gost 25.05.2016 23:15 # +4

                        Террористс вин.
                        Ответить

                        • ava bormand 25.05.2016 23:16 # +4

                          Фаер ин зе хол!
                          Ответить

                          • ava Koncord 26.05.2016 15:57 # 0

                            Ну что вы творите! Из-за вас опять подсяду на CS:GO!
                            Ответить

                            • ava guesto 26.05.2016 16:04 # +3

                              смещение GO по адресу в регистре CS?
                              Ответить

                              • ava Koncord 26.05.2016 17:01 # 0

                                ога, по этому адресу обитает путь: /home/koncord/.steam/steam/steamapps/common/Counter-Strike Global Offensive/csgo.sh
                                Ответить

                                • ava gost 26.05.2016 20:25 # 0

                                  Ого, в Линь таки завезли игорей?
                                  Ответить

                                  • ava guesto 26.05.2016 21:14 # +1

                                    каких игорей? Он конкорд

                                    а если ты про игры, то майнсвиппер еще в aptitude был
                                    https://www.debian.org/doc/manuals/aptitude/ch02s06.en.html
                                    Ответить

                                  • ava 3_14dar 26.05.2016 21:57 # 0

                                    Завезли, они тут же утонули, остался pitux races и 500 крестиков-ноликов.
                                    Ответить

                    • ava myaut 26.05.2016 12:07 # +1

                      Цикл в любом случае ну нужен:

                      switch(std::min(codes.size(), 6)) {
    case 6: ret.code5 = codes[5];  /*FALLTHROUGH*/
    case 5: ret.code4 = codes[4];  /*FALLTHROUGH*/
    case 4: ret.code3 = codes[3];  /*FALLTHROUGH*/
    case 3: ret.code2 = codes[2];  /*FALLTHROUGH*/
    case 2: ret.code1 = codes[1];  /*FALLTHROUGH*/
    case 1: ret.code0 = codes[0]; 
}
                      Ответить

              • ava guestinho 25.05.2016 22:15 # +9

                паттерн "сосни префетчер"
                Ответить

              • ava kegdan 29.05.2016 15:10 # 0

                Я думал это называется "парсер"
                Ответить

            • ava Bobik 25.05.2016 22:07 # 0

              Блин, с ## налажал :(
              Ответить

    • ava inkanus-gray 25.05.2016 22:00 # 0

      А царский сдвиг не подходит, потому что непортабельно: вдруг порядок битовых полей изменится?
      Ответить

    • ava bormand 25.05.2016 22:00 # 0

      Лень смотреть в стандарт и/или проверять. А можно описать массив битовых полей (никогда не задумывался об этом)?
      Ответить

      • ava 3_dar 25.05.2016 22:29 # +1

        работает, значит можно
        Ответить

      • ava Xom94ok 25.05.2016 22:34 # +5

        Негатив. От них и адрес-то не взять, только, разве что, дробный.
        Ответить

        • ava bormand 25.05.2016 22:39 # +3

          > дробный
          А я смогу поменять часть бита, пользуясь дробным адресом?
          Ответить

        • ava guestinho 25.05.2016 22:55 # +5

          и получилося в ответе
          полтора байта и две трети
          Ответить

          • ava gost 25.05.2016 23:17 # +1

            > получилося
            Кто тебя так научил говорить - "полчилося"?! "Получилось", ёпта, ты же на говнокодике сидишь, кресты не позорь свои!

            PS, капча - вин: http://i.prntscr.com/2fd09417b8f942c28946f72709ee5ffa.png
            Ответить

      • ava inkanus-gray 26.05.2016 00:48 # +2

        На SO советуют std::bitset, std::vector<bool> и boost::dynamic_bitset.
        Ответить

        • ava bormand 26.05.2016 07:27 # +1

          > std::vector<bool>
          Я вот до сих пор не понимаю, зачем эти байтоёбы вектор из булов испортили... Есть же отдельный битсет...
          Ответить

          • ava roman-kashitsyn 26.05.2016 09:06 # +1

            > Есть же отдельный битсет...

            Ну у битсета в compile-time размер указывается, не совсем подходит. По уму надо было отдельный dynamic_bitset запилить, но ведь так велик соблазн вкрутить куда-нибудь специализацию шаблонов, чтобы было потом, что в книжках писать...
            Ответить

    • ava codemonkey 25.05.2016 22:21 # +1

      >Almost everything about fields is implementation-specific.
      K&R

      Так что въеби Х-макро и не парься.
      Ответить

    • ava kurwa 25.05.2016 22:26 # +1

      std::bitset - не попацански?
      Ответить

      • ava codemonkey 25.05.2016 22:29 # +1

        >sexually-transmitted disease
        ненене
        Ответить

      • ava meinf 26.05.2016 00:35 # 0

        https://ideone.com/XoTbY3
        то бишь немного теряется смысл, хотя может быть этот "смысл" там нахрен и не нужен, но это уже другая история
        Ответить

        • ava kurwa 26.05.2016 00:58 # 0

          Может и не теряется. Зависит от структуры же.
          Ответить

          • ava meinf 26.05.2016 01:16 # 0

            ну, я про это и говорю, но нужны ли тогда вообще битовые поля? имхо нет
            Ответить

    • ava govnokod3r 25.05.2016 23:23 # +5

      Цикл не нужен

      int i = codes.size()

    switch(i) {
        case 0: break;
        case 6: ret.code5 = codes[--i];
        case 5: ret.code4 = codes[--i];
        case 4: ret.code3 = codes[--i];
        case 3: ret.code2 = codes[--i];
        case 2: ret.code1 = codes[--i];
        case 1: ret.code0 = codes[--i];
    }
      Ответить

      • ava bormand 25.05.2016 23:34 # +2

        case 0 лучше в конец подвинуть, имхо.
        Ответить

      • ava kurwa 25.05.2016 23:51 # +2 

        - int i = codes.size()
+ size_t i = std::min(6, codes.size());
        Ответить

        • ava Xom94ok 29.05.2016 14:23 # +1

          1. Fall-through ворнинг как отключать будете на трёх компиляторах?
          2. Qt контейнеры возвращают int для размера.
          3. Проверка размера? Ты меня сейчас три дня назад таким неявным образом тупым обозвал? Я её, как очевидную, выкинул для ясности кода. И там не просто молчаливый срез куска контейнера, а ассерт + лог, чтобы сообщить об ошибке.
          Ответить

          • ava govnokod3r 29.05.2016 15:34 # +1

            >> Fall-through ворнинг
            Не нужен

            >> как отключать будете на трёх компиляторах?
            -w
            Ответить

          • ava gost 11.10.2020 19:40 # 0

            > 1. Fall-through ворнинг как отключать будете на трёх компиляторах?
            Через аннотацию «[[fallthrough]];». Чай, не двадцатый век, пора уже «C++17» использовать.

            > 2. Qt контейнеры возвращают int для размера.
            Зачем они нужны?
            Ответить

            • ava rotoeb 11.10.2020 19:55 # 0

              Зачем вы ебёте себе мозги этой хуетой? В "PHP" нет ни компиляторов, ни контейнеров, ни аннотаций, и при этом весь веб течёт от наслаждения, как последние сучки, воздерживавшиеся год.
              Ответить

      • ava gost 26.05.2016 01:08 # +2

        Анроллнул аки царь!
        Ответить

    • ava exzerodivide 26.05.2016 10:55 # +1

      Автор почти до Duff's device дотянул :-)

      register n = (count + 7) / 8; /* count > 0 assumed */

      switch (count % 8)
      {
      case 0: do { *to = *from++;
      case 7: *to = *from++;
      case 6: *to = *from++;
      case 5: *to = *from++;
      case 4: *to = *from++;
      case 3: *to = *from++;
      case 2: *to = *from++;
      case 1: *to = *from++;
      } while (--n > 0);
      }
      Ответить

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