Discussion :

[mitje]

bonjour,

je voudrais créer une classe statique MyConnection, plutôt que de me trimbaler des instances d'une classe dynamique dans tous mes objets.
Le fait est que cette classe doit neanmoins rester paramétrable (port,ip,db,user,password).
Que la connection, ainsi créer doit rester active de sorte a ne pas avoir a se reconnecter tout le temps.

La question : est ce que c'est cohérent, possible de concevoir les choses ainsi.

Merci d'avance pour vos réponses.

[dude]

pourquoi ne pas utiliser un singleton ?

[mitje]

c'est a dire ?

je vais chercher dece côté là.

[Baptiste Wicht]

c'est a dire ?

je vais chercher dece côté là.

Un singleton permet de restreindre les instances à une seule et unique et ainsi la meme pour tout le monde.

Si tu veut faire une classe singleton, il faut déclarer ton constructeur private et faire une méthode statique getInstance() qui te permettra de renvoyer une unique instance de la classe.

Ca peut donner quelque chose du genre :

Code :

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

1
2
3
4
5
6
7
8
9
public class TaClasse {<div style="margin-left:40px">TaClasse instance = null;
 
private TaClasse{
 
}
 
public static TaClasse getInstance(){<div style="margin-left:40px">
if(instance == null){<div style="margin-left:40px">instance = new TaClasse();</div>}
 return instance;</div>}</div>}

[dude]

plus précisément :

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
public class TaClasse {
  private static TaClasse instance = null;
 
  private TaClasse(){

  }
 
  public static TaClasse getInstance(){
    if(instance == null){
      instance = new TaClasse();
    }
    return instance;
  }
}

[mitje]

Voici le code de ce que j'ai fait, si vous avez des suggestions, des critiques (même s'évère), surtout n'hesitez pas, allez y de bon coeur.
Je vous laisse éventuelement répondre et mettrais le tag resolu ensuite.
Merci à tous pour vos réponses.

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
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
/*
 * MyDBConnection.java
 *
 * Created on 7 mars 2006, 16:18
 *
 * To change this template, choose Tools | Template Manager
 * and open the template in the editor.
 */
 
package guitp;
 
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;
import javax.swing.JFrame;
 
/**
 *
 * @author lole
 */
public class MyDBConnection {
 
    /** Creates a new instance of MyDBConnection */
    private static Connection connection;
    private static ConnectDBDialog dbd;
    private static String db;
    private static String host;
    private static String port;
    private static String user;
    private static String password;
 
    /** Creates a new instance of MyDBConnection */
    private MyDBConnection() {} 
    /*private MyDBConnection(String db, String host, String port, String user, String password) {
        setParams(db, host, port, user, password); 
    }*/
 
    private static Connection init(){
 
 
        try {
            Class.forName("com.mysql.jdbc.Driver");
        } catch (Exception e){System.out.println(e);}
 
        try{
 
            //System.out.println("Connection complete");
            return DriverManager.getConnection(
                    "jdbc:mysql://" + host + ":" +port +"/" + db ,user , password
                    );
        } catch(SQLException sqle){
 
            System.out.println("Failed to get connection");
            System.out.println("Exception SQL : ");
            while (sqle != null) {
                String message = sqle.getMessage();
                String sqlState = sqle.getSQLState();
                int errorCode = sqle.getErrorCode();
                System.out.println("Message = "+message);
                System.out.println("SQLState = "+sqlState);
                System.out.println("ErrorCode = "+errorCode);
                //sqle.printStackTrace();
                sqle = sqle.getNextException();
            }
 
            return null;
 
 
        }
    }
 
    public static void showConnectDialog(MainFrame fr) {
        dbd = new ConnectDBDialog(fr);
    }
 
    public static void setParams(String dbs, String h, String p, String u, String pwd) {
 
        db = dbs;
        host = h;
        port = p; 
        user = u;
        password = pwd;
 
    }
 
    public static Connection getConnection(){
        if (connection == null) {
            connection = init(); 
        }
        return connection;
    }
 
 
    public static void destroy(){
 
        if(connection !=null){
 
            try{
                connection.close();
            } catch(Exception e){}
 
 
        }
    }
 
    public static boolean isActive() {
 
        if(connection !=null)
            return true;
        else
            return false;
 
    }
 
 
}

[Baptiste Wicht]

plus précisément :

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
public class TaClasse {
  private static TaClasse instance = null;
 
  private TaClasse(){

  }
 
  public static TaClasse getInstance(){
    if(instance == null){
      instance = new TaClasse();
    }
    return instance;
  }
}

En effet, j'avais oublié

Pour ton code, ca m'a l'air pas mal, mais c'est dommage que tu aies toutes ces méthodes statiques, mais ca a l'air d'être assez cohérent.