Discussion :

[kolorius]

Bonjour à tous,

J'ai un fichier texte dans lequel j'ai sauvegardé les valeurs des différents attributs de mes objets. Exemple : j'ai une classe Client qui a 2 attributs : String login et String mdp donc si j'ai entré un client qui a pour login "toto" et pour mdp "tata" j'ai dans mon fichier texte : "toto tata". J'aimerais, en lisant ce fichier sauvegarde, réattribuer les valeurs toto et tata respectivement aux attributs login et mdp. Cependant, je ne sais manipuler que les Fichiers et non pas les Streams.

J'espère avoir été clair x)

Merci de votre aide

[kolodz]

Tu trouvera dans les FAQ Java tout ce qui t'es nécessaire pour lire ton fichier texte :
Comment lire un fichier ligne par ligne ?
Pour ce qui est de l'affectation :

Code :

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

1
2
3
4
5
String line, login, pass;
// Affecation pour la démonstration
line = "toto tata";
login = line.split(" ")[0]; // toto
pass = line.split(" ")[1]; // tata

Tu ne devrais pas avoir de problème à lier les deux.

Cordialement,
Patrick Kolodziejczyk.

[kolorius]

Bonjour kolodz,

Merci de ton aide, j'étais parvenu à la même solution à savoir utiliser split seulement lorsque j'exécute j'obtiens l'erreur suivante :

Code :

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

1
2
3
4
5
6
7
8
Exception in thread "main" java.lang.OutOfMemoryError: Java heap space
	at java.util.ArrayList.subList(ArrayList.java:955)
	at java.lang.String.split(String.java:2311)
	at java.lang.String.split(String.java:2355)
	at projetcpo.listeClients.charger(listeClients.java:72)
	at projetcpo.Camping.chargerclient(Camping.java:51)
	at projetcpo.ProjetCPO.main(ProjetCPO.java:40)
Java Result: 1

[kolodz]

Peut-on voir ton code ?
Il semble que Java manque de mémoire pour générer les nouveaux String. C'est possible, mais pas sur un petit fichier. Il faudrait que celui-ci soit conséquent ou qu'il y ai une boucle infini !

Cordialement,
Patrick Kolodziejczyk.

[kolorius]

Oui j'ai essayé d'allouer plus de mémoire à la JVM dans les propriétés du projet mais ça ne change rien :/

Voici la méthode charger qui appelle split:

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
public void charger() throws IOException{
          try{
          FileReader fich = new FileReader(sauvegardeClients);
          BufferedReader br = new BufferedReader(fich);
          String ligne = br.readLine(), login, mdp;
          while(ligne != null){
              login = ligne.split("/")[0];
              mdp = ligne.split("/")[1];
              Client c = new Client(login, mdp);
              liste1.add(c);
              br.close();
          }
          fich.close();
          }catch(IOException ex){
              System.out.println("Chargement non fait");
          }
      }

[thelvin]

Hello,

tu pourrais nous décrire quand est-ce que la boucle while() s'arrête ?

[kolorius]

Elle s'arrête quand la ligne suivante dans le fichier texte est vide?

[thelvin]

Euh.... Okay.

Et tu pourrais nous expliquer comment ça marche ?

Par exemple je vois pas le rapport entre ligne != null et la prochaine ligne. Là comme ça, ça ressemble plus à la ligne actuelle.

[eulbobo]

Code :

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

1
2
 
          String ligne = br.readLine(), login, mdp;

Je te donne mon avis sur cette ligne...
Déclarer plusieurs variables sur une ligne, c'est moche, mais les déclarer et les instancier sur une seule ligne c'est encore pire !
Question de lisibilité, essaye toujours de séparer tes déclarations :

Code :

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

1
2
3
4
 
          String ligne = br.readLine()
          String login;
          String mdp;

Ici, tu as une chaine que tu découpes deux fois exactement de la même façon : pourquoi ne pas la découper qu'une seule fois ?

Code :

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

1
2
3
 
          login = ligne.split("/")[0];
          mdp = ligne.split("/")[1];

A transformer en :

Code :

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

1
2
3
4
 
          String[] splitted = ligne.split("/");
          login = splitted[0];
          mdp = splitted[1];

Et dernier point, la condition de sortie de ta boucle n'est pas bonne : dans le code actuel, tu vas remplir ta liste avec toujours la première ligne de ton fichier source... En boucle... A l'infini. D'où l'erreur.
Faut juste que tu oublie pas de relire la ligne d'après à la fin de ta boucle (ou dans la déclaration de la boucle avec une syntaxe que je trouve horrible mais qui est utilisée partout) :

Code :

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

1
2
3
4
 
	while ((ligne = br.readLine()) != null){
		// boucle
	}

SPOILER ALERT : à l'exécution, si tu changes que ça, tu vas encore avoir un problème à l'exécution ^^

[kolorius]

Salut eulbobo,

Merci beaucoup pour ton aide, j'ai modifié mon code ainsi :

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
public void charger() throws IOException{
          try{
          FileReader fich = new FileReader(sauvegardeClients);
          BufferedReader br = new BufferedReader(fich);
          String ligne = br.readLine();
          String login;
          String mdp;
          while(ligne != null){
              String[] splitted = ligne.split("/");
              login = splitted[0];
              mdp = splitted[1];
              Client c = new Client(login, mdp);
              liste1.add(c);
              ligne = br.readLine();
          }
          fich.close();
          }catch(IOException ex){
              System.out.println("Chargement non fait");
          }
      }

et ça a l'air de fonctionner

[kolodz]

eulbobo te proposait une version de la boucle un peu plus propre :

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
public void charger() throws IOException{
          try{
          FileReader fich = new FileReader(sauvegardeClients);
          BufferedReader br = new BufferedReader(fich);
          String ligne;
          String login;
          String mdp;
          while ((ligne = br.readLine()) != null){
              String[] splitted = ligne.split("/");
              login = splitted[0];
              mdp = splitted[1];
              Client c = new Client(login, mdp);
              liste1.add(c);
          }
          fich.close();
          }catch(IOException ex){
              System.out.println("Chargement non fait");
          }
      }

L'idée étant de bien avoir la lecture de la nouvelle ligne et le test au niveau de la structure while et non dans le code interne au while.

Cordialement,
Patrick Kolodziejczyk.