cs_depression
Messages postés100Date d'inscriptionmardi 7 novembre 2000StatutMembreDernière intervention13 juillet 2009
-
12 mars 2008 à 18:41
kiki2sirom
Messages postés153Date d'inscriptionmardi 17 août 2004StatutMembreDernière intervention23 décembre 2010
-
17 mars 2008 à 18:09
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
kiki2sirom
Messages postés153Date d'inscriptionmardi 17 août 2004StatutMembreDernière intervention23 décembre 2010 17 mars 2008 à 18:09
Oui ma foi, d'accord à Malalam, vu que c'est qqch de très débutant...
utilise également "include" pour tes paramètres de connexion à MySQL ou autre fonction à réutiliser
Dis-toi que dès que tu écris 2 fois le même paquet de code, il doit y avoir, la plupart du temps, un moyen de le contourner :
exemple sur chaque page, tu fais :
mysql_connect("localhost", "root", "");
mysql_select_db("mysql");
user.php : utilise echo plutôt que print : plus rapide ici
Par contre, par rapport à d'autres "têtes" ou qui se le croient comme tel, déjà tu utilises bien <?php : bon départ
Bon courage
kiki
windows972
Messages postés28Date d'inscriptiondimanche 17 juin 2007StatutMembreDernière intervention12 août 2009 15 mars 2008 à 16:17
merci pour les critique tu as raison je suis encore debutant ^^
malalam
Messages postés10839Date d'inscriptionlundi 24 février 2003StatutMembreDernière intervention 2 mars 201025 12 mars 2008 à 20:37
Hello,
il y a beaucoup de chats ici...et dans ton code, pas mal de problèmes.
Le point positif, c'est que tu fais des isset() à tout va. C'est rare pour un débutant.
Parfois, tu en fais un peu trop : si isset($a) renvoie true, alors $a == null renvoie false dans tous les cas.
Ton code est séquentiel : même pas l'ombre d'une fonction...difficile à intégrer dans un applicatif web existant.
Tu forcesl'utilisation de mysql.
Tu n'échappes que rarement les valeurs que tu insères en base.
Aucune vérification sur les accès à la base.
Ouvrir et fermer un tag php juste pour une accolade...ahem...
Bref...c'est vraiment light...mais d'un autre côté, tu n'en fais pas trop. Ceci dit, pose toi la question : qu'est-ce que ton code va apporter par rapport aux autres déjà présents sur CS ?
cs_depression
Messages postés100Date d'inscriptionmardi 7 novembre 2000StatutMembreDernière intervention13 juillet 2009 12 mars 2008 à 18:41
"La version 1.1 sortiras Bientot :
-supression des Bug"
Comment dire... Une release stable n'est pas sensée être livrée avec des bugs déja connus.
17 mars 2008 à 18:09
utilise également "include" pour tes paramètres de connexion à MySQL ou autre fonction à réutiliser
Dis-toi que dès que tu écris 2 fois le même paquet de code, il doit y avoir, la plupart du temps, un moyen de le contourner :
exemple sur chaque page, tu fais :
mysql_connect("localhost", "root", "");
mysql_select_db("mysql");
user.php : utilise echo plutôt que print : plus rapide ici
Par contre, par rapport à d'autres "têtes" ou qui se le croient comme tel, déjà tu utilises bien <?php : bon départ
Bon courage
kiki
15 mars 2008 à 16:17
12 mars 2008 à 20:37
il y a beaucoup de chats ici...et dans ton code, pas mal de problèmes.
Le point positif, c'est que tu fais des isset() à tout va. C'est rare pour un débutant.
Parfois, tu en fais un peu trop : si isset($a) renvoie true, alors $a == null renvoie false dans tous les cas.
Ton code est séquentiel : même pas l'ombre d'une fonction...difficile à intégrer dans un applicatif web existant.
Tu forcesl'utilisation de mysql.
Tu n'échappes que rarement les valeurs que tu insères en base.
Aucune vérification sur les accès à la base.
Ouvrir et fermer un tag php juste pour une accolade...ahem...
Bref...c'est vraiment light...mais d'un autre côté, tu n'en fais pas trop. Ceci dit, pose toi la question : qu'est-ce que ton code va apporter par rapport aux autres déjà présents sur CS ?
12 mars 2008 à 18:41
-supression des Bug"
Comment dire... Une release stable n'est pas sensée être livrée avec des bugs déja connus.