Discussion :

[bouuuh]

Bonjour,

avant de me lancer j'aimerai avoir votre avis sur la meilleure façon de fonctionner pour tester la présence ou non de séparateur dans ma chaine de caractère.

Je m'explique : pour des autorisations j'ai un input qui doit recevoir un/des numéros de département(s).

Syntaxe correct : 01,02,03... ou 01

Comment s'assurer que cette chaine de caractère a bien comme séparateurs une virgule si il y a 2 départements indiqués ?


Merci à tous

[Bovino]

strpos

[bouuuh]

Merci je regarde et je déposerai ma fonction si sa interesse

[bouuuh]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
 
<?php
function controle_seprateur($DepartementAutorise){
 
	//Compter le nombre de caractères dans ma chaine
	$NbrCaractere = strlen($DepartementAutorise);
	$Erreur = true;
 
	if($NbrCaractere < 2){
		//Indique une erreur car le département doit être au minimum de 2 caractères
		$Erreur = true;
	}
 
	if($NbrCaractere == 2){
		//Indique que nous avons uniquement un département d'indiquer
		//tester si c'est bien un numéros
		if(is_numeric($DepartementAutorise) == TRUE){ $Erreur = false; 	}
		else{ $Erreur = true; }
	}
	else{
		if((($NbrCaractere % 3) == 0) || (($NbrCaractere % 2) == 0)){
		//Indique que la chaine de caractere est un multiple de 3 ce qui est impossible
		//ex : 01, --> reste 0 erreur ; 01,02, --> reste 0 erreur
		$Erreur = true;		
		}else{
		//Indique que nous avons bien le bon nombre de caractères
		//tester la présence de la   ,   
			$requis   = ',';
			$presence = strpos($DepartementAutorise, $requis);
 
			if(($presence === false)){
				//Virgule non trouvé donc erreur
				$Erreur = true;
			}else{
				$Erreur = false;
			}
				//On continu uniquement si le nbr de caractere est supérieur a 5 car besoin de bouclé au minimum 2 fois !
				if($NbrCaractere > 5){
					//Boucler en fonction du nombre de caractere - 2->0 ; 5->1 ; 8->2 ... soit +3 a chaque fois
					//Déterminer le nombre de boucle
					$ValeurParDefaut = $NbrCaractere;
					$i = 0;
					while(($ValeurParDefaut - 3) == 2)
					{
					  $i++; // incrémentation d'une boucle
					}
 
					$j = 0;
					//Nous avons déterminer notre notre de boucle ($i)
					while (($j == $i) && ($Erreur == false))
					{
						if(($presence % 3) != 0){
						//Virgule n'étant pas à la bonne position - ex : 0,1,21 ...
							$Erreur = true;
						}else{
							$Erreur = false;
						}
						$j++; // incrémentation d'une boucle
					}
 
				}
		}
	}
	return $Erreur;
}
?>

[Bovino]

A priori, ta fonction est correcte, mais attention

Code :

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

if(($presence == false))

ne convient pas, si la première occurrence trouvée est le premier caractère, strpos renvoie 0, or

Code :

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

0 == false

c'est pourquoi il est fortement conseillé (comme précisé dans la doc) d'utiliser un comparateur strict

Code :

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

if(($presence === false))

[bouuuh]

Tu veux dire si c'est ce cas la :

,01,02 ?

[Bovino]

Oui, c'est ça.
Dans cet exemple, strpos renverra 0 et ta comparaison

Code :

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

if(($presence == false))

sera évaluée à vrai car 0 est ce que l'on appelle une falsy value, c'est-à-dire une valeur considérée comme false.

[bouuuh]

Je vais corrigé.
J'ai testé je n'ai pas cette erreur car avec le modulo % 2 j'ai indiqué que si le nombre est multiple de 3 ou paire il y a forcement une erreur :

Bonne syntaxe : 01,02 ---> 5 caractères .. 8,11 (impaire)


,01,02 ---> 6 caractères donc erreur


Script modifié avec ta remarque