Utiliser un attribut dans plusieurs classes

Version imprimable

Bonjour,

je voudrais pouvoir utiliser un attribut d'une classe (ici une LinkedList) depuis une méthode d'une 2e classe.

Par exemple:

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
 
 
public class Plateau {
	public LinkedList<Integer> listeCellule;
 
	public Plateau() {
		listeCellule = new LinkedList<Integer>();
 
		for( int i = 0; i<12 ; i++){
 
			listeCellule.add(4);
		}
	}
}
 
public class Joueur {
	protected String Nom;
	protected int Gain;
 
	public Joueur(String nom) {
		this.Nom = nom;
		this.Gain = 0;
	}
 
	private LinkedList<Integer> jouerCoup(int CelluleAJouer){
		//j'ai besoin de listeCellule ici
	}
}

Faut-il que j'instancie obligatoirement un Plateau dans classe Joueur?

Merci

08/10/2014, 18h19
Lucio_

Effectivement il faut une instance de Plateau afin d’utiliser la liste de cellule a cette endroit précis. Le plus simple étant passer le Plateau comme paramètre du constructeur.

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
 
 
public class Joueur {
	protected String Nom;
	protected int Gain;
        private Plateau plateau;
	public Joueur(String nom, Plateau p) {
		this.Nom = nom;
		this.Gain = 0;
		plateau =p;
	}
 
       //retourne un élement de la liste 
	private Integer jouerCoup(int CelluleAJouer){
		//j'ai besoin de listeCellule ici
          return this.plateau.getListeCellule().get(CelluleAJouer);
	}
}
 
 
public class Plateau {
	public LinkedList<Integer> listeCellule;
 
	public Plateau() {
		listeCellule = new LinkedList<Integer>();
 
		for( int i = 0; i<12 ; i++){
 
			listeCellule.add(4);
		}
	}
 
       public LinkedList<Integer> getListeCellule(){
	return listeCellule;
       }
 
}

09/10/2014, 13h51
wax78

C'est une idée, mais tu peux aussi le passer a la méthode jouerCoup.

Code:

private Integer jouerCoup(Plateau plateau, int CelluleAJouer){...}
18/10/2014, 15h22
SamusStrife
Tu as plusieurs solutions pour faire ce dont tu as besoin:
- Tu peux mettre ta liste en static pour pouvoir y accéder sans l'instance de la class Plateau
- Vu que tu m as l'air de programmer un jeu, un plateau est unique on est d'accord, donc tu peux mettre ta class Plateau en Singleton (plus logique à mon sens) et accéder facilement a ta liste (et ca pourrai te faciliter la vie pour le reste du code ;)
Voilà tu as le choix ;)

Citation:

Faut-il que j'instancie obligatoirement un Plateau dans classe Joueur?

Il est de loin préférable que un Joueur manipule une instance de Plateau.
Donc oui, il est préférable que tu instancie un Plateau.

Cela peut-être fait lors de l'instanciation du joueur :
(En reprenant le code de armel18 )
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 public Joueur(String nom) { this.Nom = nom; this.Gain = 0; plateau = new Plateau(); } public Joueur(String nom, Plateau p) { if( p == null){ throw new Exception("Un joueur a besoin d'un plateau pour existé !"); } this.Nom = nom; this.Gain = 0; plateau =p; }
Ainsi ton joueur peux faire :
Code:

1 2 3 4 private LinkedList<Integer> jouerCoup(int CelluleAJouer){ this.plateau.getListeCellule(); // TODO utilise la liste et ne pas juste l'appeler. }
Pour ce qui est de l'utilisation static, cela revient à avoir une liste de cellules sans plus de structure logique. Et l'utilisation du singleton n'est pas plus simple que l'utilisation d'une propriété ou d'un passage de référence. Et cela bloque des possibilités ! Comme avoir deux joueurs qui jouent en même temps sur des plateaux différents !

Cordialement,
Patrick Kolodziejczyk.

20/10/2014, 15h10
SamusStrife

Citation:

Envoyé par kolodz

Pour ce qui est de l'utilisation static, cela revient à avoir une liste de cellules sans plus de structure logique. Et l'utilisation du singleton n'est pas plus simple que l'utilisation d'une propriété ou d'un passage de référence. Et cela bloque des possibilités ! Comme avoir deux joueurs qui jouent en même temps sur des plateaux différents !

Je suis d'accord, surtout pour le static (je proposais une solution différente de ce qui avait été dit ;) ). Pour le Singleton, c'est surtout si le Plateau dans son programme est utilisé de façon unique. Évidemment si il compte faire plusieurs parties sur plusieurs plateaux le Singleton est à éviter. En quoi je juge que c'est plus simple? Parce que ça limite les erreurs (à mon sens), facilite la relecture du code et sa compréhension (tant que son utilisation est logique et nécessaire on est d'accord)
20/10/2014, 21h48
wax78

Surtout que selon le cas, un joeur peut jouer sur plusieurs plateaux. Donc autant lui passer le plateau au moment voulu. Avec ou sans singleton, static ou pas static. (mais avec static plus besoin de le passer le plateau... donc perte d'interet.)