Souci de grand débutant sur partie de code PHP

**Gilles_75** · 11/09/2019, 20h01

Bonjour

Je reprend une partie de code d'un développeur qui est parti et je suis ultra, ultra débutant
Voici la partie de 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
			if ($tab_regions_dest)
			{
				foreach ($tab_regions_dest as $region)
				{
 
					$guides_region = WP_Query_Destination_guides ('region', $region['ID']) ;
 
				if ($guides_region)
					{
						$guides['region']['titre']['fr-FR'] = ($IDd == 8493 ? 'Etats' : 'Régions') ;  
						$guides['region']['titre']['en-GB'] = ($IDd == 12129 ? 'States' : 'Regions') ; 
						$guides['region']['donnees'][] = array('nom'			=> $region['nom'][$cl]	,
															   'liste_guides'	=> $guides_region
															  ) ;
					}
 
				}
			}

Pour résumer
J'aimerais insérer d'autres tests dans cette boucle
Par exemple ceci
Je rajoute une condition ligne verte

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
		if ($tab_regions_dest)
			{
				foreach ($tab_regions_dest as $region)
				{

					$guides_region = WP_Query_Destination_guides ('region', $region['ID']) ;

				if ($guides_region)
					{
						$guides['region']['titre']['fr-FR'] = ($IDd == 8493 ? 'Etats' : 'Régions') ;  
						$guides['region']['titre']['en-GB'] = ($IDd == 12129 ? 'States' : 'Regions') ; 
						$guides['region']['titre']['fr-FR'] = ($IDd == 55555 ? 'Emirats' : 'Régions') ;  
						$guides['region']['titre']['en-GB'] = ($IDd == 66666 ? 'Emirates' : 'Regions') ;
						$guides['region']['donnees'][] = array('nom'			=> $region['nom'][$cl]	,
															   'liste_guides'	=> $guides_region
															  ) ;
					}
					
				}
			}

Mon souci c'est que le code ne prend en compte que la dernière condition du code ( la ligne en vert et ne tient pas compte de la ligne en rouge qu'il traite par défaut ligne bleue
Je ne sais pas trop si j'ai été assez clair
J'aimerais donc faire
( Si
xxxx alors
et
Si
yyyyy alors
et
SI
zzzzz alors
)
Sinon

Merci pour votre aide

Invité · 11/09/2019, 20h14

Bonjour,
En gardant la syntaxe des conditions ternaires :

Code php :

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

1
2
$guides['region']['titre']['fr-FR'] = ($IDd == 8493 ? 'Etats' : ($IDd == 55555 ? 'Emirats' : 'Régions')) ;  
$guides['region']['titre']['en-GB'] = ($IDd == 12129 ? 'States' : ($IDd == 66666 ? 'Emirates' : 'Regions')) ;

**Dendrite** · 11/09/2019, 20h44

Si tu es ultra débutant, n'hésite pas à réécrire un code que toi tu comprends...
On ne peut pas sauter les étapes comme ça.
Si j'ai bien compris (suis pas sûre), ça donnerait ça :

Code PHP :

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

1
2
3
4
5
6
7
8
9
10
11
12
if($IDd == 8493 || $IDd == 12129){
  $guides['region']['titre']['fr-FR'] = 'Etats';
  $guides['region']['titre']['en-GB'] = 'States';
}
elseif ($IDd == 55555 || $IDd == 66666){
  $guides['region']['titre']['fr-FR'] = 'Emirats';
  $guides['region']['titre']['en-GB'] = 'Emirates';
}
else{
  $guides['region']['titre']['fr-FR'] = 'Régions';
  $guides['region']['titre']['en-GB'] = 'Regions';
}

Invité · 11/09/2019, 21h11

C'est plutôt :

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
if ($IDd == 8493)
{
   $guides['region']['titre']['fr-FR'] = 'Etats';
} elseif ($IDd == 55555) {
   $guides['region']['titre']['fr-FR'] = 'Emirats';  
} else {
   $guides['region']['titre']['fr-FR'] =  'Régions';  
}
if ($IDd == 12129)
{
   $guides['region']['titre']['en-GB'] = 'States';
} elseif ($IDd == 66666) {
   $guides['region']['titre']['en-GB'] = 'Emirates';  
} else {
   $guides['region']['titre']['en-GB'] =  'Regions';  
}

**Gilles_75** · 11/09/2019, 21h31

Envoyé par jreaux62

Bonjour,
En gardant la syntaxe des conditions ternaires :

Code php :

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

1
2
$guides['region']['titre']['fr-FR'] = ($IDd == 8493 ? 'Etats' : ($IDd == 55555 ? 'Emirats' : 'Régions')) ;  
$guides['region']['titre']['en-GB'] = ($IDd == 12129 ? 'States' : ($IDd == 66666 ? 'Emirates' : 'Regions')) ;

Merci beaucoup, cette première solution fonctionne très bien

**Gilles_75** · 11/09/2019, 21h32

Envoyé par jreaux62

