Bug si je déclare des variables inutilisées

**covin85** · 04/06/2009, 22h31

Bonjour, Le contexte est le suivant :
J'utilise en php une boucle "while" afin d'afficher un certain nombre de menu (avec toggle).

Je n'y connais rien en Java.
Le nombre de menu est donc indéfini et peut varier entre 0 et 10.

J'ai réussi à modifier le script java afin d'afficher les plusieurs menus.
Le problème c'est que je déclare en java des variables (donc 10) qui ne sont pas toutes utilisées.

Et dans ce cas les menus toggle ne fonctionne pas.

J'ai une variable sur ma page PHP qui m'indique le nombre de menu à afficher.
Mais je n'arrive pas à la récupéré dans le script Java.

La page PHP :

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
 
<head>
	<script type="text/javascript" src="mootools.js"></script>
	<script type="text/javascript" src="demo.js"></script>
</head>
 
<?
echo "<strong>Questions : </strong><br /><br />";
$reqcomi2q = mysql_query("SELECT * FROM qr WHERE titre = '".$data['titre']."' && qr = 1");   	
while ($myrowq = mysql_fetch_array($reqcomi2q)) {
                $jing = $jing+1 ;               /* $jing indique le nombre de menu */
			    $jong = "v_toggle$jing";
				$jung = $jung+1 ;
				$jang = "vertical_slide$jung";
	echo '<div class="marginbottom">';
		echo "<a id='$jong' href='#'>".$myrowq["texte"]."</a>";
	echo '</div>';
	echo "<div id='$jang'>";
		echo $myrowq["pres"];
		echo '</div>';
		        $jung = $jung+1 ;
				$jang = "vertical_slide$jung";
	echo "<div id='$jang'>";
		echo 'REPONSSE';
	echo '</div><br>';
	}
?>

Le script Java (extrait) demo.js :

Code :

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

1
2
3
4
 
var myVerticalSlide1 = new Fx.Slide('vertical_slide1').hide();
...
var myVerticalSlide10 = new Fx.Slide('vertical_slide10').hide();

Avez vous une idée?

**SpaceFrog** · 05/06/2009, 08h28

Je n'y connais rien en Java.

ça tombe bien moi non plus, masi c'est une chance ton code est en javascript ...

pas toutes utilisées ?
faudrait eclaircir un peu ce point ...

tu parles de cette partie là ?

Code :

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

1
2
3
var myVerticalSlide1 = new Fx.Slide('vertical_slide1').hide();
...
var myVerticalSlide10 = new Fx.Slide('vertical_slide10').hide();

pourquoi declarer 10 menu si tu n'en utilise que 6 ?
fais une boucle qui n'en declare que 6
sinon il va falloir aller bidouiller dans motools pour rajouter un test d'existence ...

**covin85** · 05/06/2009, 09h01

Envoyé par SpaceFrog

pourquoi declarer 10 menu si tu n'en utilise que 6 ?
fais une boucle qui n'en declare que 6

C'est justement ce que j'aimerais faire, en utilisant la variable $jing qui indique le nombre de menu.

Mais je ne sais pas comment récupérer cette variable qui est dans un while PHP et l'intégré dans le JAva.

Et je ne suis même pas sur que cela fonctionne

**k1rby** · 05/06/2009, 09h10

Dans javaSCRIPT

Code PHP :

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

1
2
3
4
5
 
<?php $nbMenus=10; ?>
<script type="texte/javascript">
var nbMenus=<?php echo nbMenus; ?>;
</script>

**Federico_muy_bien** · 05/06/2009, 09h15

Je suis débutant

et je fais souvent des trucs compliqués pour pas grand chose mais bon je me lance ...

Dans le PHP tu pourrais faire un

Code php :

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

1
2
 
echo "<input type='hidden' value='".$nbre_menu."' id='nbre' />" ;

Puis dans demo.js au moment du chargement de la page, tu prend
nbre = document.GetElementByID('nbre') ;

et avec ça une boucle où tu déclare le nombre de menus necessaires ...

**andry.aime** · 05/06/2009, 09h33

Envoyé par Federico_muy_bien

document.GetElementByID('nbre')

Code :

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

document.getElementById('nbre')

**covin85** · 05/06/2009, 12h12

$jing était une variable dans le while, je les remplacer par la variable $datanb :

Code :

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

1
2
3
 
$reqcomi2q = mysql_query("SELECT * FROM qr WHERE titre = '".$data['titre']."' && qr = 1"); 
$datanb = mysql_num_rows($reqcomi2q);

Au début du while de ma page php j'ai donc ajouter :

Code :

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

1
2
 
echo "<input type='hidden' value='".$datanb."' id='nbre' />" ;

Puis sur demo.js :

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
 
nbre = document.getElementById('nbre')
 
    if(nbre == 1)
	{
	var myVerticalSlide1 = new Fx.Slide('vertical_slide1').hide();
	var myVerticalSlide2 = new Fx.Slide('vertical_slide2').hide();
	}
 
    if(nbre == 2)
	{
	var myVerticalSlide1 = new Fx.Slide('vertical_slide1').hide();
	var myVerticalSlide2 = new Fx.Slide('vertical_slide2').hide();
    var myVerticalSlide3 = new Fx.Slide('vertical_slide3').hide();
	var myVerticalSlide4 = new Fx.Slide('vertical_slide4').hide();
	}
...

Mais ça Bug toujours (l'affichage n'est pas correcte) .

J'ai donc fait plusieurs test, $datanb est une valeur correcte et fixe, j'ai tenté de déplacé ce code dans le while après la déclaration de la variable :

Code :

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

1
2
3
 
<script type="text/javascript" src="mootools.js"></script>
<script type="text/javascript" src="demo.js"></script>

Mais toujours rien, je pense que dans la page demo.js, le if n'est pas exécuté.

**covin85** · 07/06/2009, 17h52

Personne n'a de solution?

**Pikrass** · 07/06/2009, 18h12

Remplace cette ligne :

Code :

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

nbre = document.getElementById('nbre')

Par celle-ci :

Code :

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

nbre = document.getElementById('nbre').getAttribute('value');

Ensuite, tu te compliques un peu la vie non ? A la place d'utiliser une variable pour chaque Slide, tu ne peux pas les mettre dans un array ?

Code :

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

1
2
3
4
var myVerticalSlides = new Array();
for(a=0 ; a<nbre ; a++) {
    myVerticalSlides[a] = new Fx.Slide('vertical_slide'+(a+1)).hide();
}

**covin85** · 08/06/2009, 17h10

Merci cela fonctionne

Merci

**SpaceFrog** · 09/06/2009, 08h31

Code :

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

nbre = document.getElementById('nbre').value;

sera plus universel

Bug si je déclare des variables inutilisées

JavaScript

Discussions similaires

Partager

Partager