Discussion :

[kalimmba]

Voila mon code
Pour l'instant je veux lire un fichier et afficher chaque ligne
Puis récupérer les infos séparées par un : ou un ;

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
StreamReader lectureFic = new StreamReader("C:\\Users\\...\\list_users.txt");
 
            // Nombre de lignes
            string Lignes = "";
            int i = 0;
 
            while(Lignes!=null)
            {
                Lignes = lectureFic.ReadLine();
                i++;
                // Affiche les lignes
                Console.WriteLine(Lignes);
 
                if(Lignes!=null)
                {
                    //Type local ou remote
                char sep = ':';
                string[] types = Lignes.Split(sep);
                    foreach(string type in types)
                    {
                        Console.WriteLine(type);
                    }
                }
 
 
            }

Sauf que
1/ mon fichier contient x lignes et le résultat est toujours x+1 lignes
2/ J'ai un message d'erreur : Exception non gérée*: System.NullReferenceException: La référence d'objet n'est pas définie à une instance d'un objet sur la ligne string[] types = Lignes.Split(sep);Si j'ai bien compris ce message apaprait si un des objets est null mais là je vois pas ce qui est null

[PixelJuice]

Bonjour,

qu'entends tu par lignes + 1 ? Tu parles du vide qui est affiché ? Ca vient du fait que tu écris la ligne sans vérifier si elle est nulle, il suffit de descendre le WriteLine() dans le bloc if

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
                if (Lignes != null)
                {
                    // Affiche les lignes
                    Console.WriteLine(Lignes);
 
                    //Type local ou remote
                    char sep = ':';
 
                    string[] types = Lignes.Split(sep);
 
                    foreach (string type in types)
                    {
                        Console.WriteLine(type);
                    }
                }

Pour de ce qui est de l'exception, pourrais-tu montrer le contenu de ton fichier (ou une partie si il est volumineux), peut être qu'il y a quelque chose qui n'est pas prévu. J'ai testé ton code vite fait avec une liste très simple avec des : et il n'y a pas de problème.

[kalimmba]

Mon fichier comporte 3 lignes et il me dit qu'il y en a 4, donc a priori une 'null'.
Serait ce la cause du message d'erreur ?

Si je mets le WriteLine(Lignes) dans le if, j'ai le même résultat
Si je sors le Writeline(type) du foreach j'ai une autre erreur puisque type n'est valable que dans la boucle
Ou alors j'ai mal compris tes indications

Merci en tout cas

[Max]

Salut

Pour le mauvais nombre de lignes, cela vient de l'ordre de tes instructions et de ta condition de sortie. Elles ne s'enchaînent pas bien :

Code :

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

1
2
3
4
5
while (Lignes!=null) {
	Lignes = lectureFic.ReadLine();
	i++;
	// ...
}

Si tu déroules ta boucle :

• Lignes est initialisé à "", tu rentres donc ta boucle
• 1er passage : une ligne est lue, c'est OK, i passe à 1
• 2e passage : une ligne est lue, c'est OK, i passe à 2
• 3e passage : une ligne est lue, c'est OK, i passe à 3
• 4e passage : une ligne est lue, c'est KO c'est la fin du fichier donc Lignes passe à null, mais i passe quand même à 4
• tu sors de la boucle vu que Lignes vaut null

C'est bien à cause de ça que tu obtiens un System.NullReferenceException : tu essaies de faire un Split sur Lignes qui vaut null.

Voici une meilleure façon simple de lire un fichier et compter ses lignes :

Code :

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

1
2
3
4
5
6
7
8
using (StreamReader reader = new StreamReader("...")) {
	string currentLine;
	int i = 0;
	while ((currentLine = reader.ReadLine()) != null) {
		// Ici tu ne rentres dans ce bloc que lorsque la ligne est valable
		i++;
	}
}

[kalimmba]

Super merci beaucoup