BLOCK POUR LES PORTAILS NUKED-KLAN

Commentaire de malalam le 24/02/2006 13:41:13 administrateur CS

Hello,

Les petites critiques :

+ if (eregi("blok.php", $_SERVER['PHP_SELF'])) {
        die ("You cannot open this page directly");
    }

=> eregi est inutile pour çà. Il y a tout un tas d'autres moyens moins gourmands. Allez, un pas terrible (il y a mieux ):
if (basename ($_SERVER['PHP_SELF']) === 'block.php') {}

+ Evite les SELECT * :-)
+ le html n'est pas joli du tout du tout (balises en majuscule, attributs dépréciés...) En fait, parfois il est ok, parfois pas du tout...comme si certaines parties avaient été développées il y a très longtemps, et d'autres plus récemment ;-)
+ tu devrais utiliser des boucles php quand tu as des listes déroulantes à rallonge à créer, ce serait plus simple.
+ 'global' est loin d'être indispensable. On aurait préféré un fichier de configuration avec des constantes par exemple. D'autant plus que tu utilises déjà des constantes...pourquoi ne pas avoir sorti tout ça dans un fichier de conf?

Bon enfin, je ne vais pas plus loin dans les détails : ton code est orginal, le screenshot est joli :-) Et il y a pire niveau programmation PHP.
C'est pas mal, donc :-)

Commentaire de clenio le 27/02/2006 09:09:35

J'ai oublié de préciser que c'est mon premier code php. Donc je vous remercie pour vos commentaires. :)

Pour le if (eregi()){}, si vous pouviez me dire exactement à quoi cela sert, ca serait sympa. Si je l'ai mis, c'est parce que j'ai constaté que tout les blocks pour Nuken Klan commence par cela (donc pour homogénéïser mon code avec les autres sources).

Qu'as tu contre le "select *" malalam ? Il est vrai que j'aurai pu lister tous les champs de la table, mais je trouve cela plus compréhensible. Et si la structure de la table change, ca permet de péréniser le code ;).
Quand au HTML, ouai je sais... je vais d'ailleurs le faire tout beau et changer le ZIP.

Je vais tenir compte de vos remarques avant de poster la version 2.0 de mon code! La nouvelle version intègre une liste déroulante et permet de changer dynamiquement les entrées affichées (ajout de javascript).

Merci encore
Clénio

Commentaire de clenio le 27/02/2006 11:19:13

Ok
Merci Malalam, je vais modifier le code en conséquence! :)
lol, par contre, pour la v2.0, j'ai bien remplacer le select * par un select et le nom des champs. Je la poste dès que j'ai commenté.

Tant que j'y suis, je prévois un v2.1 qui ajoutera un lien dynamique au texte (lien vers une screen pour montrer le boss mort dans mon cas).

Commentaire de clenio le 07/03/2006 08:41:44

j'y pensais mais je manque de tps (mes journées devraient faire 48 voir 96 heures des fois ... ;) ).
Entre l'électronique, la prog, le montage vidéo ET le boulot (mdr), je manque un peu de tps pour tout faire. Mais je vais essayer de trouver le tps.

bye Loxyey

Commentaire de clenio le 07/03/2006 09:17:05

lien direct vers la v2.0 :

http://www.phpcs.com/code.aspx?ID=36418

Commentaire de kankrelune le 22/04/2006 17:47:44

D'une part parce que c'est le script de Clenio et d'autre part parce que si on commençait à mettre à jour tous les scripts auxquels on voit des corrections à apporter on y passerait nos journées... qui plus est je n'utilise pas ce CMS je ne vais donc pas l'installer juste pour corriger un script qui, d'ailleurs, tourne en l'état il me semble... mais libre à toi de mettre le script en question à jour si ça te chante... .. .

@ tchaOo°