Discussion :

[flavors]

Bonjour.

J'ai une petite question concernant la bonne pratique pour la "mise en page" (je ne suis pas sûr que ce soit le bon terme) du code dans le cas d'un if assez long.
Je m'explique, je dois faire un test sur des données postées d'un formulaire.
J'ai des arrays et je suis dans l'impossibilité de les tester avec "empty" comme ils sont issues du formulaire.

Je me demande donc quelle est la meilleur pratique.
Faut-il mieux avoir un if long comme ceci:

Code :

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

1
2
 
if(empty($_POST['id']) && empty($_POST['num']) && empty(array_filter($_POST['bb'])) && empty(array_filter($_POST['aa'])) && empty(array_filter($_POST['b4'])) && empty(array_filter($_POST['b5'])) && empty(array_filter($_POST['b6'])) && empty(array_filter($_POST['b8'])) && empty(array_filter($_POST['a8'])) && empty(array_filter($_POST['a9'])) && empty(array_filter($_POST['a0'])) && empty(array_filter($_POST['a1'])) && empty(array_filter($_POST['a2']))) {

ou mettre les arrays dans des variables

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
$tcPOST = array_filter($_POST['tc']);
				$bbPOST = array_filter($_POST['bb']);
				$aaPOST = array_filter($_POST['aa']);
				$a1POST = array_filter($_POST['a1']);
				$a2POST = array_filter($_POST['a2']);
				$a3POST = array_filter($_POST['a3']);
				$b1POST = array_filter($_POST['b1']);
				$b2POST = array_filter($_POST['b2']);
				$b3POST = array_filter($_POST['b3']);
				$b4POST = array_filter($_POST['b4']);
				$b5POST = array_filter($_POST['b5']);
 
				if(empty($_POST['id']) && empty($_POST['num']) && empty($bbPOST) && empty($aaPOST) && empty($a1POST) && empty($a2POST) && empty($a3POST) && empty($b1POST) && empty($b2POST) && empty($b3POST) && empty($b4POST) && empty($b5POST) ) {

Si quelqu'un peut me dire comment faut-il mieux procéder, ce serait gentil.

Merci.

[Invité]

Bonjour,

pourquoi as-tu besoin d'utiliser array_filter() ?
Tu veux tester que TOUTES les valeurs reçues pour chaque array ne sont pas vide ?

Sinon, pour chaque array, si tu veux AU MOINS une valeur, empty suffit.

D'autre part, on peut simplifier l'écriture :

if( !empty( $var1, $var2, $array1, $array2,......... ) ){

[grunk]

D'autre part, on peut simplifier l'écriture :

Code :

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

if( !empty( $var1, $var2, $array1, $array2,......... ) ){

empty() ne supporte pas plusieurs paramètres

[Invité]

Oups! J'ai naïvement cru qu'il fonctionnait comme isset().

Voilà une modification à prévoir pour PHP 8 !



OK. Je remballe mes gaules...

[flavors]

Bonjour et merci pour vos réponses.

Effectivement je veux tester si toutes mes valeurs sont bien vides.
Dans un 1er temps j'avais utilisé empty mais les arrays provenant d'un formulaire, ils ne sont jamais vide. J'ai donc vu que l'utilisation d'array_filter sans callback permet de suppimer les "vides"(en plus des 0, FALSE), seulement l'utilisation de cette fonction rallonge encore mon if (qui n'étais pas court de base). C'est pour ça que je demande si il y a une meilleur façon de procéder ou sinon de savoir s'il fallait mettre les $_POST filtrés dans des variables pour raccourcir le if ou tout laisser en brut dans le if.

La condition fonctionne bien, c'est plus une question sur la "bonne pratique" qui donne un code plus propre.

[grunk]

Tu t'en rend compte, une multitude de condition est illisible.
Dans ce cas il est sans doute plus judicieux de dédier une fonction à la tache.

Quelque chose comme ça devrait faire le job (pas testé)

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
if(areFieldsEmpty()) {
    //....
}
 
function areFieldsEmpty() {
    foreach($_POST as $field) {
		$result = null;
		if(is_array($field)) {
			$result = empty(array_filter($field));
		} else {
			$result = empty($field);
		}
 
		if(!$result)
			return false;
    }
 
	return true;
}

[flavors]

Merci pour votre réponse.

Effectivement le fait de passer par une boucle et de ce servir du retour de la fonction pour la condition est plus pratique.
J'essaie d'utiliser des boucles de plus en plus pour rendre mon code le plus lisible possible, mais je n'ai pas toujours le reflex (ça va venir).

Merci encore et bonne journée

[rawsrc]

Salut,

juste en passant,

il est simple d'y parvenir en utilisant la puissance des fonctions tableaux de PHP :
Quelque chose dans ce genre :

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
// suppose que tu aies des données de cet acabit
// et que tu ne veuilles conserver que les données  !empty()
$data = [
    'tc' => [],
    'bb' => [1, 2, 3],
    'aa' => [],
    'a1' => [],
    'a2' => [4, false, 6],
    'a3' => [],
    'b1' => [],
    'b2' => [],
    'b3' => [7, 8, false],
    'b4' => [],
    'b5' => []
];
 
// ici tu listes les clés qui subiront le traitement afin de ne pas impacter les autres
$keys = ['tc', 'bb', 'aa', 'a1', 'a2', 'a3', 'b1', 'b2', 'b3', 'b4', 'b5'];
// et là tu extrais et filtres en une passe
$x = array_filter(array_map('array_filter', array_intersect_key($data, array_flip($keys))));

Tu obtiens :

Code :

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

1
2
3
4
5
$x = [
    'bb' => [1, 2, 3],
    'a2' => [4, 6],
    'b3' => [7, 8]
];

Plus simple je ne vois pas...

[sabotage]

Dans un 1er temps j'avais utilisé empty mais les arrays provenant d'un formulaire, ils ne sont jamais vide.

Je ne comprends pas cette assertion :
- si l'utilisateur ne remplit rien, ça sera empty
- si l'utilisateur met 0, ça sera empty
- si l'utilisateur modifie le formulaire en retirant le champs, ça sera empty

[flavors]

Bonjour et merci pour les réponses.

@Sabotage
Je ne comprends pas car quand je fais un test du style :

Code :

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

1
2
3
 
if(empty($_POST['a1']))
	echo "vide";

avec a1 issue d'un formulaire

Code :

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

1
2
3
4
5
6
 
<form>
	<input id="a1_1" type="text"  name="a1[]" />
	-> 
	<input id="a1_2" type="text"  name="a1[]" /> 
</form>

si je ne remplie pas les champs, le test n'est pas validé même quand je met les champs à 0 (à moins que je fasse une mauvaise utilisation de "empty").
C'est pourquoi j'utilise array_filter.

[grunk]

Plus simple je ne vois pas...

Un code concis est rarement très lisible. 4 fonctions imbriquées ça demande quand même une certaines gymnastique d'esprit quand tu dois le relire

[rawsrc]

@grunk

ouaip, mais ça dépend si tu as fait ou pas PHP en deuxième langue vivante ;-)

[Celira]

Effectivement, un tableau d'input text de ce type :

Code :

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

1
2
3
4
5
<form>
	<input id="a1_1" type="text"  name="a1[]" />
	-> 
	<input id="a1_2" type="text"  name="a1[]" /> 
</form>

ne sera pas vide. Il sera rempli de champs vides, ce qui n'est pas la même chose.

La seule exception est si il s'agit d'un tableau de checkboxes, car les checkboxes ne sont pas envoyées si elles ne sont pas cochées. Donc un tableau de checkboxes non cochées sera vide.