Commentaires sur : fonction de Luhn en php

Par : Francis Hilaire

Francis Hilaire — Wed, 19 Oct 2011 07:42:58 +0000

@Poulpos : Ce n’est qu’un court code très ciblé que vous pouvez modifier et réutiliser, il n’est pas destiné à contrôler autre chose que la validité d’une chaîne de caractère qui aura dut être contrôlée en amont par vos différents validateurs de saisies.

Par : Poulpos

Poulpos — Wed, 19 Oct 2011 01:13:46 +0000

Intéressant : en cas de paramètre vide, la fonction renvoie quand même true.
[ isLuhnNum('') => true ]

Penser à ajouter ceci en début de fonction pour remédier simplement au problème

if($length==0){ return false; }

Par : Francis Hilaire

Francis Hilaire — Tue, 14 Oct 2008 14:09:16 +0000

Melnofil : Merci de ton intervention, j’avoue que c’est l’habitude qui m’as fait mettre ce $ret… Il est vrai qu’il est complètement inutile.

Par : Melnofil

Melnofil — Tue, 14 Oct 2008 14:02:04 +0000

Francis Hilaire :
J’ai eu du mal à voir du premier coup comment circule la variable « $ret » tout au long de ton programme, pourquoi ne pas utiliser tout simplement :
return ($tot % 10 == 0);

As-tu une raison particulière d’appeler la fonction « substr » au lieu de l’opérateur « crochets » ?
$digit = $num[$i];

Xfigpower :
Je paraphrase Francis Hilaire en ajoutant que la division est généralement l’opération de base la plus couteuse en temps processeur (quelque soit le language ou le processeur utilisé). Pire, ta multiplication par 2 est effectuée deux fois de suite pour rien !

Par : Francis Hilaire

Francis Hilaire — Mon, 09 Jun 2008 18:15:52 +0000

Merci pour cette précision Andras

Par : andras

andras — Mon, 09 Jun 2008 13:59:16 +0000

hum, il me semble bien qu’il y a une erreur dans ton implementation : ça ne marche qu’avec les nombres qui ont un nombre de chiffre pair (comme les CB) mais pas avec ceux qui ont un nombre de chiffres impair (comme les numéros SIREN).

il faut remplacer la ligne :
if (($i % 2) == 0)
par la ligne suivante :
if ((($length – $i) % 2) == 0)

et là, ça marche pour tous les cas, il me semble.

Par : Francis Hilaire

Francis Hilaire — Thu, 13 Mar 2008 17:17:29 +0000

Moi c’est le (2*$digit)/10 qui me gène, ça donne forcément un float lorsque tu fais par exemple $digit = 1; Si tu travailles comme ça dans un autre language tel que Java ou C# ça ne passera jamais. Bon il est vrai que là on est censé recevoir un int et qu’on l’interprète comme un string donc bon…

Le -9 est déjà plus mathématique car on a forcément un chiffre compris entre 0 et 18 lorsqu’on multiplie par 2 $digit et qu’il correspond exactement à ce que l’algorithme doit faire.

C’est clair, c’est plus rapide, mais pas à comprendre pour un néophyte…

Par : Xfigpower

Xfigpower — Thu, 13 Mar 2008 12:38:37 +0000

Moi, j’utilise perso une version plus courte (je n’aime pas beaucoup le « -9″ dans l’algo)

//Fonction algorithme de Luhn
function isCB($numCB)
{
//longueur de la chaine $numCB
$lengthCB = strlen($numCB);

//resultat de l’addition de tous les chiffres
$tot = 0;

for($i=$lengthCB-1;$i>=0;$i–)
{
$digit = $numCB[$i];

$tot += ($i % 2)?((2*$digit)%10 + (2*$digit)/10):$digit;
}

return !($tot % 10);
}