[C#]Lecture de fichiers

Version imprimable

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:

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:

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:

1:root 2:pass 3:0

et apres j'ai :

Code:

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.

bonjour,
essaie :

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

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

15/02/2007, 17h02
jmathon

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

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 :roll:
Je sais pas pourquoi tu utilises un fichier binaire 8O.
Je te propose une methode qui est beaucoup plus elegante.
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 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:

1 2 3 BinaryReader input = null; BinaryWriter output = null;
Je te conseille d'utiliser toujours un

Code:

try/catch/finally

si tu traites l'exception et un

Code:

using { ... }

si tu la traites pas.

J'espere que ces petits conseils vont t'aider :)
A+

15/02/2007, 18h03
scifire
notalp pourquoi tu utilises un ArrayList c'est beaucoup mieux d'utiliser un List<string> 8O (en C# 2.0 bien sur).
Si une exception se leve tu vas pas fermer ton flux (monStreamReader) - finally :roll:
Code:

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:

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

:roll:

No hard feelings :)
16/02/2007, 10h25
notalp

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 ...
16/02/2007, 23h24
scifire

Premierement merci :)

Citation:

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 :lol:
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

Citation:

"users.dat"

de jmathon
A+
18/02/2007, 11h40
jmathon

j'ai utilisé la solution de notalp
Etant debutant en C# je la trouve plus "simple" :p

Merci bien !