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

    +163

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    CTvoid cSG5Exception::Get (CTstatus& Status,
                           CTsymbol& Error,
                           CTsymbol& SubError,
                           string& Description)
{
   if (Description.c_str() != NULL) {
       Description = m_description;
   }
   Status = m_status;
   Error = m_error;
   SubError = m_subError;
}

    итолианский с++ панимаш, рагацци.
    что они хотели в строках 6-8 сказать кануло в лету.
    и вместо 4 человеческих геттеров один бесполезный заху*чили. (бесполезный тем что четыре переменные всегда надо под рукой иметь.)

    Запостил: Dummy00001, 24 Января 2011

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

    • ava guest 24.01.2011 17:45 # 0 

      CTstatus& Status,
CTsymbol& Error,
CTsymbol& SubError,
      Уберэксепшон, надэксепшон, эксепшон, субэксепшон, андерэксепшон.
      Ответить

      • ava guest 24.01.2011 17:58 # 0

        Вообще, хорошая традиция заводить один класс эксепшон на все случаи жизни... :D
        Ответить

        • ava Lure Of Chaos 24.01.2011 18:03 # −1

          или даже не класс. Насколько я помню, плюсы позволяют в качестве эксепшона кидать что угодно
          Ответить

        • ava Dummy00001 24.01.2011 18:16 # +3

          к слову там этот класс именно так и используется.

          там в двух разных краях проекта две несовместимые крайности: (выше) один эксепшн на все случаи жизни и по эксепшену на каждый уникальный случай. в добавок, в паре мест, в новом коде на это дело народ забил и пользуется третьей более осмысленой иерархией эксепшенов наследованой от std::exception. вообщем, разброд и шатание традиционные коммерческому софту который уже раз пять руки менял.
          Ответить

    • ava gegMOPO4 24.01.2011 21:00 # +2

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

    • ava guest 25.01.2011 12:02 # 0

      GПокемоновая обработка ошибок.
      Ответить

    • ava TheCalligrapher 26.01.2011 21:34 # +2

      Метод `c_str()` класса `std::string` никогда не возвращает null pointer. Для пустой строки, например, возвращается указатель на строку "", но не null pointer. Что имел в виду аффтар - не ясно.
      Ответить

    • ava increazon 28.01.2011 23:01 # −5

      показать все, что скрытоНе уверен что исключения необходимы. Удалить полностью функцию, и пусть рантайм сам разбирается.
      Ответить

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