Passer un ResultSet en paramètre ? Rediriger vers une Jsp ou une servlet ?

Version imprimable

Bonjour,

J'ai une base de données de client, sous SQL Server 05. J'y accède sans soucis via le code Java.
Sur un module de mon projet, il est possible pour l'utilisateur de rechercher un client de la BD, selon plusieurs critères.

L'utilisateur remplit un formulaire (rechercheClient.jsp). Lors de la validation, il est renvoyé vers une servlet (RechercheClient.java). Cette servlet trouve les clients, et affiche une page avec des out.println.
Cela n'est pas super esthétique, je trouve :
Voici un bout du code :
Code:

1 2 3 4 5 6 7 8 9 10 affichage += "<html>"; affichage += " <head>"; affichage += " <meta http-equiv='Content-Type' content='text/html; charset=ISO-8859-1'>"; affichage += " <link href='sos.css' rel='stylesheet' type='text/css'>"; affichage += " </head>"; affichage += " <body>"; //etc... out.println(affichage)
Est-ce raisonnable d'afficher une page uniquement à l'aide de out.println ?
Ou bien vaut-il mieux que la servlet redirige encore vers une JSP, qui récupère le ResultSet et affiche le tout ?

Merci de vos conseils.

13/06/2008, 15h05
Monkey_D.Luffy
Bonjour,

Je fais une page de recherche de client dans une base SQL Server. L'utilisateur remplit un formulaire de recherche, et en validant il doit avoir la page de résultat qui s'affiche.

Dans ma page rechercheClient.Jsp, je récupère les paramètres entrés dans le formulaire.
Je redirige ensuite vers la servlet RechercheClient.java.
Celle-ci se connecte à la base, recherche les clients, range le résultat dans un ResultSet, et je fais un :
Code:

1 2 3 ResultSet res = connexion.rechercheClient(numRecherche, nomRecherche); session.setAttribute("resultatClient", res);
Ensuite, je forward vers resultatClient.jsp. Cette JSP récupère le ResultSet et affiche les champs :
Code:

1 2 3 4 5 ResultSet res = (ResultSet)session.getAttribute("resultatClient"); while(res.next()) { out.println("Resultset res = " + res.getInt(1) + "<br />"); i++; }
Ai-je bien fait, où y a-t-il un moyen plus propre ?
Merci.
13/06/2008, 17h26
supermanu
Tout d'abord il est en effet bien mieux d'utiliser une jsp plutôt que de faire des out.println :)

Pour ce qui est de passer directement le resultset à la jsp, là par contre ce n'est pas conseillé. En effet si un jour tu décides de changer la manière dont tu récupère tes données (utilisation de procédures stockées, utilisation d'un framework de mapping...) tu devras refaire toutes tes jsp 8O
Car tu auras un couplage fort entre ta couche de présentation et ta couche de données.

Donc pour améliorer un peu cette situation, tu peux créer un bean tout simple :
Code:

1 2 3 4 5 6 7 8 9 public class ResultatClient { private int id; private String nom; public int getId() {...} public void setId(int _id) {...} ... }
Dans ta servlet tu rempli un bean ResultatClient à partir du resultset. Et c'est ce bean que tu envoie à la vue.

Ce n'est pas la perfection mais c'est déjà mieux :P
13/06/2008, 17h37
supermanu

Et d'ailleurs, si c'est possible, tu peux faire en sorte que ta méthode connexion.rechercheClient renvoie un bean ResultatClient, comme ça la transformation du resultset en ResultatClient se fait dans un seul endroit...

Enfin, il vaudrait mieux utiliser request.setAttribute(...) plutôt que session.setAttribute(..). Le résultat de ta recherche est quelque chose de temporaire, que tu veux juste afficher dans la prochaine jsp. Pas besoins de rendre ce résultat disponible dans toute l'application.
13/06/2008, 18h10
Monkey_D.Luffy

Merci pour la suggestion. :)
Donc le bean ResultatClient devrait contenir un attribut de type ResultSet ?

Et aussi, un dernier petit truc qui m'embête : comment fermer la connexion ? Car j'ouvre une connexion dans la servlet. Ensuite après la redirection vers la JSP, la connexion doit rester ouverte (pour res.next()).

A moins que le bean soit en fait une arrayList ?
Encore merci.
13/06/2008, 18h18
supermanu

Tu peux montrer le code avant connexion.rechercheClient (pour voir de quel type est l'attribut connexion).

Non, il n'y a pas de type resultSet dans le bean.

L'idée c'est de transformer le resultset que tu obtiens en un simple bean java.

Si fonctionnellement ta méthode renvoi plusieurs clients, alors tu peux créer une ArrayList dans laquelle tu va mettre tes objets ResultatClient.
Essai de trouver comment parcourir un resultset ;)

Une fois que tu as le resultset, tu peux fermer la connexion. L'objet contient tout ce dont tu as besoins, il n'est plus nécessaire d'accéder à la base.
13/06/2008, 20h38
Monkey_D.Luffy

Ah merci, je vois.
Pour le parcours de la resultset, c'est pas compliqué. Pour ranger les résultats non plus, c'est pas compliqué.
Au pire j'ai un bean Client déjà défini, je peux faire alors une arrayList d'objets Client.

C'est vrai que c'est pratique car je peux fermer la connexion juste après le remplissage de l'arrayList.

Pour connexion, il est de type SqlConnection, une classe que j'ai écrite. Cette classe contient l'url de la BD SQL, le login et le password, ainsi qu'un statement et un preparedStatement. C'est cette classe qui fait toutes les requêtes sur la base SQL Server.

Merci beaucoup, j'ai maintenant une idée de comment améliorer mon code. :)
A+.