mon code de modification ne marche pas

Version imprimable

3 pièce(s) jointe(s)

Bonjour je suis actuellement en stage dans une entreprise de développement informatique ils m'ont donné la tache de programmer une application de gestion des cartes d'abonnements des trains de l'oncf , bref tout allé bien jusqu'à ce que je bloc sur cette erreur bon voilà je m'explique j'ai crée une interface qui permet au administrateur de crée des vendeurs ( vendeurs de cartes d'abonnement des trains) voici l'interface :

Pièce jointe 488408

voici mon code jsp :

Code:

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
<%@page contentType="text/html" pageEncoding="UTF-8"%>
<!DOCTYPE html>
<html>
    <head>
        <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
        <link href="Css/Css_creation_vendeur.css" rel="stylesheet">
        <link rel="stylesheet" href="https://stackpath.bootstrapcdn.com/bootstrap/4.3.1/css/bootstrap.min.css" integrity="sha384-ggOyR0iXCbMQv3Xipma34MD+dH/1fQ784/j6cY/iJTQUOhcWr7x9JvoRxT2MZw1T" crossorigin="anonymous">
        <script src="https://code.jquery.com/jquery-3.3.1.slim.min.js" integrity="sha384-q8i/X+965DzO0rT7abK41JStQIAqVgRVzpbzo5smXKp4YfRvH+8abtTE1Pi6jizo" crossorigin="anonymous"></script>
        <script src="https://cdnjs.cloudflare.com/ajax/libs/popper.js/1.14.7/umd/popper.min.js" integrity="sha384-UO2eT0CpHqdSJQ6hJty5KVphtPhzWj9WO1clHTMGa3JDZwrnQq4sF86dIHNDz0W1" crossorigin="anonymous"></script>
 
        <script src="https://stackpath.bootstrapcdn.com/bootstrap/4.3.1/js/bootstrap.min.js" integrity="sha384-JjSmVgyd0p3pXB1rRibZUAYoIIy6OrQ6VrjIEaFf/nJGzIxFDsf4x0xIM+B07jRM" crossorigin="anonymous"></script>
        <title>JSP Page</title>
    </head>
    <body>
        <div id="header_creation_vendeur">
            <h2>Creation vendeur</h2>
        </div>
 
        <form method="post" action="Creation_vendeur">
 
            <input type="hidden" name="action" value="ajouter"/>
            <input type="hidden" value="modifier" name="action"/>
        <div id="div_cv1">
            <div class="form-group form-inline">
            <label>Matricule</label>
            <input type="text" name="matricule" class="form-control form-control-md" id="matricule"/>
             <input type="submit" class="btn btn-warning btn-lg"  action="" value="Rechercher" id="rech">
            </div>
        </div>
 
        <div id="div_cv2">
            <div class="form-group form-inline">
            <label>Matricule</label>
            <input type="text" name="matricule2" class="form-control form-control-md" id="matricule2"/>
            </div>
 
            <div class="form-group form-inline">
            <label>Login</label> 
            <input type="text" name="logincv" class="form-control form-control-md" id="logincv"/> 
            </div>
 
            <div class="form-group form-inline">
            <label>Mot de passe</label>
            <input type="password" name="password" class="form-control form-control-md" id="password"/> 
            </div>
 
 
            <div class="form-group form-inline">
 
                <label>Role</label>
 
                <div class="form-check-input">
 
                    <input type="radio" class='form-check-input' value="2" name='radio' id="radiovendeur"/>
 
                </div>
 
                <label class="form-check-label">
                        Vendeur
                </label>
 
                <div class="form-check-input">
 
                    <input type="radio" class='form-check-input' value="1"  name='radio' id="radioadmin"/>
 
                </div>
 
                <label class="form-check-label">
                        Administrateur
                </label>
            </div>
 
 
            <div class="form-group form-inline">
 
                <label>Etat</label>
 
                <div class="form-check-input">
 
                     <input type="radio" class='form-check-input' value="0"  name='radio2' id="radioactif"/>
 
                </div>
 
                <label class="form-check-label">
                        Actif
                </label>
 
                <div class="form-check-input">
 
                     <input type="radio" class='form-check-input' value="1" name='radio2' id="radioinactif"/>
 
                </div>
 
                <label class="form-check-label">
                        Inactif
                </label>
 
 
            </div>
        </div>
 
        <div id="buttoncv">
            <div class="form-group form-inline">
 
                <input type="submit" class="btn btn-warning btn-lg"   action="ajouter" value="Ajouter">
               <input type="reset" class="btn btn-warning btn-lg"   value="Effacer">
               <input type="submit" class="btn btn-warning btn-lg"  action="modifier" value="modifier">
            </div>
        </div>
 
        </form>
    </body>
</html>

J'ai crée une servlet ou je souhaite mettre tout mon code ( d'ajout , modification , recherche) c'est pour celà que j'ai mis pour le moment 2 input de type hidden voici ma servlet :

