Discussion :

[seb1395]

Bonjour à tous, voici le code qui fonctionne actuellement :

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
if($row[2]=="" || $statut=="1"){
$etat=utf8_encode($row2[2]);
$action=utf8_encode($row2[3]);
$couleur = $row2[4];
}
elseif($statut=="2"){
$etat=utf8_encode($row2[2]);
$action=utf8_encode($row2[3]);
$couleur = $row2[4];
}
elseif($statut=="3"){
$etat=utf8_encode($row2[2]);
$action=utf8_encode($row2[3]);
$couleur = $row2[4];
}
elseif($statut=="4"){
$etat=utf8_encode($row2[2]);
$action=utf8_encode($row2[3]);
$couleur = $row2[4];
}
elseif($statut=="5"){
$etat=utf8_encode($row2[2]);
$action=utf8_encode($row2[3]);
$couleur = $row2[4];
}
else{
$etat="Problème d'état" ;
 
}

y aurait-il une façon plus simple de coder cet extrait afin d'allerger le code?

Merci de votre aide

[RideKick]

utiliser switch ?

[Cemos]

Et à moins que mes yeux ne me jouent un mauvais tour, tu implémentes le même code dans chacune de tes conditions, où est l'interêt ?

[seb1395]

et bien parce que ma base de donnée est ainsi faite :

ID. nom Etat action
1 nom1 etat1 action1
2 nom2 etat2 actiont2
3 nom1 etat3 action3


Et que ma requete est la suivant :

$rqt2 = mysql_query('SELECT * FROM mabase where IDstatut='.$statut.'');
$row2=@mysql_fetch_row($rqt2);


donc si le statut = 1
j'écris état1

si le statut = 2
j'écris état2


Et dans mes conditions:
$etat=utf8_encode($row2[2]);
$action=utf8_encode($row2[3]);

revient à dire : tu m'affiches la colone2 dont la ligne est 3 par exemple.

[Cemos]

Oui mais ce que je veux dire, c'est que tu peux regrouper le traitement en une seule condition.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
elseif($statut=="2"){
$etat=utf8_encode($row2[2]);
$action=utf8_encode($row2[3]);
$couleur = $row2[4];
}
elseif($statut=="3"){
$etat=utf8_encode($row2[2]);
$action=utf8_encode($row2[3]);
$couleur = $row2[4];
}

que le statut soit égal à 2 ou à 3, tu effectues les mêmes instructions.

Code :

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

1
2
3
4
5
6
 
elseif($statut==3 || $statut==2){
$etat=utf8_encode($row2[2]);
$action=utf8_encode($row2[3]);
$couleur = $row2[4];
}

et pour l'ensemble de ton code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 
if($row[2]=="" || ($statut>=1 && $statut<=5))
{
      $etat=utf8_encode($row2[2]);
      $action=utf8_encode($row2[3]); 
      $couleur = $row2[4];
}
else
{
      $etat="Problème d'état" ;
}

[seb1395]

Nickel c'est ce que je n'arrivais à pas synthétiser.

Merci beaucoup!