Retour des valeurs vraies pour les espaces dans les variables

Question

J'ai une fonction "isValid" qui est responsable de la recherche d'espaces dans toutes les variables fournies. Je veux que la fonction retourne vrai si aucune des variables ne contient d'espaces. Serait-ce la bonne approche?

return !(strpos($this->email, " ") || strpos($this->confirm_password1, " ") || strpos($this->handle, " ") || strpos($this->phone, " ")); 

Answer 1

Afin qu'il soit facilement maintenable je ferais quelque chose comme ce qui suit:

public function isValid() 
{ 
    $properties = array(
     'email', 
     'confirm_password1', 
     'handle', 
     'phone', 
    ); 

    foreach ($properties as $property) { 
     if ($this->containsSpace($property)) { 
      return false; 
     } 
    } 

    return true; 
} 

private function containsSpace($property) 
{ 
    return !(strpos($this->$property, '') === false); 
} 

Notez le === false as check for strpos(). Démonstration: http://codepad.viper-7.com/0ZgNZq.

Notez que cela ne vérifie que les espaces et pas les autres espaces possibles. Si vous voulez vérifier les autres caractères blancs Je suggère d'utiliser un simple regex:

private function containsSpace($property) 
{ 
    return (bool)preg_match('/\s/', $this->$property); 
} 

Et enfin, si vous voulez vraiment le faire à votre façon originale que je ferais quelque chose comme:

public function isValid() 
{ 
    return strpos($this->email, " ") === false && strpos($this->confirm_password1, " ") === false && strpos($this->handle, " ") === false && strpos($this->phone, " ") === false; 
} 

Answer 2

function isValid($var) { 
    if (strpos($var, ' ') === false) { 
    // Does not contain a space character. 
    return true; 
    } 
    return false; 
} 

Answer 3

return (strpos($this->email, " ") === false && strpos($this->confirm_password1, " ") === false && strpos($this->handle, " ") === false && strpos($this->phone, " ") === false) 

mais cela rend le code assez illisible ...