Problème radio button espaces et quotes [Résolu]

gamiel · 12/12/2011, 15h36

Bonjour à tous,

J'ai un problème dans ce code et j'ai vraiment besoin d'aide car j'ai passé vraiment beaucoup de temps, mais la je tourne en rond.

Cette page affiche l'ensemble des matieres disponibles pour la classe de terminale, que l'on pourra choisir à l'aide de radio button.
La valeur du radio button sera ajoutée ensuite dans la base de donnée.
Mon probleme est le suivant:
Soit, si la matière s'appelle -l'espace- la valeur recupérée ne sera que le L.
Cependant, quand je rajoute un addslashes par contre, la matiere -l'espace va marcher, mais si la matière s'appelle -bonjour les amis- la valeur recupérée ne sera que -bonjour-.
Je pense que le probleme vient de la ligne 76, mais impossible de faire marcher une matière avec plusieurs mots, ou dans l'autre cas, les quotes, c'est soit l'un sois l'autre en fonction du addslashes à la ligne 76.
J'espère que j'ai été clair.

HELP! j'ai tout essayé!!

Merci beaucoup

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
 
<?php
 
	session_start();
	if (!isset($_SESSION['username'])) {
		header ('Location: logout.php');
		exit();
	}
	?>
 
 
<?php
	include('config.php')
	?>
 
<?php
	$sql = "SELECT ClasseEtud FROM users WHERE username='".$_SESSION['username']."'";
 
	$query = mysql_query($sql);
 
	$result = mysql_fetch_array($query);
	$_SESSION['classename']=substr($_SESSION['classename'],0,1);	
 
	if($_SESSION['classename'] !="T") {
 
		echo '<h1><p>Cette page n&rsquo;est pas pour toi, tu n&rsquo;es pas un Terminale</p></h1>';
 
		exit;
 
	}
 
 
	?>
 
 
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1">
	<title>Terminales : Choix des matieres</title>
  </head>
 <body bgcolor="#99CCFF">
        <form name="session" method="POST" action="pageRemerciementsTerminales.php"/> 
 
        <br>
 
		<p align=center><font color="red" size="10" face="Arial"> Choix de l'accompagnement personnalis&eacute; pour les Terminales </font></p>
 
 
		 <br><br>
 
	<center>         <button> Valider </button></center>	<br><br>
	<?php
 
		// Connection â€¡ ta Base MySql 
			mysql_connect("localhost", "x", "y") or die ("impossible de se connecter au serveur" ); 
			mysql_select_db("accompagnementpersonnalise") or die ("impossible de se connecter a la base de donnees" ); 
 
		// Requete 
			$sql = "SELECT* FROM matiereterminales ORDER BY NomMatiere ASC"; 
		// Resultat
			$resultat=mysql_query($sql); 
		// Nombre de rÃˆponse â€¡ ta question 
			$nombre_de_reponse=mysql_num_rows($resultat); 
 
 
 
 
 
		// Pour lister l'ensemble des reponses a la requete 
			while($ligne = mysql_fetch_array($resultat)) 
			{  
	?>  
 
						<table border="1" align="center">
			<tr  valign="middle" align="center">
				<td width="20"> <input type="radio" name="option" value=<?php echo "$ligne[NomMatiere]" ?>> </td>
				<td width="400"><?php echo "$ligne[NomMatiere]"?> </td>   
				<td width="200"><?php echo "$ligne[Effectif] / $ligne[EffectifTotal]" ?></td>
			</tr>	
		</table>
 
 
 
 
		<?php } ?> 
 
 
 
<br><br>
<center><a href="logout.php"><button> Abandonner </button></a></center>		 
 
 
  </body>
 
</html>

Benjamin Delespierre · 12/12/2011, 16h09

Non, tu n'a pas essayé la solution qui fonctionne.

Pour rappel, en HTML les valeurs des attributs sont notées entre doubles quotes

Donc ta ligne doit être:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

<td width="20"><input type="radio" name="option" value="<?php echo $ligne['NomMatiere'] ?>" /></td>

gamiel · 12/12/2011, 21h01

Un énorme merci, ca marche...
tu es parfait...
j'en reviens toujours pas...

gamiel · 12/12/2011, 21h35

Est-ce normal que si la matière s'appelle par exemple:

