Discussion :

[math567]

Bonjour,
J'ai une fonction qui vérifie si le code postal rentré dans un champ est valide ou non. Le problème c'est que lorsque je rentre u code postal de plus de 5 chiffres, la fonction me considère que c'est valide...
Où est mon erreur ??? Merci d'avance.

Ma fonction :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
function validCodePostal($cp) {
	$ereg  = '([0-9]{4,5})';   // format d'un code postal belge et français
	// test du code postal
	if (ereg($ereg, $cp)) {
		return true;
	}
	else {
		return false;
	}
}

[Halex78]

Hello, ereg est dépréciée, utilise plutôt les PCRE.
La syntaxe que tu utilises match les mots faisant de 4 à 5 lettres ne contenant que des chiffres dans la chaine que tu lui passes en paramètre.
Donc si ta chaine est une suite de 6 chiffres, il te retournera un résultat positif puisqu'effectivement de telles suites de chiffres sont trouvables dans une suite à 6 chiffres.
En supposant que la chaine n'est que le code postal, ceci devrait fonctionner :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
function validCodePostal($cp){
if(preg_match('#^[0-9]{4,5}$#',$cp){return true;}else{return false;}
}

[ascito]

tu peu faire comme ca aussi ( avais pas vu que tu avais complété ta réponse )

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
if( !is_numeric($string) || ( strlen($string)!=4  && strlen($string)!=5) )
{
	return false;
}

[math567]

Merci...

[Halex78]

Oui ascito, ta méthode évite l'utilisation des regex, mais ton code renvoie "rien" ou "false",et donc jamais "true", autant s'épargner l'utilisation du if :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

is_numeric($string) && ( strlen($string)==4  || strlen($string)==5) //Cette expression renvoie déjà le booléen qui nous intéresse...