Verification de format (formulaire)

Version imprimable

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

Bonjour,

Je suis pas tres bon en js !
Jessaie dadapter une formule pour verifier un format.
Voila mon script :

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
function Verification()
{
erreur = "";
var regExpFrenchDate=/^(0[1-9]|[12][0-9]|3[01])[- /.](0[1-9]|1[012])[- /.](19|20)\d\d$/g;
var regExpCampagne=/^(0[1-9]|1[012])[- /.](19|20)\d\d/g;
with (document.Fiche)
{
	if( (CAMPAGNE.value == "00-0000") || (CAMPAGNE.value == "") )
	{
	erreur = erreur + "Campagne non renseignée\n"
	}
	if( (CAMPAGNE.value != regExpCampagne) )
	{
	erreur = erreur + "Campagne format\n"
	}
	if(REF.value == "")
	{
	erreur = erreur + "Code article non renseigné\n"
	}
	if(acheteur.value == "aucun")
	{
	erreur = erreur + "Acheteur non sélectionné\n"
	}
	if(Categorie.value == "aucun")
	{
	erreur = erreur + "Catégorie non sélectionnée\n"
	}
	if(SSCategorie.value == "aucun")
	{
	erreur = erreur + "Sous catégorie non sélectionnée\n"
	}
	if(DOUANE.value == "")
	{
	erreur = erreur + "Nomenclature douanière non renseigné\n"
	}
}
if (erreur!="")
alert(erreur);
else
Fiche.submit()
}

En fait je bute sur la variable regExpCampagne qui devrait correspondre a lexpression 00-0000.

?

28/02/2006, 17h58
Bernard Grosdoy
En lisant rapidement j'ai l'impression que tu tests :
Code:

1 2 00-0000 != "/^(0[1-9]|[12][0-9]|3[01])[- /.](0[1-9]|1[012])[- /.](19|20)\d\d$/g";
A priori c'est faut.

Je crois que c'est plutot le format que l'on test
pour les mail par exemple j'utilise plutôt search

j'essairais donc un truc du genre :
Code:

1 2 3 4 5 valeurCampagne = CAMPAGNE.value; if (valeurCampagne.search(regExpCampagne) != -1) return true; else return false;
Cela dit, je ne manipule pas bien les expressions regulieres
28/02/2006, 18h13
Hephaistos007

Code:

CAMPAGNE.value != regExpCampagne)

Je ne suis pas un grand connaisseur de Javascript mais ce genre de comparaison me gène. La tu testes simplement la non égalité de 2 chaines de caractères. A aucun moment tu n'indique que regExpCampagne est une expression régulière. L'interpreteur de script n'est pas encore assez intelligent pour deviner ce que tu souhaites faire. :)

Il existe probablement des fonctions du type regexp(CAMPAGNE.value,regExpCampagne) qui indique explicitement que tu cherches à faire une comparaison par rapport à un motif.
01/03/2006, 08h39
SpaceFrog

fais un tour sur les tutos de regExp ...
la comparaison du format d'un chaine avec un pattern se fait plutot avec un test ou match ...
01/03/2006, 10h18
dafalri

Meric pour vos réponse.
Javoue que je nai pas tout compris a quest ce que vous disez ! :wink:
je vais regarder le tuto et je vous tiens au courant
01/03/2006, 10h58
dafalri

Jai lu le tuto sur regexp ... 8O
Ca a lair fortiche quand meme !
Si quelquun peut me venir en aide ... je voudrais juste tester un format dun champs de formulaire.
Je voudrais etre sur que le champs CAMPAGNE soit au format 00-0000
Est-ce que

Code:

/(0[1-9]|1[012])[- /.](19|20)\d\d/g;

est correct ? Est-ce que ca correspond a 00-0000 ?
01/03/2006, 11h19
dafalri
Jai change mon code :

Code:

var regExpCampagne= new RegExp(/(0[1-9]|1[012])[- /.](19|20)\d\d/g);