"a nous pa'ris" avec des guillemets, la valeur du radio button devient nulle?

Benjamin Delespierre · 12/12/2011, 21h48

non ça n'est pas normal. normalement les guillemets passent, vérifie ce que tu reçoit dans $_REQUEST.

gamiel · 12/12/2011, 22h04

En fait quand l'eleve tente de choisir une matiere avec des guillemets comme "a nous p'aris" , c'est une fenetre javascript qui s'ouvre qui me dit, "vous devez selectionner votre matiere" et donc que le radio_selected est nulle
cf ligne 116....

je sais que ca fait beaucoup de code à lire , mais je te remercies par avance...

Ca c'est la pageRemerciementsTerminale.php qui est appelée apres la page que j'ai posté au dessus

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
<?php
 
	session_start();
	if (!isset($_SESSION['username'])) {
		header ('Location: logout.php');
		exit();
	}
	?>
 
 
<?php
	include('config.php')
	?>
 
<?php
	$sql = "SELECT ClasseEtud FROM users WHERE username='".$_SESSION['username']."'";
 
	$query = mysql_query($sql);
 
	$result = mysql_fetch_array($query);
	$_SESSION['classename']=substr($_SESSION['classename'],0,1);	
 
	if($_SESSION['classename'] !="T") {
 
		echo '<h1><p>Cette page n&rsquo;est pas pour toi, tu n&rsquo;es pas un Terminale</p></h1>';
 
		exit;
 
	}
 
 
	?>
<html>
 
  <head>
 
	<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1">
 
 
	<title>Remerciements</title>
 
</head>
 
  <body>
	<form name="session" method="POST" /> 
	<?php if(isset($_SESSION['username'])){$login = htmlentities ($_SESSION['username'] , ENT_QUOTES, 'UTF-8');} ?>
   <?php
          //connection au serveur
  		$cnx = mysql_connect( "localhost", "x", "y" ) ;
 
  	  //sélection de la base de données:
  		$db  = mysql_select_db( "accompagnementpersonnalise" ) ;
 
  	 //récupération des valeurs du radio button qui a été coché 
		$selected_radio = $_POST["option"]; 
 
	//récupération du LOGIN de l'élève
	//	$username= $_POST['username'];
 
		$sql4= 'SELECT MatiereChoisie
		     FROM users
		     WHERE username = "'.$login.'"'; 
 
         $requete4 = mysql_query($sql4, $cnx) or die( mysql_error() ) ;
         $resultat4 = mysql_result($requete4,0);
 
 
		if ($resultat4 != null){
			?><script language='Javascript'>
				alert("Vous avez deja selectionne votre matiere" );
				location.href = "logout.php";
			</script>
			<?php }   
 
 
		else {
 $selected_radio=stripslashes($selected_radio);
 
	 $sql1= 'SELECT Effectif
		     FROM matiereterminales
		     WHERE NomMatiere = "'.$selected_radio.'"';
 
         $requete1 = mysql_query($sql1, $cnx) or die( mysql_error() ) ;
         $resultat1 = mysql_result($requete1,0);
		 $resultat1--; 
 
 
 
 
		 if ($selected_radio != null && ($resultat1>0 || $resultat1==0)) {
   $selected_radio=stripslashes($selected_radio);
			 $sql2 = 'UPDATE matiereterminales  
		              SET   Effectif = "'.$resultat1.'"
					  WHERE NomMatiere = "'.$selected_radio.'"';
 
 
 
			 $requete2 = mysql_query($sql2, $cnx) or die( mysql_error() ) ;
 
 
			  $sql3 = 'UPDATE users
					  SET MatiereChoisie = "'.$selected_radio.'"
					  WHERE username = "'.$login.'"';
 
		     $requete3 = mysql_query($sql3, $cnx) or die( mysql_error() ) ;
 
			 $sql= 'SELECT NumeroSalle
						   FROM matiereterminales
						   WHERE NomMatiere = "'.$selected_radio.'"'; 
 
					$requete = mysql_query($sql, $cnx) or die( mysql_error() ) ;
					$salle = mysql_result($requete,0);
	} 
 
  else{
		if ($selected_radio == null){
			?><script language='Javascript'>
				alert("Veuillez sélectionner une matière" );
				location.href = "pageTerminales.php";
			</script>
			<?php }    
		else { ?>
				<script language='Javascript'>
				location.href = "pageTerminales.php";
				alert("Il n'y a plus de places disponibles pour ce theme" );
 
				</script>
 
		<?php } 
	}
 
 
 
 
	//affichage des résultats, pour savoir si la modification a marchée:
  	if($requete2 )
  		{
  		 // echo("La décrémentation a bien eu lieu") ;
 		 }
  	else
  		{
    		  echo("La décrémentation a échoué") ;
  		}
 
	}
   ?>
 
 
 
	<center>
        	<img  width="400" height ="350" src="http://www.aquaportail.com/pictures1107/wistiti57_1310278497-diplome-bacalaureat-specimen.jpg" />
 
		<img  width="400" height ="350" src="http://thevenin.zephirin.eu/__oneclick_uploads/2009/09/numerisation0005.jpg" />
	</center>
 
 	<p align="center"> Merci d'avoir validé votre Accompagnement Personnalisé par th&egrave;me. Vous avez sélectionné : <font color="red"><?php echo stripslashes(stripslashes($selected_radio)) ?></font>. </p>
 	<p align="center"> Vous serez dans la salle : <font color="red"><?php echo "$salle" ?></font>. </p>
