Forum d'entraide PHPFrance

Bonjour,

Je souhaites ajouter unje condition à cette ligne :

if (isset($_GET['id']))
 {
                 $id = $_GET['id'];  //ICI
				 }
				 else
				 {
                 $id = $_POST['id'];  //ICI
				 }

Je souhaite dire que si $id = rien alors $id= 18

Pourriez vous me dire comment faire ?
merci d'avance

if ($id == '')
{
   $id = 18;
}

Tu avais déjà fait la condition en français, j'ai du mal à saisir là où tu bloquais en fait.

Je vais me faire fusillier par Genova, mais préfère faire : empty($var), plutôt que $var == ''. Si jamais ta valeur est null, true, ou false, le test de Genova ne risque pas de marcher

@HyWaN


Normalement

if (empty($var))

et

if ($var == '')

c'est la même chose, car PHP considère NULL, FALSE, 0 et ' comme la même chose dans le cadre d'une comparaison sans les types.

Me trompe-je ?

Euh sûrement. Tu me fous le doute du coup.

PHP est amené à être moins souple sur le typage, alors autant utiliser les fonctions appropriées maintenant (jm'en sors pas trop mal là )

Mais quand même, préfère empty à une comparaison simple à chaîne vide. Tu es sûr de ton résultat comme ça.

Bon merci pour la réponse !!

Mais je mets quoi alors ???

Je te propose ceci :

$id = (isset($_GET['id']) && !empty($_GET['id'])) ? $_GET['id'] : ((isset($_POST['id']) && !empty($_POST['id'])) ? $_POST['id'] : 18);

En d'autre termes (si jamais ce code te semble digne des plus obscurs codes des geek maniaco-délirants) :
Si $_GET['id'] est défini et non vide, on assigne la valeur à $id, sinon, si $_POST['id'] est défini et non vide, alors on l'assigne à $id et si rien de tout ça n'est vrai, alors on affecte 18 à $id.

Ceci est un opérateur ternaire (avec un second ternaire imbriqué. Au besoin, regarde dans la FAQ, il y a un sujet qui traite des opérateurs ternaires.

merde, jsuis un geek obscure maniaco-délirant ... c'est grave docteur ?
Y en a-t-il d'autre parmi ce forum ? Je dénonce (et toc) Cyrano d'en être un déjà ! Plus on est de fous, euh de maniaco-délirants, plus on rit ! (c'est bien connu).

Pour des raisons de clarté, vous pouvez vous passer d'isset() puisqu'empty() vérifie si la variable est définie sans émettre d'erreur.

Voici en plus une bonne raison d'utiliser empty : il vérifie aussi l'existence d'une variable

+1 pour Hubert.

Euh oui, mais non.. c'est pas non plus à généraliser...

Si la variable n'est pas définie, empty() retournera vrai sans emettre d'avertissement, mais si la variable est appellée ensuite vous y aurez droit quand même.

Ce n'est vallable, comme le précise Hubert, que pour plus de lisibilité dans ce cas : le isSet() n'est pas nécessaire si l'on teste juste la variable avec empty(), mais cette fonction ne dispense pas de l'appel a isSet() pour vérifier que la variable existe avant de pouvoir l'utiliser