Discussion :

[Gouyon]

Salut à tous

J'ai une classe que je sauvegarde dans un fichier en écrivant dans le fichier la valeur de chaque variable privée. Parmi ces variables deux sont du type enum.

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
 
public class Pion {
 
	private TypeCamps Camps; //enum
	private int Valeur;
	private TypePion Type;//enum
	private int X, Y;
 
	public Pion(TypeCamps CA, int V, TypePion Ty, int Xo, int Yo, boolean InitGraph) {
		Camps = CA;
		Valeur = V;
		Type = Ty;
		X = Xo;
		Y = Yo;
........
	}
 
 
 
 
public void Sauve(FileOutputStream Fich){
 
		try {
			Fich.write(Camps.ordinal());
			Fich.write(Type.ordinal());
			Fich.write(Valeur);
			Fich.write(X);
			Fich.write(Y);			
		} catch (IOException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
	}
public void Charge(FileInputStream Fich){
 
	try {
		Camps=Fich.read();		
		Type=Fich.read();
		Valeur=Fich.read();
		X=Fich.read();
		Y=Fich.read();
	} catch (IOException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	}
}

Ça se passe bien pour l'enregistrement car j'enregistre (sauf erreur de ma part) la valeur ordinale de mon enum. Par contre pour la lecture je ne sais pas comment faire? J'ai essayé quelque chose comme ça

Code :

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

1
2
 
Camps=(TypeCamps)Fich.read();

mais visiblement il n'en veut pas.
voici comment sont déclarés mes enum

Code :

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

1
2
3
4
5
6
7
8
 
public enum TypePion {
	Rond,Carre,Triangle,Pyramide;
}
 
public enum TypeCamps {
	Rouge,Bleu;
}

Merci d'avance pour vos réponses.

[le y@m's]

Si tu as stocké l'ordinal alors tu récupère l'ordinal lors du read soit un int. Tu ne peux pas caster un int en une enum. Pour récupérer la valeur de l'enum correspondant à l'ordinal tu peux utiliser la méthode values()

Code :

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

1
2
int ordinal = Fich.read();
Camps = TypeCamps.values()[ordinal];

Pourquoi n'utilises tu pas la sérialisation pour sauvegarder ton objet dans un fichier ?
Note que par convention le nom des membres d'une classe commence généralement par une minuscule .

[professeur shadoko]

ne passe pas par l'ordinal (on ne doit pas utiliser cette méthode) mais utilise tout bonnement toString().
tu stockes ainsi le nom de l'instance et après

Code :

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

1
2
3
 
String chaineLue = ******* ;
TypePion pion = TypePion.valueOf(chaineLue) ;

edit: pour les enums la tradition des noms est plutot: tout en majuscule. donc ROND, CARRE, PYRAMIDE.

[adiGuba]

Salut,


Tout d'abord quelques remarques :

• Il est préférable de respecter les conventions de nommage Java. Sinon la lecture du code en devient ambigue...
• Ton traitement des exceptions laisse à désirer. En cas de problème tu te contentes d'affichier un message sur la console. Donc au niveau software tu n'as aucun moyen de détecter l'exception
  Il serait préférable de la laisser reonter...
• Pourquoi utiliser directement des FileInputStream/FileOutputStream ? Si tu ne souhaites lire que des fichiers simples utilises des File/String représentant le nom du fichier et ouvre le toi-même. Si tu veux un truc plus générique utiliser des InputStream/OutputStream...

Sinon pour ton problème : chaque enum possède une méthode static retournant un tableau contenant toutes ses valeurs :

Code :

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

Camps = TypeCamps.values(Fich.read());

a++

[edit] grillé... et bien

[Gouyon]

Il est préférable de respecter les conventions de nommage Java. Sinon la lecture du code en devient ambigue...

Je viens de me lancer en java et je dois avouer que je ne me suis pas trop penché sur les conventions. J'ai plus l'habitude de coder en C et je met des majuscules car je trouve ça plus lisible. Mais promis je ne le ferais plus

Ton traitement des exceptions laisse à désirer. En cas de problème tu te contentes d'affichier un message sur la console. Donc au niveau software tu n'as aucun moyen de détecter l'exception
Il serait préférable de la laisser reonter...

Exact mais ce n'est pas ma préoccupation principale pour le moment je règlerais ça par la suite. Il est d'ailleurs for probable que je pose quelques questions à ce sujet .

Pourquoi utiliser directement des FileInputStream/FileOutputStream ? Si tu ne souhaites lire que des fichiers simples utilises des File/String représentant le nom du fichier et ouvre le toi-même. Si tu veux un truc plus générique utiliser des InputStream/OutputStream...

Là encore n'ayant pas une bonne connaissance de java j'ai été au plus court ce qui n'est pas forcément le plus judicieux. Vu que le y@m's me suggère d'utiliser la sérialisation pour sauvegarder mon objet dans un fichier.

Sinon pour ton problème : chaque enum possède une méthode static retournant un tableau contenant toutes ses valeurs :

Code :

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

Camps = TypeCamps.values(Fich.read());

Il semble préférer une syntax de ce type

Code :

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

Camps=TypeCamps.values()[ord];

Donc à priori problème résolu (je vérifie et je met la balise résolue)

[Gouyon]

Ca a bien marché mais j'ai finalement changé mon fusil d'épaule et utilisé la sérialisation.

Merci à tous