Discussion :

[luckyalan]

bonjour,
je fais les exercices d'un livre que le possède et je n'arrive pas à faire marcher 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
44
45
46
47
48
49
50
51
52
import java.io.*;
import java.util.*;
public class GestionFichier {
		public static void main (String [] arg) throws IOException {
		Scanner lectureClavier = new Scanner(System.in);
		Fichier f = new Fichier();
		System.out.print("Entrer le nom du fichier : ");
		String nomFichier = lectureClavier.next();
		f.ouvrir(nomFichier, "Ecriture");
		for (int i=0;i<5;i++) f.ecrire(i);
		f.fermer();
 
		f.ouvrir(nomFichier,"Lecture");
		String chaine ="";
		do{
			chaine =f.lire();
			if(chaine!= null)System.out.println(chaine);
		}while (chaine !=null);
		f.fermer();
		}	
}
 
class Fichier{
	private BufferedWriter fW;
	private BufferedReader fR;
	private char mode;
	public void ouvrir(String nomDuFichier, String s) throws IOException{
		mode = (s.toUpperCase()).charAt(0);
		if (mode=='R'|| mode == 'L')
		fR = new BufferedReader(new FileReader(new File(nomDuFichier)));
		else if (mode == 'W' || mode == 'E')
		fW = new BufferedWriter(new FileWriter(new File(nomDuFichier)));
	}
	public void ecrire(int tmp) throws IOException{
		String chaine = "";
		chaine = String.valueOf(tmp);
		if (chaine !=null) {
			fW.write(chaine,0,chaine.length());
			fW.newLine();
		}
 
	}
	public String lire() throws IOException{
		String chaine=fR.readLine();
		return chaine;
	}
	public void fermer() throws IOException{
		if (mode == 'R'|| mode == 'L') fR.close();
		else if (mode == 'W'|| mode == 'E') fW.close();
 
	}
}

il me dit dans la ligne class Fichier que "le type Fichier est déjà défini" et ca plante !
je pense que cela vient de la facon que j'ai utilisé pour mettre plusiseurs classes dans le meme fichier. à l'origine j'avais meme appele ma class Fichier public class Fichier, mais j'ai vu que je ne pouvais mettre qu'une class public par source !
ca fait 2 heure que je me bagarre à deplacer les accolades et à renommer mais rien de marche
que faire ?
merci

[joel.drigo]

Salut,

Tu as déjà une autre classe dans le même package qui a ce nom (Fichier). Si le but de l'exercice est de mettre la classe Fichier dans le même fichier que la classe GestionFichier (ce qu'on ne fait jamais en pratique, sauf dans des petits programmes de test), il te faut supprimer l'autre classe Fichier qui se trouve dans le même package.

Tu as l'option également de faire une classe interne statique :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
package fr.developpez.exemples;
 
public class GestionFichier {
 
	/* ... */
 
	public static class Fichier {
 
		/* ... */
 
	}
 
}

Dans ce cas, tu pourras avoir une classe fr.developpez.exemples.Fichier, et ce ne sera pas la même que fr.developpez.exemples.GestionFichier.Fichier.

[joel.drigo]

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
 
class Fichier{
	private BufferedWriter fW;
	private BufferedReader fR;
	private char mode;
	public void ouvrir(String nomDuFichier, String s) throws IOException{
		mode = (s.toUpperCase()).charAt(0);
		if (mode=='R'|| mode == 'L')
		fR = new BufferedReader(new FileReader(new File(nomDuFichier)));
		else if (mode == 'W' || mode == 'E')
		fW = new BufferedWriter(new FileWriter(new File(nomDuFichier)));
	}
	public void ecrire(int tmp) throws IOException{
		String chaine = "";
		chaine = String.valueOf(tmp);
		if (chaine !=null) {
			fW.write(chaine,0,chaine.length());
			fW.newLine();
		}
 
	}
	public String lire() throws IOException{
		String chaine=fR.readLine();
		return chaine;
	}
	public void fermer() throws IOException{.;
		if (mode == 'R'|| mode == 'L') fR.close();
		else if (mode == 'W'|| mode == 'E') fW.close();
 
	}
}

