Discussion :

[moimp]

Bonjour,

J'ai une fonction de contrôle des codes postaux en fonction du pays de destination.
Lorsque le nombre de caractères est fixe, ma fonction accepte les codes postaux trop longs. Pourquoi?

Code php :

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
67
68
69
70
71
72
// Contrôle du code postal
function ctl_postcode($postcode, $country='') {
	// voir https://fr.wikipedia.org/wiki/Code_postal
	$country = strtolower($country);
	// Codes à 3 chiffres
	if (in_array($country, ['mg'])) $pattern = "#^\d{3}$#";
	// Codes à 4 chiffres
	if (in_array($country, ['at', 'au', 'az', 'be', 'ch', 'dk', 'fl', 'hu', 'lu', 'md', 'mk', 'ph', 'tn', 'za'])) $pattern = "#^\d{4}$#";
	// Codes à 5 chiffres
	elseif (in_array($country, ['ba', 'de', 'dz', 'es', 'fi', 'fr', 'gt', 'hr', 'it', 'lk', 'lt', 'ma', 'me', 'my', 'mx', 'rs', 'sa', 'ua'])) $pattern = "#^\d{5}$#";
	// Codes à 6 chiffres
	elseif (in_array($country, ['by', 'cn', 'co', 'ec', 'in', 'kz', 'kg', 'ro', 'sg', 'uz'])) $pattern = "#^\d{6}$#";
	elseif (in_array($country, ['il'])) $pattern = "#^\d{7}$#";
	else {
		switch ($country) {
			case 'ar': // Argentine
				$pattern = "#^[A-Z][0-9]{4}[A-Z]{3}$|^[0-9]{4}$#";
				break;
			case 'bj': // Bénin
				$pattern = "#^\d{2}[A-Z]{2}\d{4}$#";
				break;
			 // Brésil + Cambodge CCCCC-CCC
			case 'br':
			case 'kh':
				$pattern = "#^\d{5}\-\d{3}$#";
				break;
			case 'ca':
				$pattern = "#^[a-zA-Z]{1}[0-9]{1}[a-zA-Z]{1}[ -]{1}[0-9]{1}[a-zA-Z]{1}[0-9]{1}$#";
				break;
			case 'gb':
				$pattern = "#^[A-Z]{1,2}[A-Z0-9]?[0-9] [0-9][A-Z]{2}$#";
				break;
			// CCC CC
			case 'gr':
			case 'se':
			case 'sk':
			case 'cz':
				$pattern = "#^\d{3} \d{2}#";
				break;
			case 'ht': // Haïti
				$pattern = "#^HT-\d{4}#";
				break;
			case 'ie':
				$pattern = "#^[A-Z][0-9][A-Z0-9] [A-Z][0-9]{3}$#";
				break;
			case 'jp':
				$pattern = "#^[0-9]{3}-[0-9]{4}$#";
				break;
			case 'nl':
				$pattern = "#^[1-9][0-9]{3} [A-B]{2}$#";
				break;
			case 'pl': // Pologne CC-CCC
				$pattern = "#^[0-9]{2}-[0-9]{3}$#";
				break;
			case 'pt': // Portugal CCCC-CCC
				$pattern = "#^[0-9]{4}-[0-9]{3}$#";
				break;
			case 'us':
				$pattern = "#^\d{5}([ -][0-9]{4})?$#";
				// (^\d{5}(?:[\s]?[-\s][\s]?\d{4})?$)
				break;
			case 'ta':
				$pattern = "#^[1-9]{1}[0-9]{2}$#";
				break;
			default:
				return true;
				break;
		}
	}
	if (preg_match($pattern, $postcode)) return true;
	else return false;
}

[badaze]

Des exemples ?

Un conseil. php a la bonne idée de gérer des tableaux avec des clés alphanumériques. Pourquoi ne pas en profiter ?

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
function ctl_postcode($postcode, $country='') {

$toto = array();

$toto["at"] = "la regexp";
$toto["au"] = "la regexp";
...
$toto["za"] = "la regexp";
if (isset($toto[$country])) {
  if(preg_match($toto[$country], $postcode)) return true; else return false;
} else {
  return true;
}
}

[badaze]

Et puis comme on est bien fainéants...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<?php
$toto = array();
 
//---- les groupes
$toto = array_merge($toto,array_fill_keys(['at', 'au', 'az', 'be', 'ch', 'dk', 'fl', 'hu', 'lu', 'md', 'mk', 'ph', 'tn', 'za'], "#^\d{4}$#"));
$toto = array_merge($toto,array_fill_keys(['ba', 'de', 'dz', 'es', 'fi', 'fr', 'gt', 'hr', 'it', 'lk', 'lt', 'ma', 'me', 'my', 'mx', 'rs', 'sa', 'ua'], "#^\d{5}$#"));
$toto = array_merge($toto,array_fill_keys(['by', 'cn', 'co', 'ec', 'in', 'kz', 'kg', 'ro', 'sg', 'uz'], "#^\d{6}$#"));
 
//---- Les solitaires
$toto['ar'] = "#^[A-Z][0-9]{4}[A-Z]{3}$|^[0-9]{4}$#";
$toto['bj'] = "#^\d{2}[A-Z]{2}\d{4}$#";
 
var_dump($toto);

[Invité]

Bonjour,

@badaze : bien vu !

...Lorsque le nombre de caractères est fixe, ma fonction accepte les codes postaux trop longs...

@moimp : je ne comprends pas cette phrase.
Explique mieux (exemple ?)

[badaze]

D’autant plus qu’avec un code postal français ça fonctionne bien.

[moimp]

Merci badaze pour les suggestions d'amélioration. L'idée du tableau associatif est excellente.
Pour en revenir à la question initiale, j'ai posté ma demande parce que mon code ne marche pas à tous les coups:
Par exemple, si le pays est madagascar (code = 'mg'), l'expression accepte les codes de longueur >=3 et non pas uniquement ceux de longueur strictement égale à 3.
Un code français à 6 chiffres a également été accepté mais je n'arrive pas à reproduire le cas.

[moimp]

Je ne comprends pas très bien pourquoi les contrôles qui ne marchaient pas avec ma version fonctionnent avec la tienne alors que les regex sont les mêmes. Quoi qu'il en soit le code terminé selon la ta version est exactement deux fois moins long que le mien. Bravo et Merci.

[CosmoKnacki]

Voilà pourquoi ça ne marche pas avec le code de Madagascar:

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
 ### cette condition réussie et la pattern correcte est affectée ###
if (in_array($country, ['mg'])) $pattern = "#^\d{3}$#";
 
###  mais comme ici tu as un if au lieu d'un elseif, toutes les conditions vont être testée  ###
// Codes à 4 chiffres
if (...) ...;
// Codes à 5 chiffres
elseif (...) ...;
// Codes à 6 chiffres
elseif (...) ...;
elseif (in_array($country, ['il'])) $pattern = "#^\d{7}$#";
### comme toutes les conditions précédentes on échouées tu passes dans le else puis dans le switch ###
else {
    switch ($country) {
        case 'ar': // Argentine
	// ...
        ### pour finalement atterrir ici ou la clause "default:" te fais quitter la fonction en renvoyant "true". La pattern n'a jamais été testée.
        default:
            return true;
            break;
    }
}
 
if (preg_match($pattern, $postcode)) return true;
	else return false;