[C#]Lecture de fichiers

**jmathon** · 14/02/2007, 10h59

bonjour,
je dois faire un programme en C# ou l'une des fonctions et de lire dans un fichier.
ce fichier comporte une liste d'users et de pass sous la forme :

Code :

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

1
2
3
 
user:pass:gid
user2:pass2:gid2

le probleme est que lorsque je passe au deuxieme user il me compte le '\n' avant
ce qui fait : \nuser2

voici le code :

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
		private bool CheckUserPass(string login, string pass,Socket s)
		{
			BinaryReader input = null;
			BinaryWriter output = null;
			try
			{
				String chaine;
				input=new BinaryReader(new FileStream("users.dat",FileMode.Open,FileAccess.Read));
				while(((chaine = input.ReadString()) != null))
				{	
					String[] chaines = chaine.Split(':');
					Console.WriteLine("1:"+chaines[0]+" 2:"+chaines[1]+" 3:"+chaines[2]);
					if((login==chaines[0]) && (pass==chaines[1]))
							return true;
				}
			}
			catch(FileNotFoundException e)
			{
				//Console.Error.WriteLine(e);
				try
				{
					output= new BinaryWriter(new FileStream("users.dat",FileMode.CreateNew,FileAccess.Write));
					SendMessage("Login Administrateur : ");
					String loginAdmin = System.Text.Encoding.UTF8.GetString(System.Text.Encoding.UTF8.GetBytes(ReadString(s)));
					SendMessage("Password Administrateur : ");
					String passAdmin = System.Text.Encoding.UTF8.GetString(System.Text.Encoding.UTF8.GetBytes(ReadString(s)));
					output.Write(loginAdmin + ":" + passAdmin + ":0");
					output.Close();
					SendMessage("User Added !\n");
				}
				catch(Exception)
				{
					Console.Error.WriteLine("Impossible de créer le fichier users");
				}
 
			}
			catch(Exception e)
			{
				Console.Write(e);
			}
			return false;
 
		}

le premier affichage ce fait nikel :

Code :

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

1:root 2:pass 3:0

et apres j'ai :

Code :

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

1
2
3
 
1:
root 2:pass 3:1

j'ai un autre probleme : a la fin du fichier j'ai une exception EndOfStreamException.
c'est donc que je suis à la fin du fichier, je dois le gerer via un try catch ou il est possible de ne pas générer cette exception en utilisant une autre méthode que ReadString() ?

Merci.

**notalp** · 14/02/2007, 11h29

bonjour,
essaie :

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
public static ArrayList lectureFichier(string fichierpath) 
		{ 
			ArrayList Fichier = new ArrayList();
			try 
			{ 
				StreamReader monStreamReader = new StreamReader(fichierpath); 
				string ligne = monStreamReader.ReadLine(); 
 
				while (ligne != null) 
				{ 
					string [] tab = ligne.Split(':');
					for(int i = 0 ; i < tab.Length ; i++)
					{
						Fichier.Add(tab[i]);
					}
					//					Response.Write(ligne); 
					//					Response.Write("</br>"); 
					ligne = monStreamReader.ReadLine(); 
				} 
 
				monStreamReader.Close(); 
			} 
			catch 
			{
				Fichier.Add("Erreur");
			} 
			return(Fichier);

pour récupérer le fichier puis:

Code :

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

1
2
for (int i =0 ; i < MonArray.Length ; i+=3)
Console.WriteLine("1:"+MonArray[i]+" 2:"+MonArray[i+1]+" 3:"+MonArray[i+2]);

En esperant résoudre ton problème.
Bonne journée
Notalp

**jmathon** · 15/02/2007, 17h02

merci pour ton aide ça a l'air de fonctionner.
mais il y a un probleme avec :

MonArray.Length

la méthode n'existe pas...

**scifire** · 15/02/2007, 17h47

Un ArrayList n'a pas la propriete Length mais Count. Je vais faire un petit commentaire sur le post de notalp dans le post qui suit

Je sais pas pourquoi tu utilises un fichier binaire

.
Je te propose une methode qui est beaucoup plus elegante.

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
public static bool CheckUserPass(string username, string password)
{
    string fName = @"C:\users.txt";
    bool found = false;
 
    if (File.Exists(fName))
    {
        string line = string.Empty;
        string[] userInfo = null;
        using (StreamReader sr = new StreamReader(fName))
        {
            while (!found && (line = sr.ReadLine()) != null)
            {
                userInfo = line.Split(':');
 
                // Les trois champs sont obligatoires
                System.Diagnostics.Debug.Assert(userInfo.Length == 3);
 
                found = (userInfo[0] == username && userInfo[1] == password);
            }
        }
    }
    else
    {
        // Le fichier n'exist pas.
        // Traitement ...
    }
 
 
    return found;
}

Une autre chose.
Pourquoi tu comptes sur une exception FileNotFoundException pour traiter le fait que le fichier n'existe pas

C'est beaucoup plus facile de faire un petit test File.Exists(...).
Encore une chose.
Tu ne fermes jamais tes deux flux

Code :

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

1
2
3
 
BinaryReader input = null;
BinaryWriter output = null;

Je te conseille d'utiliser toujours un

Code :

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

try/catch/finally

si tu traites l'exception et un

Code :

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

using { ... }

si tu la traites pas.

J'espere que ces petits conseils vont t'aider

A+

**scifire** · 15/02/2007, 18h03

notalp pourquoi tu utilises un ArrayList c'est beaucoup mieux d'utiliser un List<string>

(en C# 2.0 bien sur).
Si une exception se leve tu vas pas fermer ton flux (monStreamReader) - finally

Code :

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

1
2
3
4
catch 
{
	Fichier.Add("Erreur");
}

C'est pas la maniere de traiter l'exception

Comme ca tu dis qu'il y a un utilisateur dont le pseudo est "Erreur" mais il n'a ni un mot de passe ni une connexion. Une exception sera levee au moment ou tu fais :

Code :

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

Console.WriteLine("1:"+MonArray[i]+" 2:"+MonArray[i+1]+" 3:"+MonArray[i+2]);

No hard feelings

**notalp** · 16/02/2007, 10h25

Bonjour scifire,
Tu m'as l'air très calé en c#, tu devrais proposé des réponses plus souvent

Pour info :
scifire je suis en 1.1 sans VS.
En plus pour le moment je suis sous blocnot, j'ai fait ce code pour aider jmathon rapidement pour qu'il soit débloqué. Je n'ai pas encore pu réinstaller mon poste de développement donc je ne sais meme pas si mon code compile. mais je supposais que .Lenght ou .count ne poserai pas trop de problème...
Je n'ai pas dit que mon code était le plus propre du monde mais j'ai eu au moins le mérite de lui répondre.

Si tu as une meilleure solution je t"en prie donne la lui

Bien respectueusement
Notalp

PS: quel est l'interret de tester l'existance du fichier si tu renseigne son nom e n dure??? pourquoi tu ne le passe pas par paramètre ?
PPS: pour le finally tu as raison. Mais le forum est principalement la pour aiguiller les gens. Ils peuvent aussi vérifier le code et l'améliorrer ...

**scifire** · 16/02/2007, 23h24

Premierement merci

Envoyé par notalp

PS: quel est l'interret de tester l'existance du fichier si tu renseigne son nom e n dure??? pourquoi tu ne le passe pas par paramètre ?

Meme si le nom de fichier est code en dur ca ne veut pas dire qu'il exite

Mais je suis tout a fait d'accourd avec toi moi aussi j'utiliserai un parametre dont la valeur est chargee a partir d'un fichier de configuration (XML c'est a la mode

) Honnetement je l'ai mis en dur pour rester proche du

"users.dat"

de jmathon
A+