Discussion :

[floanne]

Bonjour,

j'ai un comportement très étrange de la fonction preg_match(_all).

Dans mon scripte j'ai la fonction suivante :

Code :

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

1
2
3
4
5
6
7
function extraireCity ($City_ZIP)
{
    print "\$City_ZIP : $City_ZIP\n" ;
    preg_match ("#\d*(.*)#", $City_ZIP, $tab_tmp) ;
    print_r ($tab_tmp) ;
    return @$tab_tmp[1] ;
}

Elle est censée extraire la ville en ignorant le code postale...
Or elle affiche ceci :

Code :

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

1
2
3
4
5
6
$City_ZIP :  95124 Ermont 
Array
(
    [0] =>  95124 Ermont 
    [1] =>  95124 Ermont 
)

Par contre si j'exécute le code de test suivant :

Code :

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

1
2
3
4
5
6
7
8
 
<?php
    $City_ZIP = "76200 Dieppe" ;
    print "\$City_ZIP : $City_ZIP\n" ;
    preg_match ("#\d*(.*)#", $City_ZIP, $tab_tmp) ;
    print_r ($tab_tmp) ;
    return @$tab_tmp[1] ;
?>

j'obtiens :

Code :

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

1
2
3
4
5
6
$City_ZIP : 76200 Dieppe
Array
(
    [0] => 76200 Dieppe
    [1] =>  Dieppe
)

J'ai trituré le code dans tous les sens et j'obtiens toujours la même chose : un scripte de test qui marche et mon scripte qui ne marche pas...


Je ne comprends pas bien, si quelqu'un avait une idée...

++Florent.

[Eric2a]

Il faut peut-être remplacer les guillements par des apostrophes.

Code :

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

preg_match ('#\d*(.*)#', $City_ZIP, $tab_tmp);

ou bien, essaie ceci

Code :

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

preg_match('/[0-9]+ (.+)$/', $City_ZIP, $tab_tmp);

[s.n.a.f.u]

C'est plus simple et pourtant plus sournois.

Essaies ce code qui devrait éclairer ta route :

Code :

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

1
2
3
4
5
6
7
<?php
 
    $City_ZIP = " 76200 Dieppe" ;
    print "\$City_ZIP : $City_ZIP\n" ;
    preg_match ("#\d*(.*)#", $City_ZIP, $tab_tmp) ;
    print_r ($tab_tmp) ;
    return @$tab_tmp[1] ;

[Jihnn]

D'où l'intérêt d'éviter le plus souvent possible d'utiliser le point, qui est parfois trop gourmand. En fait, c'est surtout dû au fait que tu le couples avec le quantificateur *.

s.n.a.f.u a très bien résumé le problème : la partie \d* étant facultatif et ton pattern n'étant pas particulièrement clair, lorsque d'autres caractères se trouvent devant ta chaîne, ça ne retourne pas le résultat escompté...

À priori, une ville ne pouvant pas contenir de chiffres, tu devrais plutôt utiliser une classe un peu moins gourmande ([a-z-] par exemple). Et puisqu'il semble nécessaire que tu aies le code postal ET la ville dans ta chaîne, évite * et utilise plutôt +.

[floanne]

Bonjour,

merci à tous pour vos réponses.
Je voudrais précisé que j'ai beaucoup de chaines à traiter, certaines avec le code postale d'autres sans. D'où l'étoile.
Sinon je ne comprends pas ta réponse s.n.a.f.u : tu me propose de mettre un espace devant ma chaine à matcher ?

Moi ce qui me perturbe c'est que mon code marche très bien dans un petit script de test et dès que je l'intègre à mon script principale le résultat diffère... C'est embêtant tout de même : ça veut dire que la fonction preg_match est sujette à un effet de bord.

++Florent.

[floanne]

Bon, j'ai compris où il y avait une différence.

Si dans mon script principal j'écrase ma chaine par "76200 Dieppe" ça marche.

Code :

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

1
2
3
4
5
6
7
8
function extraireCity ($City_ZIP)
{
    print "\$City_ZIP : $City_ZIP\n" ;
    $City_ZIP = "76200 Dieppe" ;
    preg_match ("#\d*(.*)#", $City_ZIP, $tab_tmp) ;
    print_r ($tab_tmp) ;
    return @$tab_tmp[1] ;
}

Bon le problème c'est que j'écrase ma chaine à matcher

Ça doit être un problème d'encoding ou un truc dans se genre car mes chaine viennent d'un fichier excel. Ce qui est curieux c'est que toutes mes autre regex marchent très bien...

Si vous avez une idée...

++Florent.

[s.n.a.f.u]

Sinon je ne comprends pas ta réponse s.n.a.f.u : tu me propose de mettre un espace devant ma chaine à matcher ?

Non, c'est le contraire : je voulais mettre en avant le fait que c'était l'espace devant qui mettait la grouille.
Je te conseillerai donc de restreindre un peu ton motif :

Code :

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

    preg_match ("#\d+ (\w+)#", $City_ZIP, $tab_tmp) ;