Discussion :

[hazem2410]

Bonjour Tous Le monde

J'ai un champ de saisie, au sein duquel l'utilisateur doit saisir uniquement un code composer de 10 chiffres sans aucun autre caractère ou bien lettre...

J'ai eu l'idée de faire un test, pour savoir si le code saisie est acceptable dans la forme ou non?

Voici un exemple:

Code :

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

1
2
3
4
5
6
 
 
            if (5 is decimal)
            {
                Label1.Text = "good";
            }

Ceci ne marche pas!

Quelqu'un aurait une solution SVP

[bizet]

Salut

je te conseille d'utiliser les regexp.
Tu trouveras des exemples très complet sur le net.

bon courage

[calagan99]

Comme le dit Bizet, avec cette regexp (^[0-9]{1,10}$) tu vérifieras que ton champs contient entre 1 et 10 des chiffres.

Ex :

Code :

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

1
2
System.Text.RegularExpressions.Regex MyRegex= new System.Text.RegularExpressions.Regex(@"^[0-9]{1,10}$");
bool IsCorrect = MyRegex.IsMatch("la valeur de la saisie");

[Arnard]

1. TryParse sera plus juste

2. decimal est un type à virgule flottante pour les calculs scientifiques et financiers. 5 est un int

3. Si tu testes dans le OnKeyPressed/OnKeyQuelqueChose, méfie-toi du copier-coller

4. la remarque sur les regex est pertinente

[calagan99]

@Arnard,
j'avais un doute sur le TryParse sur une string genre "0000012345".
Je viens de vérifier et ça passe en effet.
Par contre, le TryParse renvoie aussi true sur une saisie de type "00000123456".
Or, hazem2410 veut maximum 10 caractères.

Donc je pense que Bizet avait vu juste et que la regexp est toute indiquée.

[Arnard]

Ta remarque est pertinente calagan99, mais au vu du code posté, j'émets l'hypothèse qu'il teste via des appels OnKey... donc exécutés à chaque touche... De plus, un mTextbox.Text.Length peut renvoyer la longueur...

Par contre, si 10 caractères, mise sur uint32.TryParse, tu t'exposes à un overflow avec un int.tryParse

[calagan99]

Ta remarque est pertinente calagan99, mais au vu du code posté, j'émets l'hypothèse qu'il teste via des appels OnKey... donc exécutés à chaque touche... De plus, un mTextbox.Text.Length peut renvoyer la longueur...

Exact

Par contre, si 10 caractères, mise sur uint32.TryParse, tu t'exposes à un overflow avec un int.tryParse

Encore exact.
Mais cette fois-ci, il est clair que la regex s'impose

[Arnard]

Après, la solution qu'il choisit... La remarque avait surtout pour but de lui signifier de ne pas stocker dans un int, mais qu'un decimal n'est pas là pour ça...

De plus, à moins qu'il ne choisisse de stocker dans une String derrière (choix étrange mais pourquoi pas), il n'échappera alors pas au Parse

[calagan99]

Au moins, il aura toutes les cartes en mains.
Je pense que l'on a été assez précis

[hazem2410]

Merci calagan99 pour la réponse. Mais, STP, j'ai besoin de plus de précision.

J'ai un champ TextBox1 de saisie, alors comment l’intégrer dans le code?

[calagan99]

Merci calagan99 pour la réponse. Mais, STP, j'ai besoin de plus de précision.

J'ai un champ TextBox1 de saisie, alors comment l’intégrer dans le code?

Tu plaisantes ?
Comment je peux être plus précis que ça :

Code :

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

1
2
System.Text.RegularExpressions.Regex MyRegex= new System.Text.RegularExpressions.Regex(@"^[0-9]{1,10}$");
bool IsCorrect = MyRegex.IsMatch("la valeur de la saisie");

Tu as juste à y remplacer "la valeur de la saisie" par le contenu de ton Textbox1... (TextBox1.Text)...

[hazem2410]

Je m'excuse j'ai pas lu bien l'entre cotes

Mais ce code accepte même des codes à moins de 10 chiffres!

[Morgan7469]

Et bien il suffit de modifier la regexp en faisant quelque chose comme :

Code :

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

@"^[0-9]{10}$"

Au lieu de :

Code :

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

@"^[0-9]{1,10}$"

[calagan99]

Code :

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

1
2
System.Text.RegularExpressions.Regex MyRegex= new System.Text.RegularExpressions.Regex(@"^[0-9]{10}$");
bool IsCorrect = MyRegex.IsMatch("la valeur de la saisie");

Un petit effort d'interprétation t'aurais suffit pour comprendre le rôle de "{1,10}" dans la regex...
En passant donc à "{10}", tu ne valideras que des chaines numériques de 10 caractères.

[Arnard]

Au moins, il aura toutes les cartes en mains.
Je pense que l'on a été assez précis

Pas suffisamment il faut croire