23-10-2007, 11:24 AM
juste une petite critique rapide: il manque de commentaires. Pour l'instant c'est frais dans ta tete, mais dans 6 mois, je t'assure, ca sera plus... chaud.
Plus tu as de commentaires pertinents, mieux tu t'en sors. Remarque 2: essaie de sortir le blabla qui s'affiche a l'écran du code. Quitte a faire nue petite fonction qui reprend tes messages et qui passe des valeurs en paramètres. Ca permet de regrouper tes messages, donc plus de clarté, et de ne pas mélanger code / page.
23-10-2007, 11:28 AM
joshua a écrit :juste une petite critique rapide: il manque de commentaires. Pour l'instant c'est frais dans ta tete, mais dans 6 mois, je t'assure, ca sera plus... chaud. J'y avais pas pensé. Comment faire pour sauvegarder le rapport de combat? pour qu'il soit visible par la partie adverse? Fichier txt ou dans la base de donnée? Merci
23-10-2007, 01:55 PM
Citation :Comment faire pour sauvegarder le rapport de combat? pour qu'il soit visible par la partie adverse?Pour ma part, je sauve les "actions" et les résultats de chaque coup dans un historique (dans une table de la BDD qui ne sert qu'a ça). ça me permet de "re-sortir" le rapport de combat pour l'adversaire, faire des statistique sur le % de coups critiques, la force de frappe moyenne, etc...
Je signale que je ne détiens pas la vérité unique et absolue, je peux me tromper. La critique peut aussi être constructive. Critiquez moi!
La quête d'Ewilan http://easy2hack.ma-soiree.be
23-10-2007, 03:18 PM
et de toute facon tu es obligé de faire une sauvegarde quelque part. Ne serait-ce que pour debugguer apres plainte du joueur :d
23-10-2007, 04:36 PM
De mon coté, j'ai préféré faire un fichier pour le rapport , avec quelques str_replace si c'est un camp ou l'autre qui affiche le rapport.
Pour les stats, j'ai une table à coté.
23-10-2007, 04:51 PM
Moi je suis aussi pour les commentaires
Toi tu comprends ton script mais nous on a un peu de mal a suivre la ^^
Je voudrais pas être méchant, mais un copier-coller de 565 lignes, je veux bien compter les lignes mais faire une critique constructive sur le code ça me parait tendu
Vite fait comme ça tu as regardé le source de la page générée ? Tu as des affichages batards en plein milieu, puis d'un coup un <html> qui vient se poser là genre «faites comme si j'étais là depuis le début, vous inquiétez pas». Sinon ça part d'un bon sentiment le mysql_close() avant <html>, mais dans ce cas ça ne sert à rien Plus généralement, une fonction ne devrait jamais utiliser "echo", mais uniquement générer des chaines de caractère avant de les retourner. Le "echo" doit être réservé à la partie présentation de ton script (entre <body> et </body>...). Ah oui et les variables globales, saylemal, une ou deux comme ça pourquoi pas, mais cette flopée de globales est un nid à bugs.
Ressources [PHP][MySQL][prototype.js]
23-10-2007, 05:58 PM
naholyr a écrit :Ah oui et les variables globales, saylemal, une ou deux comme ça pourquoi pas, mais cette flopée de globales est un nid à bugs.T'es hype Naholyr ! :o p
23-10-2007, 07:48 PM
C'est une nouvelle mode de venir déposer au moins 100 lignes de code et demander si c'est bien ?
Parce que, bien que je ne veux pas être totalement méchant, on n'est pas (du moins moi) réellement là pour sécuriser/optimiser/organiser tout votre code. On (toujours au moins moi-même) veut bien aider mais pas faire à la place des autres. On ne connait pas forcément toutes les astuces. Mais on peut les chercher et réfléchir (oui, oui, c'est possible en prenant le temps de lire ce que d'autres ont fait et en lisant la documentation). Enfin, bon. |
|
Sujets apparemment similaires… | |||||
Sujet | Auteur | Réponses | Affichages | Dernier message | |
Critique de mon Premier Vrai Script de POO | Eluox | 21 | 6 797 |
30-05-2008, 06:38 PM Dernier message: Cartman34 |
|
Algorithm/fonction D'un Code D'attaque | covin85 | 16 | 6 703 |
22-08-2007, 10:54 PM Dernier message: elazard |
|
code d'attaque bis | simonhibou89 | 1 | 1 826 |
27-01-2007, 09:43 AM Dernier message: tashimitsu |
|
[résolu] code d'attaque | simonhibou89 | 16 | 5 380 |
21-12-2006, 12:16 PM Dernier message: simonhibou89 |