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

    +165

    1. 1
    throw new std::bad_alloc();

    У нас в кодеконвеншене принято передавать исключения по указателю. Вчера обнаружил вот это во многих перегрузках оператора new, да и в прочих местах по проекту.

    Запостил: Говногость, 01 Июля 2010

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

    • ava itman 01.07.2010 01:44 # 0

      Мило, а пацаны-то не знают и ловят-поди по ссылке :-(((
      Ответить

    • ava cfdev 01.07.2010 01:46 # 0

      Что-то такое уже было.
      Ответить

    • ava absolut 01.07.2010 07:40 # 0

      А зачем по указателю ? Это ж надо не забыть потом ещё удалить.
      Ответить

      • ava telnet 01.07.2010 07:45 # −3

        (не в тот ГК коммен запостил, сорри =3)
        Ответить

    • ava Говногость 01.07.2010 09:11 # 0

      Ещё весело то, что bad_alloc - исключение ошибки выделения памяти по new.
      Ответить

      • ava absolut 01.07.2010 21:45 # −1

        В данном случае bad_alloc может значить всё что угодно.
        Ответить

        • ava Говногость 01.07.2010 21:49 # 0

          Не просто 
          bad_alloc
          , а 
          std::bad_alloc
          .
          Стандартную библиотеку С++ вам нужно повторить чуток...
          Ответить

          • ava absolut 01.07.2010 22:02 # −2 

            try
{
   throw std::bad_alloc();
}
catch( std::bad_alloc& err )
{
   std::cout << "Нехватка памяти ?\n";
}


            И где тут ошибка выделения памяти ?
            Ответить

            • ava Говногость 01.07.2010 22:13 # 0

              Основной комментарий к коду прочтите:
              >Вчера обнаружил вот это во многих перегрузках оператора new, да и в прочих местах по проекту.

              Знаете, я тоже так могу сказать, что генерю исключение ArrayOutOfRange просто от нефиг делать везде по коду, что-бы его потом поймать try catch просто так. Это уже больше похоже на индуский стиль, где платят за строчки кода. В двойне говнокод.
              Ответить

              • ava absolut 01.07.2010 22:23 # −2

                Так а я о чем говорю. Следуя тому же комментарию к коду : "да и в прочих местах по проекту."
                Рукописная генерация исключения ещё ни о чем не говорит. Вот если бы родной не перегруженный new это генерил, тогда да - ошибка выделения памяти. А если уж кодер сам throw вызвал, то может быть всё что угодно, независимо от класса исключения. В гк именно этот самописный вызов.
                Ответить

    • ava cfdev 01.07.2010 11:10 # +3

      > У нас в кодеконвеншене принято передавать исключения по указателю

      Ужас.
      Ответить

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