Discussion :

[joel.drigo]

Bah, voilà, dans ta classe Connexion :

Code :

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

1
2
3
4
5
6
7
8
package gestionoutilbricolage;
import java.sql.*;
 
public class Connexion {
 
    /* ... */
 
}

tu ajoutes toutes les méthodes que tu as besoin.

Par exemple pour setAutoCommit, tout simplement :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
package gestionoutilbricolage;
import java.sql.*;
 
public class Connexion {
    /* ... */
 
    public void setAutoCommit(boolean autoCommit) throws SQLException {
        conn.setAutoCommit(autoCommit);
    }
 
}

Pareil pour les autres (rollback, commit...).

Et tu peux mettre directement les 2 méthodes que j'ai donné (printSQLException et ignoreSQLException) dans cette classe, et modifier tes appels JDBCTutorialUtilities.printSQLException([...]); par Connexion.printSQLException([...]);.

[samo36]

Par exemple pour setAutoCommit, tout simplement :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
package gestionoutilbricolage;
import java.sql.*;
 
public class Connexion {
    /* ... */
 
    public void setAutoCommit(boolean autoCommit) throws SQLException {
        conn.setAutoCommit(autoCommit);
    }
 
}

Pareil pour les autres (rollback, commit...).

Pour JDBCTutorialUtilities.printSQLException, le problème a été résolu avec ta méthode.
Par contre pour rollback, commit, prestatement j'ai un toujours le problème, j'ai fait ceci:

Code :

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

1
2
3
4
5
6
7
8
9
public void rollback(java.sql.Savepoint s) throws SQLException {
       conn.rollback(rollback);
     }
     public void commit(java.sql.Savepoint s) throws SQLException {
       conn.commit(commit);
}   
     public void  prestatement(java.sql.Savepoint s) throws SQLException {
       conn.prestatement(prestatement);
}

J'ai cherché sur internet une façon de déclarer roolback et commit, j'ai pas trouvé. J'ai juste trouvé cela:

http://www.tutorialspoint.com/javaex...c_rollback.htm

[joel.drigo]

Les méthodes commit() et rollback() existent sans argument : il te suffit de faire :

Code :

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

1
2
3
4
5
6
public void rollback() throws SQLException {
       conn.rollback();
     }
     public void commit() throws SQLException {
       conn.commit();
    }

Les versions avec SavePoint permettent d'utiliser une transaction, qu'il faudra initialiser avec un startTransaction(), mais tu n'utilises pas ça dans ton programme, si ?

Sinon, il suffit d'écrire correctement ces méthodes :

Code :

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

1
2
3
 public void rollback(java.sql.Savepoint s) throws SQLException {
       conn.rollback(rollback);
     }

la méthode rollback(Savepoint) de Connection attend un paramètre de type Savepoint, que tu as appelé s... puis rollback... Quand on déclare une variable avec un nom, on l'utilise avec ce même nom !

Code :

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

1
2
3
public void rollback(java.sql.Savepoint s) throws SQLException {
       conn.rollback(s);
     }

Tout simplement.

Par contre, prestatement(Savepoint), ça n'existe pas. Mais si c'est de prepareStatement(String) dont tu parles, une méthodes qui prend une requête SQL sous forme de String en paramètre pour créer un objet de type PreparedStatement, alors c'est le même principe, et il suffit de respecter les types des paramètres, et de ne pas changer de nom en cours de route.

Code :

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

1
2
3
4
 
     public PreparedStatement prepareStatement(String sql) throws SQLException {
       return conn.prepareStatement(sql);
}

[samo36]

Merci, j'ai une dernière question pourrais-je faire la même chose pour delete juste en remplaçant update par delete?

[joel.drigo]

C'est le même principe, oui, pour exécuter un delete que pour exécuer un update (et même un insert en fait), mais il ne suffit pas de remplacer "update" par "delete" : la syntaxe SQL est légèrement différente.

