Discussion :

[stefan73]

Bonjour,

Je reviens vers vous pour utiliser un singleton. (J'en comprends l''utiliter)

Mais c'est le mécanisme qui est pour moi encore un peu abscon !


J'ai trouvé pleins de tutoriels sur Developpez et sdz

Mais je suis toujours dans l'incompréhension de comment l'appeler

J'ai créé un projet test sous Netbeans avec deux classe
- 1er main.java
- 2e singleton.java

Exemple du 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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
 
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.SQLException;
 
/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
/**
 *
 * @author stefan
 */
public class Singleton {
    private String name = "";
    private static String url = "jdbc:sqlite:test.db";
 
    private static Connection connect;
 
/**
 
        /**
         * Constructeur privé
         */
	private Singleton(){
		try {
			connect = DriverManager.getConnection(url);
		} catch (SQLException e) {
			e.printStackTrace();
		}
	}
 
	/**
         * Méthode qui va nous retourner notre instance
         * et la créer si elle n'existe pas...
         * @return
         */
	public static Connection getInstance(){
		if(connect == null){
			new Singleton();
		}
		return connect;	
	}	
 
    }

Et dans mon main :

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
33
34
35
36
37
38
39
40
41
42
/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
 
import java.sql.Connection;
 
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import java.util.logging.Level;
import java.util.logging.Logger;
 
/**
 *
 * @author stefan
 */
public class main {
 
    public static void main(String[] args) {
        try {
			Statement rs = Connection.getInstance().prepareStatement("SELECT * FROM mytable");
 
            //Affichage des résultats
 
            while (rs.next()) {
                int id = rs.getInt(1);
                String name = rs.getString(2);
                System.out.println("Id = " + id + ", nom = " + name);
            }
 
            //Fermeture de la connexion à la base
 
 
        } catch (ClassNotFoundException ex) {
            Logger.getLogger(main.class.getName()).log(Level.SEVERE, null, ex);
        }
    }
 
}

et j'ai une une erreur au niveau du getInstance() ?

je suis un peu perdu. si vous pouviez me remettre sur les rails


Merci

[Robin56]

et j'ai une une erreur au niveau du getInstance() ?

je suis un peu perdu. si vous pouviez me remettre sur les rails

Que dit ton erreur ? A première vue, ton singleton respecte la syntaxe :
- l'instance de la classe en attribut "private static"
- un getter de l'instance en "public static"
- au sein du getter, une vérification si l'instance de classe est null, si oui, création de l'instance

Et au sein de ton main(), ton singleton est correctement appelé.

[thelvin]

Tu t'es trompé de nom de classe. C'est Singleton.getInstance()

Par ailleurs, ce n'est pas le sujet, mais il faudrait déclarer connect avec volatile, sinon il peut y avoir conflit en cas de plusieurs threads :

Code :

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

private static volatile Connection connect;

Bon, et puis quand on y réfléchit, une seule connexion SQL pour toute une application, en général, c'est bof.

[stefan73]

voici l'erreur

Exception in thread "main" java.lang.RuntimeException: Uncompilable source code - exception java.lang.ClassNotFoundException is never thrown in body of corresponding try statement
at main.main(main.java:24)
Java Result: 1

Une capture d'écran de la classe main.

[Robin56]

Tu t'es trompé de nom de classe. C'est Singleton.getInstance()

Ah oui j'suis con tout simplement

[stefan73]

Merci à vous deux.

quand à mon erreur (c'est ça quand on reprends des classe sur le web sans bien comprendre le fonctionnement) ;(

Il me reste encore une erreur à résoudre et je vous laisse tranquille

merci à vous

[jojodu31]

Bon, et puis quand on y réfléchit, une seule connexion SQL pour toute une application, en général, c'est bof.

+1, pour moi c'est l'exemple typique d'une mauvaise utilisation des Singleton que tout le monde veut mettre partout. Tu vas garder ta référence de connexion alors que ce n'est pas forcément nécessaire... de plus le getInstance() qui renvoie un autre type que le Singleton lui même c'est un peu strange

[tchize_]

Bon alors plusieurs remarques:

1) le volatile, désolé, mais inutile dans le cas présent. L'instance est privée et donc nécessite de passer par une méthode, donc d'office il n'y a rien de mis en cache dans les registres (nettoyage en début de méthode)

2) En général, un Singleton retourne une instance de lui même et pas une instance d'autre chose.

3) Tu crée un constructeur que tu n'appelle que pour ses effets de bord, mauvais design

3) Ce n'est pas comme ça qu'on instancie proprement un singleton. La méthode recommandée pour éviter les soucis entre les thread c'est

Code :

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

1
2
3
4
public class Singleton {
    private static Singleton instance = new Singleton();
    public Singleton getInstance(){return instance;}
}

4) Une connexion globale, c'est la dernière des plus mauvaise idée. Que vous fassiez un singleton pour stocker vos informations de connexion, à la rigueur, mais pas une connexion unique pour toute la durée de vie de l'application.

5) Evitez les constructeur ou les méthodes d'instance qui manipulent des champs statiques. Laissez les statiques aux méthodes statiques et les champs d'instance aux instances. chacun de son coté et tout ira bien

[stefan73]

OK !

Comment dire, je suis encore plus dans la confusion qu'avant de poser ma question cétait pas le but ! mais au moins j'ai plusieurs avis sur le Singleton

je vais juste me faire une tit application pour gérer ma compta, gestion, factures...

[sinok]

Dans le cas d'une connexion à une base de données, ce n'est pas la connexion qui doit être contenue dans le singleton, mais une classe dont le rôle est de créer des connexions à la volée et de fournir des instances des classes permettant la récupération des données depuis la base (classe appelée communément Factory).

En général cela rentre dans le cadre du pattern DAO expliqué dans le tutoriel suivant: http://cyrille-herby.developpez.com/...c-pattern-dao/

[OButterlin]

Le but du singleton est d'avoir une seule instance d'une classe par classloader et utilisable par d'autres classes de l'application.
Donc, on créé une classe et on masque le constructeur par private (histoire de ne pas pouvoir faire un new LaClasse()).
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 MaClasse
{
   private static MaClasse maClasse;
 
   private MaClasse()
   {
      ...
   }
 
   public static MaClasse getInstance()
   {
      if (maClasse == null)
      {
         maClasse = new MaClasse();
      }
      return maClasse;
   }
 
   public UnType uneMethodeUtile(...)
   {
      ...
      return unType;
   }
}

la méthode getInstance() permet de récupérer l'instance "unique" de MaClasse. La première fois, maClasse est NULL donc on créé l'instance.

Pour l'utilisation, l'appel se fait par

Code :

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

1
2
3
4
5
6
7
8
9
10
 
public UneAutreClasse
{
   public void uneMethode(...)
   {
      UnType leType = MaClasse.getInstance().uneMethodeUtile();
      ...
   }
 
}

Généralement, on utilise le singleton pour des classes utilitaires, utilisées partout.
Pour le cas de l'acquisition d'une connexion à la DB, on peut utiliser le principe.

Petit complément au cas où...

Une méthode ou une propriété static n'a pas besoin d'instance pour être accessible. Elle est cependant limitée aux propriétés et méthodes static.
Donc, MaClasse.getInstance() est accessible sans instance de MaClasse.
getInstance() retourne une instance de MaClasse.
Comme cette méthode centralise l'accès et la création de l'instance, on n'a qu'une seule instance de MaClasse après le premier appel de getInstance().
L'instance de MaClasse peut utiliser toutes les propriétés et méthodes de la classe, static ou non.

[stefan73]

Merci je vais digérer tout cela !

stefan