Tester le lancement d'une fonction

Version imprimable

Bonjour

J'ai le code suivant sur l'évènement onclick d'un bouton dasn un formulaire et j'aimerai bloquer l'exécution de la fonction si la condition est remplie mais la fonction d'exécute quand même quelle est la solution merci

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
function Ajax(Div,Nom,Entite,Tel,Iso,Reso)
{
if ((document.forms.perso.nom.value == "") && (document.perso.elements['ag'].options[document.perso.elements['ag'].selectedIndex].value == "" ) && (document.forms.perso.tel.value == "") && (document.forms.perso.iso.value == "") && (document.forms.perso.reso.value = ""))
{
Alert('Un critère doit être renseigné');
return false;
}
else
{
var xhr = getXhr()
// On défini ce qu'on va faire quand on aura la réponse
xhr.onreadystatechange = function()
{
// On ne fait quelque chose que si on a tout reçu et que le serveur est ok
if(xhr.readyState == 4)
{
document.getElementById(Div).innerHTML=xhr.responseText;
}
}
xhr.open("GET","Code.asp?Nom="+Nom+"&Entite="+Entite+"&Tel="+Tel+"&Iso="+Iso+"&Reso="+Reso,true);
xhr.send(null);
}
}

Bonjour,
en essayant une syntaxe plus "récente" ?
Code:

1 2 3 if ((document.forms['perso'].elements['nom'].value == "") && (document.forms['perso'].elements['ag'].options[document.forms['perso'].elements['ag'].selectedIndex].value == "" ) && (document.forms['perso'].elements['tel'].value == "") && (document.forms['perso'].elements['iso'].value == "") && (document.forms['perso'].elements['reso'].value = "")) { ...
et en t'y tenant ;)

EDIT : la fonction sera lancée si au moins un des champs est renseigné. C'est bien le but ?

A+

02/02/2009, 15h32
vva

merci pour le code mais cela ne fonctionne pas plus

effectivement si la fonction doit être stoppée si tous les champs sont vides
02/02/2009, 15h38
E.Bzz

:koi:

A part la syntaxe un peu ancienne, ça aurait déjà dû (pu) fonctionner avant :?

On peut voir le HTML (généré) des champs associés ?

A+

bien sur voici la page

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
 
<!--	Insertion de la page de style	-->
<link rel="stylesheet" type="text/css" href="../../style.css" />
<!--	Insertion des script JS communs	-->
<script LANGUAGE="javascript" src="../../Source.js"></script>
<script type="text/JavaScript">
function Ajax(Div,Nom,Entite,Tel,Iso,Reso)
{
 
if ((document.forms['perso'].elements['nom'].value == "") && (document.forms['perso'].elements['ag'].options[document.forms['perso'].elements['ag'].selectedIndex].value == "" ) && (document.forms['perso'].elements['tel'].value == "") && (document.forms['perso'].elements['iso'].value == "") && (document.forms['perso'].elements['reso'].value = ""))
	{
	Alert('Un critère doit être renseigné');
	return false;
	}
else
	{
	var xhr = getXhr()
	// On défini ce qu'on va faire quand on aura la réponse
	xhr.onreadystatechange = function()
	{
		// On ne fait quelque chose que si on a tout reçu et que le serveur est ok
		if(xhr.readyState == 4)
		{
		document.getElementById(Div).innerHTML=xhr.responseText;
		}
	}
	xhr.open("GET","Code.asp?Nom="+Nom+"&Entite="+Entite+"&Tel="+Tel+"&Iso="+Iso+"&Reso="+Reso,true);
	xhr.send(null);
	}
}
</script>
<form name="perso">
<table border="0">
<tr>
	<td align="center">Nom et prénom</td>
	<td align="center">Entité</td>
	<td align="center">Téléphone direct</td>
	<td align="center">Initiales ISO</td>
	<td WIDTH="15">&nbsp;</td>
	<td align="center">ID Centris</td>
</tr>
<tr>
	<td><input type="text" name="nom" size="27"></td>
	<td align="center"><!--Importation de la combo des entités--><select name=ag><option value=''></option><option value='73' >Agence de Direction Ouest                                                       </option></select></td>
	<td align="center"><input type="text" name="tel" size="15"></td>
	<td align="center"><input type="text" name="iso" size="2"></td>
	<td WIDTH="15"></td>
	<td align="center"><input type="text" name="reso" size="5"></td>
	<td><img src="../../images/valid.jpg" border="0" alt="Rechercher" name="Valid" onclick="Ajax('collabResult',document.forms.perso.nom.value,ag.options[ag.options.selectedIndex].value,document.forms.perso.tel.value,document.forms.perso.iso.value,document.forms.perso.reso.value);" WIDTH="25" HEIGHT="26"></td>
	<td><img src="../../images/cancel.jpg" border="0" alt="Effacer" name="Cancel" onclick="location.replace('ListeCollab.asp');" WIDTH="25" HEIGHT="25"></td>
</tr>
</table>
</form>
<hr />
<div id="collabResult"></div>
<script Language="JavaScript">ZoneFocus(document.perso.nom)</script>

02/02/2009, 15h47
Bovino

Tu mets des &&, donc le return false ne sera réalisé que quand TOUS les champs seront vides...
C'est ploutôt des || (ou) qu'il faut mettre !
02/02/2009, 15h49
E.Bzz

Citation:

Envoyé par Bovino

C'est ploutôt des || (ou) qu'il faut mettre !

8O

Je savais que j'avais du mal le lundi, mais là :aie:

Merci Bovino ;)
03/02/2009, 09h07
vva

