Instance d'une classe

Bonjour,

Quelques remarques :

1) Une liste, comme toutes les collections devraient être paramétrées avec le type de leur contenu.
C'est d'autant plus important ici qu'en réalité dans ton exemple ce ne sont pas des objets Secretaire que tu stockes dans ta liste, mais des WeakReference, j'y reviendrais.

2) Vu ce que l'on veut faire ici, il est intéressant de mettre la liste final et il serait également bon de renvoyer une copie de la liste dans le getInstances, sinon autant mettre la liste public....

3) Pour faire ton patron singleton en Java, il faudrait que la classe Secretaire soit final, et que le constructeur sans argument soit private, sinon on pourrait facilement (par héritage par exemple) créer un nouveau constructeur pour avoir un objet Secretaire qui ne serait pas inséré dans la liste.

Cependant, quitte à mettre la classe final, on peut du coup gérer l'ajout dans la liste directement dans le constructeur et ainsi éviter la méthode nouvelleSecretaire.

4) Les WeakReference sont un peu maladroites ici, mais ça veut dire que si l'objet Secretaire est garbage collecté, la référence n'existera plus mais la liste va continuer à la contenir, on pourrait donc se retrouver avec une liste qui continue à grossir quitte à ne contenir que des WeakReference vides...

Je pense qu'il serait plus propre d'utiliser la méthode finalize afin de supprimer directement la référence dans la liste lorsque l'objet disparaît, même si le problème de raouia1 était sûrement de garder ces objets indéfiniment dans la liste, même s'ils ne sont plus utilisés ailleurs.

import java.util.ArrayList;
import java.util.List;

public final class Secretaire
{
    private static final List<Secretaire> instances = new ArrayList<Secretaire>();
    
    public static List<Secretaire> getInstances() {
        return new ArrayList<Secretaire>(instances);
    }
    
    public Secretaire() {
        instances.add(this);
    }
    
    @Override
    protected void finalize() throws Throwable {
        instances.remove(this);
        super.finalize();
    }
}

Edit: ainsi fait les objets ne pourront jamais être garbage collectés puisqu'ils seront dans la liste.
La bonne solution serait donc entre les deux : utiliser des WeakReference ou SoftReference pour permettre le GC de l'objet et lors du finalize supprimer de la liste la référence associée.
Mais clairement ce n'est pas d'un niveau débutant ;-)

Ça dépend de quel code on parle...

Avec le code de tksteph :

public static void main(String[] args) {
    Secretaire secretaire1 = Secretaire.nouvelleSecretaire();
    Secretaire secretaire2 = Secretaire.nouvelleSecretaire();
    for (Object obj : Secretaire.getInstances()) {
        WeakReference ref = (WeakReference) obj;
        Secretaire sec = (Secretaire) ref.get();
        System.out.println(sec);
    }
}

Avec le mien :

public static void main(String[] args) {
    Secretaire secretaire1 = new Secretaire();
    Secretaire secretaire2 = new Secretaire();
    for (Secretaire sec : Secretaire.getInstances())
        System.out.println(sec);
}