Problème de boucle while

Version imprimable

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

05/02/2014, 08h57
Invité

Problème de boucle while

Bonjour à tous,

Je tiens à préciser que je suis totalement novice autour de la programmation en C#
et je n'arrive pas à comprendre pourquoi un

Code:

MessageBox.Show("X");

fait en sorte que ma boucle while fonctionne

Mon programme permet de rechercher un mot dans un fichier txt et d'écrire la ligne associé. Si le mot existe plusieurs fois il devrait écrire la première ligne puis quand l'ulisateur clic sur suivant m'écrire la deuxième ligne etc...
Le soucis étant que tant qu'il y a ce pop up (MessageBox) la boucle fonctionne mais le bouton suivant se fait court circuiter; et dès qu'il n'y est plus, impossible d'avoir les autres lignes Le programme réaffiche toujours la première ligne.

Alors j'ai pensé à mettre une pause "attente d'un clic" mais toutes mes tentatives ont échoué pour que cela fonctionne.

Mes questions sont donc:
_Comment ce fait il qu'un MessageBox fasse réagir ma boucle comme ça ?
_Auriez vous une piste pour que je puisse réalisé une "pause" afin d'attendre que l'utilisateur clic sur suivant ou reface une recherche ?

Metallode689
05/02/2014, 09h00
Invité

Bonjour,

Afin que nous puissions t'aider, pourrais-tu nous fournir le code C# de ta fameuse boucle s'il te plait?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
while ((ligne = fichier1.ReadLine()) != null && counter <= 1000)
{
     Match mat = Regex.Match(ligne, conver1, RegexOptions.IgnoreCase);
     If (mat.Success)
     {
         if (counter <= 1000)
         { 
               Solution.Text = ligne;
               MessageBox.Show("X");
               t++;
          }
          else
          {
               MessageBox.Show("erreur", exMessage);
          }
          counter++;
      }
      if ((mat.Success == false) && t == 0)
      {
          MessageBox.Show("Default introuvable bla bla bla");
          counter++;
      }
}

Ici la recherche fonctionne bien avec un afreux pop up qui apparait pour chaque ligne et court circuite par la même occasion la possibilité de pouvoir cliqué sur suivant
le counter sera ensuite remplacé par une variable plus tard

05/02/2014, 09h25
Invité

Dans ton code, je ne vois aucune interaction avec le bouton "suivant" que tu mentionnes... S'il n'y a pas d'interaction dans cette boucle avec le bouton "suivant", je ne vois pas comment ce que tu veux faire est réalisable.

As-tu le reste du code de ton formulaire, et en particulier le code du Handler du bouton "suivant'?
05/02/2014, 09h31
PixelJuice

Le fichier que tu lis est-il volumineux ? Car si la boucle while s’éternise , il est normal que cela t'affiche la première ligne car l'interface est "freeze", et un MessageBox.Show() permet une attente ,la boucle continue uniquement si tu fermes la boite de dialogue.C'est pour cela que ça fonctionne quand t'en mets une.

Essaye de donner le code complet qui concerne ce morceau de code.
05/02/2014, 09h32
Invité

effectivement c'est parce que je n'ai pas trouvé le moyen de rajouter cette intéraction comme mentionné dans mon premier post

il faudrait que ce while puisse se mettre en "pause" après un premier élément trouvé et que l'utilisateur puis reprendre la continuité de la boucle en cliquant sur suivant mais je n'ai pas trouvé comment réaliser cela.
05/02/2014, 09h37
PixelJuice

Citation:

Envoyé par metallode689

effectivement c'est parce que je n'ai pas trouvé le moyen de rajouter cette intéraction comme mentionné dans mon premier post

il faudrait que ce while puisse se mettre en "pause" après un premier élément trouvé et que l'utilisateur puis reprendre la continuité de la boucle en cliquant sur suivant mais je n'ai pas trouvé comment réaliser cela.

Dans ce cas il faudrait que tu lises d'un coup ton fichier , que tu reprenne ton code mais en lisant dans un tableau ou une liste de string.

Ensuite des que tu trouves quelque chose, la boucle se stop ,et tu reprends la recherche a l'aide d'un bouton.

En gardant sa valeur , ta variable qui compte reprendra la ou la recherche s'est arrêtée.
05/02/2014, 09h39
Invité

Citation:

Envoyé par PixelJuice

