Discussion :

[Maestro57]

Bonjour à tous,

Mon problème est que j'ai une bdd contenant des numéros à 4 chiffres, mais parmis ceux la, quelques un dispose d'une lettre. Par exemple : E120.
Je précise que les numéros sont du type Varchar et non integer.

Chaque mois je mets à jour la bdd via un site développez en php. J'aimerais créer une fonction qui me retourne "true" si elle a trouvée une lettre dans mon numéro et "false" dans le cas contraire.

Ce qui me permettrai de filtrer les numéros contenant une lettre car je ne veux pas d'eux dans ma base.

Voici la fonction que j'ai developpé :

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
 
function test($entree)
{
$machaine = $entree;
$tab_alpha = array('a','b','c','d','e','f','g','h','i','j','k','l','m','n','o','p','q','r','s','t','u','v','w','x','y','z');
for($i=0;$i<count($tab_alpha);$i++)
{
$find = $tab_alpha[$i];
@$pos = stristr($machaine, $find);
if($pos == false)
{
return false;
}
else
{
return true;
}
}
}

$entree = le numéro à tester.

Mais elle ne fonctionne pas, quelqu'un pourrais me proposer une fonction semblable comprenant un while plutot qu'un for ou bien corriger la mienne ?

Merci d'avance

[koopajah]

Peut-être que is_numeric() pourrait suffire dans ton cas pour savoir si ta chaine est un nombre ou non

[julp]

Les expressions régulières c'est bien aussi

Code :

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

1
2
3
function test($entree) {
    return preg_match('/[[:alpha:]]/', $entree);
}

Pour corriger votre code, l'erreur se situe dans votre algorithme car vous renvoyez systématiquement true ou false suivant que le premier caractère n'est pas alpha ou l'est :

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
function test($entree)
{
    $machaine = $entree;
    $tab_alpha = array('a','b','c','d','e','f','g','h','i','j','k','l','m','n','o','p','q','r','s','t','u','v','w','x','y','z');
    for($i=0;$i<count($tab_alpha);$i++)
    {
        $find = $tab_alpha[$i];
        $pos = stristr($machaine, $find);
        if($pos != false)
        {
            return true;
        }
    }
    return false;
}

Vous auriez pu optimiser en remplaçant stristr par stripos et tab_alpha par une chaîne :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
function test($entree)
{
    $tab_alpha = 'abcdefghijklmnopqrstuvwxyz';
    for ($i = 0, $l = strlen($tab_alpha); $i < $l; $i++)
    {
        if (stripos($entree, $tab_alpha[$i]) !== false)
        {
            return true;
        }
    }
    return false;
}

[Maestro57]

Je vous remercie tous deux...

Je ne peux cependant pas tester vos idées avant demain, donc je vous tiens au courant quand à la suite de l'histoire héhé merci

[The Nux]

Euh, je suis pas convaincu de l'existence de [:alpha:] dans les preg.... J'opterai plus pour un

Code :

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

1
2
3
function test ($str) {
    return (bool) preg_match("@[a-z]@i", $str);
}

qui teste la présence de lettres, mais sinon tu peux aussi faire un

Code :

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

1
2
3
function test ($str) {
    return !((bool) preg_match("@^[0-9]+$@", $str));
}

qui teste si on a bien des chiffres du début à la fin

[julp]

Euh, je suis pas convaincu de l'existence de [:alpha:] dans les preg....

Il vous aurait suffit d'en faire l'essai et oui ça marche car il s'agit du nom des classes POSIX prédéfinies