原則とパスワードの有効期限を尋ねることを教えてください

Question

実用的なプログラミングの原則を維持しようとしている私は、「尋ねる、尋ねないでください」という原則に基づいてユーザーパスワードの変更を扱う方法を決定しようとしています。

パスワードが30日ごとに期限切れになるユーザーオブジェクトがあります。私は、パスワードが期限切れの場合、パスワードの期限切れ/パスワードの変更表示を表示する必要があります。パスワードの期限が切れている（状態だと）オブジェクトを要求し、表示するビューを選択すると原則に違反しているようです。

この状況を処理する最善の方法は何ですか？

Answer 1

パスワードが認証されたとき、またはユーザーが最初に呼び出す関数があれば、ユーザーオブジェクトからPasswordExpired例外をスローすることができます。

Answer 2

ユーザーオブジェクトに（メンバーシッププロバイダー契約のように）ブール値を提供するValidate（）メソッドを持たせるか、Validate（）メソッドで検証の結果を示す何らかの並べ替えを返すことを検討する必要があります（OK、INVALID_PASSWORD、EXPIRED_PASSWORDなど）。

パスワードの有効期限が切れても、例外をスローすることはできません。ランタイムがスタックを巻き戻さなければならないので、それは悪いフォームであり、パフォーマンスヒットです。

Answer 3

私は個人的にarround戻り値/ 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(); 

Answer 4

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() 

、一切の例外を掲載していない、そしてそれはデメテル

の法則に従うしないでください、知らせます

Answer 5

これを解決する1つの方法は、このようなオブジェクト指向モデリングです：

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) { 
    ... 
} 
} 

このようにして、他のドメインオブジェクトに何かを尋ねるのではなく、ログインに認証するように指示します。