Discussion :

[vva]

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]

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]

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]

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]

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]

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]

merci pour le code mais cela ne fonctionne pas plus

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

A+

[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

[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+

[Bovino]

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é');