Discussion :

[zikyoubi]

Bonjour ,

Voila j'ai un petit soucis dans la modification d'un tableau dans mon application JEE , j'affiche dans des input de type text mes coordonnées de la base de donnée , je veux que quand je fait une modification sur une colone et que je clique sur Moddifier a gauche .. la case ou les cases se modifient .



Mais quand je change par exemple le nom de l'auteur et que je clique sur modifier , les changement ne se font pas (c'est comme si j'actualise) et ca me donne ce lien sur la barre d'adresse : http://localhost:8080/SiteWeb/modifi...News?idNewsM=1

Bref , voila les code que j'utilise :

-Interface DAO :

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
package com.tomo.dao;
*
import java.util.ArrayList;
*
import com.tomo.beans.News;
*
public interface NewsDAO {
*
*****
*
****void modifierNews( News news ) throws DAOException;
*
****
*
}

-Implementation de l'interface :

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
public class NewsDAOimplement implements NewsDAO {
*
****private DAOFactory********* daoFactory;
*
****
*
****private static final String requeteModifier* = "update news set titre = ? , auteur = ? , resumer = ? , contenue = ? where idnews = ?";
*
*****
*
****public NewsDAOimplement( DAOFactory daoFactory ) {
********super();
********this.daoFactory = daoFactory;
****}
*
***
****@Override
****public void modifierNews( News news ) throws DAOException {
********Connection connect = null;
********PreparedStatement statement = null;
*
********try {
************connect = daoFactory.getConnection();
************statement = inisialisationPrepareStatement( connect, requeteModifier, true, news.getTitre(),
********************news.getAuteur(), news.getResumer(), news.getContenue(), news.getId_news() );
*
************int resultat = statement.executeUpdate();
*
************if ( resultat == 0 ) {
****************throw new DAOException( "Aucune modification n'as étais appliquée" );
************}
*
********} catch ( SQLException e ) {
************e.printStackTrace();
************throw new DAOException( "Erreur lors de la modification de l'article" + e );
********} finally {
************fermeture( statement, connect );
********}
****}
}

-Servlet de la page ou j'affiche le tableau :

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
@WebServlet( "/modificationNews" )
public class ModificationNews extends HttpServlet {
****private static final long** serialVersionUID* = 1L;
*
****private static final String vue_modifierNews* = "/WEB-INF/modifier_news.jsp";
*
****public static final String* InisialDaoFactory = "daoFactory";
*
****private static final String NewsSession****** = "sessionNews";
*
****private static final String champ_idNewsM**** = "idNewsM";
*
****private NewsDAO************ newsDAO;
*
****public ModificationNews() {
********super();
********// TODO Auto-generated constructor stub
****}
*
****public void init() throws ServletException {
********this.newsDAO = ( (DAOFactory) getServletContext().getAttribute( InisialDaoFactory ) ).getNewsDAO();
****}
*
**
****protected void doGet( HttpServletRequest request, HttpServletResponse response ) throws ServletException,
************IOException {
*
********String idnews = getValeurChamp( request, champ_idNewsM );
*
********long idNewsM = Long.parseLong( idnews );
*
********HttpSession session = request.getSession();
*
********Map<Long, News> mapNews = (HashMap<Long, News>) session.getAttribute( NewsSession );
*
********if ( idNewsM != 0 && mapNews != null ) {
*
************newsDAO.modifierNews( mapNews.get( idNewsM ) );
************session.setAttribute( NewsSession, mapNews );
********}
*
********this.getServletContext().getRequestDispatcher( vue_modifierNews ).forward( request, response );
*
****}
*
****public String getValeurChamp( HttpServletRequest request, String champ ) {
*
********String value = request.getParameter( champ );
*
********if ( value == null || value.trim().length() == 0 ) {
*
************return null;
*
********} else {
************return value;
********}
*
****}
}

