Java

Un code plus compact, utilisation d'interfaces (surement)

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Un code plus compact, utilisation d'interfaces (surement)

El_gringo

Bon, allez, j'vais essayer d'expliquer mon pb.
J'ai une quizaine de tables SQL.
Dans chacune, je vais récupérer un ou une collection de bean que g créé, qui correspondent à peu près aux colones de chaque tables.
J'ai une classe JSQLRequest qui est chargée de faire toutes les requêtes.
Cette classe à toute sorte de méthode de ce genre:

Code :

/** Récupère toutes les applications autorisées à la recherche pour une utilisateur donné.
* @param login login de l'utilisateur (code utilisateur) dont les données sont récupérées.
* @throws JLdsServletException Lancée si une erreur SQL est produite.
* @return une énumération (<CODE>Enumeration</CODE> ) de beans <CODE>Application</CODE>
*/
public Enumeration getAllAutorizedApplications (String login) throws JLdsServletException {
if (login == null)
return null;
Object[] arguments = {login};
String formatStr = Resources.getResources ().getLabel ("sqlrequest.getallautorizedappli" );
String sqlQuery = MessageFormat.format(formatStr, arguments);
Connection c = null;
Statement stmt = null;
ResultSet r = null;
try {
c = this.connectionPool.getConnection ();
stmt = c.createStatement ();
r = stmt.executeQuery (sqlQuery);
Vector vectAutorizedApps = new Vector ();
while (r.next ()) {
String codeApp = r.getString (1).trim ();
String nomApp = r.getString (2).trim ();
vectAutorizedApps.add (new Application (codeApp, nomApp));
}
return (vectAutorizedApps.elements ());
}
catch (SQLException e) {
throw new JLdsServletException ("Une erreur est survenue lors de l'execution/lecture des résultats, de la requète :\n\t" + sqlQuery, e);
}
finally {
try { if (r != null) r.close(); } catch (SQLException x) { throw new JLdsServletException ("Erreur de fermeture du ResultSet ( " + sqlQuery, x); }
try { if (stmt != null) stmt.close(); } catch (SQLException x) { throw new JLdsServletException ("Erreur de fermeture du Ctatement ( " + sqlQuery, x); }
try { if (c != null) c.close(); } catch (SQLException x) { throw new JLdsServletException ("Erreur de libération de la connection ( " + sqlQuery, x); }
}
}

Où seule les passages en gras diffèrent. vous voyez pas un moyen de ne pas réécrire tout ça pour chaque récupération de beans à partir de ma BD ?

Message édité par El_gringo le 03-07-2002 à 11:27:40

Publicité

El_gringo

Ouuu que c'est moche ici...
Déja, mon code est pas super beau avec mon éditeur, ma alors, posté ici, c une catastrophe.
Enfin, si vous arrivez qd même à lire, et que ce code vous énerve pas trop, j'attend vos suggestions...

--greg--

bah y'a pas gd chose qui se repeterais, si ce n'est la repetition de le declaration du ResultSet, Connection etc, les catch, le finally. mais ça tu y couperas pas de toutes façons.
tu pourrais betement foutre ton code d'execution de requete dans une methode.

El_gringo

--greg-- a écrit a écrit :

bah y'a pas gd chose qui se repeterais, si ce n'est la repetition de le declaration du ResultSet, Connection etc, les catch, le finally. mais ça tu y couperas pas de toutes façons.
tu pourrais betement foutre ton code d'execution de requete dans une methode.

...Qui rendrait un ResultSet ?
Mais, de toute façon, je pourrais pas inclure le try - catch - finally dedant. ça va pas réduire grand chose en fait.
J'viens d'avoir une idée qui me parait bien :
Chacun de mes beans va implémenter une interface JLdsBean, qui imposera la méthode

Code :

public void setFromResultSet (ResultSet rs)

A ce moment là, je pourrait faire, dans JSQLRequest, une méthode

Code :

public JLdsBean getOneBeanFromDB (JLdsBeans bean, String sqlQuery)

qui ressemblerai à celle que g poste avant, mais ou la 2e partie en gras serait remplacée par un appel à la méthode setFromResultSet.
c pas mal, non !?

Message édité par El_gringo le 03-07-2002 à 11:49:59

darklord

You're welcome

