[MVC] Ce code est-il conforme?

Version imprimable

Bonjour,

Débutant en JEE, j'essaye de suivre une archi MVC pour une appli que je developpe en stage.

J'ai réalisé des classes DAO, et une vue.
Pouvez vous me dire si le code de la vue contient des choses "interdites" en MVC?

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
<%@page contentType="text/html"%>
<%@page pageEncoding="UTF-8"%>
<%@ taglib uri="http://java.sun.com/jsp/jstl/sql" prefix="sql" %>
<%@ taglib uri="http://java.sun.com/jsp/jstl/core" prefix="c" %>
<%@page language="java" import = "cciacs.gesam.dao.dao.ArretDAO" %>
<%@page language="java" import = "cciacs.gesam.dao.beans.ArretBean" %>
<%@page language="java" import = "java.util.Vector" %>
 
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
   "http://www.w3.org/TR/html4/loose.dtd">
 
<jsp:useBean id="listeArrets" class="cciacs.gesam.dao.dao.ArretDAO" scope="session"/>
 
<html>
    <head>
        <title>Test</title>
    </head>
    <body>
 
        <h2>Liste des arrets</h2>
 
        <% 
            Vector v=listeArrets.getAllArretParAgent(3);
 
            for(int i=0;i<v.size();i++){
                ArretBean a = (ArretBean) v.firstElement();
        %>
        <%= a.getTypeArret()%>
        <br>
        <%}%>
    </body>
</html>

merci d'avance !

03/04/2006, 14h08
metalpetsFR
Citation:

<%
Vector v=listeArrets.getAllArretParAgent(3);

for(int i=0;i<v.size();i++){
ArretBean a = (ArretBean) v.firstElement();
%>

Je ne suis pas non plus un expert en MVC, je suis en stage aussi !!
Mais je pense, que tu devrais minimiser (voir interdire) l'utilisation de code java dans tes jsp.
Utilise plutot :
Code:

1 2 3 4 5 <c:forEach var="arret" items="${listeArret}"> <c:out value="${tache.tacDtDemande}"/> <br> </c:forEach>
Pour cela la variable listeArret etant un java.util.List (ou vector j'imagine)
Bon je sais pas ça répond a ta question, mais voila.
Sinon pour faire du "bean" MVC regarde du coté de struts.
03/04/2006, 14h16
Laurent.B

Le getAllArretParAgent(3) n'est selon moi pas très bon car un get ne doit pas prendre d'argument, de fait, il te faudrait une méthode qui te retourne directement le résultat lié à la valeur 3, de cette manière tu pourrais utiliser les tags JSTL, que tu références mais que tu n'utilises pas...
Sinon, le modèle MVC dans tout ça, personnellement je ne vois pas de problème particulier.
03/04/2006, 15h11
zl

Si tu utilises Struts, tu devrais placer toute la logique qui te permet d'appeler tes DAOs dans Actions Struts et ensuite tu construit des JavaBean qui contiennent les informations à afficher.

Zl.
03/04/2006, 16h04
vallica

Bonjour à tous,

merci pour vos réponses tout d'abord.

Pour le moment je n'utilise pas Struts, j'attend un bouquin qui arrivera surement demain pour l'apprendre.

Je voulais juste essayer de démmarer sans Struts une appli MVC au cas où je n'aurais pas le temps de le comprendre suffisement.

Citation:

Le getAllArretParAgent(3) n'est selon moi pas très bon car un get ne doit pas prendre d'argument

Tout à fait d'accord, là j'ai mis la valeur en dur pour tester, mais je compte faire un menu déroulant rempli à partir de la bdd, qui me renverra cette valeur.

Pour metalpetsFR, oui c'est vrai que ton code est bien mieux, je l'utiliserai à l'avenir.

Citation:

Si tu utilises Struts, tu devrais placer toute la logique qui te permet d'appeler tes DAOs dans Actions Struts et ensuite tu construit des JavaBean qui contiennent les informations à afficher.

Par exemple si je veux afficher un tableau, je crée un JavaBean composé d'un vecteur, que je rempli dans le ActionStruts, et j'itère sur ce JavaBean dans la JSP c'est ca?
03/04/2006, 23h16
et.rond.et.rond
Citation:

Envoyé par vallica

Par exemple si je veux afficher un tableau, je crée un JavaBean composé d'un vecteur, que je rempli dans le ActionStruts, et j'itère sur ce JavaBean dans la JSP c'est ca?

là le bean c'est plutot "ArretBean" mais d'après le nom, c'en est déjà un donc tu peux l'utiliser dans ton code

l'idée de minimiser ton code Java c'est d'obtenir un résultat de ce genre dans ta JSP
Code:

1 2 3 <tag:liste liste="listeArrets.allArretParAgent" element="a"> <affiche valeur="a.typeArret"/> </tag:liste>
le nom de mes tags est bien sur fictif ;)
04/04/2006, 06h40
vallica

ok, j'ai vu que struts-layout permettait de faire des tableaux automatiquement avec peu de balises, je vais me pencher la dessus une fois que j'aurais un peu plus appris struts, car ca m'interesse beaucoup (tableau avec tri déjà implémenté sur les colonnes, selection, ajouts/modifis...)