Нормально ли иметь много методов в классе 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 ... До тех пор, как это имеет смысл, и не нарушает принципы, ты в порядке. Просто следуйте правилам программирования и ООП.

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

Answer 2

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

Но, как правило, у вас может быть столько функций в классе, сколько вы хотите, но в конце концов вы доберетесь до точки, где у вас так много функций, что вместо простого упрощения кода вы просто делаете большой беспорядок.

Я бы поместил только несвязанный код в отдельные функции. Потому что, когда у вас есть аналогичные части кода, часто проще закодировать его в одной функции.

Answer 3

По моему опыту, вы начинаете писать свои классы, затем возвращаетесь, чтобы добавить что-то к методу, тогда вы можете расширить свой класс с помощью подобной функциональности, и, наконец, вы в конечном итоге осознаете, что он стал грязным и есть много вещей, вы можете объединиться в одну (частную) функцию, чтобы не только уменьшить длину кода, но и упростить его поддержку.

Итак, мой совет: всегда занимайте второе место, думайте о том, что вы собираетесь делать, и будет ли метод, который вы собираетесь реализовать, - это цепочка подфункций, которые сами по себе будут полезны где-то в другом месте , Постарайтесь сделать ваши функции максимально возможными, но только объявите их необходимыми как общедоступные.

Answer 4

Да. До тех пор, пока методы в классе связаны, хорошо идти. И ваш очень короткий список. Вы можете использовать GUMP Validator, если хотите проверить. Это очень просто в использовании.