Souci de grand débutant sur partie de code PHP

Version imprimable

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:

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:

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

Bonjour,
En gardant la syntaxe des conditions ternaires :

Code:

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')) ;

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:

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';
}

C'est plutôt :

Code:

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';  
}

Citation:
Envoyé par jreaux62
Bonjour,
En gardant la syntaxe des conditions ternaires :
Code:

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

Citation:

Envoyé par jreaux62

C'est plutôt :

Code:

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

Citation:
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:

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

12/09/2019, 08h06
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.

Juste pour info, on peut utiliser des arrays:

Code:

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:

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:

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.

12/09/2019, 09h50
Gilles_75

Citation:

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.
12/09/2019, 09h54
Invité

OK.
Donc, tu te contentes de copier-coller, sans chercher à comprendre... :bravo:
12/09/2019, 10h02
Gilles_75

J'ai aussi mis en place la version des switch avec initialisation des zones, merci !
12/09/2019, 10h04
Gilles_75

Citation:

Envoyé par jreaux62

OK.
Donc, tu te contentes de copier-coller, sans chercher à comprendre... :bravo:

Pourquoi dis-tu cela ?????