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

    −1

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    if (string.IsNullOrWhiteSpace(frmDelegatorLogin.ViewModel.Username))
                throw new Exception("Не введен логин");
            if (dbEmployee == null)
                throw new Exception(string.Format("Не найден пользователь с логином \"{0}\"", frmDelegatorLogin.ViewModel.Username));
            if (dbEmployee.DelegatePassword != frmDelegatorLogin.ViewModel.Password)
                throw new Exception("Указан неверный пароль");

    Вот такой вот новый метод валидации полей. Да, логин и пароль вводятся текстом, да, Exception потом ничем не ловится и пользователь получает unhandeled exception...

    Запостил: kerman, 20 Января 2016

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

    • ava Her 20.01.2016 17:12 # −1

      Нормальный код вроде.
      Раве что, пароль хранится в открытом виде
      Ответить

      • ava kerman 20.01.2016 17:31 # 0

        За исключением того, что метод bool (возвращает, залогинились или нет)
        Ответить

        • ava bormand 20.01.2016 17:39 # 0

          > возвращает, залогинились или нет
          Возвращает true, если залогинились, а если нет - ничего не возвращает (т.к. исключение)?
          Ответить

          • ava kerman 20.01.2016 18:42 # +2

            Точно. И, как следствие, ПО падает с ошибкой вместо простого мессаджбокса "неправильный пароль или логин".
            Ответить

        • ava Her 20.01.2016 18:10 # 0

          код портировался откудато? и бул оставили как дань предкам?
          Ответить

          • ava kerman 20.01.2016 18:40 # +2

            Не портировался. Это наш сумрачный гений написал с нуля. Заодно изобрёл новый подход - validation thru exception. Я ему предложу сразу диск ц форматировать, чо уж там...
            Ответить

    • ava roman-kashitsyn 20.01.2016 18:43 # +4

      Держите таблэтку
      http://martinfowler.com/articles/replaceThrowWithNotification.html
      Ответить

      • ava kerman 20.01.2016 19:44 # 0

        Спасибо за ссыль, таблэтка отправлена поциенту.
        Ответить

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