Code:

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
@Override
    protected void doPost(HttpServletRequest request, HttpServletResponse response)
            throws ServletException, IOException {
        String matricule2=request.getParameter("matricule2");
        String logincv=request.getParameter("logincv");
        String password=request.getParameter("password");
        String radio= request.getParameter("radio");
        String radio2=request.getParameter("radio2");
 
        Connection c=cnx.getcnx();
        if(request.getParameter("action").equals("ajouter")){
 
            try {
            PreparedStatement st=c.prepareStatement("insert into vendeurs values (null,?,?,?,null,null,?,null)",Statement.RETURN_GENERATED_KEYS);
            st.setString(1, logincv);
            st.setString(2, password);
            st.setString(3, matricule2);
            st.setString(4, radio2);
            st.executeUpdate();
                int key=-1;
            ResultSet rs = st.getGeneratedKeys();
            if (rs.next()) {
                key = rs.getInt(1);
            }
            System.out.print(key);
 
            PreparedStatement st1=c.prepareStatement("insert into vendeurinroles values (null,?,?)");
            st1.setInt(1, key);
            st1.setInt(2, Integer.parseInt(radio));
            int res=st1.executeUpdate();
 
            if(res>0){
                System.out.println("ajouter avec succes");
            }
 
            else {
                System.out.println("erreur");
            }
        }
 
        catch(Exception ex){
            ex.printStackTrace();
        }
 
        }
 
 
        else if(request.getParameter("action").equals("modifier")){
            try{
                PreparedStatement stm=c.prepareStatement("update vendeurs set login=? , password=? , isactive=? where matricule=?",Statement.RETURN_GENERATED_KEYS);
                stm.setString(1, logincv);
                stm.setString(2, password);
                stm.setString(3, radio2);
                stm.setString(4, matricule2);
                stm.executeUpdate();
 
                int keym=-1;
            ResultSet rs = stm.getGeneratedKeys();
            if (rs.next()) {
                keym = rs.getInt(1);
            }
            System.out.print(keym);
 
 
            PreparedStatement stm2=c.prepareStatement("update vendeurinroles set role_id=? where vendeur_id=?");
            stm2.setInt(1, Integer.parseInt(radio));
            stm2.setInt(2, keym);
            int resmodification=stm2.executeUpdate();
                if(resmodification>0){
                    System.out.println("Modifier avec succes");
                }
                else {
                System.out.println("erreur");
                }
            }
            catch(Exception ex){
            ex.printStackTrace();
        }
        }
 
 
 
        getServletContext().getRequestDispatcher("/WEB-INF/Creation_vendeur.jsp").forward(request, response);
    }

Pour que vous comprenez trés bien j'ai 2 tables la premiere vendeurs la voici :

Pièce jointe 488410

et la table vendeurinroles la voici :

Pièce jointe 488413

comme vous le vouyez le role il n'est pas dans la table vendeurs mais bien dans la table vendeurinroles c'est pour celà que dans l'insertion par exemple j'insert dans la table vendeurs on récupérent l'id du vendeur inserer et apres j'insere au vendeurs crée un role dans la table vendeurinroles avec l'id que j'ai récupére c'est pour cela que j'ai mis ce code :

Code:

1
2
3
4
5
6
 
int key=-1;
            ResultSet rs = st.getGeneratedKeys();
            if (rs.next()) {
                key = rs.getInt(1);
            }

bref j'espere que vous m'aviez compris, bon passent au probléme maintenant :