-Page JSP ou j'affiche le tableau :

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
<center>
<table border="1">
<thead>
<tr>
<th>Titre</th>
<th>Auteur</th>
<th>resumer</th>
<th>Contenue</th>
<th>Modifier</th>
<th>Supprimer</th>
</tr>
</thead>
*
*
<tfoot>
<tr>
<td></td>
<td></td>
<td></td>
<td></td>
<td></td>
<td></td>
*
</tr>
</tfoot>
*
<c:choose>
<c:when test="${empty sessionScope.sessionNews}">
<td colspan="6" align="center">Aucune News n'est disponible</td>
</c:when>
<c:otherwise>
<tbody>
<c:forEach items="${sessionScope.sessionNews}" var="mapNews">
<tr>
*
<td><input type="text" value="<c:out value="${mapNews.value.titre}"></c:out>"/></td>
<td><input type="text" value="<c:out value="${mapNews.value.auteur}"></c:out>"/></td>
<td><textarea cols="20" rows="3"><c:out value="${mapNews.value.resumer}"></c:out></textarea></td>
<td><textarea cols="20" rows="3"><c:out value="${mapNews.value.contenue}"></c:out></textarea></td>
<td> <a href="<c:url value="/modificationNews"><c:param name="idNewsM" value="${mapNews.key}"/></c:url>">Modifier</a> </td>
<td> <a href="<c:url value="#"><c:param name="idTomo" value="${mapNews.key}"/></c:url>">Supprimer</a>* </td>
</tr>
</c:forEach>
</tbody>
</c:otherwise>
</c:choose>
</table>
</center>

En attente de vos réponses , merci .

[tchize_]

Code :

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

<a href="<c:url value="/modificationNews"><c:param name="idNewsM" value="${mapNews.key}"/></c:url>">Modifier</a>

Ceci ne fait rien d'autre que de créer un lien, ça ne va surement pas soumettre ton formulaire et encore moins effectuer les modifcations demandées.

[zikyoubi]

J'avais fait supprimer avec la meme manière , je voulais testé si je pouvais faire avec Modifier aussi , en passant juste l'id de l'article .. , est ce que je dois mettre le tableau entre un form et envoyé mais donnée avec un post ?

[tchize_]

J'avais fait supprimer avec la meme manière , je voulais testé si je pouvais faire avec Modifier aussi , en passant juste l'id de l'article ..

,
En passant juste l'id, tu n'as pas les informations entrées par l'utilisateur, donc c'est forcément problématique, ton serveur ne peut pas par magie aller lire ce qui était à l'écran. Donc oui il te faut un formulaire.

[zikyoubi]

D'accord merci , je vais essayer de faire les modifications dès que je rentre chez moi et je vous met le code .

[zikyoubi]

Pardon du retard et du double post , j'ai essayer avec le form , mais j'ai un soucis avec l'id de la news , j'arrive pas a la passer au a la class de traitement des forms , je pense qu'il y as un problème de convertion ou autres ..

Voila 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
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
package com.tomo.servlets;
 
 
@WebServlet( "/modifierNews" )
public class ModifierNews extends HttpServlet {
    private static final long   serialVersionUID  = 1L;
 
    private static final String vue_modifierNews  = "/WEB-INF/modifier_news.jsp";
 
    public static final String  InisialDaoFactory = "daoFactory";
 
    private static final String NewsSession       = "sessionNews";
 
    private NewsDAO             newsDAO;
 
    private static final String FormModifier      = "ModifeForm";
    private static final String BeansModifier     = "ModifeBean";
 
    /**
     * @see HttpServlet#HttpServlet()
     */
    public ModifierNews() {
        super();
        // TODO Auto-generated constructor stub
    }
 
 
    public void init() throws ServletException {
        this.newsDAO = ( (DAOFactory) getServletContext().getAttribute( InisialDaoFactory ) ).getNewsDAO();
    }
 
 
    protected void doGet( HttpServletRequest request, HttpServletResponse response ) throws ServletException,
            IOException {
        HttpSession session = request.getSession();
 
        ArrayList<News> listNews = newsDAO.listNews();
 
        Map<Long, News> mapNews = new HashMap<Long, News>();
 
        for ( News news : listNews ) {
            mapNews.put( news.getId_news(), news );
        }
 
        session.setAttribute( NewsSession, mapNews );
 
        this.getServletContext().getRequestDispatcher( vue_modifierNews ).forward( request, response );
    }
 
    protected void doPost( HttpServletRequest request, HttpServletResponse response ) throws ServletException,
            IOException {
 
        NewsModifierForm formModif = new NewsModifierForm( newsDAO );
        News newss = formModif.FormModifierNews( request );
 
        request.setAttribute( FormModifier, formModif );
        request.setAttribute( BeansModifier, newss );
 
        HttpSession session = request.getSession();
 
        if ( formModif.getErreurs().isEmpty() ) {
 
            Map<Long, News> mapNewsModif = (HashMap<Long, News>) session.getAttribute( NewsSession );
 
            if ( mapNewsModif == null ) {
                mapNewsModif = new HashMap<Long, News>();
            }
 
            mapNewsModif.put( newss.getId_news(), newss );
 
            session.setAttribute( NewsSession, mapNewsModif );
 
            this.getServletContext().getRequestDispatcher( vue_modifierNews ).forward( request, response );
 
        } else {
            this.getServletContext().getRequestDispatcher( vue_modifierNews ).forward( request, response );
        }
 
    }
}