[samo36]

Code :

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

1
2
3
4
5
6
7
finally {
        if (stmt != null) {
            stmt.close();
        }
 
    maconnexion.setAutoCommit(true)
    }

stmt.close() et maconnexion.setAutoCommit(true) on encore unreported exception java.sql.sqlexception must be caught or declared to be thrown.

[joel.drigo]

unreported exception java.sql.sqlexception must be caught or declared to be thrown.

C'est la base çà : si une méthode soulève un exception, il faut soit faire un catch, soit un throws...

[samo36]

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
 
} finally {
        if (stmt != null) {
            try{
            stmt.close();
            }
            catch(SQLException excep) {
              Connexion.printSQLException(excep);
            }
            }
        try{
    maconnexion.setAutoCommit(true);
    }
    catch(SQLException excep) {
              Connexion.printSQLException(excep);
        }
            }
 
 
 
}

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at gestionoutilbricolage.FenetreListeOutil.<init>(FenetreListeOutil.java:49)
at gestionoutilbricolage.GestionOutilbricolageView.jButton1ActionPerformed(GestionOutilbricolageView.java:228)






merci du temps consacré depuis le début je vais trouver la méthode.

[joel.drigo]

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException

at gestionoutilbricolage.FenetreListeOutil.<init>(FenetreListeOutil.java:49)
at gestionoutilbricolage.GestionOutilbricolageView.jButton1ActionPerformed(GestionOutilbricolageView.java:228)

Cette erreur signifie que tu invoques une méthode, ou tu acccèdes à un attribut, sur une variable non initialisée (Signification de NullPointerException). Dans la stacktrace, on te donne :

• Le nom de la classe : FenetreListeOutil
• Le nom de la méthode/constructeur : <init>, qui veut dire que c'est dans le constructeur
• Le numéro de la ligne : 49
• Et même le nom du fichier : FenetreListOutil.java

Si avec ça tu n'arrives à voir de quelle variable il s'agit, donne-moi le code du constructeur, en indiquant quelle est la ligne 49.

[samo36]

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at gestionoutilbricolage.FenetreListeOutil.<init>(FenetreListeOutil.java:49)
at gestionoutilbricolage.GestionOutilbricolageView.jButton1ActionPerformed(GestionOutilbricolageView.java:228)

Je n'ai plus ce problème car j'avais oublier d'allumer phpadmin.
Par contre quand je fais un test avec run main project sur netbeans j'ai un problème. Je selectionne une ligne de mon tableau en double cliquant, je fais ma modification puis j'appuie sur le bouton modifier. Netbeans se met en mode running. Je ferme l'application puis je l'a réouvre avec run main project mais la modification n'a pas été prise en compte .

Voila le code du bouton modifier:

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
private void jButton2ActionPerformed(java.awt.event.ActionEvent evt) {                                         
// TODO add your handling code here:
  int ligneSelectionne = TableOutil.getSelectedRow(); 
String outil = TableOutil.getValueAt(ligneSelectionne, 0).toString(); 
  PreparedStatement stmt = null;
String wql = "update outil Set nom_outil = ? where nom_outil = ?";
try {
    maconnexion.setAutoCommit(false);
    stmt = maconnexion.prepareStatement(wql);
    stmt.setString(1, "outil A");
    stmt.setString(2, "outil avant");
    stmt.executeUpdate();
    maconnexion.commit();
    } catch (SQLException e ) {
      Connexion.printSQLException(e);
        if (maconnexion != null) {
            try {
                System.err.print("Transaction is being rolled back");
                maconnexion.rollback();
            } catch(SQLException excep) {
              Connexion.printSQLException(excep);
            }
        }
    } finally {
        if (stmt != null) {
            try{
            stmt.close();
            }
            catch(SQLException excep) {
              Connexion.printSQLException(excep);
            }
            }
        try{
    maconnexion.setAutoCommit(true);
    }
    catch(SQLException excep) {
              Connexion.printSQLException(excep);
        }
            }
 
 
 
}

