Discussion :

[Baptiste Wicht]

Bonjour,

J'ai un petit problème avec le singleton...

J'ai une classe singleton OpLendings qui initialise un autre singleton (OpFilms) dans ses variables d'instances. Jusque là pas de problèmes, sauf que OpFilms a aussi un OpLendinfs dans ses variables d'instances...

Vous imaginez donc le problème... J'ai une jolie StackOverflowError.

Bien sûr une solution serait de ne pas initialiser le singleton en variable d'instance mais de l'initialiser à la première utilisation ou alors de ne pas l'initialiser du tout et de toujours faire référence à getInstance()...

Vous avez une autre idée ? Parce que ces manières de faire ne me conviennent pas tout à fait...

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 class OpLendings {
	/* Instances */
	private OpFilms films = OpFilms.getInstance();
 
	/* Singleton's instance */
	private static OpLendings instance = null;
 
	private OpLendings(){
		super();
	}
 
	public static OpLendings getInstance(){
		if(instance == null){
			instance = new OpLendings();
		}
 
		return instance;
	}
}

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
public class OpFilms {
	//Liste des films
	private DataStock<OdFilmImpl> films = null;
	private DataStock<OdFilmImpl> displayList = null;
 
	/* Singletons */
	private DatabaseManager dataBase = DatabaseManager.getInstance();
	private MyLogger logger = MyLoggerFactory.getLogger(this.getClass());
 
	/* Instances */
	private OpLendings emprunts = OpLendings.getInstance();
 
	/* Singleton's instance */
	private static OpFilms instance = null;
 
	private OpFilms(){
		super();
	}
 
	public static OpFilms getInstance(){
		if(instance == null){
			instance = new OpFilms();
		}
 
		return instance;
	}
}

[natha]

Initialise tes instances dans la méthode getInstance().

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 class OpLendings {
  /* Instances */
  private static OpFilms films;
 
  /* Singleton's instance */
  private static OpLendings instance;
 
  private OpLendings(){  super(); }
 
  public static OpLendings getInstance(){
    if(instance == null){
      instance = new OpLendings();
      films = OpFilms.getInstance();
    }
    return instance;
  }
}

J'vois pas trop comment faire autrement si ce n'est dans le constructeur directement.

[Baptiste Wicht]

Bon, je réponds vachement tard, mais je m'étais pas repenché sur ce projet depuis ce moment et j'avais un peu oublié ce problème

Alors merci bien natha, ca marche très bien

[natha]

LoL ok, mieux vaut tard que jamais.
Sinon en relisant je me rend compte qu'il faudrait aussi penser à faire des méthodes synchronisées

Code :

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

1
2
3
public static synchronized OpLendings getInstance(){
    //...
}

Sinon c'est très vilain

[Baptiste Wicht]

LoL ok, mieux vaut tard que jamais.
Sinon en relisant je me rend compte qu'il faudrait aussi penser à faire des méthodes synchronisées

Code :

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

1
2
3
public static synchronized OpLendings getInstance(){
    //...
}

Sinon c'est très vilain

Pour le moment, ces méthodes ne sont pas utilisés en multithreading, donc c'est bon, mais il faudra effectivement les synchroniser quand les threads utiliseront ces méthodes.

[natha]

Pour le moment, ces méthodes ne sont pas utilisés en multithreading, donc c'est bon, mais il faudra effectivement les synchroniser quand les threads utiliseront ces méthodes.

Autant le faire de suite, c'est toujours plus ch***t de devoir mettre à jour ce genre de chose par la suite
Mais c'est toi qui voit

[Baptiste Wicht]

Autant le faire de suite, c'est toujours plus ch***t de devoir mettre à jour ce genre de chose par la suite
Mais c'est toi qui voit

C'est sûr, mais comme j'ai beaucoup de ces classes singletons et que j'ai de très nombreux appels à ces classes, j'aimerais éviter de perdre des performances pour rien.

Si j'ai besoin de multithreader cette partie-là, j'aurai pas trop de problème à trouver les méthodes à synchroniser. Mais comme tu le dis, c'est moi qui voit, un autre ferait peut-être autrement.