GÉNÉRATEUR DE GRILLES D'ACCORDS
codefalse
Messages postés
1123
Date d'inscription
mardi 8 janvier 2002
Statut
Modérateur
Dernière intervention
21 avril 2009
-
3 févr. 2008 à 18:49
malalam Messages postés 10839 Date d'inscription lundi 24 février 2003 Statut Membre Dernière intervention 2 mars 2010 - 4 févr. 2008 à 08:10
malalam Messages postés 10839 Date d'inscription lundi 24 février 2003 Statut Membre Dernière intervention 2 mars 2010 - 4 févr. 2008 à 08:10
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/45615-generateur-de-grilles-d-accords
https://codes-sources.commentcamarche.net/source/45615-generateur-de-grilles-d-accords
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 :/