Remarques sans lancer l'homebrew, juste en regardant le code :
1) Shell en lua...
2) Les mots de passes de l'admin sont stockés en clair, donc tout le monde peut les lire. Penser à au moins utiliser un hash
3) Fausse barre de chargement => utilité?
4)
- Code: Select All Code
if bar.pourcent == 400 or bar.pourcent>400
mauvais style à remplacer par exemple par :
- Code: Select All Code
if bar.pourcent >= 400
5) Peu d'organisation dans les fichiers, pas de sous dossier pour les fonds d'écran par exemple. Doublons (version PNG et JPG), fichiers inutiles dans l'archive.
6) Indentation incorrecte ou absente dans les fichiers sources.
7) lee.lua => Pourquoi utiliser des entiers pour "valeur" et pas un boolean?
8) menu.lua => Commentaires superflus les "définition de la variable x" n'apportent rien à la compréhension du code.
Ligne 22 : A chaque itération de la boucle, on remet le CPU à 333 mhz? Pourquoi ne pas le faire une seule fois?
9) script.lua : même problème qu'a la remarque 4
10) Tout le système de gestion de la souri est à revoir. Faire quelque chose de moins "dur" pour ajouter facilement des icônes. (Par exemple placer les icônes dans un tableau à deux dimensions, et traduire les coordonnées de la souri avec des divisions entières. Ainsi les coordonnées de la souris renvoient à une case du tableau)
Voilà j'espère que ces 10 petites remarques t'aideront à t'améliorer dans le monde de la programmation. Et il ne faut pas oublier qu'un bon programme commence avec un code propre.
.
par Eric de Brocart » 14 Juil 2010, 12:59 
Eric de Brocart
Administrateur principal
...
, (j'espere que) ca viendra avec le temps....
SitemapIndex