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

    +175

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    template<class Exception>
bool GenerateException(const Exception* const e)
{
	throw e;
	delete e;
	return true;
}

    Продолжение цикла говнокодов про исключения из того же самого проекта.
    Проблем от неё почти нет, тк в основном исключения генерятся так:

    ExceptionТакойто e;
    
GenerateException(&e);

    Запостил: Говногость, 10 Февраля 2011

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

    • ava guest 10.02.2011 17:16 # +1

      Исключительная каша в мозгах.
      Ответить

    • ava Lure Of Chaos 10.02.2011 20:30 # 0

      достаточно оригинальный подход. хотя и совершенно ненужный
      Ответить

    • ava guest 10.02.2011 22:52 # 0

      Здесь лишнее:
      delete e;
return true;
      Ответить

      • ava koodeer 10.02.2011 22:54 # +3

        дополнительный пакетик лапши никогда не бывает лишним!
        Ответить

    • ava TheCalligrapher 10.02.2011 23:04 # +3

      Простите, что значит "проблем от неё почти нет"? Исключение, как понял, объявляется как локальный объект, а потом бросается "по указателю". Если ловец исключения сидит снаружи, то поймает он указатель на уже уничтоженный объект (ибо стек уже развернут). Т.е. работать это в принципе не может вообще и проблем от этого будет много и сразу.

      А если это как-то работает, то только засчет того, что ловец исключения не доступается к "нутру" пойманного "по указателю" объекта. Т.е. причина "работоспособности" тут не в том, как исключение бросается, а в том, как оно ловится.
      Ответить

      • ava guest 10.02.2011 23:08 # −1

        Плюсую. Видимо не используют внутренности исключения. Просто ловят факт появления исключения.
        Или деструктор вызывается, но критического ничего не удаляет, что дальше позволяет использовать память удаленного объекта, как ещё целый.
        Ответить

        • ava gegMOPO4 11.02.2011 22:29 # 0

          По крайней мере стандартные исключения содержат в себе строку-описание и имеют нетривиальный деструктор. Так что "работает" оно только по недосмотру и капризу компилятора.
          Ответить

          • ava guest 12.02.2011 01:37 # −1

            Возможно здесь стандартные исключения и не используются, тк template<class Exception> - это шаблон, а значит, возможно, генерируемые исключения не наследуются от стандартных std::exception.
            Ответить

            • ava gegMOPO4 12.02.2011 13:42 # 0

              Предполагать-то всё можно, даже то, что этот код нигде не используется (и только это его и оправдывает). Какой смысл бросать указатель в качестве исключения, а не нормальный наследник std::exception? Какой смысл в этой обёртке вместо простого throw? Никакого.
              Ответить

              • ava guest 12.02.2011 19:35 # 0

                Вы меня спрашиваете? Это вопросы к автору кода. Я с Вами полностью согласен.
                Ответить

          • ava guest 12.02.2011 01:39 # −1

            Тут вполне могут использовать даже инстанц шаблона:
            bool GenerateException(const int* const e)
            Мы по коду узнать это здесь не можем.
            Ответить

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