par
zeus » 22 août 2007, 14:23
le caractère '-' signifiant 'soustraction', tu ne doit pas l'utiliser dans le nom de tes variables.
Remplace ta variable $mise-a-jour par $mise_a_jour ou $miseAJour.
Même remarque pour tes variables POST, essaye de ne pas mettre d'espace.
Sinon, petite optimisation, plutôt que d'écrire ton if sur une seule ligne, de cette manière
if(isset($_POST['mise a jour'])) { $mise_a_jour = addslashes($_POST['mise a jour']); } else { $mise_a_jour = ''; }
Préfère l'indenter, pour la lisibilité, de cette manière
if(isset($_POST['mise a jour'])) {
$mise-a_jour = addslashes($_POST['mise a jour']);
} else {
$mise_a_jour = '';
}
Ou, encore mieux, passe par un
opérateur ternaire
$mise_a_jour = (isset($_POST['mise a jour'])) ? addslashes($_POST['mise a jour']) : '';
le caractère '-' signifiant 'soustraction', tu ne doit pas l'utiliser dans le nom de tes variables.
Remplace ta variable $mise-a-jour par $mise_a_jour ou $miseAJour.
Même remarque pour tes variables POST, essaye de ne pas mettre d'espace. ;)
Sinon, petite optimisation, plutôt que d'écrire ton if sur une seule ligne, de cette manière
[php]if(isset($_POST['mise a jour'])) { $mise_a_jour = addslashes($_POST['mise a jour']); } else { $mise_a_jour = ''; }[/php]
Préfère l'indenter, pour la lisibilité, de cette manière
[php]if(isset($_POST['mise a jour'])) {
$mise-a_jour = addslashes($_POST['mise a jour']);
} else {
$mise_a_jour = '';
}[/php]
Ou, encore mieux, passe par un [b][url=http://www.phpfrance.com/forums/voir_sujet-14111.php]opérateur ternaire[/url][/b]
[php]$mise_a_jour = (isset($_POST['mise a jour'])) ? addslashes($_POST['mise a jour']) : '';[/php]