Discussion :

[ultimax]

Bonjour,
j'ai créé un singleton en définissant également des variables A et B.

par commodité je les ai déclaré en static au lieu de définir des getter/setter.

cependant en lançant findBugs il me donne ce code erreur

ST: Write to static field from instance method (ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD)

This instance method writes to a static field. This is tricky to get correct if multiple instances are being manipulated, and generally bad practice.

voici le code de mon singleton

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
 
public class XSingleton {
        private static XSingleton _singleton;
	private static XSearcher _searcher;
	private static XReader _reader;
 
       private XSingleton() {
		super();
	}
 
	public static final XSingleton getInstance()
	{
		return _singleton;
	}
 
        static final void initialize(XSearcher indexSearcher, XReader   indexReader)
	{
		_singleton = new XSingleton();
		_searcher = indexSearcher;
		_reader = indexReader;
	}
 
....
}

Ma question: quelle est le best practice pour les singletons vis a vis de ces attributs?

dois je enlever le mot clé static devant la définition de _searcher et de _reader et dois je déclarer à la place des getter/setter?
ou alors je ne tiens pas compte de findBugs pour ce cas

merci d'avance

[dinobogan]

par commodité je les ai déclaré en static au lieu de définir des getter/setter.

Ils sont private, donc inaccessible depuis l'extérieur.
L'instance unique est gérée par le singleton, donc inutile de mettre static ces données membres.
De plus, ton code est bancal. Si quelqu'un appelle une seconde fois la méthode d'initialisation, ton singleton change pour tout le monde.
La création du singleton doit se faire soit à la déclaration de la données, soit dans la méthode getInstance.

[ultimax]

Merci de ta réponse rapide.

pour info la méthode initialise est appelé une fois au démarrage de l'application.

mais effectivement rien n'empêche un dév de la rappeler dans son code.

le souci est que si j'enlève le mot clé static, je ne peux plus utiliser la fonction initialise. ou alors il faut que je change ma façon de faire.

appelé mon instance de singleton et seulement après mon initialise. comme cela je n'aurais plus a définir initialise en static et donc par la même occasion mes autre objets

[lucimed]

Voici un exemple :

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
public class Singleton {
 
   protected Singleton() {
     // ...
   }
 
 
   /**
    * A handle to the unique Singleton instance.
    */
   static private Singleton _instance = null;
 
   /**
    * @return The unique instance of this class.
    */
   static public Singleton instance() {
      if(null == _instance) {
         _instance = new Singleton();
      }
      return _instance;
   }
 
 
   /. ...additional methods omitted...
}

[hibour]

Faudra initialiser les autres variables non static dans ton constructeur privé .

[ultimax]

merci pour vos réponses.

je vais partir sur une définition classique de singleton,
et appeler ensuite l'initialisation de mes objets.

[dinobogan]

Tu peux faire ça :

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
public class XSingleton
{
  private static XSingleton _singleton = null;
  private XSearcher _searcher;
  private XReader _reader;
 
  private XSingleton( XSearcher s, XReader r )
  {
    _searcher = s;
    _reader = r;
  }
 
  public static final XSingleton getInstance()
  {
    if( _singleton == null )
    {
      ... gérer exception de nullité... ou pas...
    }
    return _singleton;
  }
 
  public static final synchronized XSingleton getInstance( XSearcher s, XReader r )
  {
    if( _singleton == null )
    {
      _singleton = new XSingleton( s, r );
    }
    return _singleton;
  }
 
....
}

Si tu veux malgré tout construire le singleton dans la méthode "getInstance" sans argument, n'oublie pas de la synchroniser.