Discussion :

[Fench]

Tu la place en dernier dans ton code mais c pas beau

ET le ; derrière la function ...

ET le return ...

[pasc06]

Non justement je veux que les alertes s'affichent séquentiellement du haut vers le bas car le formulaire est très long.

Par exemple si l'utilisateur a oublié de renseigner plusieurs champs, ça teste le formulaire du haut vers le bas en affichant les alertes une par une. Ceci afin que l'utilisateur s'y retrouve.

[Fench]

Non mais plus serieusement, vu la longueur du code de ta fonction
check_box_selected et le pb qui peut avoir sur les return.

ET que peut être le return de cette function n'est pas récupéré dans ta fonction principale ...

bref je mettrais ce bout de code dans ta fonction principale et je test ...
si ça marche c que le return n'est pas récupéré.

Moi j'suis pas un expert javascript (au début ton post n'étais pas sur ce forum) je n'est pas encore utilisé de return (que 2 mois que je fais du JavaSCript)

[Fench]

Parceque sinon j'aurais fait

Code :

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

1
2
 
return checkbox_selected(22,'id_region_');

[pasc06]

Le False est bien récupéré puisque ça m'affiche l'alerte et que le formulaire ne va pas vers "traitement.php".

Ce qu'il faut c'est bloquer les tests suivants !

Ca marche avec les tests précédents avec un "focus". J'ai essayé de le rajouter mais ça ne bloque toujours pas l'alerte suivante...

[E.Bzz]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
....
if (document.form1.id_ville.value == 0){
  alert("Veuillez entrer votre ville de résidence");
  return false;
}
 
if (! checkbox_selected(22,'id_region_'))
  return false;
 
if ((document.form1.jour_naissance.options[0].selected) && (document.form1.jour_naissance.value == "jour")){
  alert("Veuillez entrer votre jour de naissance");
  return false;
}
....

A+

[E.Bzz]

Parceque sinon j'aurais fait

Code :

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

1
2
 
return checkbox_selected(22,'id_region_');

Là, même si c'est bon, tu loupes le dernier contrôle

A+

[pasc06]

Merci à tous maintenant ça fonctionne !

Je remets le tout pour les prochains galériens ! :

L'appel de la fonction :

Code :

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

1
2
if (! checkbox_selected(22,'id_region_'))
return false;

Et la fonction :

Code :

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

1
2
3
4
5
6
7
8
9
function checkbox_selected(nb,idBox)
{
var i;
for (i=1;i<=nb;i++)
{
if (document.forms['form1'].elements[idBox+i].checked) return true;
}
alert("Veuillez entrer au moins une région ou vous exercez principalement");
}

Encore merci !

[E.Bzz]

Un petit oubli !
Pour rester cohérent :

Code :

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

1
2
3
4
5
6
7
8
9
10
function checkbox_selected(nb,idBox)
{
var i;
for (i=1;i<=nb;i++)
{
if (document.forms['form1'].elements[idBox+i].checked) return true;
}
alert("Veuillez entrer au moins une région ou vous exercez principalement");
return false;
}

A+

[pasc06]

Tu veux dire retirer ce return ?

SI oui j'avais fait la correction ! Le mieux serait que l'on supprime nos derniers messages afin d'avoir en dernier message la solution...!

Toujours dans un souci de cohérence...!

[Fench]

merci j'ai compris mon erreur avec

Code :

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

return checkbox_selected(22,'id_region_');

En fait je viens de comprendre de l'interêt du onsubmit="return fctdetest()" pour bien valider un formulaire.

Voilà deux choses que j'ai appris sur ce post, c cool, merci

[E.Bzz]

Tu veux dire retirer ce return ?

Non, le rajouter : il a disparu dans ta dernière version ...

A+

[pasc06]

Bin oui mais en le retirant ça marche aussi puisqu'il y a déjà un "return false" après l'appel de la fonction... Non ?

Code :

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

1
2
3
4
if (! checkbox_selected(22,'id_region_')){
	document.form1.id_region_1.focus();
 	return false;
}

J'ai retiré le "return false" qui était dans la fonction et apparemment ça marche...

[E.Bzz]

Bin oui mais en le retirant ça marche aussi puisqu'il y a déjà un return après l'appel de la fonction... Non ?

Non
Ce test

Code :

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

1
2
if (! checkbox_selected(22,'id_region_'))
return false;

repose sur le fait que checkbox_selected() renvoie false quand le test est KO.
Or elle ne renvoie plus que true ou rien.

Tu deviens donc dépendant du fait que le nav interprète comme "false" une absence de "return true", ce qui est déconseillé/dangereux/pas propre

A+

[pasc06]

Oula ça devient un peu trop technique !

Tout débutant et humble je suis...!

Donc l'appel de la fonction :

Code :

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

1
2
3
4
5
if (! checkbox_selected(22,'id_region_')){
alert("Veuillez entrer au moins une région ou vous exercez principalement");
document.form1.id_region_1.focus();
return false;
}

Et la fonction :

Code :

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

1
2
3
4
5
6
7
8
9
function checkbox_selected(nb,idBox)
{
var i;
for (i=1;i<=nb;i++)
{
if (document.forms['form1'].elements[idBox+i].checked) return true;
}
return false;
}

Ouf...!