Java

[jsp] vous pensez quoi de mon code ???

Recherche :

Mot : Pseudo : Filtrer
Page : 1 2 3 4 Page Suivante Page Précédente Bas de page
Auteur	Sujet : [jsp] vous pensez quoi de mon code ???

bibidec

Message édité par bibidec le 07-03-2005 à 16:08:45

Publicité

darklord

You're welcome

En ce qui concerne le JavaBean:

Il doit implémenter au pire java.io.Serializable
Il ne doit contenir *que* des données et rien d'autre. En particulier il ne doit pas contenir la logique permettant d'obtenir les données. C'est le role d'une classe de type helper ou factory
Le nom de l'objet n'est pas intuitif
Tu utilises en interne un tableau à deux dimensions avec un getter unique. Bien que ce soit correct d'un point de vue théorique, ne penses tu pas que tu pourrais utiliser une structure plus complexe, te permettant de récupérer des informations plus pertinentes dans ta JSP (ca t'éviterai nottamenet de jouer avec deux indices et ce genre de crasse)
Tu initalises ton tableau a 200*10. Est ce que ton tableau aura tjrs cette taille ou est ce une taille de sécurité? Si tu ne connais pas par avant la taille d'une structure, utilise un container dynamique telle qu'une ArrayList

Le principal reproche est l'accès à la BD dans le bean et ta représentation a deux dimensions plus que douteuses

pour la JSP je laisse la parole aux experts

Message édité par darklord le 19-08-2003 à 14:15:26

---------------
Just because you feel good does not make you right

bibidec

merci beaucoup je prends bonne note !

darklord

You're welcome

bibidec a écrit :

merci beaucoup je prends bonne note !

j'ai ajouté des trucs

---------------
Just because you feel good does not make you right

lorill

le html généré n'est pas valide

lorill

DarkLord a écrit :

Le principal reproche est l'accès à la BD dans le bean et ta représentation a deux dimensions plus que douteuses

comme la requete SQL dans le jsp, c'est pas top

El_gringo

Dans ta JSP t'as beaucoup trop de scriptlets (code Java entre <% et %> ).
je te conseil de ne pas utiliser le "out.println"
Genre quand tu voudrais faire ça :

Code :

<%
if (request.getParameter("ck_listeMembres" ) != null){
DF_EspacePersoBean paramBean = new DF_EspacePersoBean();
paramBean.setTabMembres("SELECT * FROM MEMBRE ORDER BY MEMBRE_NOM" );
String membres[][];
membres = paramBean.getTabMembres();
for (int i =0; i < paramBean.getLongueurTab();i++){
out.println(membres[i][0]);
out.println(membres[i][1]);
out.println(membres[i][2]);
out.println(membres[i][3]);
out.println(membres[i][4] + " " );
}
}
/*
La fonction .equals(String) renvoit "true" si une valeur non-nulle est égal à "on"
La fonction .equals(String) renvoit "false" si un valeur non-nulle n'est pas égale à "on" !
*/
if (request.getParameter("ck_listeQqch1" ) != null) {
out.print(" Ici il y a quelque chose 1 " );
}
if (request.getParameter("ck_listeQqch2" ) != null) {
out.print(" Ici il y a quelque chose 2 " );
}
%>

fais plutot :

Code :

<%
if (request.getParameter("ck_listeMembres" ) != null){
DF_EspacePersoBean paramBean = new DF_EspacePersoBean();
paramBean.setTabMembres("SELECT * FROM MEMBRE ORDER BY MEMBRE_NOM" );
String membres[][];
membres = paramBean.getTabMembres();
for (int i =0; i < paramBean.getLongueurTab();i++){
%>
<%=membres[i][0])%>
<%=membres[i][1])%>
<%=membres[i][2])%>
<%=membres[i][3])%>
<%=membres[i][4])%>
}
}
<%--
La fonction .equals(String) renvoit "true" si une valeur non-nulle est égal à "on"
La fonction .equals(String) renvoit "false" si un valeur non-nulle n'est pas égale à "on" !
--%>
<%
if (request.getParameter("ck_listeQqch1" ) != null) {
%>
Ici il y a quelque chose 1
<%
}
if (request.getParameter("ck_listeQqch2" ) != null) {
%>
Ici il y a quelque chose 2
<%
}
%>

