Discussion :

[oodini]

Hello,

Y a-t-il un moyen rapide de vérifier que la saisie dans un champ texte n'est constituée que de chiffres ou du séparateur décimal ?

Ou faut-il faire un test à chaque caractère tapé ?

Je fais un contrôle sur les chiffres et le séparateur décimal avec les événements KeyDown/KeyPress, mais j'ai un problème avec le séparateur décimal. Seule la virgule est acceptée quand je fais un Single.Parse().

[guitoux1]

avec le errorProvider tu peux

[oodini]

Apparemment, c'est plus pour informer l'utilisateur d'une erreur, mais pas pour contrôler.

[StormimOn]

Le contrôle ErrorProvider permet d'indiquer une erreur de saisie avec un petit icône et un message associé, mais il faut tout de même vérifier si la saisie est correcte ou non et le contrôle ErrorProvider n'est pas capable de faire cette vérification. Ce n'est donc pas avec un ErrorProvider que notre ami résoudra son problème

Pour vérifier que ta zone texte correspond à un décimal, tu peux essayer de faire la conversion

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
string texte = "12,5";
decimal valeur;
// sans gestion de la culture
if (!Decimal.TryParse(texte, out valeur))
{
    // texte ne représente pas un décimal
}
// avec gestion de la culture
CultureInfo ci = new CultureInfo("fr-FR");
if (!Decimal.TryParse(texte, NumberStyles.Number, ci, out valeur))
{
    // texte ne représente pas un décimal
}

J'ai ajouté l'utilisation de CultureInfo pour l'exemple surtout. Ce qu'il faut savoir c'est que la conversion sera faite suivant la Culture locale, càd suivant les paramètres linguistiques de la machine (sauf erreur de ma part). Par défaut le séparateur décimal sera très probablement la virgule en ce qui nous concerne, ce qui veut dire que les chaînes "12,5" et "12.5" ne représentent pas la même chose après conversion. Le premier est effectivement un décimal, mais pas le second à cause du séparateur qui ne correspond pas à celui attendu pour notre Culture.

Tu peux également récupérer le séparateur correspondant à la Culture locale

Code :

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

1
2
CultureInfo ci = CultureInfo.CurrentCulture;
string s = ci.NumberFormat.CurrencyDecimalSeparator;

si celui utilisé sur ta zone texte est incorrect (point au lieu de virgule et vice-versa), rien ne t'empêche de substituer le bon séparateur avant de faire la conversion.

Le "pavé" sur la Culture n'était peut être pas nécessaire mais ca t'évitera de mauvaises surprises plus tard, sait-on jamais

[guitoux1]

ok la je m'incline

Sinon il existe des contrôles tout fait qui empêchent la saisie autre qu'un type voulu, comme par exemple Dacris NetXP 3.0 (dsl, j'ai pu l'url en tête)

[oodini]

Ben en fait, l'aspect culturel était le plus intéressant.

Sauif que je fais du contrôle à la volée (revoir le message initial, que j'ai modifié), et que j'ai du mal à chopper le séparateur avec onKeyPress/onKeyDown. Ce qui m'est fourni correspond à l'énumération "Keys", alors que je m'intéresse en fait au caractère fourni, pas à la touche.

[oodini]

Bon, ben j'ai utilisé le code suivant :

Code :

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

1
2
3
4
5
6
7
8
CultureInfo ci = CultureInfo.CurrentCulture;
char séparateur = ci.NumberFormat.CurrencyDecimalSeparator[0];
 
private void tbLength_KeyPress(object sender, KeyPressEventArgs e)
{
    if (e.KeyChar <= '0' || e.KeyChar >= '9' || e.KeyChar != séparateur)
        e.Handled = true;
}

Mais ça ne marche pas du tout : ça passe systématiquement dans le if. Pourtant, quand je regarde la valeur de e.KeyChar, ça devrait être bon...

[guitoux1]

ben il passe dans le if à cause de e.KeyChar != séparateur, non?
Car du coup tout caractère autre que le séparateur est OK

[oodini]

Oui, tu as raison...

Je me fais tellement chier sur des trucs chiants, que je ne fais même plus gaffe aux trucs cons... (désolé pour le vocabulaire : je me lâche !)

Code :

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

if ( (e.KeyChar <= '0' || e.KeyChar >= '9') && e.KeyChar != séparateur && e.KeyChar != 8)

J'en ai profité pour rajouter le contrôle sur le backspace. :-)

[goutlet]

Pourquoi ne pas utiliser les expressions régulières ???

[oodini]

Parce que je suis à la bourre, et que je ne connais pas trop les expressions régulières. Mais si tu as quelque chose à proposer...

Par ailleurs, j'aimerais transformer que si on saisit un point au lieu du séparateur décimal, ce point soit également reconnu comme séparateur décimal, et s'affiche comme tel.

Après, je pense que je pourrai passer à autre chose. :-)

[guitoux1]

voilà une expression régulière pour un réel inférieur à 100 (xx ou xx,x ou xx,xx) :

Code :

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

\d{1,2}(,[\d]{1,2})?

la même, mais sans limite à 100 :

Code :

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

\d+(,[\d]{1,2})?

[oodini]

OK, merci. :-)

Si tu as également une idée pour le petit problème soulevé dans mon dernier message...

[goutlet]

En voilà une pour valider une valeur décimale :

^\d+[.]\d+$, mais remplace le point par ton "char séparateur"...

[guitoux1]

tu test si le caractère est un point : (char)Keys.OemPeriod

[oodini]

Ce que tu proposes se fait dans l'événement onKeyDown.

Je suis resté dans onKeyPress avec :

Code :

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

1
2
3
4
5
6
7
8
private void tbNum_KeyPress(object sender, KeyPressEventArgs e)
{
    if ((e.KeyChar < '0' || e.KeyChar > '9') && e.KeyChar != séparateur && e.KeyChar != 8 && e.KeyChar != '.')
        e.Handled = true;
 
    if (e.KeyChar == '.')
        e.KeyChar = séparateur;
}

[goutlet]

Ben je pense que tu as tout ce que tu désirais...

Merci à Guitoux1 pour ses précisions...

[oodini]

Merci effectivement beaucoup à tous deux !!