le premier c'est que meme si je clic sur le bouton modifier il m'execute le code de l'ajout meme si j'ai mis la condition : else if(request.getParameter("action").equals("modifier")){

il ne sors pas de la premier condition qui est de l'ajout.

le deuxieme probleme c'est que meme si je met le code de l'insertion en commentaire il passe a la deuxieme condtion qui est de la modification oui mais il ne me modifie rien il m'affiche :

Infos: -1

Infos: erreur

ce qui signifie qu'il y a une erreur dans mon code mais je ne sais pas laquel !!!

svp si quelqu'un peut m'aider le plus tot possible !!!

et merci beaucoup en avance !

Code:

1 2 <input type="hidden" name="action" value="ajouter"/> <input type="hidden" value="modifier" name="action"/>
Ton problème doit venir de ça.

Les champs cachés sont toujours envoyés

03/07/2019, 20h14
toztoztoze
Et c'est quoi la solution stp ? pourtant le bouton ajouter marche trés bien aide moi plz je dois livré cette tache à mon encadrent le plus vite possible je te serais trés reconnaissent
Citation:
Envoyé par supergeoffrey

Code:

1 2 <input type="hidden" name="action" value="ajouter"/> <input type="hidden" value="modifier" name="action"/>

Ton problème doit venir de ça.

Les champs cachés sont toujours envoyés
04/07/2019, 09h38
Cincinnatus

Salut,

1 - tu vires les champs cachés "hidden".
2 - tu fais en sorte que tes submit remplissent le champ "action" de ton formulaire au lieu d'utiliser un pseudo-attribut 'action' (?).

Salut,

Je pensais que tu aurais compris je vais t'expliquer.

A supprimer :
Code:

1 2 <input type="hidden" name="action" value="ajouter"/> <input type="hidden" value="modifier" name="action"/>
____

Remplacer:
Code:

1 2 3 <input type="submit" class="btn btn-warning btn-lg" action="ajouter" value="Ajouter"> <input type="reset" class="btn btn-warning btn-lg" value="Effacer"> <input type="submit" class="btn btn-warning btn-lg" action="modifier" value="modifier">
Par:
Code:

1 2 3 <input type="submit" name="action" class="btn btn-warning btn-lg" value="Ajouter"> <input type="reset" class="btn btn-warning btn-lg" value="Effacer"> <input type="submit" name="action" class="btn btn-warning btn-lg" value="modifier">
_____

En fait un champs de type hidden est toujours envoyé à ton serveur (sauf si tu mets l'attribut disabled).
Du coup, tu envoies plusieurs fois le paramètre action avec plusieurs valeur ("ajouter" et "modifier") et ton serveur utilise que la première valeur.

En mettant l'attribut "name" sur le type input il enrayera a ton serveur le paramètre action à ton serveur avec la valeur de value. Fait attention à la casse (Ajouter dans le HTML et ajouter dans le java)!

____

D'autre part, il est même mieux de faire
Code:

1 2 3 <input type="submit" name="ajouter" class="btn btn-warning btn-lg" value="Ajouter"> <input type="reset" class="btn btn-warning btn-lg" value="Effacer"> <input type="submit" name="modifier" class="btn btn-warning btn-lg" value="modifier">
Code:

1 2 3 4 5 6 if (request.getParameter("ajouter") != null) { ... } else if(request.getParameter("modifier") != null){ ... }
Cela permet de modifier plus facilement les actions dans des cas ou tu dois traduire ton application.

Par exemple ta version marchera en français mais pas en anglais.
Code:

1 2 3 <input type="submit" name="ajouter" class="btn btn-warning btn-lg" value="Add"> <input type="reset" class="btn btn-warning btn-lg" value="Clear"> <input type="submit" name="modifier" class="btn btn-warning btn-lg" value="Update">
La mienne marche dans les deux cas

1 pièce(s) jointe(s)

Parfait cela marche a merveille !! merci beaucoup il reste maintenant que la deuxieme erreur quand je clic sur modifier la modification ne se réalise pas il m'affiche se message :

Pièce jointe 488652

cela signifie qu'il y a une erreur dans mon code parce que j'ai mis un bout de code qui me permet de savoir si la modification c'est réaliser avec succés ou pas :

int resmodification=stm2.executeUpdate();
if(resmodification>0){
System.out.println("Modifier avec succes");
}
else {
System.out.println("erreur");
}

dans mon cas ça affiche erreur comme tu peux le voir et aussi pour l'id il m'affiche -1

int keym=-1;
ResultSet rs = stm.getGeneratedKeys();
if (rs.next()) {
keym = rs.getInt(1);
}
System.out.print(keym);

cela signifie qu'il y a une erreur qui cloche xd mais que je n'arrive pas à la savoir pouvez vous m'aidez svp ?
Citation:
Envoyé par supergeoffrey
Salut,

Je pensais que tu aurais compris je vais t'expliquer.

A supprimer :
Code:

1 2 <input type="hidden" name="action" value="ajouter"/> <input type="hidden" value="modifier" name="action"/>
____

Remplacer:
Code:

1 2 3 <input type="submit" class="btn btn-warning btn-lg" action="ajouter" value="Ajouter"> <input type="reset" class="btn btn-warning btn-lg" value="Effacer"> <input type="submit" class="btn btn-warning btn-lg" action="modifier" value="modifier">
Par:
Code:

1 2 3 <input type="submit" name="action" class="btn btn-warning btn-lg" value="Ajouter"> <input type="reset" class="btn btn-warning btn-lg" value="Effacer"> <input type="submit" name="action" class="btn btn-warning btn-lg" value="modifier">
_____

En fait un champs de type hidden est toujours envoyé à ton serveur (sauf si tu mets l'attribut disabled).
Du coup, tu envoies plusieurs fois le paramètre action avec plusieurs valeur ("ajouter" et "modifier") et ton serveur utilise que la première valeur.

En mettant l'attribut "name" sur le type input il enrayera a ton serveur le paramètre action à ton serveur avec la valeur de value. Fait attention à la casse (Ajouter dans le HTML et ajouter dans le java)!

____

D'autre part, il est même mieux de faire
Code:

1 2 3 <input type="submit" name="ajouter" class="btn btn-warning btn-lg" value="Ajouter"> <input type="reset" class="btn btn-warning btn-lg" value="Effacer"> <input type="submit" name="modifier" class="btn btn-warning btn-lg" value="modifier">
Code:

1 2 3 4 5 6 if (request.getParameter("ajouter") != null) { ... } else if(request.getParameter("modifier") != null){ ... }
Cela permet de modifier plus facilement les actions dans des cas ou tu dois traduire ton application.

Par exemple ta version marchera en français mais pas en anglais.
Code:

1 2 3 <input type="submit" name="ajouter" class="btn btn-warning btn-lg" value="Add"> <input type="reset" class="btn btn-warning btn-lg" value="Clear"> <input type="submit" name="modifier" class="btn btn-warning btn-lg" value="Update">
La mienne marche dans les deux cas

05/07/2019, 09h30
supergeoffrey

Code:

PreparedStatement stm=c.prepareStatement("update vendeurs set login=? , password=? , isactive=? where matricule=?",Statement.RETURN_GENERATED_KEYS);

Ce n'est pas lié au fait que tu veux récupérer les clés.
Regarde le lien c'est en anglais mais tu trouvera ta solution.

https://stackoverflow.com/questions/...generated-keys

___

Sinon as-tu essayé ? Tu ne crées pas de clé.

Code:

PreparedStatement stm=c.prepareStatement("update vendeurs set login=? , password=? , isactive=? where matricule=?");

C'est bon j'ai réussi à résoudre le problème j'ai enlevé ce code :
Code:

1 2 3 4 5 6 int keym=-1; ResultSet rs = stm.getGeneratedKeys(); if (rs.next()) { keym = rs.getInt(1); } System.out.print(keym);
et j'ai modifié la requête voici ce que j'ai fais :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 else if(request.getParameter("action").equals("modifier")){ try{ PreparedStatement stm=c.prepareStatement("update vendeurs set login=? , password=? , isactive=? where matricule=?",Statement.RETURN_GENERATED_KEYS); stm.setString(1, logincv); stm.setString(2, password); stm.setString(3, radio2); stm.setString(4, matricule2); stm.executeUpdate(); PreparedStatement stm2=c.prepareStatement("update vendeurinroles set role_id=? where vendeur_id in(select id from vendeurs where matricule=?)"); stm2.setInt(1, Integer.parseInt(radio)); stm2.setString(2, matricule2); int resmodification=stm2.executeUpdate(); if(resmodification>0){ System.out.println("Modifier avec succes"); } else { System.out.println("erreur"); } }
Merci beaucoup pour votre aide !!!

Citation:

Envoyé par supergeoffrey

Code:

PreparedStatement stm=c.prepareStatement("update vendeurs set login=? , password=? , isactive=? where matricule=?",Statement.RETURN_GENERATED_KEYS);

Ce n'est pas lié au fait que tu veux récupérer les clés.
Regarde le lien c'est en anglais mais tu trouvera ta solution.

https://stackoverflow.com/questions/...generated-keys

___

Sinon as-tu essayé ? Tu ne crées pas de clé.

Code:

PreparedStatement stm=c.prepareStatement("update vendeurs set login=? , password=? , isactive=? where matricule=?");