Discussion :

[Floflo18]

Bonjour a tous,

J'ai atuellement un soucis de passage dans mon else if alors que le if lui fonctionne tres bien et je n'ai pas trouve de reponse a cela, voice le 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
else if (isset($_GET['op_list'])) {
	$machine = htmlentities($_GET['machine']);
	$machine = substr($machine, 4);
 
	if($machine = 'MECOFs' || $machine = 'MECOF1' || $machine = 'MECOF2' || $machine = 'MECOF3' || $machine = 'MECOF4') {
		$machine = 'mecof';
	} else if($machine = 'MILL TURNs' || $machine = 'WFL1' || $machine = 'WFL2' || $machine = 'WFL3' || $machine = 'NILES1') {
		$machine = 'millturn';
	}
 
	$reference = htmlentities($_GET['reference']);
	$i = 0;
 
    $op_req = $dbh->query('SELECT num_op FROM part_info WHERE machine_type="' .$machine. '" AND part_ref = "' .$reference.'"');	
	while ($line = $op_req->fetch()) {
		$i++;
		$json[$i][0]=$line["num_op"];
	}
	$op_req->closeCursor();
}

Je resort dans tout les cas machine= mecof, mais impossible de passé en millturn.

Merci de vos reponses.

[eleydet]

Bonjour,

Ce code est en JavaScript? Hum, j'ai quelques doutes... Ceci étant, la comparaison se fait avec un double égal.

Code :

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

if($machine == 'MECOFs' || $machine == 'MECOF1' || ...

[Vil'Coyote]

je ne sais pas si php à évoluer sans que je le vois mais normalement chez moi un elseif c'est sans espace donc if elseif else mais un else if j'crois pas que ça marche.
a ça s'ajoute comme indiqué le test == au lieu de =

[sabotage]

"else if" fonctionne aussi avec restrictions, donc autant utiliser "elseif"

http://php.net/manual/fr/control-structures.elseif.php

[rawsrc]

Salut,

tu peux faire plus lisible :

Code :

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

1
2
3
4
5
if (in_array($machine, ['MECOFs', 'MECOF1', 'MECOF2', 'MECOF3', 'MECOF4'], true)) {
    $machine = 'mecof';
} elseif (in_array($machine, ['MILL TURNs', 'WFL1', 'WFL2', 'WFL3', 'NILES1'], true)) {
    $machine = 'millturn';
}

[Invité]

• "else if" ouvre une nouvelle condition "if" après le "else".
• "elseif" est la suite logique du "if" initiale.

"else if" est la conséquence du fait de ne pas mettre systématiquement les accolades {...} !
Ca revient à écrire :

Code :

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

1
2
3
4
5
6
} else {
  if (.....)
   {
      //......
   }
}

BREF : il faut mettre les accolades { ... } , ou la version "moderne" if(...): ... :endif;

[Celira]

ou la version "moderne" if(...): ... :endif;

Ou pas :

Quand vous utilisez ":" pour définir votre condition if/elseif, vous ne devez pas séparer else if en deux mots,

Sinon, pour la condition, on pouvait aussi faire un switch avec des cas vides

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
switch ($machine) {
    case 'MECOFs':
    case 'MECOF1':
    case 'MECOF2':
    case 'MECOF3':
    case 'MECOF4':
        $machine = 'mecof';
        break;
    case 'WFL1':
    case 'WFL2':
    case 'WFL3':
    case 'NILES1':
    case 'MILL TURNs':
        $machine = 'millturn';
        break;
}