Си диез / Говнокод #19337 Ссылка на оригинал

0

  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 kerman, (Updated )

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

  • ava Her # Ссылка на оригинал 0

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

    • ava kerman # Ссылка на оригинал 0 ⇈

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

      • ava bormand # Ссылка на оригинал 0 ⇈

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

        • ava kerman # Ссылка на оригинал 0 ⇈

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

      • ava Her # Ссылка на оригинал 0 ⇈

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

        • ava kerman # Ссылка на оригинал 0 ⇈

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

    • ava kerman # Ссылка на оригинал 0 ⇈

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

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

Помни, guest, за тобой могут следить!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