<p align="center"><font size="10" color="red"> Merci de vous déconnecter en cliquant sur le lien ci-dessous </p>
 
	<CENTER><a href="logout.php" style="color:blue;text-decoration:none;"><button>Se déconnecter</button></a></CENTER>
 
 
 
 </form>
  </body>
 
</html>

clovis802 · 13/12/2011, 00h45

On ne place plus de point virgule après l'include() ?

Benjamin Delespierre · 13/12/2011, 10h11

Ce n'est pas obligatoire si tu fermes la balise PHP juste derrière. Je serais toi je le mettrai quand même.

Pour ton code je jetterai un oeil quand j'aurais 5min.

Celira · 13/12/2011, 11h18

Sans avoir épluché le code, bêtement je ferais un var_dump du radio pour voir ce qu'il ya dedans :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
$selected_radio = $_POST["option"];
var_dump($selected_radio);

gamiel · 13/12/2011, 14h25

le var_dump me retourne ceci:

string(0) "" avec une erreur mysql evidamment.

La valeur du radio button avec des guillemets est donc bien vide!

rawsrc · 13/12/2011, 18h18

Bonjour,

J'ai parcouru ton code et je dois avouer que c'est vachement confus :
- tu as une variable $result qui n'est jamais utilisée (ligne 20),
- tu fais un htmlentities() sur la variable $login (ligne 46) avant d'utiliser le résultat dans un WHERE (ligne 62),
- tu fais un mysql_connect() en plein milieu de ton code (ligne 49) alors qu'un mysql_query() a déjà été exécuté (ligne 18),
- le bloc entre les lignes 15 et 32 est incompréhensible,

Des remarques comme ça, il y en a encore à la pelle, cela serait sympa si tu veux être aidé de poster un code propre et surtout avec une logique qui puisse être saisie ou à défaut des commentaires qui l'expliquent

gamiel · 13/12/2011, 18h29

tu as tout a fait raison, a propos de la propreté du code.
Je suis tout nouveau dans le Php, donc ce qui est propre ou pas, je n'en ai pas la moindre idée.
Mais je suis ouvert aux critiques.

Le bloc de la ligne 15 a 32, récupere la classe de l'éleve, (TS1,TS2...TES, )
recupere une sous chaine qui est la premiere lettre de la classe
car il y a des 2ndes , des 1ere et des terminales.
d'ou le

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if($_SESSION['classename'] !="T")

qui correspond aux terminales dans ce cas.
A part ca, pour la variable result tu as raison, mais j'avais vu ca ailleurs, donc en attendant de m'ameliorer, jai pas tout chercher à comprendre.

Pour le reste des remarques, je sais pas trop quoi repondre voire quoi faire.

En tout cas, merci d'avoir pris le temps de lire.

Benjamin Delespierre · 13/12/2011, 18h47

Montre nous le code de construction du formulaire.

rawsrc · 13/12/2011, 18h49

Bon on va reprendre juste le début : (à gauche ce sont les n° de lignes) :

