1. C# / Говнокод #6258

    +109

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    try
 {
//тут другой гавнокод связаный с регистрацией
}
catch(ApplicationException)
{
    this.ModelState.AddModelError("DuplicateUser", Strings.DuplicateUser);
    return View(model);
}

    Вот так, если приложение падает значит юзверь уже существует....

    Запостил: biff, 07 Апреля 2011

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

    • ava Govnocoder#0xFF 07.04.2011 10:16 # +3

      Гавнокодер детектед!
      Ответить

    • ava guest 07.04.2011 10:47 # 0

      Эксепшен плохой, а так-то в чем проблема?
      Ответить

    • ava AxisPod 07.04.2011 10:58 # +1

      Хм, я так понимаю, что автор данной заметки считает, что try..catch используется только для падения приложений? Разве что класс исключения неудачно выбран.

      Вообще я бы посоветовал про исключения почитать, а в данном случае я так понимаю был вызов метода регистрации пользователя, который вернул исключение Т.К. пользователь уже существует.
      Ответить

      • ava Lure Of Chaos 07.04.2011 12:44 # +3

        видимо, где комментарий, там стопицот строчек, и где-то там очень внутри кидается одно из "стандартных" исключений, типа ошибки скула или деления на нуль, и вся форма падала. Г_а_внокодер, вместо обработки исключения в месте его генерирования, ловит общий класс исключений, безосновательно делает вывод, что причина ошибки одна, и рекордит сообщение в новом красивеньком вьювере.
        Предчувствую, что само приложение теперь работает очаровательно - например, при отсутствии связи с бд выдается ошибка "юзверь уже существует" вместо "не могу подключиться к базе"
        Ответить

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