Tester le lancement d'une fonction

**vva** · 02/02/2009, 15h58

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 :

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

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);
}
}

**E.Bzz** · 02/02/2009, 16h29

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

Code :

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

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+

**vva** · 02/02/2009, 16h32

merci pour le code mais cela ne fonctionne pas plus

effectivement si la fonction doit être stoppée si tous les champs sont vides

**E.Bzz** · 02/02/2009, 16h38

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+

**vva** · 02/02/2009, 16h41

bien sur voici la page

Code :

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

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>

**Bovino** · 02/02/2009, 16h47

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 !

**E.Bzz** · 02/02/2009, 16h49

Envoyé par Bovino

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

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

Merci Bovino

**vva** · 03/02/2009, 10h07

bonjour

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

**E.Bzz** · 03/02/2009, 10h13

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

Envoyé par vva

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

Ca par contre, c'est pas normal

A+

**E.Bzz** · 03/02/2009, 10h14

Envoyé par vva

merci pour le code mais cela ne fonctionne pas plus

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

A+

**vva** · 03/02/2009, 10h17

à l'exécution si je mets les || j'ai object expected et avec les && pas d'erreur mais pas de blocage de la fonction

**E.Bzz** · 03/02/2009, 10h29

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

Code :

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

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+

**Bovino** · 03/02/2009, 10h31

Code :

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

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

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

Code :

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

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

**vva** · 03/02/2009, 10h35

Envoyé par E.Bzz

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

Code :

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

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

**vva** · 03/02/2009, 10h37

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.

**E.Bzz** · 03/02/2009, 10h40

Envoyé par vva

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

Code :

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

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

La même syntaxe partout

A+

**Bovino** · 03/02/2009, 10h42

Envoyé par vva

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

Ben... oui

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

C'est ce que tu demandais.

**vva** · 03/02/2009, 10h47

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 :

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

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

**E.Bzz** · 03/02/2009, 10h48

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+

**E.Bzz** · 03/02/2009, 10h49

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+

Tester le lancement d'une fonction

JavaScript

Discussions similaires

Partager

Partager