3 : démarrage de la session
4-7 : si la variable $_SESSION['username'] n'est pas définie, tu rediriges vers le script de DECONNEXION ! (logout.php)

12 : inclusion du fichier de configuration : tu configures quoi dedans ?
16-20 : code inutile car rien de ce qui est produit n'est utilisé par ailleurs
21 : Tu redéfinies $_SESSION['classename'] à partir de $_SESSION['classename']

Cela suppose que $_SESSION['classename'] a déjà été définie ailleurs
23 : si la nouvelle valeur de $_SESSION['classename'] diffère de 'T', tu arrêtes le script
37 : tu codes le rendu en charset=ISO-8859-1 alors que ta base semble être en utf-8, tu y gagnerais en harmonisant

Bon je vais m'arrêter et te conseiller de bouqiner un peu sous peine de te noyer très vite et de jeter l'éponge en moins de temps qu'il ne faut pour l'écrire.
Ce site regorge de supers tutos sur le PHP, vas-y plonges toi dedans : cours

Bon courage

gamiel · 13/12/2011, 19h35

Non mais attendez, mon site comporte plus de 30 pages, je ne vais pas toutes les poster!

Ce que tu m'as expliqué, merci beaucoup, mais c'est moi qui ai codé, donc je sais quand meme un minimum ce que j'ai codé.

Bien evidamment, les élèves doivent se connecter pour choisir leur matiere, donc la variable classname est définie dans la page de connexion!

Si la classe differe de T c'est a dire que ce n'est pas un Terminale, alors on le previent, comme spécifié dans le 1er code que j'ai posté.

Tu as raison pour le codage ISO.

bien evidamment que si le username n'est pas défini, c'est a dire que quelqu'un veut arriver à une page qui necessitait auparavant une authentification, alors il est deconnécté.

Bref, je sais ce que j'ai écrit merci, je ne sais pas ce qui est "propre" et ce qui ne l'est pas car je suis encore débutant.
Pas besoin de m'expliquer ligne par ligne

,
cependant je te remercie d'avoir lu et pris le temps de lire surtout.

gamiel · 13/12/2011, 19h36

Benjamin quel formulaire veux tu que je poste?

Benjamin Delespierre · 13/12/2011, 23h36

Celui qui produit le fameux bouton radio. Même s'il est vrai que ton code est assez confus, le problème ne doit pas venir de là (y'en à d'autres mais ça fera l'objet d'autres discussions).

gamiel · 14/12/2011, 21h08

Alors, la page insertionTerminale1.php correspond au formulaire d'insertion par l'admin de matiere dans la base.
c'est ces matieres que l'élève verra avec un radio button pour n'en choisir qu'une seule.

La page insertionTerminale2 ,c'est celle qui va les inserer dans la base voici le code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
<?php
 
	session_start();
	if (!isset($_SESSION['username'])) {
		header ('Location: logout.php');
		exit();
	}
	?>
<?php
 
	if ($_SESSION['username'] !='administrateur')
	{
		header ('Location: logout.php');
		exit();
	}
	?>
 
 
<html>
  <head>
    <title>insertion de mati&egrave;re</title>
  </head>
<body bgcolor="#CCFFFF"/> 
 
 <table height="100" width= "800" border="1" align="center">
   <tr>
 	 <center><th><font color="red" size=10> Confirmation de votre insertion de mati&egrave;re pour la classe de Terminale</font></th></center>
   </tr>
 
 </table>
 
 
