Discussion :

[ekinoxe]

Bonjour

J'ai un petit problème avec le controle de mon formulaire que je développe sous asp.

voici le type de controle que j'utilise

Code :

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

1
2
3
4
5
6
7
8
9
/**Méthode vérifiant que la valeur saisie soit bien un entier*/
   function isInteger(valeur,formulaire){
      var re = /^\d+$/;  
       if(!re.test(valeur)){
	  	alert("Vous n'avez pas saisie d'entier !!!");
           return false;
      }
	   return true;
   }

et voici un epartie du code de mon formulaire

Code :

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

response.write "<td><input type=textarea SIZE="&length&" MAXLENGTH="&length&" name="&champ&" onchange='javascript:isInteger(this.value)'> valeur entière"

comme vous pouvez le remarquer je récupere les valeur a l'aide d'une requete.

Le probleme avce ce controle c'est que le message d'erreur apparait et averti l'utiisateur qu'il sait tromper, mais je voudrait en + qu'il efface le champ ou il s'est trompé et repositionne le cureur sur ce champ.

Si quelqu'un voit comment faire ou veut des précision faite le savoir :p

Merci

[denisC]

response.write "<td><input type=textarea SIZE="&length&" MAXLENGTH="&length&" name="&champ&" onchange='javascript:isInteger(this.value)'> valeur entière"

Le javascript est complétement inutile....

Sinon, en modifiant légerement ton code (en passant l'element plutot que sa valeur):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
   function isInteger(element){
      var re = /^\d+$/;  
       if(!re.test(element.value)){
	  alert("Vous n'avez pas saisie d'entier !!!");
           element.value='';
           element.focus();
           return false;
      }
      return true;
   }

Ca devrait le faire (en tout cas pas loin...)

[ekinoxe]

ok je vais tester, par contre que faut t'il mettre en parametre sur l'événement Onchange ? on vire this.value je pense non ?

[denisC]

ok je vais tester, par contre que faut t'il mettre en parametre sur l'événement Onchange ? on vire this.value je pense non ?

Tu mets this (l'element du formulaire) comme paramètre d'appel de la méthode sur le onchange.

[ekinoxe]

bon alors bilan apres essai :
zone de texte

Code :

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

<td><input type=textarea SIZE="&length&" MAXLENGTH="&length&" name="&champ&" value="&data&" onchange='isInteger(this)'> valeur entière

script JS

Code :

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

1
2
3
4
5
6
7
8
9
10
 function isInteger(element){
      var re = /^\d+$/;  
       if(!re.test(element.value)){
	  	alert("Vous n'avez pas saisie d'entier !!");
		   element.value='';
           element.focus(); 
           return false;
      }
	   return true;
   }

- Le champ est bien effacé en cas d'erreur, mais le focus ne marche pas

Il faut savoir que on formulaire comporte un nombre de champ variable donc lorsque je rempli le champ 1 de type entier avec la valeur "abc" puis je clique sur le champ 2, le message d'ereur apparait, le champ 1 est effacé mais le focus reste sur le champ 2

[denisC]

Il faut savoir que on formulaire comporte un nombre de champ variable donc lorsque je rempli le champ 1 de type entier avec la valeur "abc" puis je clique sur le champ 2, le message d'ereur apparait, le champ 1 est effacé mais le focus reste sur le champ 2

Typiquement un problème de priorité.

Remplace

Code :

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

1
2
 
element.focus();

par:

Code :

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

1
2
 
window.setTimeout(function(){element.focus();},200);

ça devrait marcher (en fait, c'est un problème d'ordre en changement de focus par le script et changement par le navigateur lié à l'action utilisateur).

[ekinoxe]

OK merci sa marche nikel, par contre peut tu m'expliquez un peu le role du timer pour le focus ??

Par contre maintenant je vais devoir gérer aussi la possibilité de laisser un champ vide si la BDD l'accepte. Mais sa c'est une autre histoire ^^

Merci a toi en tout cas

[denisC]

OK merci sa marche nikel, par contre peut tu m'expliquez un peu le role du timer pour le focus ??

Tout simplement la chronologie des evenements:

1. L'utilisateur demande un changement de champs (avec Tab ou la souris)
2. declenchement de onchange: le champ est vérifié, si nécessaire le focus remis sur ce champ
3. Changement du focus par le navigateur, suite à la demande utilisateur, au champ suivant

Donc, il faut un timeout pour que la remise en place du focus ne se fasse qu'après le point 3.

[ekinoxe]

Bonjour, je reviens sur mon sujet car en fait i y a quelque chose qui me chagrine avec cette fonction

Code :

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

1
2
3
4
5
6
7
8
9
10
function isInteger(element){ 
      var re = /^\d+$/;  
       if(!re.test(element.value)){ 
        alert("Vous n'avez pas saisie d'entier !!"); 
         element.value=''; 
           element.focus(); 
           return false; 
      } 
      return true; 
   }

Exemple d'utilisaton :
zone de saisie ID (entier requis)
zone de saisie nom (varchar)
1- on entre "abc" dans la zone ID
2- on place le curseur dans la zone NOM
3- affichage du message d'erreur, supression de "abc", focus sur la zone ID
4- on place le curseur dans la zone NOM alors que rien est rempli dans ID
5- affichage du message d'erreur, supression de "abc", focus sur la zone IDCLIENT
6- on place le curseur dans a zone NOM alors que rien est rempli dans IDCLIENT
7- Le curseur se place bien dans la zone NOM

Il faudrait en fait supprimer les étapes 5 et 6. Ne pas faire de controle si la cellule est vide en fait

Si vous n'avez pas tout compris faite signe

[denisC]

4- on place le curseur dans la zone NOM alors que rien est rempli dans ID
5- affichage du message d'erreur, supression de "abc", focus sur la zone IDCLIENT

Si vous n'avez pas tout compris faite signe

Je comprends la suppression de "abc" si il n'y a rien dedans

Sinon, il suffit de rajouter le test comme quoi il y a qqch:

Code :

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

1
2
 
if(element.value.length>0&&(!re.test(element.value))){

[ekinoxe]

pas bete, j'avais pas pensé a faire un controle sur la taille de la zone.


Merci grand chef