Discussion :

[hugodu28]

bonjour,

j'ai un formulaire php en lien avec une base access, avec ce bout 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
19
20
21
22
23
24
25
26
27
28
29
 // on recupere le nombre d'agent ayant declare la disponibilite
for ($i = 300 ; $i<=327; $i++) {
 $NB{$i}=0;
 
 
// connection à la base de données:
   $cnx = odbc_connect( "DSN_disponibilite", "root", "" ) or die ("Impossible de se connecter à la base de donnée") ;
 
	  $NB = "SELECT * FROM recape_nb".$NC." WHERE num_centre=$i" ;
	//  echo $NB;
 // exécution de la requête:
    $requeteNB = odbc_do($cnx, $NB) ;
$resultNB = odbc_fetch_object($requeteNB);
 
	$centre=$resultNB->num_centre;
	if ($resultNB->NB==''){$NB{$i}=0;}else{ $NB{$i}=$resultNB->NB;}
 
 
 
if (($centre==300)AND($NB{$i}>=6))               {$B{$i} ="CSPV.JPG";}   
if (($centre==300)AND(3>$NB{$i})AND ($NB{$i}>0) ){$B{$i}= "CSPR.JPG";}
if (($centre==300)AND($NB{$i}==0))               {$B{$i}= "CSPN.JPG";} 
if (($centre==300)AND(6>$NB{$i})AND ($NB{$i}>=3) ){$B{$i}= "CSPO.JPG";}
 
if (($centre==301)AND($NB{$i}>=6))               {$B{$i} ="CSV.JPG";}
if (($centre==301)AND($NB{$i}<3) AND ($NB{$i}>0)){$B{$i}= "CSR.JPG";}
if (($centre==301)AND($NB{$i}==0)){$B{$i}= "CSN.JPG";}
if (($centre==301)AND($NB{$i}<6) AND ($NB{$i}>=3)){$B{$i}= "CSO.JPG";}
....

Ainsi en fonction du nombre d'agents declarant de la disponibilité, l'image du centre apparaitra soir en noir, rouge, orange ou vert.

Jusqu'à 9 personnes tout se passe bien, après , l'image est CSPR.JPG

Si vous avez une idée, merci

[Celira]

Attention à la priorité des opérateurs ! Il y a une différence entre AND et && :

// --------------------
// "&&" a un précédence supérieure à "and"

// Le résultat de l'expression (true && false) est assigné à $g
// Agit comme : ($g = (true && false)) => retourne false
$g = true && false;

// La constante true est assignée à $h, puis, false est ignoré
// Agit comme : (($h = true) and false) => retourne true
$h = true and false;

D'autre part, si tu n'imbriques pas tes conditions, tu risques un cas type "c'est celui qui parle en dernier qui a raison". Donc si tes conditions sont théoriquement mutuellement exclusives, le mieux c'est de les imbriquer à coup de if/else...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
if ($centre==300) {
	if ($NB{$i}>=6) {
		$B{$i} ="CSPV.JPG";
	} else if ((6>$NB{$i}) && ($NB{$i}>=3)) {
		$B{$i}= "CSPO.JPG";
	} else if ((3>$NB{$i}) && ($NB{$i}>0) ) {
		$B{$i}= "CSPR.JPG";
	} else ($NB{$i}==0) {
		$B{$i}= "CSPN.JPG";
	} 
} else if ($centre==301) {
	....
}

N'oublie pas soit d'initialiser $B{$i}, soit de prévoir les cas else

[hugodu28]

bravo et merci pour tes explications très claires.