JDBC et SQL requête UPATE en erreur sous Tomcat

**philo71** · 10/08/2017, 20h22

bonjour,

je travail sur les requêtes SQL via jdbc, jusqu'a aujourd'hui j'ai réussis les requêtes suivantes : SELECT et INSERT, je cherche a faire une requête UPDATE,

mais j'ai des problemes de convertions de la capture de la ligne dans une table assez simple. en faites depuis ma JSP ou s'affiche les champs de ma table avec l'index je veux pouvoir extraire le numéro de l'index depuis la JSP qui est une string est la convertir en integer pour faire ma requête update.

voici ma méthode update :

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
public void miseajourUtilisateur(Utilisateur utilisateur, Up_noms request) {
 
        loadDatabase();
        int id = Integer.parseInt(request.getString("id"));
        utilisateur.setId(id);
            try {
 
                PreparedStatement preparedStatement = connexion.prepareStatement("UPDATE noms SET nom = ?, prenom = ? WHERE id = ?");
 
                preparedStatement.setString(1, utilisateur.getNom());
                preparedStatement.setString(2, utilisateur.getPrenom());
                preparedStatement.setInt(3, utilisateur.getId());
 
                preparedStatement.executeUpdate();
            } catch (SQLException e) {
                e.printStackTrace();
            }
        }
 
/*  private String getString(String string) {
        // TODO Auto-generated method stub
        return getString("id");
    }*/

extrait de ma jsp

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
<form method="post" action="db_update">
        <p>
            <label for="id">id : </label>
            <input type="text" name="id" id="id" />
        </p>
        <p>
            <label for="nom">Nom : </label>
            <input type="text" name="nom" id="nom" />
        </p>
        <p>
            <label for="prenom">Prénom : </label>
            <input type="text" name="prenom" id="prenom" />
        </p>
 
        <input type="submit" name="update" id="id"/>
    </form>

je voudrais pouvoir extraire "id" de la JSP et la convertir en Integer.

mais eclipse EE me renvoit une erreur sur "GetString" du code suivant:

Code :

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

int id = Integer.parseInt(request.getString("id"));

auto completion me propose de ajouter si dessous cette fonction sur la quelle je suis en erreur et je ne sait quel type renvoyer : string ou integer

Code :

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

1
2
3
4
private String getString(String string) {
        // TODO Auto-generated method stub
        return getString("id");
    }

que faut t'il faire ?

cordialement

philippe

ps extrait de ma méthode DoPost de ma servlet

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
protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
	    Utilisateur utilisateur = new Utilisateur();
 
		//utilisateur.setid(request.hashCode(id));
 
 
	    try {
			utilisateur.setNom(request.getParameter("nom"));
		} catch (BeanException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
        utilisateur.setPrenom(request.getParameter("prenom"));
        Up_noms tableNoms = new Up_noms();
        tableNoms.miseajourUtilisateur(utilisateur, tableNoms);
        request.setAttribute("utilisateurs", tableNoms.recupererUtilisateurs());
        this.getServletContext().getRequestDispatcher("/WEB-INF/db_update.jsp").forward(request, response);
	}

**joel.drigo** · 10/08/2017, 20h59

Salut,

Déjà, pourquoi récupérer un id depuis l'instance de Up_noms pour l'injecter dans l'instance de Utilisateur, ce que tu fais dans ces lignes :

Code :

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

1
2
int id = Integer.parseInt(request.getString("id"));
        utilisateur.setId(id);

Pour ensuite faire le paramètrage de la requête UPDATE à partir de l'id provenant de l'instance de Utilisateur ici :

Code :

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

preparedStatement.setInt(3, utilisateur.getId());

au lieu de simplement affecter cette information directement dans la méthode doPost de la servlet (par utilisateur.setId( quelquechose )).

Ensuite, qu'est-ce qui est censé affecter actuellement cet id dans l'instance de Up_noms dans la servlet (je ne vois aucun code qui pourrait le faire), sachant qu'à priori il s'agit d'un paramètre du post, donc normalement récupérable par :

Code :

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

1
2
3
4
5
protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
 
String paramId = request.getParameter("id");
 
/*... reste du code de ta servlet ...*/

En faisant simplement comme ça, tu pourrais le passer directement à ton instance d'Utilisateur par :

Code :

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

1
2
3
4
5
Utilisateur utilisateur = new Utilisateur();
 
/** ... affectation nom et prénom ... **/
 
utilisateur.setId( Integer.parseInt(paramId) );

En conséquence, à priori, en faisant comme ça, ton instance de Up_noms ne servirait plus à rien dans miseajourUtilisateur().

