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

    +160

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    for( int b = 0; b < cntblocket; ++b )
{
    delete msrLst[b]->record;
    delete msrLst[b]->record;
}

    Сразу вспоминается: "- А зачем у вас тут два одинаковых jump'а стоять? - А вдруг один не сработает?"

    Видимо, совсем не выспался программист ;)

    Запостил: ftptrash, 13 Ноября 2010

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

    • ava Мистер Хэнки 13.11.2010 12:39 # +2

      копипастнул и забыл исправить
      Ответить

    • ava absolut 13.11.2010 13:10 # +8

      два джампа ничего плохого не сделают, а вот два delete - вполне могут.
      Ответить

    • ava Govnoeb 13.11.2010 13:54 # +1

      явные delete? я надеюсь, что приведенный код - часть десктруктора
      Ответить

      • ava ctm 13.11.2010 14:36 # 0

        важнее, что их 2 подряд, одинаковых. Хорошо хоть такие косяки отлавливаются довольно легко.
        Ответить

        • ava absolut 13.11.2010 14:41 # 0

          > отлавливаются довольно легко.
          с этого места поподробнее, плиз.
          Ответить

          • ava ctm 13.11.2010 15:08 # 0

            в классе msrLst[b]->record (назовем его test)
            способ 1:
            добавим static int ObjCount;
            int test::ObjCount = 0;
            в конструкторе ObjCount ++;
            в деструкторе if ((-- ObjCount) < 0) throw ....;
            при выполнении получим что такое вообще имеет место быть - теперь знаем что искать. искать придется вручную.
            способ 2:
            Делаем сингл-тон хеш-таблицу.
            в конструкторе test:
            hash[int(this)] = this;
            в деструкторе test:
            if (hash[int(this)] == NULL) throw ...; // тут получим конкретный вызов
            hash[int(this)] = NULL;
            Ответить

            • ava absolut 14.11.2010 00:04 # 0

              Хорошо, но для этого надо сначала класс локализовать.
              Ответить

            • ava Govnoeb 14.11.2010 12:35 # +1

              кидать в деструкторах - плохо.
              надо бы хотя бы добавить
              if(--ObjCount < 0 && !uncaught_exception()) throw 666;

              потом оборачивать придется все классы и структуры в countable_. т.е.
              class ... : public countable_ { ... };
              Ответить

              • ava ctm 14.11.2010 14:22 # +1

                я имел в виду идею, понятно, что ее надо довести до ума.
                Ответить

    • ava Xom94ok 13.11.2010 22:36 # +1

      Тест на трезвость :-) "Сколько операторов delete тут написано?"
      Тест на трезвость :-) "Сколько операторов delete тут написано?"
      Ответить

    • ava guest 16.12.2010 23:06 # 0

      Да тут наверняка копипаста и забыл исправить в
      delete msrLst[b]->record;
      слово record на что-то ещё.
      Ответить

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