Erreur de syntaxe sur ma requête SELECT

Version imprimable

Bonjour, j'ai 1 petit soucis avec des noms de fichiers contenu dans 1 table à afficher dans 1 select.

voici l'erreur:

Citation:

Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in C:\wamp\www\rbdc\traitementmenu.php on line 210

et mon code:
Code:

1 2 3 4 5 6 7 8 9 10 11 <?php $sqlfichiers = "select nomFichierj from fichiersj where numFiche = '${datamenu['numFiche']}'"; $reqfichiers = mysql_query($sqlfichiers) or die("Erreur : ". $sqlfichiers .'<br/>'. mysql_error()); while($datafichiers = mysql_fetch_row($reqfichiers)) { echo "<option>$datafichiers['nomFichierj']</option>"; } ?> </select>
Il doit donc afficher tout les noms de fichier "1 champ" dans le select les uns a la suite des autres.

08/03/2006, 10h49
Yogui

Salut

Je n'arrête pas de militer contre les guillemets doubles ^^ :

Code:

echo '<option>'.$datafichiers['nomFichierj'].'</option>';
08/03/2006, 11h01
vincedjs

Merci, il m'affiche pu d'erreur par contre, mon select est vide, mais quand je clic dessu il contient bien le nombre de champs correspondants dans la table. mais vide, sans espace, sans longueur lié au champ, "rien"
08/03/2006, 11h04
XtofRoland

je copie colle tjrs ce code...

mais n'est ce pas mysql_fetch_into ou mysql_fetch_array (tableau et tableau associatif) ?
08/03/2006, 11h08
Yogui

@XtofRoland : Une bonne fois pour toutes, lis le manuel sur chacune des fonctions mysql_fetch_...(), tu comprendras que chacune a son utilité mais que toutes permettent de récupérer les tuples. :D
Si notre ami préfère utiliser row (tableau indexé numériquement) plutôt qu'array ou assoc (tableaux associatifs), c'est son problème même si j'ai tendance à préférer assoc ^^

@vincedjs : Donne-nous la requête exacte stp.
08/03/2006, 11h12
XtofRoland

$datafichiers = mysql_fetch_row($reqfichiers))
si cela retourne un tableau indexe numeriquement ...

echo "<option>$datafichiers['nomFichierj']</option
intuitivement je dirait que les recuperer avec un nom c'est pas une bonne idée...

mais sans certitude...j'ai pas approfondit la :?:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
					<select name='la_valeur'> 
					<?php
					$sqlfichiers = "select nomFichierj 
								  from fichiersj 
								  where numFiche = '${datamenu['numFiche']}'";
					$reqfichiers = mysql_query($sqlfichiers) or die("Erreur : ". $sqlfichiers .'<br/>'. mysql_error());
					while($datafichiers = mysql_fetch_row($reqfichiers))
						{
						echo '<option>'.$datafichiers['nomFichierj'].'</option>';
						}
					?>
					</select>

la requete est donc

Code:

1
2
3
4
 
$sqlfichiers = "select nomFichierj 
	      from fichiersj 
	      where numFiche = '${datamenu['numFiche']}'";

08/03/2006, 11h13
Sion_Sempai
Citation:

Envoyé par Kirkis

Salut

Je n'arrête pas de militer contre les guillemets doubles ^^ :

Code:

echo '<option>'.$datafichiers['nomFichierj'].'</option>';

moi ce pour quoi je milite c'est les variables directement dans les chaines ^^'
Code:

1 2 3 $sqlfichiers = "select nomFichierj from fichiersj where numFiche = '".${datamenu['numFiche']}."'";
c'est quand même mieux et, je trouve, plus lisible, de sortir les variables :P
Et puis, ca evite souvent bien des problèmes ^^
08/03/2006, 11h17
Yogui
@Sion_Sempai : nous sommes à peu près d'accord mais préfère ce type de code :
Code:

1 2 3 $sqlfichiers = 'select nomFichierj from fichiersj where numFiche = "'.$datamenu['numFiche']."';
Ainsi, tu évites à PHP du travail inutile et tu ne perds ni ne gagnes rien en lisibilité.

@vincedjs : Merci, je sais lire. Je te demande la requête que MySQL doit exécuter, pas ton code PHP.
Place echo devant $sqlfichiers =
08/03/2006, 11h21
Sion_Sempai
Citation:
Envoyé par Kirkis

@Sion_Sempai : nous sommes à peu près d'accord mais préfère ce type de code :

Code:

1 2 3 $sqlfichiers = 'select nomFichierj from fichiersj where numFiche = "'.${datamenu['numFiche']}."';

Ainsi, tu évites à PHP du travail inutile et tu ne perds ni ne gagnes rien en lisibilité.
manque une quote :P

Code:

where numFiche = "'.${datamenu['numFiche']}.'"';

mais pourquoi dis-tu que ca evite du travail pour php ?
08/03/2006, 11h27
Yogui
Surtout que nous sommes sortis de l'intérieur des guillemets doubles, ce qui fait que les accolades sont devenues inutiles :
Code:

1 2 3 $sqlfichiers = 'select nomFichierj from fichiersj where numFiche = "'.$datamenu['numFiche'].'"';
Sinon : http://frederic.bouchery.free.fr/?2004/07/15/2-Les-Chaines-De-Caracteres-Soyons-Coherents
08/03/2006, 11h32
Sion_Sempai

effectivement... Holala, trop de petits détails d'un coup la, j'suis fatigué !! :lol:

Merci pour le lien ^^
08/03/2006, 11h43
vincedjs

ne vous battez pas pour 1 guillemet :lol:

voile la requete:

Code:

select nomFichierj from fichiersj where numFiche = "81"

Ce qui correspond bien a ma bdd
08/03/2006, 11h46
Yogui
Très bien.

Nous en venons maintenant à la très juste remarque de XtofRoland :

Citation:

Envoyé par XtofRoland

$datafichiers = mysql_fetch_row($reqfichiers))
si cela retourne un tableau indexe numeriquement ...

echo "<option>$datafichiers['nomFichierj']</option
intuitivement je dirait que les recuperer avec un nom c'est pas une bonne idée...

mais sans certitude...j'ai pas approfondit la :?:
- Soit tu utilises mysql_fetch_row($reqfichiers) avec $datafichiers[0]
- Soit tu utilises mysql_fetch_assoc($reqfichiers) avec $datafichiers['nomFichierj']

Exact, c'est bon a savoir :wink:
merci beaucoup

code final:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
<select name='la_valeur'> 
					<?php
					 $sqlfichiers = 'select nomFichierj 
                          from fichiersj 
                          where numFiche = "'.$datamenu['numFiche'].'"';
					$reqfichiers = mysql_query($sqlfichiers) or die("Erreur : ". $sqlfichiers .'<br/>'. mysql_error());
					while($datafichiers = mysql_fetch_assoc($reqfichiers))
						{
						echo '<option>'.$datafichiers['nomFichierj'].'</option>';
						}
					?>
					</select>