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

    +161

    1. 1
    2. 2
    3. 3
    std::auto_ptr<ItemDesc> desc(new ItemDesc());
...
m_items.insert(desc->m_item->m_name, desc.release());

    Да, я тоже говнокодер. А ведь предупреждали...

    Запостил: Kirinyale, 16 Февраля 2011

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

    • ava gegMOPO4 17.02.2011 12:02 # −1

      И что? В чём говнокод?
      Ответить

      • ava absolut 17.02.2011 12:17 # +1

        Может быть в desc.release() ?
        Правда хотелось бы узнать, что за тип у m_items.
        Ответить

        • ava gegMOPO4 17.02.2011 12:30 # +1

          Вот без release() был бы говнокод.
          Ответить

          • ava absolut 17.02.2011 12:32 # 0

            Аргументируйте
            Ответить

            • ava gegMOPO4 17.02.2011 12:44 # 0

              Отсутствие release() почти наверняка означало бы, что в контейнере сидят auto_ptr со всем истекающим.

              Kirinyale ниже объяснил, в чём на самом деле говнокод.
              Ответить

        • ava Kirinyale 17.02.2011 12:34 # +1

          тип m_items - boost::ptr_map
          Ответить

      • ava Kirinyale 17.02.2011 12:33 # +3

        Говнокод в том, что порядок вычисления аргументов никем не зафиксирован.

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

        • ava gegMOPO4 17.02.2011 12:38 # 0

          Да, не заметил, что в том же выражении desc используется дважды. Тогда говнокод, согласен.

          desc->m_item->m_name следовало бы выделить в отдельную переменную.
          Ответить

          • ava Kirinyale 17.02.2011 12:41 # +1

            Так и сделал, когда заметил. Точнее, выделил в переменную релизнутый указатель, чтоб не копировать строку.

            Мейерс о такой хрени писал. А я читал. Но забыл.
            Ответить

            • ava gegMOPO4 17.02.2011 12:56 # 0

              Если m_item->m_name гарантированно не вызывает исключения, то можно (и лучше) и указатель.
              Ответить

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