<?php
  //connection au serveur
  $cnx = mysql_connect( "localhost", "x", "y" ) ;
 
  //sélection de la base de données:
  $db  = mysql_select_db( "accompagnementpersonnalise" ) ;
 
  //récupération des valeurs des champs:
  //nom:
  $nomMatiere= $_POST["NomMatiere"] ;
  //$nomMatiere=addslashes($nomMatiere);
  //effectif total:
  $effectifTotal = $_POST["EffectifTotal"] ;
  //effectif
  $effectif = $_POST["Effectif"];
  //numero de salle:
  $numSalle = $_POST["NumeroSalle"] ;
 
 
   if($nomMatiere == null) {
?>  	<script type="text/javascript">
    		alert("Veuillez entrer un nom pour la matiere" );
    		location.href = "insertionTerminales1.php";
   		 </script> 
 <?php }   
 
 else if($effectif > $effectifTotal) {
?>  	<script type="text/javascript">
    		alert("L'effectif total doit être supérieur ou égal au nombre de places disponibles " );
    		location.href = "insertionTerminales1.php";
   		 </script> 
 <?php }   
  else {
 
  //création de la requête SQL:
  $sql = "INSERT  INTO matiereterminales (IdMat, NomMatiere, EffectifTotal, Effectif, NumeroSalle)
            VALUES ( '','$nomMatiere', '$effectifTotal', '$effectif', '$numSalle') " ;
 }
  //exécution de la requête SQL:
  $requete = mysql_query($sql, $cnx) or die( mysql_error() ) ;
 
  //affichage des résultats, pour savoir si l'insertion a marchée:
   if($requete)
  {
    ?> <br><br> <center> <?php echo("L'insertion a &eacute;t&eacute; correctement effectu&eacute;e") ; ?> </center>
  <?php 
  }
  else
  {
   ?> <br><br> <center> <?php echo("L'insertion a echouee") ; ?> </center>
 <?php
  }
 
 
?>
 
<br><br><br>
        <center><a href="index(admin).php"><button>Retour au menu</button></a></center>
   <br>
	<center><a href="insertionTerminales1.php"><button> Retour &agrave; l'insertion de mati&egrave;res</button></a></center>
   <br>     
        <center><a href="logout.php"><button>Se d&eacute;connecter</button></a></center>
</html>

Benjamin Delespierre · 16/12/2011, 12h40

Pour t'éviter les problèmes possibles d'encodage de caractère (ce qui pourrait être la raison de la disparition de la valeur du bouton radio), pourquoi ne pas passer l'ID de la matière plutôt que son nom ?

gamiel · 18/12/2011, 01h08

J'ai pensé à cette idée, mais je n'y arrive pas, comment récuperer au final le nom de la matiere choisie pour faire un update de la table users et rajouter donc la matiere qu'il a choisi?
Car au final la valeur du radio button c'est L'id et plus le nom de la matiere

Cela se passe dans la pageRemerciementTerminale.php
ligne 101 à 103...

Merci de m'orienter!!

12/12/2011, 16h09	#2
Benjamin Delespierre Modérateur Benjamin Delespierre Développeur Web Inscription : février 2010 Messages : 2 991 Détails du profil Informations personnelles : Nom : Benjamin Delespierre Âge : 24 Localisation : France Informations professionnelles : Activité : Développeur Web Secteur : High Tech - Opérateur de télécommunications Informations forums : Inscription : février 2010 Messages : 2 991 Points : 5 031 Points : 5 031	Non, tu n'a pas essayé la solution qui fonctionne. Pour rappel, en HTML les valeurs des attributs sont notées entre doubles quotes Donc ta ligne doit être: Code : Sélectionner tout - Visualiser dans une fenêtre à part <td width="20"><input type="radio" name="option" value="<?php echo $ligne['NomMatiere'] ?>" /></td> __________________ A la recherche d'un framework MVC facile a prendre en main ? Essayez Axiom Nouveau: la référence d'Axiom est disponible sur GitHub (je la peaufine en ce moment même). Un problème correctement identifié est à moitié résolu, évitez de poster l'intégralité de votre code avec pour seule explication "ça ne marche pas...". Pour identifier correctement vos problèmes PHP, utilisez la gestion des erreurs et xdebug. Les boutons et existent, servez-vous en
	10

12/12/2011, 21h48	#5
Benjamin Delespierre Modérateur Benjamin Delespierre Développeur Web Inscription : février 2010 Messages : 2 991 Détails du profil Informations personnelles : Nom : Benjamin Delespierre Âge : 24 Localisation : France Informations professionnelles : Activité : Développeur Web Secteur : High Tech - Opérateur de télécommunications Informations forums : Inscription : février 2010 Messages : 2 991 Points : 5 031 Points : 5 031	non ça n'est pas normal. normalement les guillemets passent, vérifie ce que tu reçoit dans $_REQUEST. __________________ A la recherche d'un framework MVC facile a prendre en main ? Essayez Axiom Nouveau: la référence d'Axiom est disponible sur GitHub (je la peaufine en ce moment même). Un problème correctement identifié est à moitié résolu, évitez de poster l'intégralité de votre code avec pour seule explication "ça ne marche pas...". Pour identifier correctement vos problèmes PHP, utilisez la gestion des erreurs et xdebug. Les boutons et existent, servez-vous en
	00