Message édité par El_gringo le 19-08-2003 à 14:24:09

bibidec

lorill a écrit :

comme la requete SQL dans le jsp, c'est pas top

ah ok,

je pensais qu'il était préférable d'accéder aux données avec mon fichier javabean...

okok....

mais comment devrais-je procéder ?
je fais l'accès dans un scriptlet ? dans un jsp ?

darklord

You're welcome

bibidec a écrit :

ah ok,

je pensais qu'il était préférable d'accéder aux données avec mon fichier javabean...

okok....

mais comment devrais-je procéder ?
je fais l'accès dans un scriptlet ? dans un jsp ?

dans une servlet

---------------
Just because you feel good does not make you right

bibidec

DarkLord a écrit :

dans une servlet

ah d'accord,

je fais l'accès à la base dans un servlet.
et je récupère mes données depuis le jsp!

donc je n'utilise plus de javabean ???

j'espère que je raisonne juste :??: :sweat:

Publicité

darklord

You're welcome

bibidec a écrit :

ah d'accord,

je fais l'accès à la base dans un servlet.
et je récupère mes données depuis le jsp!

donc je n'utilise plus de javabean ???

j'espère que je raisonne juste :??: :sweat:

presque !!!

Ton javabean est utilisé pour représenter les données. Donc c'est l'objet que tu retournes, depuis ta servlet, à ta JSP

Edit: en fait, avec ta servlet tu vas chercher les données, tu remplis un JavaBean avec les données et tu retournes le JavaBean a la JSP. Celle ci utlise les getter pour afficher les infos dans la page web. C'est une traduction simple du modèle MVC

Message édité par darklord le 19-08-2003 à 14:31:23

---------------
Just because you feel good does not make you right

El_gringo

bibidec a écrit :

ah d'accord,

je fais l'accès à la base dans un servlet.
et je récupère mes données depuis le jsp!

donc je n'utilise plus de javabean ???

j'espère que je raisonne juste :??: :sweat:

Ben si, pour stocker les données récupérées dans la servlet. Sinon tu ferais pour récupérer les données dans ta JSP !?

lorill

bibidec a écrit :

je fais l'accès dans un scriptlet ? dans un jsp ?

Non, surtout pas, cf les réponses des autres.
Le but, c'est de completement séparer l'acces aux donnée de l'affichage

bibidec

El_gringo a écrit :

Ben si, pour stocker les données récupérées dans la servlet. Sinon tu ferais pour récupérer les données dans ta JSP !?

oui mais je comprends pas quelque chose alors :

quand je suis dans ma servlet je crée une instance et j'appelle ma fonction set.
quand je suis dans ma jsp je suis pas obligé de recréer une instance pour appeler ma fonction get ?

merci pour tout vos coups de pouce !

lorill

bibidec a écrit :

quand je suis dans ma jsp je suis pas obligé de recréer une instance pour appeler ma fonction get ?

non, tu dois la récupérer (par session par exemple) mais pas la recréer.

bibidec

lorill a écrit :

non, tu dois la récupérer (par session par exemple) mais pas la recréer.

okok,

je suis donc obligé de créer une session pour récupérer ma données envoyer par le servlet à mon bean...

d'accord je vais essayer !!!
:love:

lorill

bibidec a écrit :

je suis donc obligé de créer une session pour récupérer ma données envoyer par le servlet à mon bean...

:non:

pour récupérer l'instance du bean garni par la servlet

bibidec

lorill a écrit :

:non:

pour récupérer l'instance du bean garni par la servlet

okkkkkkkkkkkkkkkkkkkkkkkkkk !
c quand meme compliqué pour faire si peu je trouve non ?

enfin si c comme cela qu'il procéder pour avoir quelque chose de propre alors c bien !

merci beaucoup

lorill

