Tester le lancement d'une fonction

Version imprimable

Voir 40 message(s) de cette discussion en une page

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+

Voir 40 message(s) de cette discussion en une page