Remarque importante : il n'y a pas de libération de ressources dans miseajourUtilisateur(), en particulier sur le PreparedStatement (à voir comment tu gères exactement ta connexion et ce que fait loadDatabase()), qu'il faudrait fermer (toujours fermer Statement, PreparedStatement et/ou ResultSet dans le cas d'un SELECT (ça va fermer automatiquement le PreparedStatement qui l'a créé)).
Au plus simple, pour faire ça, utiliser un try-with-resource (note bien les parenthèses) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
try (PreparedStatement preparedStatement = connexion.prepareStatement("UPDATE noms SET nom = ?, prenom = ? WHERE id = ?")){
 
   preparedStatement.setString(1, utilisateur.getNom());
                preparedStatement.setString(2, utilisateur.getPrenom());
                preparedStatement.setInt(3, utilisateur.getId());
 
                preparedStatement.executeUpdate();
 
} catch(SQLException e) {
     // traiter l'erreur
}

(à la fin du try, la méthode close() de preparedStatement est appelée automatiquement).

A part ça, à noter que le code suivant (qui ne servirait plus à rien si tu fais comme ci-dessus) :

Code :

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

1
2
3
private String getString(String string) {
        return getString("id");
    }

te causerait une StackOverflowError (la méthode s'appelle elle-même "infiniment" sans condition d'arrêt).

**philo71** · 10/08/2017, 22h35

J'ai effectué les changement que tu m'as conseillé !
j'ai une erreur sur : paramId

Code :

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

