Iterateur sur enum

Version imprimable

03/10/2017, 15h10
bj303931

Iterateur sur enum
Bonjour,
j'essaye d'itérer sur un énum qui représente un code-barre :oops:avec Epaisseur et Nature à l'intérieur.

Je suis bloquée à 2 endroits :( dans le méthode next.

Mes getters ne sont pas reconnus.:(

Mes enums ne passent pas dans la méthode next()...:oops:
(erreur: undefined type)

La méthode next:
Code:

1 2 3 4 5 public Code39.Epaisseur next(){ Code39.Epaisseur next=pile.pop(); descendre(E); return next.getE(); }
Bien à vous.

En terme de conception, il serait préférable de créer une classe Enum au lieu de créer une classe basique comme tu l'as fait.

Code:

1
2
3
4
5
6
7
8
9
 
public enum CodeBarre {
// ajoute tes enums
// créer une variable de type entier qui contient la valeur de l'enum
// créer un contructeur en privé avec comme paramètre la valeur
// créer une méthode get puis une méthode set qui sont des getter et setter
// créer ta méthode next() qui elle pourra appeler ton getter
// ensuite tester ta classe car la classe enum n'est pas instantiable puisqu'abstraite 
}

03/10/2017, 17h01
joel.drigo

Salut,

Au lieu de faire des phrases ("tel truc avec tel machin à l'intérieur"), ce serait mieux de nous montrer le code complet, ça nous permettrait de voir vraiment de quoi tu parles.

Si Code39.Epaisseur est une enum, il n'y a pas de raison qu'elle ait une méthode getE(), sauf si tu en as écrite une, et donc il faudrait qu'on la voit pour se faire une idée de ce qu'elle fait, ce qu'elle prend en paramètre et ce qu'elle retourne... pareil pour tout le reste (et c'est quoi E ?).

Oui, tu as raison. Il y a 2 fichiers classes: l'itérateur et la classe d'action ;)

La classe de l'itérateur

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
 
import java.util.Deque;
import java.util.LinkedList;
 
public class IterateurSymbole {
 
	private final Deque <Code39.Epaisseur> pile; 
 
	public IterateurSymbole (Code39.Epaisseur E, Code39.Nature N){
 
		pile=new LinkedList<>();
		descendre(E);
 
	}
 
	private void descendre(Code39.Epaisseur E){
		Code39.Epaisseur Ecourant = E;
 
		while(Code39.Epaisseur.getE() != null){
			pile.push(Ecourant);
		}
 
	}
 
	public boolean hasNext(){
		return !pile.isEmpty();
	}
 
	public Code39.Epaisseur next(){
		Code39.Epaisseur next=pile.pop();
		descendre(E);
		return next.getE();
	}
 
}

L'autre classe ;)

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
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
import java.awt.Graphics;
import javax.swing.JFrame;
import javax.swing.JPanel;
 
public class Code39 {
 
	public static void main(String[] args){
		System.out.println("It's run!");
		CodeBarre("0110", g);
 
	}
 
	enum Epaisseur {LARGE, ETROIT};
	Epaisseur epaisseur;
	enum Nature {BARRE, ESPACE};
	Nature nature;
 
	public Code39(){this(Epaisseur.ETROIT,Nature.ESPACE);}
	public Code39(Epaisseur E, Nature N){
		this.setE(E);
		this.setN(N);		
										}
 
	public void setE(Epaisseur epaisseur){
		this.epaisseur=epaisseur;
	}
 
	public void setN(Nature nature){
		this.nature=nature;		
	}
 
	public Epaisseur getE(){return this.epaisseur;}
	public Nature getN(){return this.nature;}
 
//Dessiner le code barre : 0 une barre étroite; 1 un espace large
	public void CodeBarre(String CB, Graphics g){
		int i=0;
		int n=CB.length();
		Epaisseur E=Epaisseur.ETROIT;
		Nature N=Nature.ESPACE;
		for(i=0; i<n; i++){
			if (CB.charAt(i)==0){
				E=Epaisseur.ETROIT;
				N=Nature.BARRE;
			}
			else{
				E=Epaisseur.LARGE;
				N=Nature.ESPACE;
			}
		}
		draw(E,N,g);
	}
 
	public void draw(Epaisseur E, Nature N, Graphics g){
		int espace=0;
		if (E==Epaisseur.ETROIT && N==Nature.BARRE){
			//Dessiner une barre étroite
			g.drawLine(10+espace,10+espace,100,100);
		}
		else if(E==Epaisseur.LARGE && N==Nature.BARRE){
			//Dessiner une barre large
			g.drawLine(25+espace,25+espace,100,100);
		}
		else if(E==Epaisseur.ETROIT && N==Nature.ESPACE){
			//Un espace petit
			espace=espace+10;		
		}
		else if(E==Epaisseur.LARGE && N==Nature.BARRE){
			//Dessiner un grand espace
			espace=espace+24;
		}
	}
 
	public String toString(){
 
		StringBuilder builder=new StringBuilder();
		toString(builder, this);
		return builder.toString(); }
 
	private static void toString(StringBuilder builder, Code39 v){
 
		builder.append('(');
		builder.append(v);
		builder.append(' ');
 
		builder.append(' ');
		builder.append(')');
	}
 
 
 
}

Sinon transact, je vais suivre ton conseil et faire une classe énum, merci.;)

03/10/2017, 18h14
joel.drigo

Là, on voit tout de suite que getE (getE et getN, que tu aurais pu appeler getEpaisseur et getNature quand même !) est une méthode de la classe Code39, pas de la classe Epaisseur, donc il n'y a aucune raison de pouvoir appeler getE sur une instance de Epaisseur (enum ou pas enum).

C'est quoi exactement que tu cherches à faire ? Itérer à priori, mais itérer sur quoi ?

Citation:

Envoyé par bj303931

Sinon transact, je vais suivre ton conseil et faire une classe énum, merci.

Oui, m'enfin, tu en as déjà deux des enum, pourquoi y aurait-il besoin d'en faire d'autres ?
04/10/2017, 13h50
Rony Rauzduel

Citation:

Oui, m'enfin, tu en as déjà deux des enum, pourquoi y aurait-il besoin d'en faire d'autres ?

En terme de bonnes pratiques à acquérir dès les débuts en programmation. Il est préférable de dissocier les entités en respectant le SRP(Single Responsability Principle) ... même pour un exercice.
ici il s'agit de créer uniquement une enum pour Epaisseur et une enum pour Nature tout simplement.
04/10/2017, 14h24
joel.drigo

Citation:

Envoyé par transact

ici il s'agit de créer uniquement une enum pour Epaisseur et une enum pour Nature tout simplement.

Elles existent déjà ces deux enum ! Si tu parles de les mettre dans un fichier .java à part, c'est un autre sujet (plus de modélisation, mais d'organisation).