Discussion :

[sam01]

Bonjour à tous,

je fais un test sur champ pour savoir s'il est vide ou rempli :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 
if(isset($_POST["telephone"])) {$telephone = $_POST['telephone'];} else { $telephone = "";}
 
// ci-dessous le code qui nous intéresse
 
if (isset($telephone))
{
if (myIsInt($telephone) == false) { $erreur[] = '<center><font color="#FF0000">Le champ <b>Telephone</b> doit être numerique!</font></center>';}
}

le souci c'est que mon champ telephone n'est pas rempli et la condition if s'exécute quand même. En gros j'ai le message d'erreur le champs doit être numérique alors qu'il n'est même pas rempli.

merci d'avance pour votre aide.

[yiannis]

bonjour,

isset ne teste pas si ta variable est vide mais si elle existe

Code :

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

if (isset($telephone) && trim($telephone) != "")

avec ca cela devrait fonctionner

[guitou12]

Moi j'aurait dit

Code :

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

if (!empty($telephone))

Mais bon ça revient au même

[Andalor]

Salut.

Je ne connais vraiment la fonction de ton code, donc je vais peut être dire une connerie.
Mais si ta variable telephone vient d'une autre page tu devrais mettre :

if (isset($_GET['telephone']))

Au lieu de

if (isset($telephone))

Enfin, j'avais un soucis similaire avec les isset en passant à la version php5 à cause des POST et des GET qui étaient devenus obligatoire.

[yiannis]

oui c'est vrai guitou12, mais le probleme avec empty(), c'est que si l'utilisateur appuie sur la barre d'espacement, l'espace n'est pas nul ni vide et la, tu ne rentrera plus dans ta condition

[guitou12]

oui c'est vrai guitou12, mais le probleme avec empty(), c'est que si l'utilisateur appuie sur la barre d'espacement, l'espace n'est pas nul ni vide et la, tu ne rentrera plus dans ta condition

C'est vrai de toute façon pour un test de numéro de téléphone ça reste léger il faudrait faire un regex ou plus simplement calculer la longueur de la chaine et vérifier qu'on a que des chiffres

[guitou12]

Salut.

Je ne connais vraiment la fonction de ton code, donc je vais peut être dire une connerie.
Mais si ta variable telephone vient d'une autre page tu devrais mettre :

if (isset($_GET['telephone']))

Au lieu de

if (isset($telephone))

Enfin, j'avais un soucis similaire avec les isset en passant à la version php5 à cause des POST et des GET qui étaient devenus obligatoire.

Salut, si tu avais lu son code tu aurais vu que $telephone est soit vide soit à la valeur de $_POST['telephone'], c'était écrit 2 lignes plus haut tu n'as même pas daigné regarder avant de répondre.......

[yiannis]

dis moi sam01, ta fonction myIsInt() fait quelque chose de particulier ou elle teste juste si $telephone est numerique?
Parce que tu sais tu a une fonction php pour le faire: is_numeric()

[sam01]

bonjour,

is-numeric n'était pas suffisant, voici ma fonction :

Code :

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

1
2
3
4
5
6
 
 
function myIsInt($x)
									{  
										return(is_numeric($x) ? intval(0+$x) == $x : false);   
									}

merci pour ta réponse. je teste cela.