VÉRIFICATION DE LA VALIDITÉ D'UN BILLET DE BANQUE

Commentaire de neigedhiver le 14/12/2007 02:55:15

Salut,

Quelques petites améliorations sur les regex : plutôt que d'en faire je ne sais pas combien dans ton script, tu peux te contenter de n'en faire qu'une.
Je crois comprendre à la lecture de ton code qu'un numéro de série est composée d'une lettre entre L et Z inclus, à l'exception de O, Q et W, suivie de 11 chiffres. Une seule regex suffit pour vérifier la validité du numéro et récupérer la lettre et le numéro :

if (preg_match('`^([^a-koqw])([0-9]{11})`i', $_POST['num'], $matches)) {
  // Numéro syntaxiquement valide
}
else {
  // Numéro invalide !
}

Si le numéro est syntaxiquement correct, $matches est alors un tableau de ce genre :

    * [0]=>S15509500765
    * [1]=>S
    * [2]=>15509500765

$matches[0] est la chaine qui correspond au motif
$matches[1] la lettre, et $matches[2] la série de chiffres.

Déjà, tu économises des perfs (une seule pcre au lieu de 3) et tu gagnes en quantité de code (et donc en lisibilité) : tu n'as plus qu'un seul test qui débouche sur le message d'erreur numéro invalide.

Ensuite, pour la valeur des lettres... Switch n'est pas optimal, puisque le script parcourt toutes les possibilités jusqu'à tomber sur la bonne.

Puisqu'on a utilisé une regex très sévère en terme de filtrage, on sait que la lettre ne peut être qu'une lettre entre L et Z sauf O, Q et W et pas un autre caractère. On peut donc utiliser directement sa valeur ascii :

$pre = ord(strtoupper($matches[1])) - 64;

Ca, c'est fait : une ligne qui te fait économiser 49 lignes de ton switch($letter)

Ensuite, pour concaténer le tout et obtenir un nombre :

$num = (int) ($matches[1] . $matches[2]);

Maintenant, plus qu'à vérifier la validité du numéro :

if ( bcmod($num, "9") == 8)

Ca, c'est beurk. Mettre 9 entre guillemets change son type en chaine de caractères. bcmod, qui opère sur des nombres, va devoir le changer encore en nombre, alors qu'il était déjà dans le bon type.

if (bcmod($num, 9) == 8)

Voilà : 3 lignes au lieu de plusieurs dizaines. T'as gagné tellement de lignes et en lisibilité que tu peux même te permettre d'ajouter des commentaires ^^ Parce que ça manque, et c'est dommage, tu t'es donné la peine de faire un code XHTML 1.0 Transitionnal valide...

Et si tu veux faire encore plus propre, tu peux mettre tout le html à la fin du script, pour bien séparer le traitement de l'affichage.

Encore un dernier détail... Dans un soucis de factorisation (qui consiste à ne pas écrire 2 fois un code identique utilisé à 2 endroits différents) tu peux modifier l'affichage du message final vrai ou faux billet.

Par exemple :

$billet = (bcmod($num, 9) == 8) ? 'vrai' : 'faux';

Utilise une classe pour la mise en forme de ton texte (et pas un h2 qui sert à identifier un titre, et uniquement à cela, et pas à écrire plus gros);

echo '<p id="resultat" class="' . $billet . '">Votre billet est ' . $billet . ' !</p>';

Dans ton css :

p#resultat {
  font-weight: bold;
  background-color: transparent;
}

p.vrai {
  color: green;
}

p.faux {
  color: red;
}

Et là, tu auras un code php performant, élégant, documenté (si tu mets des commentaires ;) ) et séparé du code html.
Quant au html, en plus d'être valide, il sera aussi sémantiquement correct.

Commentaire de neigedhiver le 14/12/2007 12:43:59

Tu voulais surement dire :
([l-z^oqw])

Les deux regex sont à peu près équivalentes :
http://www.lumadis.be/regex/test_regex.php?lang=fr&pat1=%60^([^a-koqw])([0-9]{11})%60i&pat1typ=preg_match&pat2=%60^([^a-koqw])([0-9]{11})%60i&pat2typ=preg_match&txt=S15509500765&result=Ti

La première est un coup plus rapide, un coup plus lente...

Commentaire de neigedhiver le 14/12/2007 12:48:19

Hum... Je sais pas si le lien s'affiche bien, il est peut-être trop long...
Voici : http://www.orphyx.net/shortlink.php?l=1

Commentaire de kankrelune le 14/12/2007 12:51:32

le mien est plus rapide et surtout le tien ne fait pas de différence entre 015509500765 et S15509500765... .. . ;)

@ tchaOo°

Commentaire de kankrelune le 14/12/2007 13:01:56

Au final après test...

`^([l-npr-vx-z])([0-9]{11})$`i

les deux autre laissent passer des codes invalides... .. .

@ tchaOo°

Commentaire de kankrelune le 14/12/2007 15:40:12

Muarf je suis con moi... .. .



function validateSerial($num)
{
    if(!empty($num) && preg_match('`^([l-npr-vx-z])([0-9]{11})$`i', $num, $matches))
    {
         return (bcmod(
                      str_replace(
                                  $matches[1],
                                  ord(strtoupper($matches[1])) - 64,
                                  $matches[0]), '9') === '8');
    }
    return false;
}

@ tchaOo°

Commentaire de neigedhiver le 16/12/2007 15:43:20

@Kankrelune : Bravo pour le monologue, j'aurais pas fait mieux.

@The_Magicien : Ben si on poste en commentaire, c'est pour que tu puisses bénéficier des améliorations qu'on propose. Tu ne t'appropries pas notre travail, on te le donne. C'est pas pareil.

Et puis même si tu considères que ce n'est pas toi qui a écrit ce code, il l'a été en tout cas à ton initiative : il ne me serait pas venu à l'idée de me renseigner sur la manière dont vérifier un numéro de série d'un billet Euro, de même qu'il ne me serait pas venu à l'idée non plus d'écrire un code pour le vérifier.
Les commentaires avaient donc pour but de te montrer comment faire plus simple, plus efficace et plus élégant (en plus, là, t'as directement une fonction utilisable dans n'importe quelle application). Ils avaient aussi pour vocation de le montrer à tout le monde.

Tu peux donc sans scrupule l'utiliser. Si tu ne veux pas le mettre à la place de ton code, ce que je peux comprendre, tu peux le mettre à la suite ;)
Mais je t'en prie... On se fout de droits d'auteurs sur des commentaires sur une source ^^