Bonjour,

Quelques remarques en vrac :
- Comme COD57, je confirme que le manque de filtrage anti SQL-Injection est une vulnerabilité importante... Il faudrait vraiment corriger ça.
- Pourquoi devoir modifier plein de fichiers pour la configuration du script au lieu de tout mettre dans le config.php (qui devrait servir à ça d'après son nom) ?
- A quoi servent ces lignes exactement (surtout les deux dernières...) :
$sql="SELECT * FROM travaux";
$req = mysql_query($sql) or die('Erreur SQL !<br />'.$sql.'<br />'.mysql_error());
while($data=mysql_fetch_assoc($req)){
}
- Je ne vois aucun mysql_free_result()...
- Pourquoi utiliser du XHTML qui non seulement est déprécié, mais en plus a très peux de chances d'être correctement interprété (90% des serveurs HTTP ne sont pas correctement configurés pour servir du XHTML, et le navigateur l'interprète comme du quirks mode (cf google pour les détails... de toutes façons à notre époque il n'y a pas grand intérêt à utiliser un autre doctype que celui d'html 5)).

Bien cordialement,
Eric

> Oui, il est un peu lourd avec "EVEN" et "ODD", mais l'objectif est aussi pédagogique.
> Et cela marque bien la somme pair de impair. La fonction dans le programme de production
> est bien plus barbare en terme de code :D

Si c'est pédagogique, autant ne pas montrer d'horreurs aux petits jeunes qui pourraient être tentés d'imiter ces pratiques... :o)

La cle sert surtout pour verifier après un scan que la lecture s'esrt bien passée.

Pour ton code DUN14, je suppose que tu vas utiliser GD ? ou peut-etre google charts mais tu devras ajouter le cadre et les chiffres.

Bonjour,

C'est la première fois que j'entends parler de l'EAN14... C'est utilisé où ?
Par contre, ton code marche sans doute aussi pour l'UPC (auquel il suffit de rajouter un 0 au début pour le transformer en EAN13).

Ton implémentation fonctionne, mais le coup du tableau de tableau c'est un peu lourd (et inutile).

Tu peux directement utiliser $sum[0] pour les pairs et $sum[1] pour les impairs, puis avoir dans ta boucle un :
$sum[$i%2]+=intval(substr($barcode,$i,1));
ou mieux :
$sum[$i%2]+=intval($barcode[$i]));

Eric

phpAnonyme: Desole, je pensais que tu parlais du code.

Pour ce qui est de "faire un controle", ca depend de ce que tu veux mesurer. C'est vrai que "Compteur de visites" est un peu trompeur ici, puisqu'il s'agit plus d'un Compteur de pages vues".
Un cookie permettrait d'obtenir le nombre de visiteurs uniques / une periode de temps (heure/jour/mois/session) en fonction de la duree de vie de celui-ci.

J'avoue que je ne vois pas vraiment ce qu'on peut ajouter de plus mais si tu as un exemple precis en tete je suis curieux de le connaitre :o)

cod57: Ta version n'est pas tres lisible, et si le fichier n'existe pas elle le cree vide.
Si tu veux vraiment sauver quelques caracteres pour rendre le code illisible, essaie plutot comme ca:

file_put_contents('compteur.txt',$compteur=(file_exists('compteur.txt')?file_get_contents('compteur.txt')+1:1),LOCK_EX);
echo $compteur;

PS: C'est marrant d'ecrire du code comme ca mais je ne peux que vous encourager a ne PAS le faire si vous comptez l'utiliser "en conditions reelles" :o)

phpAnonyme: Rien n'est jamais TROP minimaliste : Si tu peux obtenir le meme resultat de 2 manieres, la plus courte est toujours la meilleure (par meme resultat, j'entend "qui fonctionne dans tous les cas").

De plus, il y a beaucoup d'exemples de 2 ou 3 lignes qui sont des horreurs (offrant au choix des trous de securite ou une utilisation de ressources sans communes mesures avec la fonction remplie) et ce n'est pas evident pour un debuttant de faire le tri.

En mettant une bonne note a ces 3 lignes-ci plutot qu'a d'autres, ca permet de aussi d'indiquer qu'il s'agit d'un bout de code qui fait correctement ce qu'on attend de lui sans mauvaise surprise.

Alors d'accord ce n'est pas de la "rocket-science" mais il faut bien commencer un jour :o)

> Sinon, créer un cookie sur le Navigateur et n'incrémenter que si le cookie n'est pas présent.

C'est ce que je fais : Ca permet d'avoir un compteur genre "Cette page a ete vue xxx fois par yyy personnes differentes"...

C'est effectivement un bon exemple, et tres interessant (tout le monde n'a pas envie d'un framework de 2000 lignes de PHP pour faire des petits trucs simples).
"La perfection n'est pas atteinte quand on ne peut plus rien ajouter, mais quand on ne peut plus rien enlever" :o)

J'aurai mis la note max si tu avais mis ca dans une fonction et donne quelques explications sur le deploiement (comme choisir un repertoire dans lequel le serveur peut ecrire (et pas juste donner les droits sur le fichier comme je disais plus haut, car comme le fichier va etre cree par le premier appel du script, les droits seront forcement bons pour celui-ci)) mais FWIW je ne met pas souvent d'aussi bonnes notes ;o)

Bien vu !
Je ne connaissais pas cette nouveaute.

Auquel cas, ca reste tres simple.

Par contre, tu devrais donner quelques details sur la mise en oeuvre (comme "penser a changer les droits du fichier .txt pour que le process du serveur HTTP puisse ecrire dedans) et mettre a jour le source avec le LOCK_EX.

Eric