UN SIMPLE QCM EN LIGNE

Commentaire de TheSin le 05/07/2006 10:22:02

$mat=$qcm['matiere'];
$fil=$qcm['filiere'];
$niv=$qcm['niveau'];
pour faire juste après :
echo $mat;
echo $fil;
echo $niv;
(en enlevant l'html entre)
pourquoi pas un simple ...
echo $qcm['matiere'];
echo $qcm['filiere'];
echo $qcm['niveau'];
pas la peine de surcharger la mémoire inutilement ;)

De plus :
         $id=$_GET['id'];
         $idmat=$_GET['idmat'];
         $ntest=$_GET['ntest'];
         $mat=$_GET['mat'];
         $fil=$_GET['fil'];
         $niv=$_GET['niv'];
Tu met tout ca dans une requete mysql sans les vérifier, donc mysql injection très facile à faire ;)

Encore une petite chose ... je crois que le niveau initié est un peu exagéré ;)

Commentaire de Taka_s_tiger le 05/07/2006 13:38:19

il me semble avoir déja lu sur ce site que la virgule est à préférer plutot que l'opérateur de concaténation . (point) dans le cadre d'un echo...pour des raisons d'optimisation

quelqu'un peut confirmer ?

Commentaire de jean84 le 17/07/2006 20:43:03

2/3 trucs :

- utilise les ' plutot que les " (surtout dans une requete de type SELECT * FROM !!)
- dans connect.php
-> $cnx = mysql_connect($mysqlServerDB,$userDB,$passwdDB) or
        die("Echec de la connection");
        mysql_select_db($dataBaseNameDB) or die("Echec de la connection");

Ton mysql_select_db() doit etre de la frome : mysql_sql_db($dataBaseNameDB, $cnx);

Evite les echo en boucle :
echo '<tr height="25"><td bgcolor="#CCFFCC"><font face="@Dotum" color="#330033" size="3">';
echo '<a href="matiere.php?id='.$id.'">'.$mat.'</a>';
echo '</td><td bgcolor="#CCFFCC"><font face="@Dotum" color="#330033" size="3">';
echo $niv;
echo'</td><td bgcolor="#CCFFCC"><font face="@Dotum" color="#330033" size="3">';
echo $fil;
echo'</td></tr>';

Utilise de preference le '.' =>
Moi perso je fais sa :
$Affichage = '';
$Affichage .= '<tr height="25"><td bgcolor="#CCFFCC"><font face="@Dotum" color="#330033" size="3">';
$Affichage .= <a href="matiere.php?id='.$id.'">'.$mat.'</a>';
$Affichage .= </td><td bgcolor="#CCFFCC"><font face="@Dotum" color="#330033" size="3">';
$Affichage .= $niv;
$Affichage .= </td><td bgcolor="#CCFFCC"><font face="@Dotum" color="#330033" size="3">';
$Affichage .= $fil;
$Affichage .= </td></tr>';
echo $Affichage;
fais le en debut de page et tu verras sa sera mieux et plus digeste (a mon sens evidement)

<?php
    $score=0;
    //conection à la base
    require ("connect.php");  
    $choice = $_POST['choice'];
    //compter le nombre des questions
    $nbq=count($choice);
?>
<?php [...]
Si tu doit encore utiliser php, pourquoi referme la balise pour la re-ouvrir ??

Sinon pour la suite, je dirais que c'est sympa comme code malgre les quelques erreurs (on est la pour apprendre),  et te souhaite de continuer comme sa (par contre essaye de faire un truc un peu plus fun comme code parce que les qcm ... lol... :p)

@+

Commentaire de spidermario le 04/04/2007 17:07:21

En fait, la virgule pour les echo n'est pas toujours plus rapide que le point dans la mesure où elle doit écrire plusieurs fois sur le flux de sortie, ce qui est long.

Toutefois, en mettant

     ob_start();

au début du code et

     ob_end_flush();

à la fin, echo n'écrira pas directement sur le flux de sortie mais sur un tampon qui sera vidé à l'appel de ob_end_flush, ce qui fait que la virgule devient plus performante. On peut d'ailleurs à tout moment appeler la fonction ob_get_content pour récupérer le contenu actuel du tampon.