Code :

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

1
2
String outil = TableOutil.getValueAt(ligneSelectionne, 0).toString();
  PreparedStatement stmt = null;

J'ai local hides pour outil et stmt, mais je sais que c'est pas une erreur mais juste un avertissement. Est ce que c'est ça la cause de l'absence de prise en compte des modification que je fais sur ma ligne sélectionné.

[joel.drigo]

Je pense que ça vient du second paramètre qui est en dur. Dans le code suivant, tu récupères le nom de l'outil dans la ligne sélectionnée que tu mets dans la variable outil. Mais dans le paramètre 2, tu mets une chaine en dur ("outil avant") : cette requête modifie un outil qui s'appelle "outil avant", pour le renommer en "outil A".

Code :

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

1
2
3
4
5
6
7
8
String outil = TableOutil.getValueAt(ligneSelectionne, 0).toString(); 
  PreparedStatement stmt = null;
String wql = "update outil Set nom_outil = ? where nom_outil = ?";
try {
    maconnexion.setAutoCommit(false);
    stmt = maconnexion.prepareStatement(wql);
    stmt.setString(1, "outil A");
    stmt.setString(2, "outil avant");

Il faut donc que tu remplaces stmt.setString(2, "outil avant") par stmt.setString(2, outil).

A noter, que quelque soit la ligne que tu modifies, le nouvel outil s'appelle toujours "outil A" :

1. Il faudra probablement que tu prévois un champ pour saisir le nouveau nom
2. Cela pose évidemment un problème d'utiliser comme critère de recherche le nom de l'ancien outil. Il vaut mieux utiliser un critère qui se fonde sur l'identifiant unique de l'outil pour retrouver l'outil à modifier.

[samo36]

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
private void jButton2ActionPerformed(java.awt.event.ActionEvent evt) {                                         
// TODO add your handling code here:
  int ligneSelectionne = TableOutil.getSelectedRow(); 
String outil = TableOutil.getValueAt(ligneSelectionne, 0).toString(); 
  PreparedStatement stmt = null;
String wql = "update outil Set id_outil = ? where id_outil = ?";
try {
    maconnexion.setAutoCommit(false);
    stmt = maconnexion.prepareStatement(wql);
    stmt.setString(1, "id_outil A");
    stmt.setString(2, "id_outil" );
    stmt.executeUpdate();
    maconnexion.commit();
    } catch (SQLException e ) {
      Connexion.printSQLException(e);
        if (maconnexion != null) {
            try {
                System.err.print("Transaction is being rolled back");
                maconnexion.rollback();
            } catch(SQLException excep) {
              Connexion.printSQLException(excep);
            }
        }
    } finally {
        if (stmt != null) {
            try{
            stmt.close();
            }
            catch(SQLException excep) {
              Connexion.printSQLException(excep);
            }
            }
        try{
    maconnexion.setAutoCommit(true);
    }
    catch(SQLException excep) {
              Connexion.printSQLException(excep);
        }

}

je vais utiliser id-outil pour la recherche. Par contre un exemple pour:

Il faudra probablement que tu prévois un champ pour saisir le nouveau nom

un champ propre?


http://www.tutorialspoint.com/jdbc/j...te-records.htm

[joel.drigo]

je vais utiliser id-outil pour la recherche.

Oui, mais non.

Code :

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

1
2
 
String wql = "update outil Set id_outil = ? where id_outil = ?";

C'est bien d'avoir changer le where pour faire la recherche sur l'id, mais pourquoi avoir changer le set ? C'est le nom que tu veux changer, pas l'id ! Surtout que l'id, tu ne peux probablement pas le modifier si tu as fait un champ autoincrémental.

Et là :

Code :

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

1
2
3
 
    stmt.setString(1, "id_outil A");
    stmt.setString(2, "id_outil" );

tu n'as pas faire ce que j'ai dit : tu ne dois pas passer une chaine en dur ! Le paramètre 1 c'est une variable qui contient le nouveau nom, et le second paramètre une variable qui contient l'id de l'outil dans la ligne sélectionnée !

Par contre un exemple pour un champ

Bah, déjà, ça dépend comment tu comptes faire ton interface. On peut saisir le nom dans un formulaire, dans sa propre fenêtre, ou on pourrait le faire directement dans la JTable. Bon, comme tu as un bouton pour modifier une ligne, ce sera plus simple de le faire dans un formulaire. Un formulaire à un champ. Le plus simple est d'utiliser un JOptionPane, avec la méthode showInputDialog(). Tu as un exemple dans le tutoriel Oracle. Ici aussi.

[samo36]

p

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
rivate void jButton2ActionPerformed(java.awt.event.ActionEvent evt) {                                         
// TODO add your handling code here:
    String  nomOutil = JTNomOutil.getText();
    String  fabricantOutil = JTFabricantOutil.getText();
    String  anneeFabrication = JTAnneeFabrication.getText();
    String  prix = JTPrix.getText();
 
  int ligneSelectionne = TableOutil.getSelectedRow(); 
String outil = TableOutil.getValueAt(ligneSelectionne, 0).toString(); 
  PreparedStatement stmt = null;
 
String wql = "update outil Set nom_outil,fabricant_outil,annee_fabrication,prix_outil = ? where id_outil = ?";
try {
 
    JOptionPane jop = new JOptionPane();
    String nom = (String)JOptionPane.showInputDialog(
   null,
   "Veuillez entrer votre modification",
   "Modification",
   JOptionPane.QUESTION_MESSAGE,
   new ImageIcon( 
         getClass().getResource("arbre.gif")),
 
 
    stmt.setString(1, "");
    stmt.setString(2, "id_outil");
    stmt.executeUpdate();
    JOptionPane.showMessageDialog(mainFrame, "Votre requête a été exécuté" );
        JTNomOutil.setText("");
        JTFabricantOutil.setText("");
        JTAnneeFabrication.setText("");
        JTPrix.setText("");
 
    maconnexion.commit();
    } catch (SQLException e ) {
      Connexion.printSQLException(e);
        if (maconnexion != null) {
            try {
                System.err.print("Transaction is being rolled back");
                maconnexion.rollback();
            } catch(SQLException excep) {
              Connexion.printSQLException(excep);
            }
        }
    } finally {
        if (stmt != null) {
            try{
            stmt.close();
            }
            catch(SQLException excep) {
              Connexion.printSQLException(excep);
            }
            }
        try{
    maconnexion.setAutoCommit(true);
    }
    catch(SQLException excep) {
              Connexion.printSQLException(excep);
        }
            }
 
 
 
}

Je veux pouvoir changer le nom de l'outil son prix etc...) donc j ai mis:

Code :

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

String wql = "update outil Set nom_outil,fabricant_outil,annee_fabrication,prix_outil = ? where id_outil = ?";

Par contre, netbeans me dit que

'void' type not allowed here ')' expected.

pour

Code :

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

 stmt.setString(1, "");

?

[joel.drigo]

Chaque ? dans le SQL d'un PreparedStatement représente la valeur d'UN paramètre ! Donc, il faut autant de ? que de paramètres !

Et, donc, autant de setString( index du paramètre, valeur du paramètre ).

[samo36]

Chaque ? dans le SQL d'un PreparedStatement représente la valeur d'UN paramètre ! Donc, il faut autant de ? que de paramètres !

Et, donc, autant de setString( index du paramètre, valeur du paramètre ).

je ne sais plus la ligne pour me connecter à ma base de donnée est ce celui la:

Code :

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

1
2
 
stmt=maconnexion.ObtenirConnexion().createStatement();