Discussion :

[theo16]

Bonjour,

Problème encore de débutant !

J'ai fait un projet test où j'ai une classe ConnectMysql qui comporte une méthode connect, une méthode close et une autre qui retourne un Resulset d'une requête passée en paramètre.

Dans ce projet j'ai fait une autre classe main qui me teste la connexion et une requête et tout va bien.

Mais comme cette classe connexion me servira à différents projets, je l'ai mise dans un répertoire fr/pl/mysql dans le répertoire de l'espace de travail.

J'ai commencé un autre projet de gestion d'une table qui me permet de voir la programmation java sur pas mal de fonctionnalités.

Dans une classe TableTest, j'ai un constructeur avec tous les champs de la table, ainsi que tous les getters et setters.

Puis j'ai une méthode InsertTable qui m’insère un enregistrement dans ma table.


Ma méthode connect de ma classe ConnectMysql

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
 
public class ConnectMysql {    
	   private String base = "";
	   private String user = "";
	   private String pwd = "";
	   private Connection dbConnect = null;
	   private Statement dbStatement = null;
...
 public Boolean connect() {
    try {
	Class.forName("com.mysql.jdbc.Driver");
	System.out.println("Driver Mysql trouvé");
    }
     catch (ClassNotFoundException e) {
        System.err.print("ClassNotFoundException: ");
        System.err.println(e.getMessage());
        return false;
    } 	
    try {
       this.dbConnect = DriverManager.getConnection("jdbc:mysql://localhost/" + this.base, this.user, this.pwd);
                                // note la base, user et pwd viennent des paramètres du constructeur 
       System.out.println("Base " + this.base + " connectée");
       this.dbStatement = this.dbConnect.createStatement();
       return true;
    } 
    catch(SQLException ex) {
       System.err.println("SQLException: " + ex.getMessage());
    }
    return false;
 }

Ma classe tableTest pour gérer ma table

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
 
 
public class TableTest {
    private int id;
    private String nom;             // Nom
    private String prenom;          // Prenom
    .....
 
    public TableTest(int id, String nom, String prenom, .....) {
       super();
       this.id = id;
       this.nom = nom;
       this.prenom = prenom;
       .....
    }
    public String getNom() {
       return nom;
    }
    public void setNom(String nom) {
       this.nom = nom;
    }
    ......
 
    public void insertTable() {
        String sqlTmp = "";
        sqlTmp += "insert into tabletest values (";
        sqlTmp += "'" + this.getNom() + "', ";
        sqlTmp += "'" + this.getPrenom() + "', ";
        ....
        sqlTmp += "'" + this.getDivers() + "' )";
        try {
           dbStatement.executeUpdate(sqlTmp);     // erreur dbStatement cannot be resolved
        } catch (SQLException e) {
            e.printStackTrace();
        }
   }
}

J'ai incorporé ma classe ConnectMysql dans mon projet de gestion de table avec Link source où j'ai mis le chemin de Linker Folder location, donc fr/pl/mysql sous mon workspace et m'a donc donné mon folder name : mysql où se trouve ma classe ConnectMysql.

Et j'ai une erreur "dbStatement cannot be resolved" sur la ligne dbStatement.executeUpdate(sqlTmp);

J'ai un import java.sql.* dans les deux fichiers java.
J'ai essayé de mettre un import fr.pl.mysql ou rajouter la classe, essayé aussi de mettre package fr.pl.mysql, et j'ai toujours cette erreur.

Pouvez vous me dire ce qui se passe ??
Merci d'avance.
Cordialement.

[tchize_]

Tu n'a défini nulle part dans ta classe TableTest de variable dbStatement. Les champs des classes que tu importe n'apparaissent pas miraculeusement dans la classe qui fait l'import. L'import sert juste à voir la classe, pas à fusionner les classes ou que sais-je que tu imagine

[theo16]

Bonjour,

J'ai donc rajouté la ligne "Statement dbStatement = dbConnect.createStatement();" avant ma ligne d'erreur "dbStatement.executeUpdate(sqlTmp);", mais c'est maintenant cette ligne rajoutée qui est en erreur "dbConnect cannot be resolved".

Faut il redéclarer toutes les variables de connexion ? et si oui comment celles ci peuvent elles faire alors référence à la connexion effective de ma classe ConnectMysql ??

Merci de vos réponses.
Cordialement.

[jeanClude45]

Dans TableTest t'as pas de Connection dbConnect donc logique qu'il te dise qu'il ne la trouve pas ...

Fais comme ça :


private Connection dbConnect;
public TableTest(Connection dbConnect, ....){
this.dbConnect = dbConnect;
}

et dans ton autre classe TableTest t = new TableTest(dbConnect , ..)

[theo16]

En faisant ce que tu m'as dit :

private Connection dbConnect;
public TableTest(Connection dbConnect, ....){
this.dbConnect = dbConnect;
}

Je n'ai plus d'erreur sur ma ligne "Statement dbStatement = dbConnect.createStatement();"

Mais que veux tu dire par "dans ton autre classe TableTest" ? je n'ai qu'une seule classe TableTest !!

En définitive, pouvez vous m'expliquer que faut il déclarer dans deux classes qui sont sur 2 fichiers séparés :

Une classe de connexion à Mysql qui me servira pour tous les projets utilisant Mysql
et dans chaque classe utilisant cette classe connexion pour chacun de mes projets ??

Merci d'avance.

[tchize_]

Ce que tu ne comprends pas c'est que les variable sont dans l'autre classe (ici MYsqlConnect) restent dans cette classe. JeanClude te propose une solution (pas propre à mon sens) qui consiste à passer les information de connexion à TableTest. Mais d'une manière général, c'est à toi d'adapter MysqlConnect pour que soit il rendre les variable dont tu as besoin visible (getters) soit qu'il contienne directement la méthode insertTable, plutot que de la foutre dans TableTest soit encore faire une méthode a mi chemin dans MysqlConnect qui prend le sql en paramètre et l'exécute.

Code :

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

1
2
3
4
5
6
public class ConnectMysql { 
//....
 
public void executeSQLUpdate(String sql) throws SQLException {
     dbStatement.executeUpdate(sqlTmp);
}

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 TableTest {
    private int id;
    private String nom;             // Nom
    private String prenom;          // Prenom
    private ConnectMysql connectMysql;
    .....
 
    public TableTest(int id, String nom, String prenom, ....., ConnectMysql  connectMysql ) {
       ....
       this.connectMysql  = ConnectMysql ;
       .....
    }
 
    public void insertTable() {
        String sqlTmp = "";
        sqlTmp = .....;
 
        try {
           connectMysql.executeSQLUpdate(sqlTmp);     
        } catch (SQLException e) {
            e.printStackTrace();
        }
   }
}

Accessoirement c'est aussi un très mauvaise idée de réaliser des requetes SQL à la volée en concaténant des String avec des valeurs probablement soumises par l'utilisateur. Qu'arrivera-t-il le jour où l'utilisateur tapera
"; drop database;
comme prénom? utilise des preparedStatement pour éviter les attaques par injection SQL

[theo16]

Merci de cette réponse et de cet exemple.

Je dois donc rajouter cette méthode executeSQLUpdate dans ma classe ConnectMysql et l'appeler dans ma méthode insertTable, c'est çà ?
et cela me servira alors pour toutes les maj : insert, update et delete, c'est encore çà ?

Désolé de toutes ces questions qui doivent vous paraître basiques, mais pour moi c'est tout nouveau et je m'y perds encore un peu.

Merci de vos réponses.

[tchize_]

par exemple oui.

[theo16]

Merci à tous et bonne soirée.