un bean qui prend un resultset beurk.

Par contre un bean qui a des méthodes load, save, update ca c'est cool. Et tu peux faire appel à une classe d'utilistaires SQL pour la connection et les autre brols.

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

El_gringo

DarkLord a écrit a écrit :

un bean qui prend un resultset beurk.

Par contre un bean qui a des méthodes load, save, update ca c'est cool. Et tu peux faire appel à une classe d'utilistaires SQL pour la connection et les autre brols.

Et !? tu mettrais quoi comme paramètre de load ? c cette méthode qui serait chargée des accès BD !? j'croyais qu'y fallait pas faire ça non plus ds un bean.
Tu peut détailler un peu tt ça ?
Et une classe d'utilitaires SQL, genre !? j'vois pas bien ce qu'elle pourrait m'apporter comme utilitaire...

darklord

You're welcome

bin normallement un bean contient toute les infos sur ce que tu veux décrire (un client, un colis, une société). Certains mettent également les méthodes d'accès au bean lui permettant de s'updater, se loader tout seul dans la bd.

Comme tout bean a un constructeur vide tu peux faire

Code :

ClientBean client = new ClientBean();
client.load(124644567);
// où 124644567 est la clé primaire permettant de retrouver le client
System.out.println("Loaded: "+client.getName());

Et pour la classe d'utilitaires c'est par exemple une classe qui te donnerait une instance de Connection sur laquelle tu pourrais déclarer ton statemment etc ...

Ainsi tu centralies l'information propre au bean dans le bean. Ceci n'est pas vraiment valalbe si tu veux commencer à sérialiser tes beans (auquel cas le bean doit rester purement un objet contennat des données). Dans ce cas tu dois faire appel à une classe de type helper (ClientBeanHelper) qui fera le job pour toi.

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

El_gringo

DarkLord a écrit a écrit :

bin normallement un bean contient toute les infos sur ce que tu veux décrire (un client, un colis, une société). Certains mettent également les méthodes d'accès au bean lui permettant de s'updater, se loader tout seul dans la bd.

Comme tout bean a un constructeur vide tu peux faire

Code :

ClientBean client = new ClientBean();
client.load(124644567);
// où 124644567 est la clé primaire permettant de retrouver le client
System.out.println("Loaded: "+client.getName());

haaa... là t chiant ! Au début, j'avais fait un truc comme ça (méthode load) et tu m'as dit que, non, c'est mauvais, et que je devais pas faire ça. Du coup, j't'ai écouté, g recommencé en sortant les accès BD du bean. Et maintenant tu me dis que c pas bien grave !?

darklord

You're welcome

j'ai dis ca dépend ce que tu veux faire. si tu as présenté ton projet comme utilisant la serialization ou autre, c'est normal que je te l'ai déconseillé. Je connais pas ton projet moi.

Et puis si je suis chiant, je ne te réponds plus comme ca le problème est résolu :kaola:

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

El_gringo

DarkLord a écrit a écrit :

j'ai dis ca dépend ce que tu veux faire. si tu as présenté ton projet comme utilisant la serialization ou autre, c'est normal que je te l'ai déconseillé. Je connais pas ton projet moi.

Et puis si je suis chiant, je ne te réponds plus comme ca le problème est résolu :kaola:

Nooooon, 'faut pas te vexer Darky... mais sur ce coup là, j'étais pas content, comprend moi. :sweat:
Mais 'faut pas que tu m'laisses, j'ai besoin de toi moi.

Publicité

--greg--

:bounce: spéciale dédicace à Darklord

FORUM HardWare.fr

Programmation

Java

Un code plus compact, utilisation d'interfaces (surement)

Sujets relatifs
Optimiser le code	[Delphi] Marquer un temps d'attente dans le code !
[Delphi] Utilisation de ShellExecute !	[VBA] Excel, créer une feuille à la volée et lui affecter du code
exist'il un code speciale pour afficher le sigle euros en HTML??	Utilisation de "SecurityManager"
Probleme de code !	[MUTEX] difference entre ce bout de code et un mutex
Urgent (Aide sur utilisation de PWS)	[Java] utilisation de thread
Plus de sujets relatifs à : Un code plus compact, utilisation d'interfaces (surement)

Page générée en 0.063 secondes