Discussion :

[Nicolas94]

Bonjour,

Je dois tester une variable avec un script javascript.
En fait j'y arrive d'une manière mais de l'autre qui semble plus propre ca ne marche pas.

Code qui fonctionne :

Code :

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

1
2
3
4
5
6
7
8
 
function validInscription() {
	var login = document.getElementById('inscription').login.value;
	if (document.getElementById('inscription').login.value=="") {
		alert("loginvide");
		return false;
	}
}

Code qui ne fonctionne pas:

Code :

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

1
2
3
4
5
6
7
8
 
function validInscription() {
	var login = document.getElementById('inscription').login.value;
	if (login=="") {
		alert("loginvide");
		return false;
	}
}

Je ne vois pas d'ou vient le problème :p
Merci de votre aide.

Cordialement,
Nico

[johannesb]

Code qui fonctionne :

Code :

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

1
2
3
4
5
6
7
8
 
function validInscription() {
	var login = document.getElementById('inscription').login.value;
	if (document.getElementById('inscription').login.value=="") {
		alert("loginvide");
		return false;
	}
}

Qu'entendez-vous par "fonctionne" ?

Qu'est-ce que la fonction est censé retourner comme résultat si la valeur du champ n'est pas vide ?

Ici, faute d'un return explicite, elle renvoie undefined, ce qui sera facilement confondu avec false. Mais je ne sais pas si c'est ça, votre problème.

[RomainVALERI]

Heu...

Code :

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

1
2
3
4
5
6
7
8
function validInscription() {
	var login = document.getElementById('inscription').login.value;
	if (document.getElementById('inscription').login.value=="") {
		alert("loginvide");
		return false;
	}
}

Code qui ne fonctionne pas:

Code :

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

1
2
3
4
5
6
7
8
function validInscription() {
	var login = document.getElementById('inscription').login.value;
	if (login=="") {
		alert("loginvide");
		return false;
	}
}

Si j'ai bien compris, tu fais référence à un input dans un formulaire (dont l'id est inscription), en utilisant la propriété name. Mais pourquoi ne pas directement donner un id à ton input et le référencer :

Code javascript :

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

var login = document.getElementById("idDeTonInput").value;

? ^^
Ca y gagnerait en simplicité/lisibilité, et au niveau compatibilité entre navigateurs je ne suis pas sûr que la syntaxe basée sur le name soit très fiable...

[Nicolas94]

Bonjour,

Merci de vos remarques.
J'utilise un getElementById car je ne peux pas utiliser de name=.. dans la déclaration de mon formulaire pour des soucis de validation xhtml.

Pour les soucis de navigateurs le code fonctionne donc je ne vois pas le rapport.

Mon problème est juste que la seconde façon d'écrire le code est plus "propre", car elle reprend une variable ou est est affectée une valeur.

Cependant ça ne fonctionne pas, et je ne sais pas pourquoi.

Merci Svp de se concentrer sur ça sans vouloir changer ce qui ne nécessite pas (il me semble) de l'être.

Cordialement,
Nico.

[johannesb]

J'utilise un getElementById car je ne peux pas utiliser de name=.. dans la déclaration de mon formulaire pour des soucis de validation xhtml.

Je ne comprends pas le souci de validation, mais en tout état de cause, c'est bien d'utiliser document.getElementById qui est régi par une norme W3C plutôt que telle ou telle fantaisie plus ou moins documentée de tel ou tel navigateur.

Pour les soucis de navigateurs le code fonctionne donc je ne vois pas le rapport.

Moi, mon souci, c'est que je ne comprends pas le vôtre

Qu'appelez-vous "fonctionner" ? Qu'est-ce que votre fonction est censée faire, qu'elle fait dans le premier cas et non dans le second ?

[vermine]

Bonjour,

Dites-nous aussi ce que retourne un alert de:

Code :

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

document.getElementById('inscription').login.value

et de login après affectation.

[RomainVALERI]

Et puis tu nous montres une fonction JS de validation d'éléments HTML dont nous n'avons pas le code...
Poste un extrait de l'html correspondant à ton formulaire, qui *je suppose mais c'est lourd d'être obligé de supposer* doit peut-être appeler la fonction de validation comme ça ? (mais avec ou sans le return ça fait une différence ^^)

Code html :

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

<form id="inscription" action="blabla.php" onsubmit="return validInscription();">

Sinon, à part ça : je m'accroche encore un peu parce que je sais que la communication par forum interposé n'est pas forcément évidente, mais si tu te vexes encore je passe à autre chose et on en parle plus hein...

[Bovino]

Le nom de ta variable est le même que celui de ton champ, cela perturbe JavaScript qui ne sait pas quel objet choisir.

Essaye avec

Code :

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

1
2
3
4
5
6
7
function validInscription() {
	var mon_login = document.getElementById('inscription').login.value;
	if (mon_login=="") {
		alert("loginvide");
		return false;
	}
}

[johannesb]

Dites-nous aussi ce que retourne un alert de:

Code :

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

document.getElementById('inscription').login.value

et de login après affectation.

Dans la mesure où il y a très peu de chances pour que l'objet DOM retourné par document.getElementById('inscription'), à supposer qu'il existe tout court et qu'il existe en plus déjà au moment de l'appel de la fonction, ait une propriété appelée "login" laquelle aurait elle-même une propriété appelée "value", ça m'intéresserait aussi !

[Nicolas94]

Merci Bovino pour la réponse.
Mon problème était en effet là ou tu le dis.

Merci aux autres aussi de vous être penché sur le problème.

Cordialement,
Nico.