bonjour

Citation:

Envoyé par Bovino

Tu mets des &&, donc le return false ne sera réalisé que quand TOUS les champs seront vides...
C'est ploutôt des || (ou) qu'il faut mettre !

justement le code doit stoper le changement de page si tous les champs sont vides
et si je mets les || j'ai une erreur sur la ligne
03/02/2009, 09h13
E.Bzz

Citation:

Envoyé par vva

justement le code doit stoper le changement de page si tous les champs sont vides

Ouf !
Ce lundi n'était pas si terrible :mrgreen:

Citation:

Envoyé par vva

et si je mets les || j'ai une erreur sur la ligne

Ca par contre, c'est pas normal :?

A+
03/02/2009, 09h14
E.Bzz

Citation:

Envoyé par vva

merci pour le code mais cela ne fonctionne pas plus

Donc : une erreur ?
Au chargement, à l'exécution ?

A+
03/02/2009, 09h17
vva

à l'exécution si je mets les || j'ai object expected et avec les && pas d'erreur mais pas de blocage de la fonction
03/02/2009, 09h29
E.Bzz

Il faudrait utiliser la même syntaxe dans l'appel de la fonction

Code:

onclick="Ajax('collabResult',document.forms.perso.nom.value,ag.options[ag.options.selectedIndex].value,document.forms.perso.tel.value,document.forms.perso.iso.value,document.forms.perso.reso.value);"

(ainsi que dans celui de ZoneFocus() )

A+
03/02/2009, 09h31
Bovino

Code:

Alert('Un critère doit être renseigné');

JavaScript est sensible à la casse et tu es trop zélé avec les majuscules ;)

Code:

alert('Un critère doit être renseigné');
03/02/2009, 09h35
vva

Citation:

Envoyé par E.Bzz

Il faudrait utiliser la même syntaxe dans l'appel de la fonction

Code:

onclick="Ajax('collabResult',document.forms.perso.nom.value,ag.options[ag.options.selectedIndex].value,document.forms.perso.tel.value,document.forms.perso.iso.value,document.forms.perso.reso.value);"

(ainsi que dans celui de ZoneFocus() )

A+

désolé mais je ne comprends pas ce que tu me demandes
03/02/2009, 09h37
vva

exact bovino je suis un peu léger avec la syntaxe, cela fait un moment que je n'ai pas refait de JS et je dois me faire violence

avec le alert au lieu de Alert je n'ai plus d'erreur par contre la fonction est bloquée si tous les champs ne sont pas renseignés.
03/02/2009, 09h40
E.Bzz

Citation:

Envoyé par vva

désolé mais je ne comprends pas ce que tu me demandes

Code:

onclick="Ajax('collabResult',document.forms['perso'].elements['nom'].value, ....

La même syntaxe partout ;)

A+
03/02/2009, 09h42
Bovino

Citation:

Envoyé par vva

la fonction est bloquée si tous les champs ne sont pas renseignés.

Ben... oui :koi:

Citation:

justement le code doit stoper le changement de page si tous les champs sont vides

C'est ce que tu demandais.
:?
03/02/2009, 09h47
vva

je sèche complet, je viens même de changer de méthode en récupérant les valeur passées en paramètre à la fonction car il s'agit des même et pas mieux

Code:

if ((Nom == "") && (Entite == "" ) && (Tel == "") && (Iso == "") && (Reso = "") && (Lieu == "" ))
03/02/2009, 09h48
E.Bzz
Bon, comme il y a ambigüité, autant récapituler :
- Tous les champs vides => alert()
- Un champ (ou plus) est renseigné => Ajax()
C'est ça ?

Si oui, qu'est ce que tu obtiens quand :
- Tous les champs vides
- Un champ (ou plus) est renseigné
? :?

A+
03/02/2009, 09h49
E.Bzz

Citation:

Envoyé par vva

en récupérant les valeur passées en paramètre à la fonction car il s'agit des même

En ayant changé la syntaxe dans l'appel de la fonction ??

A+
03/02/2009, 09h53
vva
Citation:
Envoyé par E.Bzz

Bon, comme il y a ambigüité, autant récapituler :

Tous les champs vides => alert()
Un champ (ou plus) est renseigné => Ajax()

C'est ça ?

exact tu résume bien la situation

Si oui, qu'est ce que tu obtiens quand :

Tous les champs vides
Un champ (ou plus) est renseigné

? :?

A+
le code n'est apparemment pas pris en compte, j'ai essayé juste avec un test sur le champ nom et pas de problème, si vide blocage de la fonction mais si j'ajoute un autre champ, le test passe outre
03/02/2009, 09h56
E.Bzz

Citation:

Envoyé par E.Bzz

En ayant changé la syntaxe dans l'appel de la fonction ??

A+
03/02/2009, 10h02
vva

j'ai modifié l'appel de la fonction de cette manière

Code:

onclick="Ajax('collabResult',document.forms['perso'].elements['nom'].value,document.forms['perso'].elements['ag'].options[document.forms['perso'].elements['ag'].selectedIndex].value,document.forms['perso'].elements['tel'].value,document.forms['perso'].elements['iso'].value,document.forms['perso'].elements['reso'].value,document.forms['perso'].elements['agabv'].options[document.forms['perso'].elements['agabv'].selectedIndex].value);"
03/02/2009, 10h10
E.Bzz

OK pour les paramètres.

Mais dis donc, je remarque que maintenant mais elle est un peu light la structure de la page :?
Je ne vois pas de tag <html> ni <body> ...

On a tout le code là ?

Faudrait peut être nous redonner une version actualisée, d'ailleurs.

A+
03/02/2009, 10h18
vva

il y a effectivement des balises manquantes car la page est imbriquée dans une page mère par un server.execute en asp
03/02/2009, 10h21
E.Bzz

Citation:

Envoyé par vva

il y a effectivement des balises manquantes car la page est imbriquée dans une page mère par un server.execute en asp

Ah 8O

Ca aurait été intéressant de la savoir dès le début :roll:

Il y a tout le code de la page affiché dans le nav, là ?

A+

