Discussion :

[inter-rupteur]

Bonjour à tous,

voilà, j'ai écrit ce bout de code et j'aurais aimé savoir si c'est correct ou si y a moyen de l'optimiser svp !!!

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
 
 
 $SELECsqL0 = "SELECT * FROM testA where ma_bd = 5";
 $SELECsqL1 = "SELECT * FROM testB where mb_jt = '$VAL_CO'";
 
foreach ($CONNpdo1->query($SELECsqL0) as $row)
          {
$incremrow = $row['user_jt'];			
echo "
  <div class=\"niveau\" id=\"COGE0\">
	<span class=\"plus\" id=\"COVDEPn".$incremrow."\" onClick=\"nivoplus('COVDEP','$incremrow');return(false)\">+</span>
	<a href=\"\">$row[ma_lib]</a><br />
	<ul class=\"niveau\" id=\"COVDEP".$incremrow."\">";
 foreach ($CONNpdo1->query($SELECsqL1) as $row)
    {
	echo "<img src=\"pics/archi/h1.png\" width=\"14\" height=\"19\" align=\"middle\"/> $row[nom] $row[prenom]<br />";
    }
echo "</ul></div>\n";
           }	
 
$CONNpdo1 = null;

merci à vous pour votre aide !!

Inter-Rupteur

[sabotage]

Repeter X fois la même requête est evidemment mauvais.
Si tu as besoin d'afficher X fois le résultat de $SELECsqL1, il faut executer une fois cette requete, construire et stocker ce que tu veux afficher et l'afficher ensuite X fois.

[inter-rupteur]

tu ferais cela comment car je comprends le principe, mais pour le mettre en oeuvre, c'est une autre histoire...

[ThomasR]

Bonjour,

Comme ceci tout simplement :

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
 
 
 
 $SELECsqL0 = "SELECT * FROM testA where ma_bd = 5";
 $SELECsqL1 = "SELECT * FROM testB where mb_jt = '$VAL_CO'";
 $rows1 = $CONNpdo1->query($SELECsqL0);
 $rows2 = $CONNpdo1->query($SELECsqL1);
foreach ($rows1 as $row1){
    $incremrow = $row['user_jt'];			
    echo "
        <div class=\"niveau\" id=\"COGE0\">
    	<span class=\"plus\" id=\"COVDEPn".$incremrow."\"     onClick=\"nivoplus('COVDEP','$incremrow');return(false)\">+</span>
	<a href=\"\">$row1[ma_lib]</a><br />
	<ul class=\"niveau\" id=\"COVDEP".$incremrow."\">";
    foreach ($rows2 as $row2){
        // etc
    }
}

De plus, les deux boucles ne devrait pas utiliser la même variable $row.

[inter-rupteur]

Merci ThomasR, j'étais pas trop loin !!

[inter-rupteur]

j'ai un souci de nouveau... J'essaye de mettre en place une boucle imbriquée dans le 2eme Foreach, mais ca passe pas... J'ai testé avec un foreach, for et un while, le résultat est tjrs le même (voir ci-dessous : résultat normal) !!!

Je remet le code avec le 2eme foreach :

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
 
$SELECsqL0 = "SELECT * FROM testA where ma_bd = 5";
$SELECsqL1 = "SELECT * FROM testB where mb_jt = '$VAL_CO'";
 
 $rows1 = $CONNpdo1->query($SELECsqL0);
 $rows2 = $CONNpdo1->query($SELECsqL1);
 
foreach ($rows1 as $row1){
    $incremrow = $row['user_jt'];			
    echo "
        <div class=\"niveau\" id=\"COGE0\">
    	<span class=\"plus\" id=\"COVDEPn".$incremrow."\"     onClick=\"nivoplus('COVDEP','$incremrow');return(false)\">+</span>
	<a href=\"\">$row1[ma_lib]</a><br />
	<ul class=\"niveau\" id=\"COVDEP".$incremrow."\">";
 
foreach ($rows2 as $row2)
           {		
   echo "<img src=\"pics/archi/h1.png\" width=\"14\" height=\"19\" align=\"middle\"/>$row[nom] $row[prenom]<br />";
	}				
 
 echo "</ul>";
         }

voici le résultat normal :



le probleme est que test1 doit récupérer que les AAAA-3 et test2 que les IIII-4, etc... J'ai testé avec un 3eme foreach imbriqué dans le 2eme foreach, et le résultat était le même... Idem pour un for et while !! Donc là je séche...

voici 2 tables de ma base de données ou se trouve user_jt (la jointure entre les deux tables) que je n'arrive pas à utiliser afin de classer les datas en fonction de test1, test2, etc... :

table testA


table testB


J'ai aussi un fichier var-test.php avec des valeurs déclarées :

Code :

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

1
2
3
4
5
 
$test  = 2;
$test1 = 3;
$test2 = 4;
etc...

J'espère que j'ai été kler...

Merci à vous
Inter-Rupteur

[sabotage]

Dans un cas comme ça, je prefere une jointure a des requetes imbriqués :

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
 
$SELECsqL0 = "SELECT * FROM testA 
JOIN testB using (user_jt) 
WHERE ma_bd = 5
AND mb_jt = '$VAL_CO'
ORDER BY mlt_id";
 
 $rows1 = $CONNpdo1->query($SELECsqL0);
 
$current_mlt_id = 0;
foreach ($rows1 as $row){
    $incremrow = $row['user_jt'];			
    if ($row['mlt_id'] != $current_mlt_id) {
       echo "
        <div class=\"niveau\" id=\"COGE0\">
    	<span class=\"plus\" id=\"COVDEPn".$incremrow."\"     onClick=\"nivoplus('COVDEP','$incremrow');return(false)\">+</span>
	<a href=\"\">$row[ma_lib]</a><br />
	<ul class=\"niveau\" id=\"COVDEP".$incremrow."\">";
 
        $current_mlt_id = $row['mlt_id'];
    }
 
   echo "<img src=\"pics/archi/h1.png\" width=\"14\" height=\"19\" align=\"middle\"/>$row[nom] $row[prenom]<br />";
 
 
 echo "</ul>";
}

[inter-rupteur]

Bonjour Sabotage,

et merci pour ta réponse !! J'avais pensé à une solution de ce type auparavant et je l'ai abondonné car le résultat était complétement différent. Je viens de tester la tienne, et on s'y rapproche un peu plus malgré encore un souci. Voilà le résultat :






Une partie des datas se retrouvent en dehors...

Merci à toi
Inter-Rupteur

[sabotage]

Il faut que tu places les elements HTML au bon endroit, j'ai juste repris ce que tu avais écris sans l'adapter.

[inter-rupteur]

J'ai effectué les modifications aux HTML comme convenu !
Par contre, il a fallu que je rajoute une condition dans le 1er If pour gérer les fins de </ul>, sinon j'avais un espace entre Test (parent) et les autres test1, etc...

Code :

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

1
2
3
4
5
6
7
8
 
if (($row['user_jt']-1) == $ul_jt)
	  {  
	  }
	  else
	  {
		echo "</ul>";
	  }

Donc là ca tourne !! J'en ai plusieurs comme ceux là à faire mais avec des arbos plus profondes !! J'espère que le modéle sera portable !!
Je te tiendrai au courant !!

Merci beaucoup Sabotage !

Inter-rupteur