bof, c'est pas si compliqué que ca...

Servlet (Controleur)
--> Recupere tes donnés
--> Garni ton bean (Modele)
--> Mets le bean en session
--> Forwarde vers la JSP (Vue)

JSP
--> Lit le bean a partir de la session
--> Fait l'affichage

benou

bibidec a écrit :

je suis donc obligé de créer une session pour récupérer ma données envoyer par le servlet à mon bean...

:non:
stocker le bean dans la session est une possibilité.

d'ailleur, ca a du sens que si tu es susceptible de t'en reservir par la suite ... sinon, autant stocker le bean en tant qu'attribut de la requete (request.setAttribute).

---------------
ma vie, mon oeuvre - HomePlayer

benou

bibidec a écrit :

c quand meme compliqué pour faire si peu je trouve non ?

c'est la façon propre de faire les choses ... [:spamafote]
c'est sûr que pour des petits trucs comme ca, c'est lourd mais si tu ne fais pas ca pour des plus gros projet ca fait du code spagheti

---------------
ma vie, mon oeuvre - HomePlayer

El_gringo

Moi je trouve quand même que dans certain cas, la processus est lourd pour pas grand chose. C'est mal, mais il m'est arrivé de lire un fichier dans une JSP.

Et sinon, t'avais lu mon 1er post dans la topic, bibidec ?

benou

en utilisant la JSTL, tu pourrais bcp simplifier le codage de ta page jsp.

Si tu veux faire les trucs rapidement, tu peux notament écrire ta page qui fait un accès BDD sans écrire un seule ligne de code java. Mais bon, c'est plus propre de passer par le modèle MVC comme ca t'as déjà été expliqué ...

doc rapide et bien faite sur la jstl : http://www.javaworld.com/javaworld [...] -jstl.html

---------------
ma vie, mon oeuvre - HomePlayer

bibidec

El_gringo a écrit :

oui oui,
je l'avais lu!
j'ai également pris note et je mettrai cela en application dès que j'aurai des infos à afficher dans mes scriptlets...
merci bcp...

benou

El_gringo a écrit :

Moi je trouve quand même que dans certain cas, la processus est lourd pour pas grand chose. C'est mal, mais il m'est arrivé de lire un fichier dans une JSP.

logiquement, tu mets un place un modèle MVC que dans les projets de taille déjà importante, sinon tu fais du JSP+JSTL ou JSP+scriplet crados

---------------
ma vie, mon oeuvre - HomePlayer

_guigui_

bibidec a écrit :

okkkkkkkkkkkkkkkkkkkkkkkkkk !
c quand meme compliqué pour faire si peu je trouve non ?

enfin si c comme cela qu'il procéder pour avoir quelque chose de propre alors c bien !

merci beaucoup

bah non c'est pas compliqué, ca simplifie beaucoup la lecture de la JSP, et comme on part du principe que celui qui travaille sur la JSP est un designer HTML ca lui facilite grandement la vie!

du coup pour récupérer le bean dans ta jsp tu fais juste ca:

Code :

<jsp:useBean id="paramBean" class="DF_EspacePersoBean" scope="request"/>

