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

    +7

    1. 1
    value = *((const char*)(*it).second.value);

    void* -> const char* -> char

    Запостил: ptr2ptr, 04 Августа 2016

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

    • ava j123123 04.08.2016 07:28 # +16

      Где здесь C++, ptr2ptr?
      Ответить

      • ava Antervis 04.08.2016 07:35 # +17

        Скорее всего it это итератор на что-то вроде std::map<whatever_t, void*>

        Возможно можно было вместо (*it). написать it-> Иначе хз в чем именно гк
        Ответить

        • ava bormand 04.08.2016 21:03 # +16

          > в чем именно гк
          В потенциальном UB'е, например. Вдруг в it->second.value был нулл или слишком короткий буфер. Я бы въебал как минимум ассёрт.
          Ответить

          • ava Soul_re@ver 04.08.2016 21:10 # +17

            > Я бы въебал как минимум ассёрт

            Я бы просто въебал. Нехуй void* использовать. По крайней мере можно было выделить эту хуйню в функцию.
            Ответить

            • ava bormand 04.08.2016 21:12 # +15

              > выделить эту хуйню в функцию
              Ну, по-хорошему, это должно быть в акцессоре у того, у чего брали value (а само value - в привате)...
              Ответить

              • ava Soul_re@ver 04.08.2016 21:28 # +16

                Я надеюсь что это дроч приватных полей внутри члена класса. Иначе вопрос — где здесь С++?
                Ответить

            • ava bormand 04.08.2016 21:19 # +19

              З.Ы. Кресты - говно. В джаве/шарпе нельзя выябываться и оптимизировать - код получается унылым, медленным, жрущим память, но пишется быстро и с первого раза...

              А на крестах я сегодня ведь день проебал чтобы скрестить ужа с ежом - чтобы и память лишний раз не аллоцировало, и исключения не кидало (но ошибки не теряло), и кишки сишной реализации наружу не торчали, и типобезопасно было, и все типичные юзкейсы няшно и кратко смотрелись...

              В итоге закончилось всё void* в единственном приватном поле у всех публичных классов этого модуля ;(
              Ответить

              • ava Soul_re@ver 04.08.2016 21:27 # +16

                А чем any не подошёл? Типобезопасности побольше, исключения опциональны.

                > Кресты - говно
                Удивил. Всё говно, кроме говна — оно ценное удобрение. Но не всё. Некоторое говно — говно.
                Ответить

                • ava bormand 04.08.2016 21:39 # +16

                  > чем any не подошёл
                  чтобы и память лишний раз не аллоцировало

                  Там void* не потому что много разных вариантов, а потому что структуру в хедере палить нельзя (она сишная, т.е. попадёт в корневой неймспейс). А саму структуру в any сунуть тоже нельзя, т.к. класс не должен контролировать её время жизни (чё-то типа итератора)...
                  Ответить

                  • ava Soul_re@ver 04.08.2016 21:48 # +16

                    Этакий доморощённый type-erasure? У меня где-то валялся any_pointer. type_info + void* + касты с проверкой типа положенного значения. Хотя если void* только для скрытия типа, то и это не нужно, да.

                    А не хочется палить даже имя структуры? Потому что для объявления указателя достаточно объявления struct kokoko;
                    Ответить

                    • ava bormand 04.08.2016 21:58 # +16

                      Да просто имя сишное, без неймспейсов. Хуй знает до какого угла проекта оно доползёт из этого хедера...
                      Ответить

    • ava guest 04.08.2016 23:24 # +16

      -> для слабаков
      Ответить

      • ava inkanus-gray 04.08.2016 23:48 # +18

        А вдруг -> перегружен? Ну а вдруг?
        Ответить

        • ava bormand 05.08.2016 18:28 # +16

          > А вдруг -> перегружен? Ну а вдруг?
          А вдруг * перегружена? Ну а вдруг?

          З.Ы. Их как бы парочкой обычно и перегружают.
          Ответить

          • ava kurwa 05.08.2016 21:22 # +16

            да и название it намекает, что не вдруг, а перегружены
            Ответить

            • ava bormand 05.08.2016 21:23 # +16

              А вдруг там тупо указатель в качестве итератора?
              Ответить

              • ava kurwa 05.08.2016 21:24 # +16

                Возможно в этом и состоит говнокод?
                Ответить

          • ava kegdan 06.08.2016 04:12 # +16

            А в ppCьке нет принудительной парной перегрузки?
            Ответить

            • ava bormand 06.08.2016 08:06 # +17

              Нет конечно, зачем отбирать возможность разложить грабли?
              Ответить

              • ava kegdan 06.08.2016 08:28 # +16

                Иногда мне кажется, что ppCьку специально делали максимальным говном
                Ответить

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