Le form :

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
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
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
package com.tomo.forms;
 
import java.util.HashMap;
import java.util.Map;
 
import javax.servlet.http.HttpServletRequest;
 
import com.tomo.beans.News;
import com.tomo.dao.NewsDAO;
 
public class NewsModifierForm {
 
    // attribus:
    private String              champ_idNews   = "idNewsN";
    private String              champ_titre    = "titreN";
    private String              champ_auteur   = "auteurN";
    private String              champ_resumer  = "resumerN";
    private String              champ_contenue = "contenueN";
 
    private NewsDAO             newdao;
 
    private Map<String, String> erreurs        = new HashMap<String, String>();
 
    public String               resultat;
 
    public Map<String, String> getErreurs() {
        return erreurs;
    }
 
    public String getResultat() {
        return resultat;
    }
 
    public NewsModifierForm( NewsDAO newdao ) {
        super();
        this.newdao = newdao;
    }
 
    // Method principale :
 
    public News FormModifierNews( HttpServletRequest request ) {
 
        News news = new News();
 
        String titre = getValeurChamp( request, champ_titre );
        String auteur = getValeurChamp( request, champ_auteur );
        String contenue = getValeurChamp( request, champ_contenue );
        String resumerr = getValeurChamp( request, champ_resumer );
 
        String id = getValeurChamp( request, champ_idNews );
 
        long idNews = Long.parseLong( id );
 
 
 
        if ( erreurs.isEmpty() ) {
 
            TraiterAuteur( auteur, news );
            TraiterTitre( titre, news );
            TraiterContenue( contenue, news );
            TraiterResumer( resumerr, news );
            news.setId_news( idNews );
 
        }
 
        if ( erreurs.isEmpty() ) {
            newdao.modifierNews( news );
            resultat = "Modification de l'article Réussie";
        } else {
            resultat = "Modification de l'article Echoué";
        }
 
        return news;
 
    }
 
    // Methode utilitaires :
 
    public void TraiterTitre( String titre, News news ) {
        try {
 
            validationTitre( titre );
 
        } catch ( FormsException e ) {
            setErreurs( champ_titre, e.getMessage() );
        }
 
        news.setTitre( titre );
    }
 
    public void TraiterAuteur( String auteur, News news ) {
 
        try {
            validationAuteur( auteur );
        } catch ( FormsException e ) {
            setErreurs( champ_auteur, e.getMessage() );
        }
 
        news.setAuteur( auteur );
    }
 
    public void TraiterContenue( String contenue, News news ) {
 
        try {
            validationContenue( contenue );
        } catch ( FormsException e ) {
            setErreurs( champ_contenue, e.getMessage() );
        }
 
        news.setContenue( contenue );
 
    }
 
    public void TraiterResumer( String resumer, News news ) {
 
        try {
 
            validationResumer( resumer );
 
        } catch ( FormsException e ) {
            setErreurs( champ_resumer, e.getMessage() );
        }
 
        news.setResumer( resumer );
    }
 
    public void validationContenue( String contenue ) throws FormsException {
 
        if ( contenue != null ) {
 
            if ( contenue.trim().length() < 10 ) {
                throw new FormsException( "Le contenue de l'article doit dépasser 10 caractères" );
            }
 
        } else {
            throw new FormsException( "Veuillez entrez le contenue de l'article" );
        }
 
    }
 
    public void validationAuteur( String auteur ) throws FormsException {
 
        if ( auteur != null ) {
 
            if ( auteur.trim().length() > 30 ) {
                throw new FormsException( "Le nom de l'auteur ne doit pas dépasser 30 caractères" );
            }
 
        } else {
            throw new FormsException( "Veuillez entrez le nom de l'auteur" );
        }
 
    }
 
    public void validationResumer( String resume ) throws FormsException {
 
        if ( resume != null ) {
            if ( resume.trim().length() > 200 ) {
                throw new FormsException( "Le resumer ne doit pas dépasser 150 caractères" );
            }
        } else {
            throw new FormsException( "Veuillez entrez le résumer de l'article" );
        }
 
    }
 
    public void validationTitre( String titre ) throws FormsException {
 
        if ( titre != null ) {
 
            if ( titre.trim().length() > 30 ) {
                throw new FormsException( "Le titre ne doit pas dépasser 30 caractères" );
            }
 
        } else {
            throw new FormsException( "Veuillez entrez le titre de l'aticle" );
        }
 
    }
 
