malalam
Messages postés10839Date d'inscriptionlundi 24 février 2003StatutMembreDernière intervention 2 mars 201025 4 févr. 2008 à 08:10
Hello,
echo, tout comme print d'ailleurs, accepte les parenthèses. C'est simplement historique.
Néanmoins, en effet, ça ne facilite pas la lecture.
A ce qui a déjà été dit, j'ajouterai que ton en-tête html indique que ta page est en xhtml, mais que tu ne respectes pas la syntaxe xhtml : tous les éléments doivent être fermés :
=>
=>
Et que tu utilises des éléments dépréciés :
=>
Un élément <form> n'a pas d'attribut "name".
Un formulaire se met en forme via des <fieldset>, des <label>, des ...pas des tableaux, normalement. Or là, tu peux très bien te passer des tableaux.
Tu devrais aussi créer une css et sortir tous tes les attributs style que tu as mis : ce serait bien plus simple à modifier, et cela allègerait grandement ton code.
Le bon côté des choses, c'est que ton code est original :-)
coucou747
Messages postés12303Date d'inscriptionmardi 10 février 2004StatutMembreDernière intervention30 juillet 201244 3 févr. 2008 à 23:09
moi ce qui me choque (mis a part l'utilite du php ici... qui se resume a proposer un "enregistrer") c'est plus les lignes :
l'utilisation des echo ici est etrange : tu l'utilises comme si c'etait une fonction (avec des parentheses) or c'est pas une fonction... ensuite, tu utilises les "..." pour delemiter une chaine qui ne contient pas de variable, je m'explique :
toi tu fais :
$variable="toto";
echo "la valeur est ".$toto;
tu pourrais faire :
echo "la valeur est $toto";
ou mieux : (plus rapide a l'execution et plus lisible):
echo 'la valeur est '.$toto;
cs_shemesh
Messages postés1Date d'inscriptionlundi 25 septembre 2006StatutMembreDernière intervention 3 février 2008 3 févr. 2008 à 21:09
Quand un gros débutat comme moi poste timidement son code c'est bien pour entendre ce genre de réponse, merci pour vos conseils et ne vous excusez pas, je suis là pour apprendre...
nacedo57400
Messages postés39Date d'inscriptionsamedi 7 janvier 2006StatutMembreDernière intervention13 janvier 2009 3 févr. 2008 à 19:35
J'suis tout a fait d'accord avec CODEFALSE, ton php n'est pas spécialement utile.
tu aurrai pu utiliser la librairie GD afin de dessiner de véritable partition avec different type de note dessus.
l'idée de basse est bonne mais il faut encor l'améliorer
++
codefalse
Messages postés1123Date d'inscriptionmardi 8 janvier 2002StatutModérateurDernière intervention21 avril 20091 3 févr. 2008 à 18:49
Tu ne vérifie pas les données provenant de l'utilisateur, du coup ton système est faillible aux attaques de type xss. Protège les avec des fonctins du type htmlentities et compagnie.
De plus ton code ne fait rien de plus que proposer un formulaire et récuperer les valeurs pour les afficher ensuite. Tu ne fait aucun traitement derriere.
Ne le prends pas mal, mais ton script, au niveau php, ne sert à rien :/
4 févr. 2008 à 08:10
echo, tout comme print d'ailleurs, accepte les parenthèses. C'est simplement historique.
Néanmoins, en effet, ça ne facilite pas la lecture.
A ce qui a déjà été dit, j'ajouterai que ton en-tête html indique que ta page est en xhtml, mais que tu ne respectes pas la syntaxe xhtml : tous les éléments doivent être fermés :
=>
=>
Et que tu utilises des éléments dépréciés :
=>
Un élément <form> n'a pas d'attribut "name".
Un formulaire se met en forme via des <fieldset>, des <label>, des ...pas des tableaux, normalement. Or là, tu peux très bien te passer des tableaux.
Tu devrais aussi créer une css et sortir tous tes les attributs style que tu as mis : ce serait bien plus simple à modifier, et cela allègerait grandement ton code.
Le bon côté des choses, c'est que ton code est original :-)
3 févr. 2008 à 23:09
# echo( "Titre : ".$titre_grille." - \n" ) ;
# echo( "Page : ".$page."
\n" ) ;
# echo( "Auteur : ".$auteur." - \n" ) ;
# echo( "Date : ".$date."
\n" ) ;
# echo( "Lien web : ".$web."
\n" ) ;
l'utilisation des echo ici est etrange : tu l'utilises comme si c'etait une fonction (avec des parentheses) or c'est pas une fonction... ensuite, tu utilises les "..." pour delemiter une chaine qui ne contient pas de variable, je m'explique :
toi tu fais :
$variable="toto";
echo "la valeur est ".$toto;
tu pourrais faire :
echo "la valeur est $toto";
ou mieux : (plus rapide a l'execution et plus lisible):
echo 'la valeur est '.$toto;
3 févr. 2008 à 21:09
3 févr. 2008 à 19:35
tu aurrai pu utiliser la librairie GD afin de dessiner de véritable partition avec different type de note dessus.
l'idée de basse est bonne mais il faut encor l'améliorer
++
3 févr. 2008 à 18:49
# $titre_grille = $_POST['titre_grille'] ;
# $page = $_POST['page'] ;
# $auteur = $_POST['auteur'] ;
# $date = $_POST['date'] ;
# ...
Tu ne vérifie pas les données provenant de l'utilisateur, du coup ton système est faillible aux attaques de type xss. Protège les avec des fonctins du type htmlentities et compagnie.
De plus ton code ne fait rien de plus que proposer un formulaire et récuperer les valeurs pour les afficher ensuite. Tu ne fait aucun traitement derriere.
Ne le prends pas mal, mais ton script, au niveau php, ne sert à rien :/