Discussion :

[weed]

Bonjour,


Je me suis créé une classe, MaConnexion, qui me permettra de gerer ma connexion à la base de donnée.

Dans le fichier main, je me suis créé une instance de ma classe MaConnexion l'affectant à la variable maSQL :
MaConnexion maSQL = new MaConnexion();

puis je veux appeller la méthode insert2(). Utilisateur de netbean6, j'ecris donc ma variable maSQL puis le . et apres je fais Alt+Espace pour choisir ma méthode insert2();

Je ne comprends absolument pas pourquoi Netbean m'indique que le package maSQL n'existe pas. Comme vous pouvez le voire sur le screen, la classe MaConnexion et Main sont dans le meme package gestionemploi2. Dans le fichier Main et Maconexion, j'ai preciser : package gestionemploi2;





Ma classe :

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
 
 package gestionemploi2;
 
import java.sql.*;
import java.util.logging.Level;
import java.util.logging.Logger;
 
 
public class MaConnexion {
 
    private Connection myConnection;
 
    public MaConnexion() {
        try {
            Class.forName("com.mysql.jdbc.Driver");
            myConnection=DriverManager.getConnection("jdbc:mysql://localhost/java","root", "");
        } catch (SQLException ex) {
            Logger.getLogger(MaConnexion.class.getName()).log(Level.SEVERE, null, ex);
        } catch (ClassNotFoundException ex) {
            Logger.getLogger(MaConnexion.class.getName()).log(Level.SEVERE, null, ex);
        }
    }
    public void Insert2 () {
        String q = "INSERT INTO `entreprise` ( `ENTREPRISE_ID` , `NOM` ) VALUES ('', 'coucouc');";
 
        try {
            Statement stmt = myConnection.createStatement();
             stmt.executeUpdate(q);
        } catch (SQLException ex) {
            Logger.getLogger(MaConnexion.class.getName()).log(Level.SEVERE, null, ex);
        }
 
    }   
}

et mon fichier main :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 
package gestionemploi2;
 
public class Main {
 
    MaConnexion maSQL = new MaConnexion();
    maSQL.insert2();
 
}

[adiGuba]

Salut,


Insert2 n'est pas la même chose que insert2...


Quand à ta classe MyConnexion elle a un gros problème : aucune ressource n'est jamais libérer !

Comment libérer proprement les ressources ?

a++

[weed]

Salut,


Insert2 n'est pas la même chose que insert2...


Quand à ta classe MyConnexion elle a un gros problème : aucune ressource n'est jamais libérer !

Comment libérer proprement les ressources ?

a++

Et bien curieusement, comme je l'ai dit, c'est Netbean qui m'a rajouter le nom de la méthode (Alt + Espace). Ils s'est donc trompé en mettant un i minuscule. Bizzare.


Pour la libération des ressources. N'est ce pas suffisant de créer une méthode close.

Code :

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

1
2
3
4
5
6
7
8
 private Connection myConnection;
 
 public void Close () {
        try {
            myConnection.close();
        } catch (SQLException ex) {
            Logger.getLogger(MaConnexion.class.getName()).log(Level.SEVERE, null, ex);
        }

Mais dans ce cas, il faudrait que cette méthode soit dans un bloc finally pour être sur à 100% qu'elle va s'exécuter même en cas d'exception.

Seul souci, la portée des variables. Elles sont limitées à l'interieur d'un bloc. Il font donc penser à creer sa variable maSQL en dehors du catch.

Code :

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

1
2
3
4
5
6
7
         MaConnexion maSQL = new MaConnexion();
         try {
            maSQL.Insert("INSERT INTO `entreprise` ( `ENTREPRISE_ID` , `NOM` ) VALUES ('', 'c moi');");
         }
         finally {
            maSQL.Close();
        }

Corrige moi si je me trompe. Sinon un grand merci à toi.

[adiGuba]

Pour la libération des ressources. N'est ce pas suffisant de créer une méthode close.

Code :

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

1
2
3
4
5
6
7
8
 private Connection myConnection;
 
 public void Close () {
        try {
            myConnection.close();
        } catch (SQLException ex) {
            Logger.getLogger(MaConnexion.class.getName()).log(Level.SEVERE, null, ex);
        }

Oui... mais ici tu ne fermes que la Connexion et il vaut mieux fermer également les Statement/ResultSet, par 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
	public void Insert2() {
		String q = "INSERT INTO `entreprise` ( `ENTREPRISE_ID` , `NOM` ) VALUES ('', 'coucouc');";
 
		try {
			Statement stmt = myConnection.createStatement();
			try {
				stmt.executeUpdate(q);
			} finally {
				stmt.close();
			}
		} catch (SQLException ex) {
			Logger.getLogger(MaConnexion.class.getName()).log(Level.SEVERE,null, ex);
		}
	}

C'est d'autant plus vrai lorsque tu réutilises la même connexion pendant un certain temps (et que tu crées plusieurs Statement sans les fermer...)

Mais dans ce cas, il faudrait que cette méthode soit dans un bloc finally pour être sur à 100% qu'elle va s'exécuter même en cas d'exception.

Oui tout à fait !

Et si tu es parano tu peux aussi redéfinir finalize() pour (re)forcer le close().

Seul souci, la portée des variables. Elles sont limitées à l'interieur d'un bloc. Il font donc penser à creer sa variable maSQL en dehors du catch.

Oui tout à fait.



Sinon juste un détail : les nom de variables et de méthodes commencent par une minuscule en Java

a++