Discussion :

[kamallll]

Bonjour,

Je suis débutant dans javascript, et je voulais demander votre aide, j'ai une liste déroulante que je remplis à partir d'une base de données et une autre page ou j'affiche un tableau de données récupérés de la base de données.

Ma question:

là j'ai mis le select avec le onchange mais j'ai un soucis au niveau de la fonction du js et j'aimerai votre avis dessus.

Merci d'avance.


*) Mon select :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 <select  id="numBC" name="numBC" onchange="changementType();">
      <option value="0"> ---- Choisissez un bon de commande ----</option>
	<?php
           // Je récupère le contenu du select :
		$bdd=connectBD();	
		$req = $bdd->query('SELECT  DISTINCT(numBC) FROM commandefournisseur ');							
		while ($donnees = $req->fetch()){
				$numBC=$donnees['numBC'];
	?>							
		<option value="<?php echo $numBC; ?>"> <?php echo $numBC; ?></option>
	<?php 
		}
	$req->closeCursor();
	?>
</select>

*) Ma fonction JS :

Code :

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

1
2
3
4
5
6
7
8
9
function changementType() { 
	var numBC = document.getElementById("numBC"); 
	var div = document.getElementById("detailComm"); 
		if (numBC != 0) { 
			div.style="display:block"; 
		} else { 
			div.style="display:none"; 
		} 
}

*) Ma div qui contient le tableau a afficher :

Code html :

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

1
2
3
4
5
6
7
<div  id="detailComm" style="display:none" class="table-responsive">
<table class="table table-striped table-bordered">
..
.. (je fais des traitement de base de donnees en utilisant le $numBC récupere ds le select)
..
</table>
</div>

[Bovino]

Code :

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

var numBC = document.getElementById("numBC");

Là, ta variable numBC référence un objet DOM de type HTMLSelectElement.

Code :

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

if (numBC != 0)

donc numBC ne peut jamais être égal à 0 ! C'est la propriété value de l'objet qu'il faut tester...

D'autre part

Code :

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

div.style="display:block";

Cette façon de faire, sans pour autant être réellement fausse, est malgré tout maladroite. Il est préférable de modifier les propriétés de l'objet style.

Code :

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

div.style.display = "block";

[kamallll]

j'ai corrigé la seconde remarque, mais pour la première je n'ai pas de contrôle sur la valeur de numBC, elle est remplie de la base de données, donc dans mon javascript je veux qu'elle affiche le tableau de données dans la div en fonction de la valeur du numBC, par exemple elle affichera tous les détails de la commande qui aura le numBC = AB123 sachant que l'utilisateur a sélectionné AB123 dans la liste déroulante. chaque fois que la valeur selectionné dans la liste déroulante va changer, les informations que contiendra le tableau seront differentes (vu que je les récupere via une requete sql)

et le 0 c'est la valeur de ce choix là : <option value="0"> ---- Choisissez un bon de commande ----</option>

(et merci pour votre réponse)

[Bovino]

mais pour la première je n'ai pas de contrôle sur la valeur de numBC

Je crois que tu n'as pas compris ce que j'ai dit...

Je ne te dis pas de modifier cette variable, je te dis juste que ce qu'elle contient est un objet DOM, qui ne peut donc jamais valoir une valeur numérique, contrairement à la propriété value de cet objet !

[Spartacusply]

Pour faire une métaphore :

Code :

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

1
2
if (personne == 72) //Une personne = 72 ? Ca ne veut rien dire.
if (personne.poids == 72) //Alors que ceci est parfaitement compréhensible, si la personne pèse 72 kilos...

C'est idem avec ton formulaire, si numBC != 0, comprends pas, par contre si numBC.value != 0 c'est quand même beaucoup mieux.