Discussion :

[brice97431]

BOnjour, voilà j'ai un code qui est très répétitif et qui s'avère long quand j'ai beaucoup de cas différents , quelqu'un pourrait il me dire comment l'optimiser svp
Voici le code :

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
if($l==1){
		echo"accueil";
		$query='SELECT nom_zone, id_rubrique_zone
				FROM zone
				INNER JOIN rubrique_zone
				ON zone.id = rubrique_zone.id_zone
				WHERE rubrique_zone.id_rubrique = '.$l.'';
	}
  else if($l==2){
		echo 'actu';
		$query='SELECT nom_zone, id_rubrique_zone
				FROM zone
				INNER JOIN rubrique_zone
				ON zone.id = rubrique_zone.id_zone
				WHERE rubrique_zone.id_rubrique = '.$l.'';
	}
  else if($l==3){
		echo 'actu';
		$query='SELECT nom_zone 
				FROM zone
				INNER JOIN rubrique_zone
				ON zone.id = rubrique_zone.id_zone
				WHERE rubrique_zone.id_rubrique = '.$l.'';
	}
  else if($l==4){
		echo 'sport';
		$query='SELECT nom_zone 
				FROM zone
				INNER JOIN rubrique_zone
				ON zone.id = rubrique_zone.id_zone
				WHERE rubrique_zone.id_rubrique ='.$l.'';
	}

[stealth35]

pas besoin de répéter la requete c'est la même, met la après la condition

[rawsrc]

Bonjour,

factoriser tout ça n'est pas très compliqué quand même, non ?

Code :

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

1
2
3
4
5
6
7
8
9
<?php
$rubriques = array(1 => 'accueil', 'actu', 'actu', 'sport');
echo $rubriques[$l];
$query='SELECT nom_zone 
            FROM zone
            INNER JOIN rubrique_zone
            ON zone.id = rubrique_zone.id_zone
            WHERE rubrique_zone.id_rubrique ='.$l;
?>

[Celira]

Et pour le cas où tu aurais réellement besoin d'une structure type if ($n==1){} elseif ($n==2){} elseif ($n==3){}..., tu as le switch qui est prévu pour

[zuymanto]

raswrc pour la factorisation sur un code basique comme celui là ça passe mais si ce qu'il voulais faire était vraiment à base de if avec des traitements complexes autres que echo un switch serait meilleur.
Une solution serait:

<?php
function r($p1, $p2){
return "MA REQUETE AVEC PARAMETRE $p1, $p2";
}
switch($v){
case : --- $req = r(...)
}
?>

[brice97431]

merci pour vos réponse j'avais aussi penser au switch mais il s'avère que j'ai une soixantaine de cas à traiter et c'était toujours aussi long qu'avec plein de "if" mais j'avais pas penser au tableau donc merci à rawsrc