Optimiser une liste de IF

Version imprimable

Bonjour,

Voici le début de mon bout de code PHP :
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 <?php $i0 = $_POST['id']; if (!empty($i0)) { $i01 = trim($_POST['i1']); } $i02 = ""; if ($i01 == "001A ") { $i02 = "Ok"; } if ($i01 == "002B ") { $i02 = "Ok"; } if ($i01 == "003C ") { $i02 = "Ok"; } if ($i01 == "004D ") { $i02 = "Ok"; } if ($i01 == "021E ") { $i02 = "Ok"; } if ($i01 == "037F ") { $i02 = "Ok"; } if ($i01 == "077Z ") { $i02 = "Ok"; } if ($i01 == "123O ") { $i02 = "Ok"; } if ($i01 == "321V ") { $i02 = "Ok"; } if ($i01 == "007Z ") { $i02 = "Ok"; } if ($i02 == "") { echo <<<END $i01 référence à revoir . . . END; } else { echo <<<END $i01 écrire dans fichier à finir . . . END; } ?>
Ma question, j'aimerais bien réduire les lignes de IF :

Code:

if ($i01 == "000A ") { $i02 = "Ok"; }

pour info : ce bout de code PHP est généré par Excel avec VBA, chaque matin . . .

le formulaire PHP, me permet de récupérer une liste de référence, et chaque matin,

je génère le fichier PHP, avec une nouvelle liste de IF

Merci . . .

:?

Salut,

comme ça :

Code:

1
2
$data = ['001A ', '002B ', '003C ', '004D ', '021E ', '037F ', '077Z ', '123O ', '321V ', '007Z '];
$i02  = in_array($i01, $data, true) ? 'Ok' : '';

Il y a aussi le switch qui est prévu pour ça, j'ai également ajouter un false plutôt que ''

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
 
<?php
 
$i0 = $_POST['id'];
 
if (!empty($i0)) $i01 = trim($_POST['i1']); break;
 
switch ($i01) {
    case "001A": 
        $i02 = "Ok"; 
        break;
    case "002B": 
        $i02 = "Ok"; 
        break;
    case "003C": 
        $i02 = "Ok"; 
        break;
    case "004D": 
        $i02 = "Ok"; 
        break;
    case "021E": 
        $i02 = "Ok"; 
        break;
    case "037F": 
        $i02 = "Ok"; 
        break;
    case "077Z": 
        $i02 = "Ok"; 
        break;
    case "123O": 
        $i02 = "Ok"; 
        break;
    case "321V": 
        $i02 = "Ok"; 
        break;
    case "007Z": 
        $i02 = "Ok"; 
        break;
    default:
      $i02 = false;
}
 
if ($i02 == false) 
 
echo <<<END

 $i01 référence à revoir . . .

END;
 
} else {
 
echo <<<END

 $i01 écrire dans fichier à finir . . .

END;
 
}
 
?>

Question de goût :)

23/05/2019, 10h35
zipbox

Super merci . . .

Cool . . .

;)

Citation:

Envoyé par MaitrePylos

Il y a aussi le switch qui est prévu pour ça...

On peut simplifier et "factoriser" :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 <?php switch ($i01) { case "001A": case "002B": case "003C": case "004D": case "021E": case "037F": case "077Z": case "123O": case "321V": case "007Z": $i02 = "Ok"; break; default: $i02 = false; }
Mais dans le contexte, la méthode de rawsrc est plus concise.

N.B. PAR CONTRE, il faudrait s'assurer de la présence ou pas d'un espace :

c'est "001A " ou "001A" ??
...

N.B.2 @MaitrePylos
Je ne suis pas fan d'affecter une chaine ("Ok") et/ou un booléen (false) à la même variable...
Restons cohérent :

"Ok" ou ""
true ou false