voici le code source de la page

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
<%@ Language=VBScript %>
<!--#include file ="bib.asp"-->
<!--	Insertion de la page de style	-->
<link rel="stylesheet" type="text/css" href="../../style.css" />
<!--	Insertion des script JS communs	-->
<script LANGUAGE="javascript" src="../../Source.js"></script>
<script type="text/JavaScript">
function Ajax(Div,Nom,Entite,Tel,Iso,Reso,Lieu)
{
//if ((document.forms['perso'].elements['nom'].value == "") && (document.forms['perso'].elements['ag'].options[document.forms['perso'].elements['ag'].selectedIndex].value == "" ) && (document.forms['perso'].elements['tel'].value == "") && (document.forms['perso'].elements['iso'].value == "") && (document.forms['perso'].elements['reso'].value = "") && (document.forms['perso'].elements['agabv'].options[document.forms['perso'].elements['agabv'].selectedIndex].value == "" ))
if ((Nom == "") && (Entite == "" ) && (Tel == "") && (Iso == "") && (Reso = "") && (Lieu == "" ))
{
	alert('Il faut au moins renseigner un critère');
	return false;
}
else
{
	var xhr = getXhr()
	// On défini ce qu'on va faire quand on aura la réponse
	xhr.onreadystatechange = function()
	{
		// On ne fait quelque chose que si on a tout reçu et que le serveur est ok
		if(xhr.readyState == 4)
		{
		document.getElementById(Div).innerHTML=xhr.responseText;
		}
	}
	xhr.open("GET","Code.asp?Nom="+Nom+"&Entite="+Entite+"&Tel="+Tel+"&Iso="+Iso+"&Reso="+Reso+"&Lieu="+Lieu,true);
	xhr.send(null);
}
}
</script>
<form name="perso">
<table border="0">
<tr>
	<td align="center"><%=sNom%></td>
	<td align="center"><%=sAffectation%></td>
	<td align="center"><%=sLieu%></td>
	<td align="center"><%=sTel%></td>
	<td align="center"><%=sInit%></td>
	<td WIDTH="15">&nbsp;</td>
	<td align="center"><%=sReso%></td>
</tr>
<tr>
	<td><input type="text" name="nom" size="27"></td>
	<td align="center"><!--Importation de la combo des entités--><!--#include file ="../../Modules/ComboAG.asp"--></td>
	<td align="center"><!--Importation de la combo des lieux de travail--><!--#include file ="../../Modules/ComboAG_ABV.asp"--></td>
	<td align="center"><input type="text" name="tel" size="15"></td>
	<td align="center"><input type="text" name="iso" size="2"></td>
	<td WIDTH="15"></td>
	<td align="center"><input type="text" name="reso" size="5"></td>
	<td><img src="../../images/valid.jpg" border="0" alt="<%=sValider%>" name="Valid" onclick="Ajax('collabResult',document.forms['perso'].elements['nom'].value,document.forms['perso'].elements['ag'].options[document.forms['perso'].elements['ag'].selectedIndex].value,document.forms['perso'].elements['tel'].value,document.forms['perso'].elements['iso'].value,document.forms['perso'].elements['reso'].value,document.forms['perso'].elements['agabv'].options[document.forms['perso'].elements['agabv'].selectedIndex].value);" WIDTH="25" HEIGHT="26"></td>	
	<td><img src="../../images/cancel.jpg" border="0" alt="<%=sEffacer%>" name="Cancel" onclick="location.replace('ListeCollab.asp');" WIDTH="25" HEIGHT="25"></td>
</tr>
</table>
</form>
<hr />
<div id="collabResult"></div>
<script Language="JavaScript">ZoneFocus(document.perso.nom)</script>

03/02/2009, 10h35
E.Bzz

OK
Je pense donc qu'il te faut revoir la génération initiale de la page pour obtenir une page HTML complète et valide.

Tout débugage correct est impossible autrement ...

(ne serait-ce que parce que la syntaxe JS "document." fait référence à l'élément <body> alors que celui-ci n'est pas présent dans ta page)

A+
03/02/2009, 10h40
vva

question de novice

ok mais pourquoi alors la fonction ajax fonctionne, elle transmet les valeurs correctes à la page asp et le retour d'info s'affiche correctement ?
03/02/2009, 10h49
vva

ok je viens de trouver l'erreur en fait le lundi est dur mais le mardi aussi

Code:

if ((Nom == "") && (Entite == "" ) && (Tel == "") && (Iso == "") && (Reso = "") && (Lieu == "" ))

en regardant de plus près on voit un espace de trop dans la dernière condition et surtout il manque un = au test de reso:evilred::evilred::evilred:
03/02/2009, 10h52
vva

au fait j'ai oublié MERCI à tous les deux
03/02/2009, 10h52
E.Bzz

:ccool:

... ce qui ne remet pas en cause ma dernière remarque ;)

A+