13/12/2011, 10h11	#8
Benjamin Delespierre Modérateur Benjamin Delespierre Développeur Web Inscription : février 2010 Messages : 2 991 Détails du profil Informations personnelles : Nom : Benjamin Delespierre Âge : 24 Localisation : France Informations professionnelles : Activité : Développeur Web Secteur : High Tech - Opérateur de télécommunications Informations forums : Inscription : février 2010 Messages : 2 991 Points : 5 031 Points : 5 031	Ce n'est pas obligatoire si tu fermes la balise PHP juste derrière. Je serais toi je le mettrai quand même. Pour ton code je jetterai un oeil quand j'aurais 5min. __________________ A la recherche d'un framework MVC facile a prendre en main ? Essayez Axiom Nouveau: la référence d'Axiom est disponible sur GitHub (je la peaufine en ce moment même). Un problème correctement identifié est à moitié résolu, évitez de poster l'intégralité de votre code avec pour seule explication "ça ne marche pas...". Pour identifier correctement vos problèmes PHP, utilisez la gestion des erreurs et xdebug. Les boutons et existent, servez-vous en
	00

13/12/2011, 18h18	#11
rawsrc Expert Confirmé Martin Dev indep Inscription : mars 2004 Messages : 1 462 Détails du profil Informations personnelles : Nom : Martin Âge : 35 Localisation : France, Bouches du Rhône (Provence Alpes Côte d'Azur) Informations professionnelles : Activité : Dev indep Informations forums : Inscription : mars 2004 Messages : 1 462 Points : 2 552 Points : 2 552	Bonjour, J'ai parcouru ton code et je dois avouer que c'est vachement confus : - tu as une variable $result qui n'est jamais utilisée (ligne 20), - tu fais un htmlentities() sur la variable $login (ligne 46) avant d'utiliser le résultat dans un WHERE (ligne 62), - tu fais un mysql_connect() en plein milieu de ton code (ligne 49) alors qu'un mysql_query() a déjà été exécuté (ligne 18), - le bloc entre les lignes 15 et 32 est incompréhensible, Des remarques comme ça, il y en a encore à la pelle, cela serait sympa si tu veux être aidé de poster un code propre et surtout avec une logique qui puisse être saisie ou à défaut des commentaires qui l'expliquent __________________ # Dans la Création, tout est permis mais tout n'est pas utile...
	00

13/12/2011, 18h29	#12
gamiel Invité régulier Gam Inscription : avril 2010 Messages : 27 Détails du profil Informations personnelles : Nom : Gam Localisation : France Informations forums : Inscription : avril 2010 Messages : 27 Points : 5 Points : 5	tu as tout a fait raison, a propos de la propreté du code. Je suis tout nouveau dans le Php, donc ce qui est propre ou pas, je n'en ai pas la moindre idée. Mais je suis ouvert aux critiques. Le bloc de la ligne 15 a 32, récupere la classe de l'éleve, (TS1,TS2...TES, ) recupere une sous chaine qui est la premiere lettre de la classe car il y a des 2ndes , des 1ere et des terminales. d'ou le Code : Sélectionner tout - Visualiser dans une fenêtre à part if($_SESSION['classename'] !="T") qui correspond aux terminales dans ce cas. A part ca, pour la variable result tu as raison, mais j'avais vu ca ailleurs, donc en attendant de m'ameliorer, jai pas tout chercher à comprendre. Pour le reste des remarques, je sais pas trop quoi repondre voire quoi faire. En tout cas, merci d'avoir pris le temps de lire.
	00

12/12/2011, 21h01	#3
gamiel Invité régulier Gam Inscription : avril 2010 Messages : 27 Détails du profil Informations personnelles : Nom : Gam Localisation : France Informations forums : Inscription : avril 2010 Messages : 27 Points : 5 Points : 5	Un énorme merci, ca marche... tu es parfait... j'en reviens toujours pas...
	00

12/12/2011, 21h35	#4
gamiel Invité régulier Gam Inscription : avril 2010 Messages : 27 Détails du profil Informations personnelles : Nom : Gam Localisation : France Informations forums : Inscription : avril 2010 Messages : 27 Points : 5 Points : 5	Est-ce normal que si la matière s'appelle par exemple: "a nous pa'ris" avec des guillemets, la valeur du radio button devient nulle?
	00

13/12/2011, 00h45	#7
clovis802 Invité régulier Chercheur en informatique Inscription : décembre 2011 Messages : 13 Détails du profil Informations personnelles : Sexe : Localisation : France, Bas Rhin (Alsace) Informations professionnelles : Activité : Chercheur en informatique Secteur : Conseil Informations forums : Inscription : décembre 2011 Messages : 13 Points : 9 Points : 9	On ne place plus de point virgule après l'include() ?
	00

13/12/2011, 14h25	#10
gamiel Invité régulier Gam Inscription : avril 2010 Messages : 27 Détails du profil Informations personnelles : Nom : Gam Localisation : France Informations forums : Inscription : avril 2010 Messages : 27 Points : 5 Points : 5	le var_dump me retourne ceci: string(0) "" avec une erreur mysql evidamment. La valeur du radio button avec des guillemets est donc bien vide!
	00

13/12/2011, 18h47	#13
Benjamin Delespierre Modérateur Benjamin Delespierre Développeur Web Inscription : février 2010 Messages : 2 991 Détails du profil Informations personnelles : Nom : Benjamin Delespierre Âge : 24 Localisation : France Informations professionnelles : Activité : Développeur Web Secteur : High Tech - Opérateur de télécommunications Informations forums : Inscription : février 2010 Messages : 2 991 Points : 5 031 Points : 5 031	Montre nous le code de construction du formulaire. __________________ A la recherche d'un framework MVC facile a prendre en main ? Essayez Axiom Nouveau: la référence d'Axiom est disponible sur GitHub (je la peaufine en ce moment même). Un problème correctement identifié est à moitié résolu, évitez de poster l'intégralité de votre code avec pour seule explication "ça ne marche pas...". Pour identifier correctement vos problèmes PHP, utilisez la gestion des erreurs et xdebug. Les boutons et existent, servez-vous en
	00

13/12/2011, 18h49	#14
rawsrc Expert Confirmé Martin Dev indep Inscription : mars 2004 Messages : 1 462 Détails du profil Informations personnelles : Nom : Martin Âge : 35 Localisation : France, Bouches du Rhône (Provence Alpes Côte d'Azur) Informations professionnelles : Activité : Dev indep Informations forums : Inscription : mars 2004 Messages : 1 462 Points : 2 552 Points : 2 552	Bon on va reprendre juste le début : (à gauche ce sont les n° de lignes) : 3 : démarrage de la session 4-7 : si la variable $_SESSION['username'] n'est pas définie, tu rediriges vers le script de DECONNEXION ! (logout.php) 12 : inclusion du fichier de configuration : tu configures quoi dedans ? 16-20 : code inutile car rien de ce qui est produit n'est utilisé par ailleurs 21 : Tu redéfinies $_SESSION['classename'] à partir de $_SESSION['classename'] Cela suppose que $_SESSION['classename'] a déjà été définie ailleurs 23 : si la nouvelle valeur de $_SESSION['classename'] diffère de 'T', tu arrêtes le script 37 : tu codes le rendu en charset=ISO-8859-1 alors que ta base semble être en utf-8, tu y gagnerais en harmonisant Bon je vais m'arrêter et te conseiller de bouqiner un peu sous peine de te noyer très vite et de jeter l'éponge en moins de temps qu'il ne faut pour l'écrire. Ce site regorge de supers tutos sur le PHP, vas-y plonges toi dedans : cours Bon courage __________________ # Dans la Création, tout est permis mais tout n'est pas utile...
	00

13/12/2011, 19h35	#15
gamiel Invité régulier Gam Inscription : avril 2010 Messages : 27 Détails du profil Informations personnelles : Nom : Gam Localisation : France Informations forums : Inscription : avril 2010 Messages : 27 Points : 5 Points : 5	Non mais attendez, mon site comporte plus de 30 pages, je ne vais pas toutes les poster! Ce que tu m'as expliqué, merci beaucoup, mais c'est moi qui ai codé, donc je sais quand meme un minimum ce que j'ai codé. Bien evidamment, les élèves doivent se connecter pour choisir leur matiere, donc la variable classname est définie dans la page de connexion! Si la classe differe de T c'est a dire que ce n'est pas un Terminale, alors on le previent, comme spécifié dans le 1er code que j'ai posté. Tu as raison pour le codage ISO. bien evidamment que si le username n'est pas défini, c'est a dire que quelqu'un veut arriver à une page qui necessitait auparavant une authentification, alors il est deconnécté. Bref, je sais ce que j'ai écrit merci, je ne sais pas ce qui est "propre" et ce qui ne l'est pas car je suis encore débutant. Pas besoin de m'expliquer ligne par ligne , cependant je te remercie d'avoir lu et pris le temps de lire surtout.
	00

13/12/2011, 19h36	#16
gamiel Invité régulier Gam Inscription : avril 2010 Messages : 27 Détails du profil Informations personnelles : Nom : Gam Localisation : France Informations forums : Inscription : avril 2010 Messages : 27 Points : 5 Points : 5	Benjamin quel formulaire veux tu que je poste?
	00

13/12/2011, 23h36	#17
Benjamin Delespierre Modérateur Benjamin Delespierre Développeur Web Inscription : février 2010 Messages : 2 991 Détails du profil Informations personnelles : Nom : Benjamin Delespierre Âge : 24 Localisation : France Informations professionnelles : Activité : Développeur Web Secteur : High Tech - Opérateur de télécommunications Informations forums : Inscription : février 2010 Messages : 2 991 Points : 5 031 Points : 5 031	Celui qui produit le fameux bouton radio. Même s'il est vrai que ton code est assez confus, le problème ne doit pas venir de là (y'en à d'autres mais ça fera l'objet d'autres discussions). __________________ A la recherche d'un framework MVC facile a prendre en main ? Essayez Axiom Nouveau: la référence d'Axiom est disponible sur GitHub (je la peaufine en ce moment même). Un problème correctement identifié est à moitié résolu, évitez de poster l'intégralité de votre code avec pour seule explication "ça ne marche pas...". Pour identifier correctement vos problèmes PHP, utilisez la gestion des erreurs et xdebug. Les boutons et existent, servez-vous en
	00

16/12/2011, 12h40	#19
Benjamin Delespierre Modérateur Benjamin Delespierre Développeur Web Inscription : février 2010 Messages : 2 991 Détails du profil Informations personnelles : Nom : Benjamin Delespierre Âge : 24 Localisation : France Informations professionnelles : Activité : Développeur Web Secteur : High Tech - Opérateur de télécommunications Informations forums : Inscription : février 2010 Messages : 2 991 Points : 5 031 Points : 5 031	Pour t'éviter les problèmes possibles d'encodage de caractère (ce qui pourrait être la raison de la disparition de la valeur du bouton radio), pourquoi ne pas passer l'ID de la matière plutôt que son nom ? __________________ A la recherche d'un framework MVC facile a prendre en main ? Essayez Axiom Nouveau: la référence d'Axiom est disponible sur GitHub (je la peaufine en ce moment même). Un problème correctement identifié est à moitié résolu, évitez de poster l'intégralité de votre code avec pour seule explication "ça ne marche pas...". Pour identifier correctement vos problèmes PHP, utilisez la gestion des erreurs et xdebug. Les boutons et existent, servez-vous en
	00

18/12/2011, 01h08	#20
gamiel Invité régulier Gam Inscription : avril 2010 Messages : 27 Détails du profil Informations personnelles : Nom : Gam Localisation : France Informations forums : Inscription : avril 2010 Messages : 27 Points : 5 Points : 5	J'ai pensé à cette idée, mais je n'y arrive pas, comment récuperer au final le nom de la matiere choisie pour faire un update de la table users et rajouter donc la matiere qu'il a choisi? Car au final la valeur du radio button c'est L'id et plus le nom de la matiere Cela se passe dans la pageRemerciementTerminale.php ligne 101 à 103... Merci de m'orienter!!
	00

Outils de la discussion
Afficher une version imprimable Envoyer un lien vers cette page par email