Event lost focus appelé en boucle

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Event lost focus appelé en boucle

LePhasme

Les Belges domineront le monde

Je débute en C#, pour vérifier qu'une textbox ne contient une valeur numérique j'utilise le code suivant :

Code :

private void check_value(object sender)
{
TextBox tb = (TextBox)sender ;
MessageBoxResult result;
if (!Utility.isNumeric(tb.Text)){
result = MessageBox.Show("Vous ne pouvez entrer que des chiffres" );
tb.Focus();
}
}
private void Amount_1_LostFocus(object sender, RoutedEventArgs e)
{
check_value(sender);
}

Problème, quand la valeur contient une lettre, l'event Lostfocus de la textbox est appelé en boucle et je ne comprends pas pourquoi vu que je lui donne le focus juste avant...

Message édité par LePhasme le 21-02-2011 à 04:47:02

---------------
Instagram - Mon PVT en Australie.

Publicité

deliriumtremens

sic transit intestinal...

Je dirais au premier coup d'oeil que tu ne précises pas, dans ton Amount_1_LostFocus, que l'événement est consommé ou non selon le test fait dans check_value.

(je ne sais plus s'il y a un e.Handled avec LostFocus, mais c'est dans l'esprit).

Donc il repart pour un tour (en gros).

Autres remarques :
L'intérêt d'avoir une méthode à part (check_value) me paraît aussi assez limité dans ton cas, vu que tu castes explicitement ton sender en TextBox...
Autant tout faire dans Amount_Lost_Focus dans ce cas-là (tu peux appliquer la même méthode à plusieurs TextBox - ou, pour être plus précis mais moins clair, donner la même méthode en paramètre à plusieurs délégués avec la même signature liés à des TextBox différents).

Pas besoin de passer par MessageBoxResult : tu ne fais rien de result...
Un MessageBox.Show("xxx" ) te suffit pour un message d'avertissement.

Cela mis à part, je préfère (est-ce rationnel) utiliser des événements validated ou validating pour ce genre de choses. Mais bon ça dépend de l'usage.

LePhasme

Les Belges domineront le monde

J'ai essayé de rajouter un e.handled = true dans le lostfocus mais ça n'a rien changé.

J'utilisais une méthode à part parce que j'ai plusieurs champs et j'aurais juste à ajouter l'appel à la méthode dans le lostFocus des autres champs quand j'aurais réussi à trouver la cause du problème.
Mais effectivement ce serait sans doute plus propre d'utilisé un delegate.

Je n'ai pas d'évènement validated/validating attaché à ma textbox.

Message cité 1 fois

---------------
Instagram - Mon PVT en Australie.

deliriumtremens

sic transit intestinal...

Edit : tu fais du c#, mais quel techno, quel version du framework, en fait ?

LePhasme a écrit :

Bon :
Pour chaque TextBox

Code :

myTextBox.Validating += new CancelEventHandler(this.ValidatingTextBox);
myTextBox2.Validating += new CancelEventHandler(this.ValidatingTextBox);

Tu n'as pas besoin d'avoir une méthode Validating_TextBox
et une autre Validating_TextBox1 !

Ensuite

Code :

public void ValidatingTextBox(object sender, CancelEventArgs e) {
TextBox tb = (TextBox)sender;
if (!Utility.isNumeric(tb.Text)) {
tb.Text = string.Empty; // eventuellement, tu nettoies ta boîte
MessageBox.Show("Vous ne pouvez entrer que des chiffres" );
e.Cancel = true;
}
}

Message édité par deliriumtremens le 22-02-2011 à 11:02:19

deliriumtremens

sic transit intestinal...

Ah ouais, tu fais du WPF ^^

Bon ben tu n'as pas de validating, c'est sûr.

Mais la logique devrait rester presque la même

Code :

myTextBox.LostFocus+= new RoutedEventHandler(this.TextBoxLostFocus); // je suppose qu'il s'agit d'un RoutedEventHandler ??
myTextBox2.LostFocus+= new RoutedEventHandler(this.TextBoxLostFocus);
...
myTextBox2354.LostFocus += new RoutedEventHandler(this.TextBoxLostFocus);

Code :

public void TextBoxLostFocus(object sender, RoutedEventArgs e) {
TextBox tb = (TextBox)sender;
int val;
if(!Utility.IsNumeric(tb.Text)) {
tb.Text =string.Empty;//vidage éventuel
e.Handled = true;
MessageBox.Show("erreur" );
//tu n'as pas besoin de refaire un Focus
}
}

Message édité par deliriumtremens le 22-02-2011 à 11:16:25

deliriumtremens

sic transit intestinal...

Ah, ben testé, ça marche à peu près, mais on perd le focus...

Le mieux serait peut-être donc de faire plutôt un TextBoxChanged

(La même méthode pour toutes tes Textbox, on l'aura compris).

Code :

private void TextBoxTextChanged(object sender, TextChangedEventArgs e)
{
TextBox tb = (TextBox)sender;
double val;
if (tb.Text.Length > 0 && !Double.TryParse(tb.Text, out val))
{
MessageBox.Show("erreur" );
tb.Text = string.Empty;//bourrin, mais ça évite de garder des saletés.
}
}

Message édité par deliriumtremens le 22-02-2011 à 11:46:59

LePhasme

Les Belges domineront le monde

Le problème c'est qu'effacer tout pour une erreur de frappe c'est franchement moyen...
Du coup je vais essayer de passer par des bindings validations vu que c'est une application pour apprendre de toute façon...

Un grand merci pour tes réponses

Message édité par LePhasme le 23-02-2011 à 01:26:47

---------------
Instagram - Mon PVT en Australie.

FORUM HardWare.fr

Programmation

C#/.NET managed

Event lost focus appelé en boucle

Sujets relatifs
[Résolu] Boucle infinie d'appel de scripts.	vérifierr stppp uune boucle d'étoile croissante
boucle for python	Empecher prise de focus
Probleme : Tableau perd ses valeurs une fois sortie de la boucle	Besoin d'aide sur une boucle "if else if else"
Batch : Compteur dans une boucle for	Scan d'un dossier image, lecture des images une par une en boucle
Greasemonkey Boucle executée une seule fois	[RESOLU]KeyDown_Event - comment retourner une combinaison de touches
Plus de sujets relatifs à : Event lost focus appelé en boucle

Page générée en 0.087 secondes