fonction if imbriqués

Version imprimable

bonjour
j'essaye en vain de faire une fonction imbriquée:(
mon but est de connaitre la "caf" d'une famille suivant le nombre d'enfant de moins de 16 ans:
1 enfant = 0
2 enfants=124
3 enfants=275
4 enfants=433
5 enfants=590
les variables $AgeEnfant ont par defaut 17 ans et commence par $AgeEnfant0
j'ai donc écris
Code:

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 function cafok { $AllocFam_ok =0; // bnp if ($bank=='bnp1' or $bank=='bnp2' ') { if ($AgeEnfant1<16) { if ($AgeEnfant2<16) {if ($AgeEnfant3<16) {if ($AgeEnfant4<16) {$AllocFam_ok=590;} } else{$AllocFam_ok=433;} } else{$AllocFam_ok=275;} } else{$AllocFam_ok=124;} } } return $AllocFam_ok; }
la réponse est erronnée (590 dans ts les cas)
il y a évidement une erreur mais le débutant que je suis ne la trouve pas!
merci de votre aide:mouarf:

21/06/2010, 15h45
antoinev2
Bonjour,

si je peux me permettre, c'est un peu "fouillis", prends l'habitude de respecter l'indentation : ton code sera plus clair et donc, plus facile à débugger : tu dois avoir fait une petite erreur qui apparaîtra clairement.

un exemple d'indentation respectée :
Code:

1 2 3 4 5 6 7 if (condition1) { instruction1 ; } elseif (condition2) { instruction2 ; } else { instruction3 ; }
(copié de la faq)

Accessoirement, je dirais qu'il y a un guillemet simple de trop dans ce test :

Code:

if ($bank=='bnp1' or $bank=='bnp2' ')
21/06/2010, 17h00
kogoi

fonction if imbriqué

bjr
merci pour votre reponse mais j'ai due mal m'expliquer
concernant le guillemet, c'est parce que j'ai tronqué la condition qui était trop longue et j'

fonction if imbriqué

désolé erreur de manip!!!
je repete donc
bjr
merci pour votre reponse mais j'ai due mal m'expliquer
concernant le guillemet, c'est parce que j'ai tronqué la condition qui était trop longue et j' ai laissé ce guillemet par erreur
l'erreur est apres;
actuellement j'utilise bien elseif mais je dois tester tous les cas
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 if ($NbEnfant=="2") { // bnp hypo /sygma hypo conso GE if (($AgeEnfant0<16 and $AgeEnfant1<16) and ($bank=='bnp1' or $bank=='bnp2' or $bank=='bnp3' or $bank=='bnp4' or $bank=='sygmahypo' or $bank=='sygmaconso' or $bank=='GEPLUS' or $bank=='GERAC' or $bank=='GEPREM' or $bank=='GE1'or $bank=='GE2'or $bank=='GE3')) {$AllocFam_ok=124;} elseif ($NbEnfant=="3") { elseif (($AgeEnfant0<16 and $AgeEnfant1<16 and $AgeEnfant2<16) and ($bank=='bnp1' or $bank=='bnp2' or $bank=='bnp3' or $bank=='bnp4' or $bank=='sygmahypo' or $bank=='sygmaconso' or $bank=='GEPLUS' or $bank=='GERAC' or $bank=='GEPREM' or $bank=='GE1'or $bank=='GE2'or $bank=='GE3')) {$AllocFam_ok=275;} elseif (($AgeEnfant0<16 and $AgeEnfant1<16 and $AgeEnfant2>=16) and ($bank=='bnp1' or $bank=='bnp2' or $bank=='bnp3' or $bank=='bnp4' or $bank=='sygmahypo' or $bank=='sygmaconso' or $bank=='GEPLUS' or $bank=='GERAC' or $bank=='GEPREM' or $bank=='GE1'or $bank=='GE2'or $bank=='GE3')) {$AllocFam_ok=124;}
etc ( je ne vais pas tout marquer , il y a + de 200 lignes)
cela est actuellement donc tres lourd
je voulais clarifier mon script en utilisant des if imbriqués mais pour une raison que j'ignore, cela ne marche pas
j'attends vos remarques:mouarf:

Il semblerait que tu te sois emellé dans tes accolades, voici le code corrigé :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
if ($bank=='bnp1' or $bank=='bnp2') {
	if ($AgeEnfant1<16) {
		if ($AgeEnfant2<16) {
			if ($AgeEnfant3<16) {
				if ($AgeEnfant4<16) {
					$AllocFam_ok=590;
				}
				else {
					$AllocFam_ok=433;
				}
			else {
				$AllocFam_ok=275;
			}
		else{
			$AllocFam_ok=124;
		}
	}
}

FONCTION IF IMBRIQUE

merci à toi, mais j'ai encore une erreur
Parse error: parse error in C:\xampp\htdocs\essai2\Fonction.php on line 182
la ligne 182 en occurence c'est le else qui precede le 275
Je te note la fonction

Code:

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
function cafok ($NbEnfant,$AgeEnfant0,$AgeEnfant1,$AgeEnfant2,$AgeEnfant3,$AgeEnfant4,$AgeEnfant5 )
{
$AllocFam_ok =0;
 
 
if ($bank=='bnp1' or $bank=='bnp2') {
	if ($AgeEnfant1<16) {
		if ($AgeEnfant2<16) {
			if ($AgeEnfant3<16) {
				if ($AgeEnfant4<16) {
					$AllocFam_ok=590;
				}
				else {
					$AllocFam_ok=433;
				}
			else {
				$AllocFam_ok=275;
			}
		else{
			$AllocFam_ok=124;
		}
	}
}
 
 return $AllocFam_ok;
 
}

je ne vois pas d'erreur !!!

Salut,

Je pense que pour simplifier la chose, il vaut mieux d'abord compter le nombre d'enfants effectivement âgés de moins de 16 ans.

Exemple minimaliste
Code:

1 2 3 4 5 6 7 $tab=array( $AgeEnfant0, $AgeEnfant1, $AgeEnfant2, $AgeEnfant3, $AgeEnfant4, $AgeEnfant5 ); $Enfants=0; foreach($tab as $age){ if($age<16)$Enfants++; }
Ensuite, tout devient plus clair
Code:

1 2 3 4 5 6 7 8 9 10 $AllocFam_ok=0; switch($Enfants){ case 2:$AllocFam_ok=124; Break; case 3:$AllocFam_ok=275; Break; case 4:$AllocFam_ok=433; Break; case 5:$AllocFam_ok=590; }
EDIT : J'avais oublié les 'case'... Merci Thes32 !

@Eric2a : la syntaxe correcte de switch c'est

Code:

1
2
3
4
5
6
7
8
9
 
switch( $a )
{
  case 1:
   //bloc de code
   //break;
  case 3:
   //etc..
}

22/06/2010, 11h31
Eric2a

:oops:

Code corrigé !

Merci à toi

FONCTION IF IMBRIQUE

salut et merci à eric2A et thes32:ccool:
la solution d'utiliser swish est une bonne idée mais la fonction en question est plus complexe que cela :
je n'ai donné que l'exemple d'une banque mais chacune prend un age différent pour calculer la caf
telle banque prend les enfants de moins de 16 ans
telle autre prend les enfants de moins de 10 ans
etc
c'est pourquoi j'ai utilisé un if imbriqué
en clair
Code:

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 33 34 35 36 if ($bank=='bnp1' or $bank=='bnp2') { if ($AgeEnfant1<16) { if ($AgeEnfant2<16) { if ($AgeEnfant3<16) { if ($AgeEnfant4<16) { $AllocFam_ok=590; } else { $AllocFam_ok=433; } else { $AllocFam_ok=275; } else{ $AllocFam_ok=124; } } } elseif ($bank=='xxxx' or $bank=='yyyy') { if ($AgeEnfant1<15) { if ($AgeEnfant2<15) { if ($AgeEnfant3<15) { if ($AgeEnfant4<15) { $AllocFam_ok=590; } else { $AllocFam_ok=433; } else { $AllocFam_ok=275; } else{ $AllocFam_ok=124; } } }
etc etc....
mais pourquoi une erreur sur la ligne
Code:

1 2 3 else { $AllocFam_ok=275; }
le code me semble pourtant correct:calim2::calim2::calim2:
merci

on ne peut pas avoir plusieurs clause else qui se suivent comme tu le fais. La logique des if c'est

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
if( $condition )
{
  //le premier bloc
}
else if( $condition2 )
{
  //le second bloc alternatif
}
else if( $conditionN )
{
  //le Nième bloc alternatif
}
else
{
  //bloc s'exécutant si aucune condition n'est remplie
}

22/06/2010, 14h23
antoinev2
Citation:
Envoyé par kogoi

mais pourquoi une erreur sur la ligne

Code:

1 2 3 else { $AllocFam_ok=275; }

le code me semble pourtant correct:calim2::calim2::calim2:
merci
Cet extrait de code me semble correct en lui-même.
Il faudrait qu'on voie le code dans son intégralité pour voir où se situe une erreur de logique, par exemple.
Si ton codes est très long, tu peux le mettre en PJ.

Par contre, une solution telle que proposée par Eric2a permet un code plus court.
A part l'âge des enfants, qui dépend de la banque, as-tu d'autres conditions à vérifier dans ce code?

Si tu n'as pas d'autres conditions à vérifier, tu pourrais avoir :

une première fonction qui reçoit en argument le nom de la banque, et qui retourne l'âge maximum des enfants.

une deuxième fonction qui reçoit en argument : l'âge maximum des enfants, et l'âge de chacun des enfants

et simplement un bout de code qui appelle ces fonctions.

FONCTION IF IMBRIQUE

Comme demandé la fonction complete avec toujours la même erreur sur sur ligne 182 ( 2eme else)

Code:

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
function cafok ($bank,$NbEnfant,$AgeEnfant0,$AgeEnfant1,$AgeEnfant2,$AgeEnfant3,$AgeEnfant4,$AgeEnfant5, ){
$AllocFam_ok =0;
 
if ($bank=='bnp1' or $bank=='bnp2') {
	if ($AgeEnfant1<16) {
		if ($AgeEnfant2<16) {
			if ($AgeEnfant3<16) {
				if ($AgeEnfant4<16) {
					$AllocFam_ok=590;
				}
				else {
					$AllocFam_ok=433;
				}
			else {
				$AllocFam_ok=275;
			}
		else{
			$AllocFam_ok=124;
		}
	}
}
elseif ($bank=='bnpconso') {
	if ($AgeEnfant1<6) {
		if ($AgeEnfant2<6) {
			if ($AgeEnfant3<6) {
				if ($AgeEnfant4<6) {
					$AllocFam_ok=590;
				}
				else {
					$AllocFam_ok=433;
				}
			else {
				$AllocFam_ok=275;
			}
		else{
			$AllocFam_ok=124;
		}
	}
}
elseif ($bank=='brookeo') {
	if ($AgeEnfant1<9) {
		if ($AgeEnfant2<9) {
			if ($AgeEnfant3<9) {
				if ($AgeEnfant4<9) {
					$AllocFam_ok=590;
				}
				else {
					$AllocFam_ok=433;
				}
			else {
				$AllocFam_ok=275;
			}
		else{
			$AllocFam_ok=124;
		}
	}
}
elseif ($bank=='ccfp1' or $bank=='ccfp2' or $bank=='ccfp3') {
	if ($AgeEnfant1<15) {
		if ($AgeEnfant2<15) {
			if ($AgeEnfant3<15) {
				if ($AgeEnfant4<15) {
					$AllocFam_ok=590;
				}
				else {
					$AllocFam_ok=433;
				}
			else {
				$AllocFam_ok=275;
			}
		else{
			$AllocFam_ok=124;
		}
	}
}
elseif ($bank=='cfcal' or $bank=='wash') {
	if ($AgeEnfant1<10) {
		if ($AgeEnfant2<10) {
			if ($AgeEnfant3<10) {
				if ($AgeEnfant4<10) {
					$AllocFam_ok=590;
				}
				else {
					$AllocFam_ok=433;
				}
			else {
				$AllocFam_ok=275;
			}
		else{
			$AllocFam_ok=124;
		}
	}
}
elseif ($bank=='creatis') {
	if ($AgeEnfant1<14) {
		if ($AgeEnfant2<14) {
			if ($AgeEnfant3<14) {
				if ($AgeEnfant4<14) {
					$AllocFam_ok=590;
				}
				else {
					$AllocFam_ok=433;
				}
			else {
				$AllocFam_ok=275;
			}
		else{
			$AllocFam_ok=124;
		}
	}
}
 
elseif ($bank=='lift') {
	if ($AgeEnfant1<13) {
		if ($AgeEnfant2<13) {
			if ($AgeEnfant3<13) {
				if ($AgeEnfant4<13) {
					$AllocFam_ok=590;
				}
				else {
					$AllocFam_ok=433;
				}
			else {
				$AllocFam_ok=275;
			}
		else{
			$AllocFam_ok=124;
		}
	}
}
 return $AllocFam_ok;
 
}

le code est très simple et répétitif ,c'est pourquoi je n'en avais donné qu'une partie
merci

FONCTION IF IMBRIQUE

re
pour info, je vous rappelle mon code initial qui me semblait + logique (accolades placées differement)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
if ($bank=='bnp1' or $bank=='bnp2') {
	if ($AgeEnfant1<16) {
				if ($AgeEnfant2<16) {
					if ($AgeEnfant3<16){
						if ($AgeEnfant4<16){$AllocFam_ok=590;}
						else {$AllocFam_ok=450;}
									}
					else {$AllocFam_ok=250;}
								}
				else {$AllocFam_ok=120;}
						}
								}	
elseif ($bank=='bnpconso') {
	if ($AgeEnfant1<6) {
			if ($AgeEnfant2<6) {
					if ($AgeEnfant3<6){
						if ($AgeEnfant4<6){$AllocFam_ok=590;}
						else {$AllocFam_ok=450;}
								}
					else {$AllocFam_ok=250;}
							}
			else {$AllocFam_ok=120;}
					}
						}

etc
mais ne fonctionne pas (donne 590 à chaque fois)

Essaie ce code...

Code:

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
33
34
35
36
37
38
39
40
41
42
function cafok ($bank,$NbEnfant,$AgeEnfant0,$AgeEnfant1,$AgeEnfant2,$AgeEnfant3,$AgeEnfant4,$AgeEnfant5){
	// Détermination de l'âge maximum pour la banque
	$tab=array(
		$AgeEnfant0,$AgeEnfant1,$AgeEnfant2,$AgeEnfant3,$AgeEnfant4,$AgeEnfant5
	);
	$AgeMaxi=17;
	if($bank=='bnpconso')
		$AgeMaxi=6;
	elseif($bank=='brookeo')
		$AgeMaxi=9;
	elseif($bank=='cfcal' or $bank=='wash')
		$AgeMaxi=10;
	elseif($bank=='lift') 
		$AgeMaxi=13;
	elseif($bank=='creatis')
		$AgeMaxi=14;
	elseif($bank=='ccfp1' or $bank=='ccfp2' or $bank=='ccfp3')
		$AgeMaxi=15;
	elseif($bank=='bnp1' or $bank=='bnp2')
		$AgeMaxi=16;
 
	if($AgeMaxi>16)return 0; // Faut changer de banque ?
 
	// Compter le nombre d'enfants pris en compte par la banque
	$Enfants=0;
	foreach($tab as $age){
		if($age<$AgeMaxi)$Enfants++;
	}
 
	// Obtenir le coêf. en fonction du nombre d'enfants
	$AllocFam_ok=0;
	switch($Enfants){
		case 2:$AllocFam_ok=124;
			Break;
		case 3:$AllocFam_ok=275;
			Break;
		case 4:$AllocFam_ok=433;
			Break;
		case 5:$AllocFam_ok=590;
	}
	return $AllocFam_ok;
}

22/06/2010, 16h38
kogoi

FONCTION IF IMBRIQUE

merci pour votre persévérence !!!!!
mais votre script donnait 0 dans tous les cas:cry:
j'ai donc vérifié en amont et l'erreur etait dans la récup des variables:oops::oops::oops::oops::oops:
maintenant tout fonctionne parfaitement et c'est beaucoup plus clean
TROP FORT:ccool::ccool::ccool::ccool:
vive ce forum et encore merci à tous
pat

Pour info : dans ton code, il manquait certaines accolades :

exemple : 1e premier bloc de ton code
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 if ($bank=='bnp1' or $bank=='bnp2') { if ($AgeEnfant1<16) { if ($AgeEnfant2<16) { if ($AgeEnfant3<16) { if ($AgeEnfant4<16) { $AllocFam_ok=590; } else { $AllocFam_ok=433; } else { $AllocFam_ok=275; } else{ $AllocFam_ok=124; } } } elseif ($bank=='bnpconso') {
et corrigé :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 if ($bank=='bnp1' or $bank=='bnp2') { if ($AgeEnfant1<16) { if ($AgeEnfant2<16) { if ($AgeEnfant3<16) { if ($AgeEnfant4<16) { $AllocFam_ok=590; } else { $AllocFam_ok=433; } } else { $AllocFam_ok=275; } } else{ $AllocFam_ok=124; } } }
Je te conseille d'utiliser un éditeur de texte avec coloration syntaxique.
Par exemple, Notepad++.
Tu aurais pu vérifier tes paires d'accolades rapidement.

Bon en tout cas, la solution de Eric2a est évidemment plus claire et plus facile à faire évoluer.