cette validation par regexp n'a aucun sens

Bonjour,

j'ai codé cette méthode il y a un certain temps, mais je m'interroge sur son utilité :

Code:

---

1 2 3 4 5 6 7 8

public function validString($string):bool { $pattern = '~[^[:alnum:]]~';   $valid = preg_replace($pattern, '', $string);//on retire ts les car. non alphanumériques   return ( ctype_alnum($valid) ); }

---

Etes-vous bien d'accord que cette condition ctype_alnum($valid) sera toujours vraie ? Et que donc la méthode retournera toujours true (donc ne sert à rien) ?

La première chose qui ne va pas c'est que tu mélanges validation et sanitization. Une méthode dont l'objet est la validation ne doit s'en tenir qu'à ça à mon avis. (donc exit le preg_replace).
Ensuite considère ce qui se passe lorsque la chaîne est vide (ou qu'elle ne contient aucun caractère alphanumérique).

Même si ce code, c'est n'importe quoi, je crois que mon idée était de retirer par preg_replace les caractères autorisés puis de tester si le résultat est vide ou pas. Ca tient pas la route cette approche ?

Ton idée de départ était de retirer les caractères non-autorisés puis de renvoyer le résultat de ctype_alnum() (qui renverra toujours true sauf si la chaîne modifiée est vide).
Je ne vois pas trop quel est l'intérêt par rapport à une simple test avec ctype_alnum().

Tout à fait d'accord avec toi que le code du post #1 ne présente que l'intérêt (discutable) de se compliquer la vie.

Je cherche une solution pour qu'on accepte que les caractères alphanumériques et -,_,. (par exemple) :

Code:

---

1 2 3 4 5 6

function validString($string):bool { $pattern = '~[:alnum:,\_,\-,\.]~';   return ( preg_match($pattern, '', $string) ) ; }

---

(idée générale mais pas au point car marche pas).
Pourrais-tu me corriger ?

Je parie qu'il y aura encore des gens qui vont dire "ouais, les regex, c'est compliqué, c'est pour le geek, ... etc" (je blague).

Une regex, c'est pas si difficile et surtout si tu utilises ton meilleur ami, j'ai cité regex101.com; va voir https://regex101.com/r/bPIfjr/1/

Pour un exemple : http://sandbox.onlinephpfunctions.co...4c15481b2881f5

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

<?php   echo "Il n'y a pas d'accent, devait être correct \n";   $pattern = '/^[A-Za-z0-9-_\.]*$/'; $string = 'Il-etait-une-fois-9-canards';   printf(" Résultat: %s\n", preg_match($pattern, $string) ? 'Correct!' : 'Oups; incorrect') ;   echo "Cette fois il y a un accent, incorrect \n";   $pattern = '/^[A-Za-z0-9-_\.]*$/'; $string = 'Il-était-une-fois-9-canards';   printf(" Résultat: %s\n", preg_match($pattern, $string) ? 'Correct!' : 'Oups; incorrect') ;

---

Le résultat sera correct pour le premier cas, incorrect pour l'autre.

Merci pour ta réponse.

En plus, l'ami en question (https://regex101.com/) contient une quick reference qui répond à pas mal de mes questions.

Y en a quand même pour laquelle j'ai pas trouvé de réponse : pourquoi ce pattern ne marche pas : $pattern = '/^[:alnum:-_\.]*$/'; ?

Attention avec cette traîtresse d'ancre de fin de chaîne $ qui laisse passer une nouvelle ligne: exemple.

Pour y parer, on utilise le modificateur D (Dollar End Only):

Code:

---

$pattern = '/^[A-Za-z0-9_.-]*$/D';

---

mais le mieux est à mon avis d'utiliser les ancres \A et \z qui sont plus explicites (et sans surprise):

Code:

---

$pattern = '/\A[A-Za-z0-9_.-]*\z/';

---

Citation:

---

Envoyé par laurentSc

pourquoi ce pattern ne marche pas : $pattern = '/^[:alnum:-_\.]*$/'; ?

---

Parce que c'est $pattern = '/\A[[:alnum:]_.-]*\z/';, les classes de caractères POSIX comme [:alnum:] sont notées entre crochets (en plus de ceux qui délimitent la classe de caractères dans laquelle elles figurent).

NB: le point n'est pas un caractère spécial au sein d'une classe de caractères, inutile de l'échapper. Mieux vaut placer le tiret au début ou en fin de classe si tu veux éviter de l'échapper (il n'est pas improbable qu'un jour il ne soit plus possible de le placer après un rang comme A-Z sans l'échapper).

Autre chose [:alnum:] avec le souligné _ correspond à la classe \w, donc on peut écrire la pattern de cette manière:

Code:

---

$pattern = '/\A[\w.-]*\z/';

---

J'ai remis les ancres \A et \z (que tu m'avais déjà montrées) mais il reste une erreur :

Code:

---

1 2 3 4 5 6 7 8 9

function validString($string):bool { $pattern = '~\A[A-Za-z0-9_-()\,\. \']*\z~';   return ( preg_match($pattern, $string) ) ; }   $str="jj_èkl"; echo $str."/// validString(".$str.")=".validString($str)."<br/>";

---

Citation:

---

( ! ) Warning: preg_match(): Compilation failed: range out of order in character class at offset 14 in C:\projets\dvp\divers\test.php on line 6

---

Pourquoi ?

En fouinant un peu, j'ai trouvé que l'erreur venait du non-échappement du tiret.

Par ailleurs, j'ai simplifié la regex avec le raccourci cité par Cosmo et en enlevant 2 échappements inutiles : $pattern = '/\A[\w_(),. \']*\z/';.