24-10-2007, 08:29 AM
naholyr a écrit :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 à rienPlus 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.
Merci pour ces remarques. J'avais pas été voir mon code.
Pour utiliser les variables dans une fonction il ne faut pas les définir en global avant?