    public static String getValeurChamp( HttpServletRequest request, String champ ) {
        String value = request.getParameter( champ );
 
        if ( value.trim().length() == 0 || value == null ) {
            return null;
        } else {
            return value;
        }
    }
 
    public void setErreurs( String champ, String message ) {
        erreurs.put( champ, message );
    }
 
}

La page JSP ou j'affiche le tableau :

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
<p><c:out value="${ModifeForm.resultat}"></c:out></p>
 
<center>
<form action="<c:url value="/modifierNews"/>" method="post">
 
<table border="1">
<thead>
<tr>
<th>Numéro</th>
<th>Titre</th>
<th>Auteur</th>
<th>resumer</th>
<th>Contenue</th>
<th>Modifier</th>
<th>Supprimer</th>
</tr>
</thead>
 
 
<tfoot>
<tr>
<td></td>
<td></td>
<td></td>
<td></td>
<td></td>
<td></td>
<td></td>
 
</tr>
</tfoot>
 
<c:choose>
 
<c:when test="${empty sessionScope.sessionNews}">
<td colspan="6" align="center">Aucune News n'est disponible</td>
</c:when>
 
<c:otherwise>
<tbody>
<c:forEach items="${sessionScope.sessionNews}" var="mapNews">
 
<tr>
<td ><input type="text" value="<c:out value="${mapNews.value.id_news}" > </c:out>" size="1px;" disabled="disabled" name="idNewsN"/></td>
 
<td><input type="text" value="<c:out value="${mapNews.value.titre}"></c:out>" name="titreN" size="17px;"/>
<c:out value="${ModifeForm.erreurs.titreN}"></c:out>
</td>
<td><input type="text" value="<c:out value="${mapNews.value.auteur}"></c:out>" name="auteurN" size="13px;"/>
<c:out value="${ModifeForm.erreurs.auteurN}"></c:out>
</td>
<td><textarea cols="20" rows="3" name="resumerN"><c:out value="${mapNews.value.resumer}" ></c:out></textarea>
<c:out value="${ModifeForm.erreurs.resumerN}"></c:out>
</td>
<td><textarea cols="20" rows="3" name="contenueN"><c:out value="${mapNews.value.contenue}"></c:out></textarea>
<c:out value="${ModifeForm.erreurs.contenueN}"></c:out>
</td>
<td><input type="submit" value="Modifier"/></td>
<!-- <td> <a href="<c:url value="/modificationNews"><c:param name="idNewsM" value="${mapNews.key}"/></c:url>">Modifier</a> </td> -->
<td> <a href="<c:url value="#"><c:param name="idTomo" value="${mapNews.key}"/></c:url>">Supprimer</a>  </td>
</tr>
 
</c:forEach>
</tbody>
</c:otherwise>
 
</c:choose>
 
</table>
 
</form>
 
</center>

L'erreur dans la console :

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
"Servlet.service()" pour la servlet com.tomo.servlets.ModifierNews a généré une exception
java.lang.NullPointerException
	at com.tomo.forms.NewsModifierForm.getValeurChamp(NewsModifierForm.java:181)
	at com.tomo.forms.NewsModifierForm.FormModifierNews(NewsModifierForm.java:49)
	at com.tomo.servlets.ModifierNews.doPost(ModifierNews.java:80)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:647)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:728)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:305)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:210)
	at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:222)
	at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:123)
	at org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:472)
	at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:171)
	at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:99)
	at org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:953)
	at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:118)
	at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:408)
	at org.apache.coyote.http11.AbstractHttp11Processor.process(AbstractHttp11Processor.java:1023)
	at org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:589)
	at org.apache.tomcat.util.net.JIoEndpoint$SocketProcessor.run(JIoEndpoint.java:312)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
	at java.lang.Thread.run(Unknown Source)

Image du tableau :

[jessey]

Il faudrait que tu changes l'ordre des éléments tester dans ton if.
Car si value est null, tu auras tout le temps un nullPointerException

Code :

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

1
2
3
4
5
6
7
 
public static String getValeurChamp( HttpServletRequest request, String champ ) {
        String value = request.getParameter( champ );
 
        if ( value.trim().length() == 0 || value == null ) {
            return null;
        }

[zikyoubi]

En fait je viens de me corriger , le problème venais de disabled="disabled" ! l'id ne passé pas au form .. il fallait faire a la place readonly maintenant ca marche bien , merci pour les réponses !