Discussion :

[moimp]

Bonjour,
J'ai écrit une fonction d'initialisation de variable qui semble marcher mais j'ai des doutes sur la bonne pratique et les éventuels risques. J'aimerais donc avoir un avis.

Code :

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

1
2
3
4
5
6
7
8
9
<?php
function defaultEmptyValue($var, $value=null)
{
	if (!isset($var) or empty($var)) $var = $value;
	return $var;
}
 
$maVar = @defaultEmptyValue($maVar);
?>

Mon but est le suivant:
Si la variable a une valeur significative, je n'y touche pas,
Si elle n'est pas définie ou si elle a une valeur équivalente à empty, elle prend par défaut la valeur du second paramètre.

[sabotage]

Sur la forme, plus simplement :

Code :

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

if (empty($var)) $var = $value;

[moimp]

Merci. Effectivement avec cette simplification, ça marche. Mais je ne comprends pas pourquoi la ligne 4 ne provoque pas d'erreur sans isset($var). En effet, à la ligne 8, j'ai pris la précaution de mettre un @ mais la variable n'est toujours pas définie à la ligne 4.

[mathieu]

il vaut mieux éviter d'utiliser @ et tester avec "isset" si la variable existe

Code :

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

1
2
3
if (!isset($variable) || empty($variable)) {
    $variable = $valeurParDefaut;
}

[Invité]

@mathieu,
relis la doc. :
!isset est "inclus" dans empty.

empty ( mixed $var )

Détermine si une variable est considérée comme vide.
Une variable est considérée comme vide si elle n'existe pas, ou si sa valeur équivaut à FALSE.
La fonction empty() ne génère pas d'alerte si la variable n'existe pas.

Ce qui suit est considéré comme étant vide :

• "" (une chaîne vide)
• 0 (0 en tant qu'entier)
• 0.0 (0 en tant que nombre à virgule flottante)
• "0" (0 en tant que chaîne de caractères)
• NULL
• FALSE
• array() (un tableau vide)
• $var; (une variable déclarée, mais sans valeur)

[sabotage]

Pour éviter le @, tu pourrais faire

Code :

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

1
2
3
4
5
6
function defaultEmptyValue($var, $value=null)
{
	if (empty($$var)) $$var = $value;
	return $var;
}
defaultEmptyValue('unevar', 'valeur');

Le fond de tout ça c'est quand même de savoir pourquoi tu aurais des variables inderterminées.

[mathieu]

@mathieu,
relis la doc. :
!isset est "inclus" dans empty.

ah d'accord, je n'utilise jamais la fonction "empty" et il a indiqué avoir une erreur de variable indéfinie donc je pensais que ça venait de là