C'est plutôt :

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
if ($IDd == 8493)
{
   $guides['region']['titre']['fr-FR'] = 'Etats';
} elseif ($IDd == 55555) {
   $guides['region']['titre']['fr-FR'] = 'Emirats';  
} else {
   $guides['region']['titre']['fr-FR'] =  'Régions';  
}
if ($IDd == 12129)
{
   $guides['region']['titre']['en-GB'] = 'States';
} elseif ($IDd == 66666) {
   $guides['region']['titre']['en-GB'] = 'Emirates';  
} else {
   $guides['region']['titre']['en-GB'] =  'Regions';  
}

Merci bien j'ai implémenté votre première solution

**Gilles_75** · 11/09/2019, 21h34

Envoyé par Dendrite

Si tu es ultra débutant, n'hésite pas à réécrire un code que toi tu comprends...
On ne peut pas sauter les étapes comme ça.
Si j'ai bien compris (suis pas sûre), ça donnerait ça :

Code PHP :

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

1
2
3
4
5
6
7
8
9
10
11
12
if($IDd == 8493 || $IDd == 12129){
  $guides['region']['titre']['fr-FR'] = 'Etats';
  $guides['region']['titre']['en-GB'] = 'States';
}
elseif ($IDd == 55555 || $IDd == 66666){
  $guides['region']['titre']['fr-FR'] = 'Emirats';
  $guides['region']['titre']['en-GB'] = 'Emirates';
}
else{
  $guides['region']['titre']['fr-FR'] = 'Régions';
  $guides['region']['titre']['en-GB'] = 'Regions';
}

Merci bien j'ai testé la première solution de jreaux62 qui fonctionne aussi très bien

**Dendrite** · 12/09/2019, 08h06

Merci Jreaux pour la correction.
@Gilles : que la condition ternaire fonctionne, j'en étais certaine. La question est : la comprends-tu ?
Moi, je trouve les conditions ternaires imbriquées très peu lisibles. Je ne les utilise que sur un niveau.
L'autre question : c'est que si tu fais évoluer le code de quelqu'un qui va revenir, reste dans sa logique à lui. Mais s'il ne revient pas et que tu dois t'approprier "définitivement" ses scripts, n'hésite pas à les rendre plus simples pour toi, ils n'en seront pas moins performants.

Invité · 12/09/2019, 09h34

Juste pour info, on peut utiliser des arrays:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
$titre_fr_arr = [
	0 => 'Régions',
	8493 => 'Etats',
	55555 => 'Emirats',
	];
$titre_en_arr = [
	0 => 'Regions',
	12129 => 'States',
	66666 => 'Emirates',
	];
$guides['region']['titre']['fr-FR'] = ( !empty($titre_fr_arr[$IDd]) )? $titre_fr_arr[$IDd] : $titre_fr_arr[0] ;  
$guides['region']['titre']['en-GB'] = ( !empty($titre_en_arr[$IDd]) )? $titre_en_arr[$IDd] : $titre_en_arr[0] ;

Ou encore des switch :

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
switch($IDd)
{
	case 8493:
		$guides['region']['titre']['fr-FR'] = 'Etats';
		break;
	case 55555:
		$guides['region']['titre']['fr-FR'] = 'Emirats';
		break;
	default:
		$guides['region']['titre']['fr-FR'] = 'Régions';
}
switch($IDd)
{
	case 12129:
		$guides['region']['titre']['en-GB'] = 'States';
		break;
	case 66666:
		$guides['region']['titre']['en-GB'] = 'Emirates';
		break;
	default:
		$guides['region']['titre']['en-GB'] = 'Regions';
}

ATTENTION : les 2 switch (comme les 2 if précédents), ne peuvent pas être fusionnés en un seul ici (car pour chacun, on ne passe que dans une seule condition, or on a 2 variables à remplir).

On peut néanmoins procéder autrement, en initialisant D'ABORD les variables avec leur valeur par défaut :

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
$guides['region']['titre']['fr-FR'] = 'Régions';
$guides['region']['titre']['en-GB'] = 'Regions';
switch($IDd)
{
	case 8493:
		$guides['region']['titre']['fr-FR'] = 'Etats';
		break;
	case 55555:
		$guides['region']['titre']['fr-FR'] = 'Emirats';
		break;
	case 12129:
		$guides['region']['titre']['en-GB'] = 'States';
		break;
	case 66666:
		$guides['region']['titre']['en-GB'] = 'Emirates';
		break;
}

Mais, à mon avis, c'est moins lisible, notamment pour la maintenance.

**Gilles_75** · 12/09/2019, 09h50

Envoyé par Dendrite

Merci Jreaux pour la correction.
@Gilles : que la condition ternaire fonctionne, j'en étais certaine. La question est : la comprends-tu ?
Moi, je trouve les conditions ternaires imbriquées très peu lisibles. Je ne les utilise que sur un niveau.
L'autre question : c'est que si tu fais évoluer le code de quelqu'un qui va revenir, reste dans sa logique à lui. Mais s'il ne revient pas et que tu dois t'approprier "définitivement" ses scripts, n'hésite pas à les rendre plus simples pour toi, ils n'en seront pas moins performants.

Merci pour ta version que j'ai testé et mise en place car bien plus lisible.
Je n'ai pas testé la version de @Jreaux qui doit fonctionner tout autant.