Le fichier que tu lis est-il volumineux ? Car si la boucle while s’éternise , il est normal que cela t'affiche la première ligne car l'interface est "freeze", et un MessageBox.Show() permet une attente ,la boucle continue uniquement si tu fermes la boite de dialogue.C'est pour cela que ça fonctionne quand t'en mets une.

Essaye de donner le code complet qui concerne ce morceau de code.

il n'est pas si volumineux que ça :) la boucle while ne s'éternise pas je dirais que lorsqu'il m'affiche la premiere ligne contenant le mot de recherche mon programme sort de sa boucle.

je veux bien passer une plus grande partie du code mais il va falloir que je le tape (c'est compliqué je ne peux pas faire autrement ^^).
05/02/2014, 10h06
Invité

Citation:

Envoyé par metallode689

je dirais que lorsqu'il m'affiche la premiere ligne contenant le mot de recherche mon programme sort de sa boucle.

Oui, c'est exactement ça qui se passe : toutes tes occurrences ont déjà été parcourues lorsque l'utilisateur peut cliquer sur le bouton "Suivant". C'est la raison pour laquelle un "MessageBox" arrête la boucle là où elle était.
Il y a plusieurs solutions pour résoudre ce problème, comme par exemple :
- Mettre une nouvelle fenêtre modale qui va arriver à chaque fois que l'un des mots est trouvé.
- Parcourir tout et mettre les occurrences dans une liste, comme suggéré par pixeljuice
- Ne pas faire de boucle mais retenir l'état du "match" actuel de la regex, et le faire avancer à chaque fois que l'utilisateur clique sur "suivant".

Citation:

Envoyé par metallode689

je veux bien passer une plus grande partie du code mais il va falloir que je le tape (c'est compliqué je ne peux pas faire autrement ^^).

Ah bon? Tu n'as pas la possibilité de faire un "copié/collé" depuis ton poste?

Salut,

effectivement un peu plus de code nous éclairerait car sur cette portion de code je ne vois pas ce qui cloche.
Ton explication n'est pas super clair, mais selon ce que tu as dit j'aurais fait ça:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 string ligne; string result; int counter=0; int t=0; string conver1="test"; System.IO.StreamReader fichier1 = new System.IO.StreamReader(@"C:\test.txt"); while ((ligne = fichier1.ReadLine()) != null && counter <= 1000) { Match mat = Regex.Match(ligne, conver1, RegexOptions.IgnoreCase); if (mat.Success) { t++; result = ligne; MessageBox.Show("Ligne : " +counter+ "\n\n Contenu :" + result + "\n\n" +t+"e occurence", "Recherche du mot : "+conver1 ); } counter++; } if ( t == 0) { MessageBox.Show("Default introuvable bla bla bla"); counter++; }
L: 10 : Retrait de ton 'if(counter<=1000)' (et donc du 'else' aussi), puisque c'est déjà une condition pour continuer ton while.
J'ai sorti ton 'counter++' du if, comme ça il sert à compter les lignes. (Je ne voyais pas trop l'utilité de ce dernier dans le if, étant donné que tu as déjà ton 't' pour compte le nombre de mots trouvés.

L:18: J'ai sorti le if ((mat.Success == false) && t == 0) de la boucle puisque sinon le fil d’exécution passe dedans tant qu'il n'a pas trouvé de mots, je ne vois pas l'intérêt.

Mais alors: que veux-tu mettre à la place de ton messageBox ?

Arnaud DS

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
        private void Rechercher_Click(object sender, EventArgs e)
        {
            string conver1 = textBox1.Text;
            int t = 0;
            int counter = 0;
            //on vérifie s'il y a une valeur d'entrée ou non 
            if (conver1 != "")
            {
                 if (conver1.IndexOf(TextRecup1) != 0)
                {
                    try
                    {
                        fichier1 = new StreamReader(@"C:\Users\user\Desktop\doc.txt");
                        while ((ligne = fichier1.ReadLine()) != null && counter <= 10000)
                        {
                            Match mat = Regex.Match(ligne, conver1, RegexOptions.IgnoreCase);
                                if (mat.Success)
                                {
                                    if (counter <= 10000)
                                    {
                                        Solution.Text = ligne;
                                        t++;
                                    }
                                    else
                                    {
                                        MessageBox.Show("Erreur", exMessage);
                                    }
                                    counter++;
                                }
                                if ((mat.Success == false) && counter == 10000 && t == 0)
                                {
                                    MessageBox.Show("Defaut introuvable, Remonter l'information grâce au bouton approprié.");
                                }
                                counter++;
                         }
 
                    }
                    catch (Exception ex)
                    {
                    MessageBox.Show("Une erreur est survenue");
                    MessageBox.Show(ex.Message);
                    }
                    finally
                    {
                    }
                }
                else
                {
                    MessageBox.Show("Defaut introuvable, Remonter l'information grâce au bouton approprié.");
                }
            }
            if ( counter <= 0)
            {
                MessageBox.Show("Taper un élément à rechercher");
            }
        }

Ps: je repette je suis autodidacte et novice donc je pense que ce code peut etre simplifié et mieux écrit mais pour l'instant je cherche à ce que cela fonctionne :)

Citation:

Envoyé par Ph_Gr

Oui, c'est exactement ça qui se passe : toutes tes occurrences ont déjà été parcourues lorsque l'utilisateur peut cliquer sur le bouton "Suivant". C'est la raison pour laquelle un "MessageBox" arrête la boucle là où elle était.
Il y a plusieurs solutions pour résoudre ce problème, comme par exemple :
- Mettre une nouvelle fenêtre modale qui va arriver à chaque fois que l'un des mots est trouvé.
- Parcourir tout et mettre les occurrences dans une liste, comme suggéré par pixeljuice
- Ne pas faire de boucle mais retenir l'état du "match" actuel de la regex, et le faire avancer à chaque fois que l'utilisateur clique sur "suivant".

Merci pour vos pistes :)

Citation:

Ah bon? Tu n'as pas la possibilité de faire un "copié/collé" depuis ton poste?

Pour l'histoire je suis au boulot, entreprise qui ne me permet pas d'avoir visual studio sur leur poste (sachant qu'il me demande de creer un programme chercher l'erreur [ironie]) je travail donc avec mon ordi personnel qui ne pas se connecter au réseaux de l'entreprise n'ayant pas de clé usb et ayant demandé à la moitier du service uné clé qu'ils n'ont pas. je dois faire en sorte de réécrire mon programme pour vous le montrer

05/02/2014, 10h32
Invité

Citation:

Envoyé par metallode689

Pour l'histoire je suis au boulot, entreprise qui ne me permet pas d'avoir visual studio sur leur poste (sachant qu'il me demande de creer un programme chercher l'erreur [ironie]) je travail donc avec mon ordi personnel qui ne pas se connecter au réseaux de l'entreprise n'ayant pas de clé usb et ayant demandé à la moitier du service uné clé qu'ils n'ont pas. je dois faire en sorte de réécrire mon programme pour vous le montrer

Ah d'accord! Je comprend. Et il n'y a personne auprès de qui tu peux aller demander la possibilité d'installer au moins un visual studio express sur ton poste de travail? Vu qu'il s'agit d'un travail qu'ils te demandent d'un point de vue professionnel, je trouve que c'est la moindre des choses, non? Avec Visual Studio Express, il n'y a pas de pb de licence.
05/02/2014, 10h50
Invité

Citation:

Envoyé par ArnaudDS

Mais alors: que veux-tu mettre à la place de ton messageBox ?

Merci de ta réponse,

à la place de mon MessageBox je souhaiterai mettre une interaction de l'utilisateur avec un bouton car l'utilisateur voit déjà sa recherche grace au Solution.Text = ligne.
05/02/2014, 10h52
Invité

Citation:

Envoyé par Ph_Gr

Ah d'accord! Je comprend. Et il n'y a personne auprès de qui tu peux aller demander la possibilité d'installer au moins un visual studio express sur ton poste de travail? Vu qu'il s'agit d'un travail qu'ils te demandent d'un point de vue professionnel, je trouve que c'est la moindre des choses, non? Avec Visual Studio Express, il n'y a pas de pb de licence.

non le logiciel n'apparait pas dans le catalogue et le service info n'a pas voulu "acheter" ça pour un apprentie :/
05/02/2014, 11h04
Invité

Citation:

Envoyé par metallode689

non le logiciel n'apparait pas dans le catalogue et le service info n'a pas voulu "acheter" ça pour un apprentie :/

Le catalogue? Ah tu es obligé de passer par un catalogue précis quand tu es à ton entreprise?

VSE est gratuit et on peut le télécharger ici par exemple :
http://www.microsoft.com/fr-fr/downl....aspx?id=40787
05/02/2014, 11h07
Invité

Citation:

Envoyé par Ph_Gr

Le catalogue? Ah tu es obligé de passer par un catalogue précis quand tu es à ton entreprise?

VSE est gratuit et on peut le télécharger ici par exemple :
http://www.microsoft.com/fr-fr/downl....aspx?id=40787

je n'ai pas les droits administrateur pour installer quelconque programme sur l'ordinateur je ne peux même pas changer l'heure si j'en avais envie ^^
05/02/2014, 11h14
Invité

Citation:

Envoyé par metallode689

je n'ai pas les droits administrateur pour installer quelconque programme sur l'ordinateur je ne peux même pas changer l'heure si j'en avais envie ^^

OK. Mais sinon tu peux toujours en parler à un administrateur, car en plus d'être gratuit, VSE peut être aussi utilisé dans un but lucratif.
05/02/2014, 11h22
ArnaudDS
Citation:

Envoyé par metallode689

non le logiciel n'apparait pas dans le catalogue et le service info n'a pas voulu "acheter" ça pour un apprentie :/

Ahah j'étais dans ce cas l'année dernière, la dure loi des stagiaires/apprentis... :?Mais tu devrais pouvoir demander l'installation de VSE au service IT de ta boîte, à t'entendre on dirait que tu bosses dans une cave en pédalant pour avoir du jus :mouarf: ... Tu dois être dans une grosse structure...

Je comprends mieux ton problème maintenant. J'ai peut-être une solution alternative de la même veine que Ph_Gr:

Vu que visiblement tu n'as pas de gros fichiers, tu fais d'abord un tour pour compter le nombre de lignes:
Code:

1 2 3 4 5 6 7 8 9 int nbLignes=0; string buff = ""; while (buff != null) { buff = sr.ReadLine(); if (buff == null) { break; } else {nbLignes++; } }
Ensuite au lieu de faire un while, tu fais un "for" (globalement moi je n'utilise les while qu'en dernier recours, je trouve que les for c'est beaucoup plus pratique et moins plantable. Les for, c'est fort.) avec comme limite le 'nbLignes'. Seulement, dés que tu rencontres un 'Match', tu 'break' ton for, en prenant soin de récupérer la ligne où ça c'est arrêté.

Ensuite tu mets une méthode derrière ton bouton "Suivant", qui relance un for à partir de la ligne où le for s'est arrêté (+1), etc...

Arnaud DS

Citation:

Envoyé par ArnaudDS

...au lieu de faire un while, tu fais un "for" (globalement moi je n'utilise les while qu'en dernier recours, je trouve que les for c'est beaucoup plus pratique et moins plantable. Les for, c'est fort.)

Ah oui, tout à fait d'accord, le "for" c'est le point fort de C#! :mrgreen:

Citation:

Envoyé par ArnaudDS

...Seulement, dés que tu rencontres un 'Match', tu 'break' ton for, en prenant soin de récupérer la ligne où ça c'est arrêté.

Ensuite tu mets une méthode derrière ton bouton "Suivant", qui relance un for à partir de la ligne où le for s'est arrêté (+1), etc...

Arnaud DS

C'est intéressant cette technique, mais en fait cette fonctionnalité existe déjà en .NET : il s'agit du mot clé "yield" avec comme possibilités "yield return" ou "yield break".

L'idée c'est d'avoir une fonction qui renvoie un "IEnumerable" de quelque chose et c'est capté dans un "foreach" qui appelle la fonction.

Exemple tout bête :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 private IEnumerable<string> TestChaines() { yield return "chaine 1"; yield return "chaine 2"; } private void ParcoursDeChainesTest() { foreach(string str in TestChaines()) { MessageBox.Show(str); } }
C'est exactement la mise en œuvre de ton idée, mais assez simpliste. Après il suffit d'utiliser ce principe pour le problème d'ici.

05/02/2014, 12h36
ArnaudDS

Je ne connaissais pas cette technique! C'est sûr si on peut s'éviter de l'algo inutile... ;)
Comme quoi, Microsoft fais tout à notre place! Ils sont formidables.Sans montrer le code source par contre, du coup si ça ne marche pas impossible de savoir ce qui se passe réellement en background. :aie: . Mais, allez, ne crachons pas dans la soupe.

@metallode689 : Si tu n'es pas super calé en itérateurs, algorithmique, je t'invite à faire ça en dur. Enfin, c'est ton choix, mais c'est toujours mieux de comprendre comment ça marche.

@Ph_Gr : Par contre moi je prends!

Arnaud DS

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