Discussion :

[zipbox]

Bonjour,

Voici le début de mon bout de code PHP :

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
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 :

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

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 . . .

[rawsrc]

Salut,

comme ça :

Code :

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

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

[MaitrePylos]

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

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
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

[zipbox]

Super merci . . .

Cool . . .

[Invité]

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

On peut simplifier et "factoriser" :

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
 
<?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