problème avec StreamReader sur la lecture d'un fichier texte

Bonjour je suis en train de refaire un programme dans lequel j'ai énormément de GOTO. Pour essayé de les éliminer, j'ai fais des petite fonctions.
Dans mon code pour un bouton j'ai

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

private void button1_Click(object sender, EventArgs e) { //définition des variables locale DialogResult r = DialogResult; // variable longueur pour la longueur du nom du fichier avec le chemin int longueur = 0; if (radioButton1.Checked == false && radioButton2.Checked == false && radioButton3.Checked == false && radioButton4.Checked == false) { MessageBox.Show("Vous n'avez pas choisi le type de processeur!", "ERREUR", MessageBoxButtons.OK, MessageBoxIcon.Warning); return; } //ouverture du fichier à désassembler OpenFileDialog dlg = new OpenFileDialog(); //affichage de la boite de dialogue ouverture de fichier dlg.Title = "Ouverture du fichier à désassembler"; ....... //création du constructeur de lecture du fichier à désassembler FileStream fs = new FileStream(fichier, FileMode.Open, FileAccess.Read); //création du lecteur StreamReader lecfic = new StreamReader(fs);

---

Dans une fonction j'ai fais

Code:

---

1 2 3 4 5 6 7 8 9 10 11

public void adok() { //lecture de la première ligne de code (entete) A = lecfic.ReadLine(); //lecture de la deuxième ligne début du code A = lecfic.ReadLine(); //positionnement sur le premier byte de la ligne commence à 0, 5 hexa soit 6 en décimale pb = 5; // vers la routine Boucle1 Boucle1(); // goto Boucle1; //=====> }

---

Lorsque j’exécute j'ai un message sur la première ligne A = lecfic.ReadLine(); Exception levée A = lecfic.ReadLine(); system.Nullreference exception : la référence d'objet n'est pas définie à une instance d'un objet.
J'ai du oublier quelques chose ?? Je pense que les définitions du StreamReader sont perdue en quitant la fonction bouton1. Comment les faire suivre dans mes autres fonctions ????

bah, c'est quoi A ?
sinon pour ta variable lecfic effectivement c'est un problème de portée, ici elle est déclarée dans ta fonction de click, donc n'existe plus à sa sortie.
donc il faut la déclarer à l'extérieur de la fonction en public si nécessaire, ou peut-être mieux la passer en paramètre à cette fonction qui en a besoin.
Ou encore, passer le nom du fichier a ta fonction et initié ton streamreader dans celle-ci (un commentaire indique que tu lis la 1ère ligne du fichier, or a priori, rien n'assure que l'on se trouverait au début du fichier à ce moment là)

Comme dans adok() il accède directement à lecfic sans le déclarer, il est possible qu'il y ait une variable de ce nom déclarée sur la portée de la fenêtre. Mais comme dans le gestionnaire du bouton il redéclare une variable lecfic, celle-ci masque la variable globale qui, du coup, n'est pas initialisée.

Bonjour, n'arrivant pas à faire, je reviens vers vous.
J'ai fais

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

public void adok() { //lecture de la première ligne de code (entete) Lecture(); //lecture de la deuxième ligne début du code Lecture(); //positionnement sur le premier byte de la ligne commence à 0, 5 hexa soit 6 en décimale pb = 5; // vers la routine Boucle1 Boucle1(); // goto Boucle1; //=====> } /// <summary> /// Routine lecture dans le fichier xxx.epr ou xxx.obj ou xxx.s19 /// </summary> /// <returns></returns>   public void Lecture() { //création de fs file stream pour la lecture FileStream fs = new FileStream(fichier, FileMode.Open, FileAccess.Read); //création du lecteur StreamReader lecfic = new StreamReader(fs); //lecture d'une ligne de code A = lecfic.ReadLine(); //return A; }

---

sans résultat je lis toujours la première lire du fichier en entré.
A est une variable string qui me permet de récupérer une ligne dans le fichier en entrée. Fichier texte

Code:

---

1 2 3

00 01 02 03 04 05 06 07 08 09 0A 0B 0C 0D 0E 0F 0000 0F 7F 70 01 7F 70 03 7F 70 11 7F 70 13 7F 70 06 0010 CE 00 04 FF 70 00 CE FF 1C FF 70 02 CE 00 05 FF

---

Merci de m'aider

À chaque fois que tu appelles la fonction Lecture() tu ouvres le fichier et lis la première ligne. Il te faut ouvrir le fichier une fois et garder la même source après avoir ignoré la première ligne. Par ailleurs, tu peux récupérer le contenu du fichier sous forme de lignes avec un simple File.ReadLines ou File.ReadAllLines.

A mon sens, il faut que tu comprennes ce que fait le programme et le refaire de zéro (ou presque) sans suivre la logique des goto, ça sera peut-être plus efficace, et à mon sens ici la fonction lecture n'a pas lieu d'être

Bonjour, je pense avoir réussi ce que je voulais faire.
Dans les définitions des variables j'ai fait:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

// définition pour la lecture de fichier private FileStream fs; public FileStream FS { get => fs; set => fs = value; } // définition pour lecteur du fichier à désassembler private StreamReader sr; public StreamReader SR { get => sr; set => sr = value; } // définition pour ecriture de fichier private FileStream ws; public FileStream WS { get => ws; set => ws = value; } // définition pour écriture du fichier désassemblé private StreamWriter sw; public StreamWriter SW { get => sw; set => sw = value; }

---

Dans le corps de mon bouton 1, j'ai fait :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

//création de fs file stream pour la lecture fs = new FileStream(fichier, FileMode.Open, FileAccess.Read); //création du lecteur SR = new StreamReader(fs); // variable longueur pour la longueur du nom du fichier avec le chemin int longueur = 0; //fichier égale la longueur du nom du fichier plus le passe moins les trois caractères de l'extension longueur = fichier.Length - 3; //création du chemin et du fichier *.dec en sortie contenant le désassemblage du fichier *.epr Fichier1 = string.Concat(fichier.Substring(0, longueur), "dec"); //création du constructeur écriture du fichier de désassemblage ws = new FileStream(Fichier1, FileMode.Create, FileAccess.Write); //création du writer SW = new StreamWriter(ws);

---

Maintenant lorsque je fais

Code:

---

1 2 3 4 5 6 7 8 9 10 11

public void adok() { //lecture de la première ligne de code (entete) A = SR.ReadLine(); //lecture de la deuxième ligne début du code A = SR.ReadLine(); //positionnement sur le premier byte de la ligne commence à 0, 5 hexa soit 6 en décimale pb = 5; // vers la routine Boucle1 Boucle1(); // goto Boucle1; //=====> }

---

, je récupère bien la première ligne de mon fichier texte dans ma variable string A puis la deuxième ligne.
Merci pour vos suggestions et avec acharnement j'ai résolu mon problème.
Je ferme le sujet.
Cordialement ASTVT.

Bonjour,

Code:

---

1 2 3

//fichier égale la longueur du nom du fichier plus le path moins les trois caractères de l'extension longueur = fichier.Length - 3;

---

Attention, tu pars de l'hypothèse que l'extension fait 3 caractères, ni plus ni moins.
Or rien ne te garantie que ce sera toujours le cas.
Tu devrais utiliser la classe Path pour gérer ceci, et plus explicitement Path.ChangeExtension() qui me semble tout indiqué ici.

Tatayo.

j'ai aussi du mal à voir pourquoi une variable publique et une variable privée. Si elle sont définies et utilisées uniquement dans la même portée (dans une classe par exemple), le doublon n'est pas nécessaire. Tu peux te contenter des variables privées si dans le même portée, sinon utilise les publiques (ou passe les en paramètres des fonctions)