Принцип «Говори, не спрашивай» и срок действия пароля

login
   model.validate();
   return model.show(self);

passwordExpired()
  return View("ChangePassword")

loginSuccess()
  return View("default")

class User
  show(aController)
      if passwordExpired
          return aContoller.passwordExpired()
     else return aContoller.loginSuccess()

Скажи, не спрашивай, без исключений и подчиняется закону Деметры

Вы можете сгенерировать исключение PasswordExpired из объекта пользователя, когда пароль аутентифицирован, или любую другую функцию, которую вы сначала вызываете для пользователя.

Вы должны рассмотреть возможность того, чтобы пользовательский объект имел метод Validate(), который предоставляет логическое значение (как это делает контракт поставщика членства), или рассмотреть возможность того, чтобы метод Validate() возвращал какое-то перечисление, указывающее результат проверки (ОК, INVALID_PASSWORD , EXPIRED_PASSWORD и т. д.).

Есть много вариантов - создание исключения не должно быть одним из них, если срок действия пароля истек. Это плохой тон, а также удар по производительности, поскольку среда выполнения должна раскручивать стек.

Мне лично не нравится программировать возвращаемые значения / Enum типы. Чем больше у вас типов возврата, тем больше путей вам нужно протестировать/поработать. Кроме того, использование исключений для управления потоком является плохой практикой (если вы действительно не можете найти другой вариант, но обычно есть лучший).

Просроченный пароль на самом деле не является исключительным для меня. В конце концов, это действительное состояние (иначе вы бы сделали что-то против истечения срока действия паролей вообще)

Я стараюсь не усложнять и либо возвращаю bool, либо что-то вроде Func<T>, которое может быть вызвано вызывающим кодом напрямую.

Вероятно, что-то вроде этого:

public class User
    {
        private DateTime _lastChangeDate;
        public Action Validate()
        {
            if (_lastChangeDate >= DateTime.Now.AddDays(-30))
            {
                return new Action(() => this.Login());
            }
            else
            {
                return new Action(() => this.ChangePassword());
            }
        }
        private void Login()
        {
            Console.WriteLine("Login");
        }
        private void ChangePassword()
        {
            Console.WriteLine("Change Password");
        }
    }

На стороне вызывающего абонента:

user.Validate().Invoke();

Одним из способов решения этой проблемы является объектно-ориентированное моделирование, подобное этому:

public class Login {

private String userName;
private String password;
private Date expirationDate;    

public void authenticate(String password) {
    if (this.password.equals(password) {
        redirectoToMainOrEditPage();
    } else {
        redirectToFailPage();
    }
}

private void redirectToMainOrEditPage() {
    Date today = new Date();

    if (today.before(expirationDate)) {
        redirectToMainPage();
    } else {
        redirectToEditPage();
    }
}

private void redirectToMainPage() {
    ...
}

private void redirectToEditPage() {
    ...
}

private void redirectToFailPage() {
    ...
}

public void changePassword(String newPassword) {
    ...
}

public void changeExpirationDate(Date newDate) {
    ...
}
}

Таким образом, вы ничего не спрашиваете у других объектов домена, а указываете Login для аутентификации, так как у него есть все необходимое для этого.