utilisateur.setId(Integer.parsetInt(paramId);

Je sais pas si c'est une bonne idée mais j'ai créé une java bean pour chaque requêtes, donc j'ai de la redondance de code,
j'ai créé Up_noms qui est la copie conforme de Noms simplement pour distinger les requêtes.
Mais d'après tes explications je mis prends mal, je complique trop mon code, je le reconnais !

voici ce que je fait dans mon load database !

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
package com.boticiel.bdd;
 
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import java.sql.PreparedStatement;
import java.util.ArrayList;
import java.util.List;
 
import com.boticiel.forms.BeanException;
import com.boticiel.forms.Utilisateur;
 
public class Noms {
    private Connection connexion;
 
    public List<Utilisateur> recupererUtilisateurs() {
    	List<Utilisateur> utilisateurs = new ArrayList<Utilisateur>();
        Statement statement = null;
        ResultSet resultat = null;
 
        loadDatabase();
        int id=0;
        try {
            statement = connexion.createStatement();
 
            // Exécution de la requête
            resultat = statement.executeQuery("SELECT id, nom, prenom FROM noms;");
 
            // Récupération des données
            while (resultat.next()) {
 
 
                id = resultat.getInt("id");
 
                String nom = resultat.getString("nom");
                String prenom = resultat.getString("prenom");
 
                Utilisateur utilisateur = new Utilisateur();
                utilisateur.setId(id);
                utilisateur.setNom(nom);
                utilisateur.setPrenom(prenom);
 
                utilisateurs.add(utilisateur);
            }
        } catch (SQLException | BeanException e) {
        } finally {
 
            // Fermeture de la connexion
            try {
                if (resultat != null)
                    resultat.close();
                if (statement != null)
                    statement.close();
                if (connexion != null)
                    connexion.close();
            } catch (SQLException ignore) {
            }
        }
 
        return utilisateurs;
    }
 
    private String toString(String id) {
		// TODO Auto-generated method stub
		return id;
	}
 
	private void loadDatabase() {
        // Chargement du driver
        try {
            Class.forName("com.mysql.jdbc.Driver");
        } catch (ClassNotFoundException e) {
        }
 
        try {
            connexion = DriverManager.getConnection("jdbc:mysql://localhost:3306/javaee", "root", "");
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }
 
 
}

s'il te plais quand tu me mets du code dit moi la destination, exemple servlet ou bean !
voici le code de Noms :

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
package com.boticiel.bdd;
 
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;
import java.sql.PreparedStatement;
import java.util.ArrayList;
import java.util.List;
 
import com.boticiel.forms.BeanException;
import com.boticiel.forms.Utilisateur;
 
public class Noms {
    private Connection connexion;
 
    public List<Utilisateur> recupererUtilisateurs() {
    	List<Utilisateur> utilisateurs = new ArrayList<Utilisateur>();
        Statement statement = null;
        ResultSet resultat = null;
 
        loadDatabase();
        int id=0;
        try {
            statement = connexion.createStatement();
 
            // Exécution de la requête
            resultat = statement.executeQuery("SELECT id, nom, prenom FROM noms;");
 
            // Récupération des données
            while (resultat.next()) {
 
 
                id = resultat.getInt("id");
 
                String nom = resultat.getString("nom");
                String prenom = resultat.getString("prenom");
 
                Utilisateur utilisateur = new Utilisateur();
                utilisateur.setId(id);
                utilisateur.setNom(nom);
                utilisateur.setPrenom(prenom);
 
                utilisateurs.add(utilisateur);
            }
        } catch (SQLException | BeanException e) {
        } finally {
 
            // Fermeture de la connexion
            try {
                if (resultat != null)
                    resultat.close();
                if (statement != null)
                    statement.close();
                if (connexion != null)
                    connexion.close();
            } catch (SQLException ignore) {
            }
        }
 
        return utilisateurs;
    }
 
    private String toString(String id) {
		// TODO Auto-generated method stub
		return id;
	}
 
	private void loadDatabase() {
        // Chargement du driver
        try {
            Class.forName("com.mysql.jdbc.Driver");
        } catch (ClassNotFoundException e) {
        }
 
        try {
            connexion = DriverManager.getConnection("jdbc:mysql://localhost:3306/javaee", "root", "");
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }
 
 
}

Donc d'après ce que tu m'as écrit j'ai fait des erreurs de codage, je complique trop !
voici Utilisateurs :

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
package com.boticiel.forms;
 
public class Utilisateur {
 
	private String paramId;
 
	public String getParamId() {
		return paramId;
	}
 
 
 
	public void setParamId(String paramId) {
		this.paramId = paramId;
	}
 
 
 
	private int id=0;
	private String nom;
    private String prenom;
 
	public Object getParameterInteger;
 
	public String getNom() {
        return nom;
    }
 
 
 
	public void setNom(String nom) throws BeanException {
        if (nom.length() > 10) {
            throw new BeanException("Le nom est trop grand ! (10 caractères maximum)");
        }
        else {
            this.nom = nom; 
        }
    }
    public String getPrenom() {
        return prenom;
    }
    public void setPrenom(String prenom) {
        this.prenom = prenom;
    }
 
    public int getId() {
		return id;
	}
 
 
 
	public void setId(int id) {
		this.id = id;
	}
 
 
}
 
//La DaoFactory.java change peu. On y ajoute seulement un

je prends un cours de Java EE aprés avoir lu des tas de livres mais c'est toujours pas clair dans mon esprit. désolé

Mais si tu peux m'aider j'en serais ravi.

cordialement
Philippe

**joel.drigo** · 11/08/2017, 02h34

Envoyé par philo71

s'il te plais quand tu me mets du code dit moi la destination, exemple servlet ou bean !

Je t'ai indiqué directement ou indirectement tous les "emplacements" des codes que je t'ai indiqués.

Par exemple :

Code :

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

1
2
3
4
5
protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
 
String paramId = request.getParameter("id");
 
/*... reste du code de ta servlet ...*/

Ici la méthode doPost c'est bien la méthode doPost que tu as écrite. Je n'ai jamais dit d'ajouter un attribut paramId dans Utilisateur. Dans ta classe Utilisateur, tu as déjà un attribut id (type int), et c'est cet attribut qui représente l'identifiant de ton Utilisateur.
Il suffit donc de lire le paramètre de la requête (La ligne String paramId = request.getParameter("id"); dans la méthode doPost de la servlet (méthode faite pour traiter la requête post), et de l'injecter dans l'objet Utilisateur (avec la méthode setId() de cette classe), pour le passer à ta méthode de traitement.
Donc exactement le code que je t'ai indiqué :

Code :

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

1
2
3
4
5
6
7
8
9
10
protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
 
String paramId = request.getParameter("id"); // récupération du paramètre de la requête (qui vient du formulaire posté)
 
/*... reste du code de ta servlet ...*/
Utilisateur utilisateur = new Utilisateur(); // là c'est déjà ce que tu faisais
 
/** ... affectation nom et prénom ... donc le code que tu avais écrit avec setNom et setPrenom **/
 
utilisateur.setId( Integer.parseInt(paramId) ); // ici tu injectes la valeur de paramId dans l'attribut id par le mutateur setId (en convertissant la valeur en int).

Et tu supprimes bien sûr

Code :

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

1
2
int id = Integer.parseInt(request.getString("id"));
        utilisateur.setId(id);

dans la méthode miseajourUtilisateur, puisque l'id de l'utilisateur est déjà affecté (dans la méthode doPost, ci-dessus).

Commence déjà par ça. Et tu verras les problèmes de libération de ressources ensuite.

En particulier, ce que je t'ai déjà indiqué pour fermer un Statement (ou PreparedStatement, c'est pareil). Mais aussi ici :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
	private void loadDatabase() {
        // Chargement du driver
        try {
            Class.forName("com.mysql.jdbc.Driver");
        } catch (ClassNotFoundException e) {
        }
 
        try {
            connexion = DriverManager.getConnection("jdbc:mysql://localhost:3306/javaee", "root", "");
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }

ou tu crées une nouvelle connexion à chaque fois que tu appelles loadDatabase, sans jamais la fermer, donc d'une part sans jamais libérer les ressources associées dans ton application, mais également sans que le sgbd puisse savoir que la connexion ne sert plus à rien (selon le paramétrage, soit il finira par la fermer (idle), soit il gardera toutes les connexions et tu finiras par te faire jeter pour une nouvelle connexion, puisque le nombre de connexions est forcément limité (soit par paramètre, soit par la mémoire allouée au serveur).
En général, on utilise plutôt un pool de connexions, ce qui permet d'avoir un nombre limité fixe de connexions, qu'on peut partager entre différentes requêtes, et qu'on ferme lorsqu'on arrête l'application.
Pour faire plus simple, au moins au début, tu peux faire une connexion par requête, mais il faut la fermer lorsque la requête est effectuée et traitée.
Une première bonne pratique pour éviter les problèmes est surtout de ne pas faire de variable de classe. De toute manière, il ne te sera pas possible de partager la connexion entre plusieurs requêtes simultanées, et tu éviteras ce partage en ne faisant pas de variable de classe.

Déjà ce code :

Code :

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

1
2
3
4
        try {
            Class.forName("com.mysql.jdbc.Driver");
        } catch (ClassNotFoundException e) {
        }

doit être fait au tout début et une seule fois (au démarrage de l'application). Encore que le plus simple est de ne pas le faire du tout : avec les pilotes de type 4 ce n'est nécessaire de le faire, et je suppose que tu utilises une bibliothèque_que récente, donc tout à fait capable de s'en passer.

Ainsi, pour obtenir une connexion, la méthode devient :

Code :

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

1
2
3
4
5
6
7
8
9
private Connection loadDatabase() { // ici la méthode devrait plutôt s'appeler getConnection, parce que c'est ce qu'elle fait, et en aucun cas ne charge la base
 
        try {
            return DriverManager.getConnection("jdbc:mysql://localhost:3306/javaee", "root", "");
        } catch (SQLException e) {
            e.printStackTrace(); // au minimum, ou mieux, à écrire dans un fichier de log
            return null;
        }
    }

Et donc, faire tes requêtes avec un try-with-ressources :

Code :

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

1
2
3
4
5
6
7
try(Connection connection = loadDatabase()) {
 
    try(Statement statement = connection.createStatement()) { // ou try(PreparedStatement preparedStatement = connection.prepareStatement("...la requête ici... ")) {
          /* ... faire la requête et la traiter ... */
    }
 
}

Ensuite, pour l'exercice, tu pourras voir comment faire un petit pool, ou utiliser un datasource jndi.

PS: pense que quand tu dis qu'il y a une erreur, il est pertinent de dire laquelle (message d'erreur, et stacktrace). Même si parfois on peut le deviner, il est toujours mieux d'avoir le message exact plutôt que de faire de la divination et de risquer de tomber à côté.

**philo71** · 11/08/2017, 22h01

Je t'es écouté avec attention mais dans ma servlet j'ai une erreur dans la ligne :

Code :

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

 utilisateur.setId( Integer.parsetInt(paramId) );

J'ai une erreur sur le "parsetInt" qui est souligné par eclipse :
utilisateur.setId( Integer.parsetInt(paramId) );

voila ma servlet après tes instructions.

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
protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		 String paramId = request.getParameter("id");
 
		Utilisateur utilisateur = new Utilisateur();
 
 
	    try {
			utilisateur.setNom(request.getParameter("nom"));
		} catch (BeanException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
        utilisateur.setPrenom(request.getParameter("prenom"));
        Up_noms tableNoms = new Up_noms();
        tableNoms.miseajourUtilisateur(utilisateur, tableNoms);
        request.setAttribute("utilisateurs", tableNoms.recupererUtilisateurs());
        utilisateur.setId( Integer.parsetInt(paramId) ); // ici j'ai une erreur sur le parsetInt il est souligné
        this.getServletContext().getRequestDispatcher("/WEB-INF/db_update.jsp").forward(request, response);
	}

**joel.drigo** · 12/08/2017, 00h46

C'est parseInt , pas parsetInt.

JDBC et SQL requête UPATE en erreur sous Tomcat

JDBC Java

Vue hybride

Discussions similaires

Partager

Partager