Discussion :

[moimp]

Bonjour,
Sur une structure switch à 2 niveaux, j'ai une erreur Cannot 'break' 2 levels in C:\wamp64\www\proginet\appSirep\backend\settingListUpdate.php on line 23Je ne vois pas comment coder ce type de structure. Je peux bien sûr remplacer le niveau parent par des conditions if ... elseif mais j'aimerais comprendre pourquoi mon code ne fonctionne pas.

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
switch($iPage){
	case 5:
		switch($iCurrentTab){
			case 1: // filters
			case 2: // genders
			case 3: // titles
			case 4: // standard roles
			case 5: // first contact origines
				if (strlen($post['text']) > getColumnMaxLength('sirep', $table, 'text'))
					$aErrors[] = getDBText($language,1032);
			break 1;
			case 6: // // Quality norms
				if (strlen($post['norm_code']) > getColumnMaxLength('sirep', $table, 'norm_code'))
					$aErrors[] = getDBText($language,1032);
				if (strlen($post[$language.'_title']) > getColumnMaxLength('sirep', $table, $language.'_title'))
					$aErrors[] = getDBText($language,1032);
			break 1;
			case 7: // Legal status
				if (strlen($post['legal_status']) > getColumnMaxLength('sirep', $table, 'legal_status'))
					$aErrors[] = getDBText($language,1032);
			break 1;
		}
	break 2;
	case 6:
		switch($iCurrentTab){
			case 1: // users
			break 1;
			case 2: // rights
			break 1;
		}
	break 2;
}

[cavo789]

Bonjour

Je n'essaie même pas de lire car ... beurk ;-)

Essaie de couper ton code en petites fonctions afin de le rendre et lisible et maintenable. Si tu lis l'anglais : https://github.com/jupeter/clean-cod...d-conditionals

Des if imbriqués, c'est déjà vraiment pas trop; des switch imbriqués, c'est brrrrrr.

[Séb.]

break 2 veut dire "je poursuis le traitement 2 niveaux au-dessus"
Le 1er switch ne peut donc pas contenir de break 2, seulement un break 1 (1 est la valeur par défaut)

Ceci dit je rejoins cavo789, il vaut mieux éviter ces structures et découper ton script en fonctions, ou comme tu dis en if/elseif

[moimp]

Merci à vous deux.