Problème de boucle while

Invité · 05/02/2014, 08h57

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 :

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

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

Invité · 05/02/2014, 09h00

Bonjour,

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

Invité · 05/02/2014, 09h11

Code :

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

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

Invité · 05/02/2014, 09h25

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'?

Invité · 05/02/2014, 09h32

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.

**PixelJuice** · 05/02/2014, 09h37

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.

**PixelJuice** · 05/02/2014, 09h31

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.

Invité · 05/02/2014, 09h39

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 ^^).

Invité · 05/02/2014, 10h06

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".

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?

**ArnaudDS** · 05/02/2014, 10h14

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 :

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

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

**Loudlee** · 05/02/2014, 17h06

Bonjour,

Je suis débutante également et mon code est sale, mais pour répondre à :

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...

j'aurais fais ça :

Code :

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

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
57
58
59
60
61
using System;
using System.Collections.Generic;
using System.Windows.Forms;
using System.IO;
 
namespace Recherche {
	public partial class Form1 : Form {
 
		List<String> listedFile = new List<string>();
		int lastLine = 0;
 
		public Form1() {
			InitializeComponent();
		}
 
		private void bt_file_Click(object sender, EventArgs e) {
			Stream myStream = null;
			listedFile.Clear();
			lastLine = 0;
			if ( opfile.ShowDialog() == DialogResult.OK ) {
				try {
					if ( ( myStream = opfile.OpenFile() ) != null ) {
						using ( myStream ) {
							tb_file.Text = opfile.FileName;
							StreamReader sReader = new StreamReader(tb_file.Text);
							string line;
							while ( ( line = sReader.ReadLine() ) != null ) {
								listedFile.Add(line);
							}
							sReader.Close();
						}
					}
				}
				catch ( Exception ex ) {
					MessageBox.Show("Error: Could not read file from disk. Original error: " + ex.Message);
				}
			}
		}
 
		private void bt_search_Click(object sender, EventArgs e) {
			searchNext();
		}
 
		private void bt_next_Click(object sender, EventArgs e) {
			searchNext();
		}
 
		public void searchNext() {
			if ( tb_word.Text != "" ) {
				for ( int i = lastLine; i < listedFile.Count; i++ ) {
					if ( listedFile[i].ToLower().Contains(tb_word.Text.ToLower()) ) {
						tb_find.Text = listedFile[i];
						lastLine++;
						break;
					}
					lastLine++;
				}
			}
		}
	}
}

en sachant que les variable qui commencent par "tb" sont des textbox, celles qui commencent par "bt" sont des boutons, et que "opfile" est un control "openFileDialog" (que j'utilise d'ailleurs pour la première fois) et qui a comme attributs :
.Filter = txt files (*.txt)|*.txt|All files (*.*)|*.*
.InitialDirectory = c:\\

Enjoy

Invité · 11/02/2014, 16h04

j'ai donc fini par faire une boucle for (car en C# le for c'est trop fort xD) j'avais juste zapé que dans les conditions du for on pouvait mettre une variable:

Code :

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

for (i = c; i = 50; i++)

le c pouvait donc être une variable ce qui à résolu tout mon problème de recherche merci pour toutes vos pistes.

j'ai un légé souci lorsqu'il sort de la boucle il m'indique une erreur du type "ne peut pas être null" et là le programme plante :/ mais le problème de base est donc résolu

Problème de boucle while [Débutant]

C#

Vue hybride

Discussions similaires

Partager

Partager