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

    +11

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    typedef ::boost::array<T, item_amount+1> not_raw_array;
union data
{
	 not_raw_array m_raw_array;//Most members of boost::array not suitable for call with raw array. Be careful!
};

    Запостил: LispGovno, 27 Марта 2013

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

    • ava defecate-plusplus 27.03.2013 11:26 # +5

      и да, осторожней, детка, в лесу могут быть волки
      Ответить

    • ava TarasB 27.03.2013 11:34 # +2

      чёзанах
      Ответить

    • ava LispGovno 27.03.2013 11:35 # −2

      Кстати, чуть ниже строчка:
      data m_array;
      Ответить

    • ava bormand 27.03.2013 18:41 # +2

      It's dangerous to go alone! Take this.
      Ответить

    • ava LispGovno 27.03.2013 19:09 # +3

      Никто не догадался, поэтому поясню ход мысли автора, так как я посмотрел комментарии ревизии в GIT:
      Это часть контейнера. Автор не использовал not_raw_array напрямую, а завернул его в union, чтобы иметь возможность произвольно создавать и убивать объекты в m_raw_array массиве. Иначе без union в not_raw_array объекты создадутся сами, если имеют нетривиальный конструктор. Может в С++11 оно у автора и работало, но у меня вчера бомбануло, когда я дописал конструктор в структуру (шаблонное T), что хранил в этом контейнере.
      Ответить

      • ava defecate-plusplus 27.03.2013 20:12 # +3

        леденящая душу логика
        ставлю плюс за задумку, тру какир
        Ответить

      • ava roman-kashitsyn 27.03.2013 20:27 # +2

        почему бы просто не использовать char[ sizeof(T) * ( item_amount + 1) ] и placement new ?
        Ответить

        • ava defecate-plusplus 27.03.2013 20:48 # 0

          ты что, это же reinterpret_cast делать на operator []!
          Ответить

          • ava roman-kashitsyn 27.03.2013 21:14 # 0

            Я слишком устал и зелен, чтобы понять, сарказм это или нет.
            Ответить

            • ava defecate-plusplus 27.03.2013 22:37 # 0

              ну я такого пиздеца еще не видел никогда
              хак окей
              char[...] это, как ни крути, массив чаров, и чтобы получить i-тый T & из него, нужно мало того, что развлекаться с sizeof(T), но и постоянно приводить (char *) к (T *), а (char const *) к (T const *)

              сделал бы я так же, если бы приспичило (тьфу тьфу)? нет, слишком неочевидно - не проссышь без объяснений, что это вообще, в отличие от классического char[...]
              Ответить

              • ava bormand 27.03.2013 22:57 # +1

                > развлекаться с sizeof(T)
                > постоянно приводить
                Сколько сарказма в этом предложении... Осторожнее надо быть, читатели ведь и правда могут поверить, что там что-то сложнее пары приватных методов с одним реинтерпрет кастом внутри, да сайзофа при объявлении массива...
                Ответить

                • ava roman-kashitsyn 27.03.2013 23:57 # 0

                  имхо, геморройней отслеживать, какой объект отконструировался, а какой ещё нет, с bitset<item_amount + 1> возиться на доступе и в деструкторе
                  Ответить

                  • ava bormand 28.03.2013 00:07 # +1

                    Ну это если контейнер с рандомным созданием-удалением собираем. А если, к примеру, какую-нибудь bounded queue, то там ведь и пары чисел хватит. Если аналог вектора - там вообще одно число - текущий size.
                    Ответить

        • ava TarasB 28.03.2013 09:26 # +4

          Ты чо, это же скомпилируется в С++03 и не даст почувствовать, какой крутой новый стандарт!
          Ответить

    • ava LispGovno 28.03.2013 16:22 # 0

      Ну приехали... В том контейнере используется std::uninitialized_copy в assign, а он по стандарту не гарантирует даже weak exception safety.
      То есть недоконструированные объекты в случае исключения он не откатит.

      В случае исключения я даже узнать не смогу что успело создаться, а что нет
      Ответить

      • ava LispGovno 28.03.2013 16:50 # 0

        >std::uninitialized_copy в assign
        А ещё в конструкторе и assignment operator

        У меня конечно есть исходники и сейчас поправлю, но противно.
        Что посоветуете взамен std::uninitialized_copy, окромя самописного велосипеда? Может что из буста?
        Ответить

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