On n'écrit jamais ça (ou vraiment dans des cas rares qu'on maîtrise par des précautions) : je veux parler de déclarer des writers et des readers en variable d'état. Premièrement, tu risques que tes flux ne soient pas correctement fermés (ce qui est le cas de ton programme en cas d'erreur). Le fait que des ressources reste ouvertes continuellement est problématique. En plus, le fait qu'un fichier soit ouvert en lecture et en écriture alternativement rend la classe potentiellement en erreur également (rendre une classe dépendente d'un état demande des précautions), d'autant plus que la classe permet que le même fichier soit ouvert en lecture et en écriture en même temps, ce qui est problématique. Cette structure multiplie les risques de problème d'utilisation.

Est-ce que l'enoncé demande explicitement de faire une classe spécifique avec les traitements d'écriture/lecture ? Car cela pourrait être fait de manière moins problématique avec des méthodes, genre :

Code :

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

1
2
3
private void write(BufferedWriter writer, int tmp) throws IOException {
   /* ... */
}

Sinon, une inner class limiterait les problèmes, avec un mnimum de blindage dans l'utilisage (un try/finally, ou un try-with-ressource).

[luckyalan]

merci Joel,
avec public static class Fichier effectivement ca marche
alors exactement les codes que j'ai recopié dans mon livre (anne Tasso)
sont en plusieurs morceaux (étapes)

Code :

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

1
2
3
4
5
public class Fichier{
	private BufferedWriter fW;
	private BufferedReader fR;
	private char mode;
}

Code :

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

1
2
3
4
5
6
7
public void ouvrir(String nomDuFichier, String s) throws IOException{
		mode = (s.toUpperCase()).charAt(0);
		if (mode=='R'|| mode == 'L')
		fR = new BufferedReader(new FileReader(new File(nomDuFichier)));
		else if (mode == 'W' || mode == 'E')
		fW = new BufferedWriter(new FileWriter(new File(nomDuFichier)));
	}

Code :

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

1
2
3
4
5
6
7
public void ecrire(int tmp) throws IOException{
		String chaine = "";
		chaine = String.valueOf(tmp);
		if (chaine !=null) {
			fW.write(chaine,0,chaine.length());
			fW.newLine();
		}

Code :

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

1
2
3
4
public String lire() throws IOException{
		String chaine=fR.readLine();
		return chaine;
	}

Code :

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

1
2
3
4
5
public void fermer() throws IOException{
		if (mode == 'R'|| mode == 'L') fR.close();
		else if (mode == 'W'|| mode == 'E') fW.close();
 
	}

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
import java.io.*;
import java.util.*;
public class GestionFichier {
		public static void main (String [] arg) throws IOException {
		Scanner lectureClavier = new Scanner(System.in);
		Fichier f = new Fichier();
		System.out.print("Entrer le nom du fichier : ");
		String nomFichier = lectureClavier.next();
		f.ouvrir(nomFichier, "Ecriture");
		for (int i=0;i<5;i++) f.ecrire(i);
		f.fermer();
 
		f.ouvrir(nomFichier,"Lecture");
		String chaine ="";
		do{
			chaine =f.lire();
			if(chaine!= null)System.out.println(chaine);
		}while (chaine !=null);
		f.fermer();
		}	
}

on les découvre dans l'ordre ou je les ai mis, avec les explications pas à pas.
j'aurai du je suppose les compiler un à un, je ne suis vraiement pas habitué à ce type d'écriture pas petits bouts de fichiers, j'ai surtout dans ma jeunesse fait du basic, puis du Qbasic, puis de C mais en m'en tenant aux bases élémentaires, alors la prog orientée objet et les class, ca rentre pas trop facilement
je les ai copié
1) pour apprendre
2) parce que j'envisage de lire écrire modifier un fichier texte et que cela me semblait un début, pas un aboutissement car je n'envisage pas d'écrire que des int, mais aussi des string et autres...
merci pour ton aide précieuse (comme d'hab...)