Discussion :

[Devilju69]

Bonjour je voudrais améliorer mon code car j'utilise un switch case et je pense que l'on peut regrouper les deux sans problème car quand je dois faire un changement je dois le faire deux fois !!
Quelle est la solution ??

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
<?php
switch ($_GET['choix'])
{
case '1' :
// La requête sql pour récupérer les messages de la page actuelle.
$retour_messages=mysql_query('SELECT * FROM gamme ');
 
 
while($donnees_messages=mysql_fetch_assoc($retour_messages)) // On lit les entrées une à une grâce à une boucle
{
$libelle = $donnees_messages['libelle_gamme'];
 
//Affichage du contenu
echo '<a href="produits.php?id=' . $libelle . '&choix=1"><img src="Images/img_site/'.stripslashes($donnees_messages['photo_gamme']).'.gif" width="680"></a><br /><br />
'; 
}
break;
//---------------------------------------------------------------------------------------------------------------------------------// 
case '2' :
// La requête sql pour récupérer les messages de la page actuelle.
$retour_messages=mysql_query('SELECT * FROM famille');
 
while($donnees_messages=mysql_fetch_assoc($retour_messages)) // On lit les entrées une à une grâce à une boucle
{
$libelle = $donnees_messages['libelle_famille'];
 
//Affichage du contenu
echo '<table border="1">
<tr align="center">
<td width="200"><a href="produits.php?id=' . $libelle . '&choix=2">'.stripslashes($donnees_messages['libelle_famille']).'</a></td>
<td width="200"><a href="produits.php?id=' . $libelle . '&choix=2"><img src="Images/produit/'.stripslashes($donnees_messages['photo_famille']).'.gif" width="35px"></a></td>
</tr>'; 
}
echo '</table>';
break;
}
 
include 'footer.php';
?>

[Kennel sébastien]

Salut,

Est ce que cela irai ? :

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
 
<?php
 
$choix = (int) $_GET['choix'];
 
switch ($choix) {
    case 1:
        $retour_messages = 'gamme';
        $libelle         = 'libelle_gamme';
        $photo           = 'photo_gamme';
        $fragment        = '<a href="produits.php?id=%s&choix=1">
        		              <img src="Images/img_site/%s.gif" width="680">
        		            </a><br /><br />';
      break;
    case 2:
        $retour_messages = 'famille';
        $libelle         = 'libelle_famille';
        $photo           = 'photo_famille';
        $fragment        = '<table border="1">
                              <tr align="center">
                                <td width="200">
                                  <a href="produits.php?id=%s&choix=2">%s</a>
                                </td>
                                <td width="200">
                                  <a href="produits.php?id=%s&choix=2">
                                    <img src="Images/produit/%s.gif" width="35px">
                                  </a>
                                </td>
                              </tr>
                            </table>';
      break;
}
 
$retour_messages = mysql_query('SELECT * FROM ' . $retour_messages . ' ');
 
while ($donnees_messages = mysql_fetch_assoc($retour_messages)) {
    $libelle = stripslashes($donnees_messages[$libelle]);
    $photo   = stripslashes($donnees_messages[$photo]);
 
    switch ($choix) {
        case 1:
            printf($fragment,$libelle,$photo);
          break;
        case 2:
            printf($fragment,$libelle,$libelle,$libelle,$photo);
          break;
    }
}
 
include 'footer.php';
 
?>

[Devilju69]

On peut pas faire mieux ?

[Kennel sébastien]

Tu ne peux factorisé que ce qui est identique. Si tu regardes bien, tu remarqueras que tu peux encore factorisé au niveau des fragments HTML...