(en changeant le nom de ta classe là c'est pas terrible)

bibidec

benou a écrit :

logiquement, tu mets un place un modèle MVC que dans les projets de taille déjà importante, sinon tu fais du JSP+JSTL ou JSP+scriplet crados

okok mais attendez !

admettons que je dois faire un petit projet pour une boite en jsp, c'est quand meme mieux que je le fasse avec le modèle mvc non ? meme si je mets plus de temps....autant faire qqch de vraiment propre non ?

the real moins moins

benou a écrit :

merci d'insister sur ce point. à priori ça n'a aucun sens de le mettre en session.

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

lorill

bibidec a écrit :

admettons que je dois faire un petit projet pour une boite en jsp, c'est quand meme mieux que je le fasse avec le modèle mvc non ? meme si je mets plus de temps....autant faire qqch de vraiment propre non ?

si t'as pas de contrainte de temps, oui.
si c'est juste un truc 'jetable', bah [:sinclaire]

benou

the real moins moins a écrit :

merci d'insister sur ce point. à priori ça n'a aucun sens de le mettre en session.

toutafé, à moins de vouloir gérer une sorte de cache ... mais sinon ca charge la mémoire pour rien pendant 20 minutes ...
surtout un tableau de 200*10, Ca fait quand même du 64Ko par utilisateur pour rien

Message édité par benou le 19-08-2003 à 15:41:18

---------------
ma vie, mon oeuvre - HomePlayer

_guigui_

bibidec a écrit :

oui c'est mieux, parce que quelle que soit la taille du projet, si tu veux maintenir ton application ca sera quand même plus facile si tu n'as pas trop de code java dans tes JSP.

en plus si tu veux debugger ca va etre difficile de faire du pas à pas dans les JSPs (possible ou pas suivant ton IDE, et pas trés lisible s'il sait le faire)

Message édité par _guigui_ le 19-08-2003 à 15:32:04

bibidec

[citation=491283,1]... sinon, autant stocker le bean en tant qu'attribut de la requete (request.setAttribute).
[/citation]

Je comprends pas bien ça...

the real moins moins

ben y'a un objet "requête" auquel tu peux coller des attributs (n'importe quel objet, en gros c'est une hashmap)

du coup dans ta servlet tu ferais
request.setAttribute("pouet", tonBean);

et dans ton jsp (par l'intermediaire d'un tag, là je te montre le code ) tu ferais
((TonBean) request.getAttribute("pouet" )).getTralala();

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

benou

the real moins moins a écrit :

et dans ton jsp (par l'intermediaire d'un tag, là je te montre le code ) tu ferais
((TonBean) request.getAttribute("pouet" )).getTralala();

et dans la jstl tu ferais requestScope.pouet.tralala

et en JSP2 tu ferais pouet.tralala

---------------
ma vie, mon oeuvre - HomePlayer

the real moins moins

benou a écrit :

et dans la jstl tu ferais requestScope.pouet.tralala

et en JSP2 tu ferais pouet.tralala

tu pourrais mettre un bout de contexte autour de ton exemple en jsp2? (y'a bien un tag ou qqchose non?)

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

bibidec

the real moins moins a écrit :

ah okokokokok...
je commence à y avoir plus clair !
merci à tout le monde pour vos réponses, c'est gentil!
je vais essayer de mettre ça en pratique, histoire d'avoir un modèle à suivre "propre" pour mes futurs applications !
merci!

the real moins moins

j'sais pas c'qu'on a aujourd'hui, on est vraiment sympas [:core 666]

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

_guigui_

the real moins moins a écrit :

ben y'a un objet "requête" auquel tu peux coller des attributs (n'importe quel objet, en gros c'est une hashmap)

faut qu'il soit sérialisable quand même

the real moins moins

_guigui_ a écrit :

faut qu'il soit sérialisable quand même

euh oué ptet, j'ai jamais fait gaffe à vrai dire

mais ça fait lgtps que j'ai pas fait de web [:sisicaivrai]

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

the real moins moins

http://java.sun.com/products/servl [...] ng.Object)

parlent pas de serializable [:spamafote]

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?

Publicité

Page : 1 2 3 4

Page Suivante

Page Précédente

Haut de page

FORUM HardWare.fr

Programmation

Java

[jsp] vous pensez quoi de mon code ???

Sujets relatifs
[ Linux ] Un prog pour obtenir le code assembleur d'un exécutable ?	[PHP, MYSQL] [résolu]Comment optimiser ce code? (nombreux SELECT FROM)
[delphi] longueur du code ... comment vous faites?	code CSS foireux ou bug de ie? (si c'est le cas j'ai besoin d'un fix)
VB6 - Composant FTP : Ch testeurs (code source client FTP fourni)	executer du code 1 fois sur deux dans un while
simplifier une ligne de code ASP	concours de code
Que pensez vous de ce code
Plus de sujets relatifs à : [jsp] vous pensez quoi de mon code ???

Page générée en 0.131 secondes