et :
Code:

1 2 3 4 if( (CAMPAGNE.value != regExpCampagne) ) { erreur = erreur + "Campagne format\n" }
01/03/2006, 11h20
SpaceFrog

^[0-9]{2}+\-+[0-9]{4}+$ :?:
01/03/2006, 11h22
dafalri

Jai donc change en :

Code:

var regExpCampagne= new RegExp(^[0-9]{2}+\-+[0-9]{4}+$);

Mais ca a lair detre pire parce que je nai meme pas de popup !
01/03/2006, 11h27
dafalri

En fait quand je mets

Code:

var regExpCampagne= new RegExp(/(0[1-9]|1[012])[- /.](19|20)\d\d/g);

jai mon popup derreur "Campagne format" alors que jai par defaut 00-0000 dans mon champs CAMPAGNE (en plus des autres message derreurs des autres champs obligatoires).
Et si je mets

Code:

var regExpCampagne= new RegExp(^[0-9]{2}+\-+[0-9]{4}+$);

je nai pas de popup du tout.
01/03/2006, 11h29
SpaceFrog

desolé je connais pas bien les reg exp ... mais c'est juste une question de pattern ...
JavaTwister est un crack sur le sujet ...
01/03/2006, 11h34
dafalri

ou alors il existe peut etre un autre moyen que les exp reg pour tester un format de champs ?
01/03/2006, 11h34
denisC
Citation:

Envoyé par dafalri

if( (CAMPAGNE.value != regExpCampagne) )

Auh, je pense que même sans JT, je peux dire qu'il y aun bug là....

Plutot:
Code:

1 2 if (!regExpCampagne.test(CAMPAGNE.value)) ...
01/03/2006, 11h38
SpaceFrog

oui Denis pour le test mais pour le pattern ?
01/03/2006, 11h39
dafalri

cest ce que jallais dire je crois que cest mon modele qui fait defaut ...
01/03/2006, 11h48
denisC
Citation:

Envoyé par dafalri

/(0[1-9]|1[012])[- /.](19|20)\d\d/g;

Il était pas mal, celui-là, non?

Il suffit d'y faire quelques modifs:
Code:

1 2 var regExpPattern = /((0[1-9])|(1[0-2]))-(19|20)\d{2}/;
Attention au bug de l'an 2100 :)
01/03/2006, 11h54
dafalri

Mais ya pas besoin de dire que cest une RegExp alors ?
01/03/2006, 11h58
dafalri
:?
Ca marche pas.
Code:

1 2 3 4 5 var regExpPattern = /((0[1-9])|(1[0-2]))-(19|20)\d{2}/; if (!regExpPattern.test(CAMPAGNE.value)) { erreur = erreur + "Campagne format\n" }
et dans le html jai

Code:

<input name="CAMPAGNE" type="text" id="CAMPAGNE" value="00-0000" size="5" maxlength="7" onFocus="this.value=''" />
01/03/2006, 12h02
denisC
Citation:

Envoyé par dafalri

Mais ya pas besoin de dire que cest une RegExp alors ?

Le fait de mettre la chaine entre / / reviens à faire un new RegExp.
Code:

1 2 /blabla/ = new RegExp("blabla")
:idea:
01/03/2006, 12h03
denisC
Citation:
Envoyé par dafalri

:?
Ca marche pas.

Code:

1 2 3 4 5 var regExpPattern = /((0[1-9])|(1[0-2]))-(19|20)\d{2}/; if (!regExpPattern.test(CAMPAGNE.value)) { erreur = erreur + "Campagne format\n" }

et dans le html jai

Code:

<input name="CAMPAGNE" type="text" id="CAMPAGNE" value="00-0000" size="5" maxlength="7" onFocus="this.value=''" />
un petit

Code:

document.getElementById('CAMPAGNE')

Pour acceder à un élement par son id, avant de chercher à voir sa value.

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