在php類中有很多方法是否正常？

Question

我正在研究驗證不同的東西，如空，長度等的表單驗證的PHP類。

這裏有幾個我在那裏的方法。正如標題所說，在課堂上有一堆方法是正常的嗎？ 只是回答是或否就足夠了。

//Checks if is username empty 
     function validate_empty_username(){ 
      if($this->empty_username){ 
       echo "<li>Please fill username field</li>"; 
       return false; 
      } 
     } 

     //Checks if is password empty 
     function validate_empty_password(){ 
      if($this->empty_password){ 
       echo "<li>Please fill password field</li>"; 
       return false; 
      } 
     } 

     //Checks if is email empty 
     function validate_empty_email(){ 
      if($this->empty_email){ 
       echo "<li>Please fill email field</li>"; 
       return false; 
      } 
     } 

     //Check username length is short 
     function validate_username_length_min(){ 
      if($this->get_username_length < 3){ 
       echo "<li>Username provided's too short!</li>"; 
       return false; 
      } 
     } 

     //Check password length if short 
     function validate_password_length_min(){ 
      if($this->get_password_length < 5){ 
       echo "<li>Password provided's too short!</li>"; 
       return false; 
      } 
     } 

     //Check email length if short 
     function validate_email_length_min(){ 
      if($this->get_email_length < 5){ 
       echo "<li>Email provided's too short!</li>"; 
       return false; 
      } 
     } 

Answer 1

這不是lots ......只要是有意義的，並且不違反原則的，你沒事。只需遵循編程規則和OOP。

你已經證明這些方法顯然是有點相關，如果你把它們放到一類是沒有問題的。如果這個邏輯遍佈在你的源代碼中，我認爲它會更糟糕。

Answer 2

我假設你將使用所有這些功能的驗證一次，所以我會建議把它們放在一起。

但通常只要你想，你可以有多種功能於一類，但最終你會得到一個點，你有，而不是使代碼更簡單，你只是做一個很大的混亂如此多的功能。

我只把不相關的代碼放在單獨的函數中。因爲當你有類似的代碼部分時，通常在單一函數中編寫代碼更容易。

Answer 3

在我的經驗是這樣的：你開始寫你的類，然後你回來的東西添加到一個方法，那麼你可以通過一個類似的功能擴展您的類，最後你就會意識到，它變髒並且有很多東西，你可以合併成一個（私有）函數，不僅可以減少代碼的長度，而且可以更容易維護。

所以我的建議是：總是花一點時間，想想你要做什麼，以及你要實現的方法是否是一個子函數鏈，這些子函數本身在別的地方會很有用。儘量讓自己的功能儘可能原子化，但只是將需要的功能宣佈爲公開。

Answer 4

是的。只要班級中的方法相關，就很好去了。你的是一個很短的列表。如果你想驗證，你可以使用GUMP Validator。它也很簡單。