[WINNER INSIDE] Je suis fort en algo que j'arrive à planter ma bécane

Je suis fort en algo que j'arrive à planter ma bécane [WINNER INSIDE] - Java - Programmation

Marsh Posté le 07-05-2004 à 17:23:30    

Yop !
 
Je suis en train de créer un forum en Java.
 
Le principe est simple. Il y a des messages "parents" et des messages "enfants" (c-a-d des réponses aux messages parents).
 
Et j'ai réussi à le coder d'une façon tellement merdique que j'ai un joli plantage d'Eclipse et une petite fenêtre "out of memory" qui s'affiche.
 
J'ai une page jsp toute conne :
 

Code :
  1. <%@page import="miage.bean.*"%>
  2. <%@page import="java.util.*"%>
  4. <% Vector vec = Message.getMessages(); %>
  6. <table>
  8. <% for (int i = 0; i < vec.size(); i++) { %>
  10. <tr>
  11.  <td>
  12.   <%
  13.    Message msg = (Message) vec.get(i);
  14.    out.println(msg.getTitre());
  15.   %>
  16.  </td>
  17. </tr>
  19. <%}%>
  21. </table>


 
Et voilà mon bean :
 

Code :
  1. /*
  2. * Created on 7 mai 2004
  3. *
  4. */
  5. package miage.bean;
  7. import java.sql.*;
  8. import java.util.*;
  9. import javax.naming.*;
  10. import javax.sql.DataSource;
  12. /**
  13. * @author eric.ledonge@free.fr
  14. *
  15. */
  16. public class Message implements java.io.Serializable {
  18. private int identifiant;
  19. private int identifiantPere;
  20. private String visiteur;
  21. private String date;
  22. private String titre;
  23. private String corps;
  25. private static final String select = "SELECT identifiant, identifiantPere, visiteur, date, titre, corps  FROM MESSAGE";
  27. public Message() {
  28. }
  30. public Message(ResultSet rs) throws SQLException {
  31.  init(rs);
  32. }
  34. public void init(ResultSet rs) throws SQLException {
  35.  this.identifiant = rs.getInt("identifiant" );
  36.  this.identifiantPere = rs.getInt("identifiantPere" );
  37.  this.visiteur = rs.getString("visiteur" );
  38.  this.date = rs.getString("date" );
  39.  this.visiteur = rs.getString("titre" );
  40.  this.corps = rs.getString("corps" );
  41. }
  43. static Connection getConnection() {
  44.  try {
  45.   Context ctx = new InitialContext();
  46.   if(ctx == null ) throw new Exception("Erreur : pas de context" );
  47.   System.out.println(ctx);
  48.   DataSource ds = (DataSource)ctx.lookup("java:comp/env/jdbc/forum" );
  49.   return ds.getConnection();
  50.  }
  51.  catch (Exception e) {
  52.   System.out.println("Erreur : " + e);
  53.   return null;
  54.  }
  55. }
  57. public static Vector getMessages() throws SQLException {
  58.  Connection con = null;
  59.  Statement stm = null;
  60.  ResultSet rs = null;
  61.  Vector vec = new Vector();
  62.  try {
  63.   con = getConnection();
  64.   stm = con.createStatement();
  65.   rs = stm.executeQuery(select);
  66.   while (rs.next()) {
  67.    // Je récupère un message de la base de données
  68.    Message msg = new Message(rs);
  69.    // Je vérifie l'indentifiant père du message afin de savoir si c'est un message racine
  70.    if (msg.getIdentifiantPere() == 0) {
  71.     // Si oui, j'ajoute ce message à la fin du vecteur
  72.     vec.add(msg);
  73.     System.out.println(msg);
  74.    }
  75.    else {
  76.     // Sinon, je recherche la position du père successeur de son père dans le vecteur
  77.     boolean trouve = false;
  78.     int cpt = 0;
  79.     Enumeration en = vec.elements();
  80.     while (en.hasMoreElements() || trouve == false) {
  81.      Message buf = (Message) en.nextElement();
  82.      if ((buf.getIdentifiant() + 1) == msg.getIdentifiantPere()) {
  83.       vec.add(cpt, msg);
  84.       System.out.println(msg);
  85.      }
  86.      else {
  87.       vec.add(msg);
  88.       System.out.println(msg);
  89.      }
  90.      cpt++;
  91.     }
  92.    }
  94.   }
  95.  }
  96.  finally {
  97.   if (rs != null) rs.close();
  98.   if (stm != null) stm.close();
  99.   if (con != null) con.close();
  100.  }
  101.  return vec;
  102. }
  105. public int getIdentifiantPere() {
  106.  return this.identifiantPere;
  107. }
  109. public void setIdentifiantPere(int identifiantPere) {
  110.  this.identifiantPere = identifiantPere;
  111. }
  113. public int getIdentifiant() {
  114.  return this.identifiant;
  115. }
  117. public void setIdentifiant(int identifiant) {
  118.  this.identifiant = identifiant;
  119. }
  121. public String getTitre() {
  122.  return this.titre;
  123. }
  125. }


 
Quelqu'un aurait une petite idée ?

Reply

Marsh Posté le 07-05-2004 à 17:23:30   

Reply

Marsh Posté le 07-05-2004 à 17:46:41    

Ouais, Message.getMessages() charge tous les messages de ton forum ?
C'est pas un peu complètement bourrin ?

Reply

Marsh Posté le 07-05-2004 à 17:53:59    

Peut-être que oui en effet :pt1cable:  
 
Ce code est en fait issu d'un remodelage personnel d'un exemple se trouvant dans le bouquin Tomcat par la pratique chez Eyrolles.
 
L'exemple fonctionnait bien, j'ai donc pensé le réadapter à mon pb.
 
Mais comment puis-je alors m'y prendre pour faire le traitement qui consiste à afficher dans le bon ordre les messages racineset leurs réponses ?
 
C'est dommage que ça bouffe autant de mémoire car je trouvais cette méthode pas mal :)

Reply

Marsh Posté le 09-05-2004 à 21:15:00    

tu vas me faire le plaisir de virer ton code d'acces aux données en dehors du bean [:kiki]


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

Marsh Posté le 09-05-2004 à 21:16:11    

je parie sur un stack overflow typique du récursif

Reply

Marsh Posté le 10-05-2004 à 14:45:06    

C'est quoi un bean?
Je veux dire, pour de vrai. Parce que tout ce que j'ai lu c'était imcompréhensible. Pas la peine d'implémenter ou de dériver quoi que ce soit?
 
Suffi de faire n'importe quoi en JAVA et de dire "Mais qu'est-ce que c'est que ce bean ? "

Reply

Marsh Posté le 11-05-2004 à 09:05:00    

Citation :

<% Vector vec = Message.getMessages(); %>


Les vector c mal


Message édité par meumeul le 11-05-2004 à 09:05:23
Reply

Marsh Posté le 11-05-2004 à 09:05:35    

meumeul a écrit :

Citation :

<% Vector vec = Message.getMessages(); %>


Les vector c mal


:??: explique...

Reply

Marsh Posté le 11-05-2004 à 11:32:33    

JagStang a écrit :

:??: explique...

c'est un moyen courant de se la péter dans le milieu des semi-noobs qui viennent d'apprendre qu'il vaut mieux utiliser un ArrayList qui est un vector non synchronizés, meme milieu qui croit que la synchronization tue les performances.
Jusqu'a ce qu'ils se rendent compte qu'ils gagneraient autant à utiliser LinkedList dans les 3/4 des cas :ange:

Reply

Marsh Posté le 11-05-2004 à 11:41:38    

the real moins moins a écrit :

c'est un moyen courant de se la péter dans le milieu des semi-noobs qui viennent d'apprendre qu'il vaut mieux utiliser un ArrayList qui est un vector non synchronizés, meme milieu qui croit que la synchronization tue les performances.
Jusqu'a ce qu'ils se rendent compte qu'ils gagneraient autant à utiliser LinkedList dans les 3/4 des cas :ange:


 
LinkedList et ArrayList c'est pas du tout la même chose : le choix est purement algorithmique, on voit ça sur les bans de la fac ou autre (cout insertion/deletion, etc.), aucune raison qu'on fasse chier qqun avec ça ici?
 
Synchronized ou non, c'est une question technique sur le langage, c'est normal que l'on en parle plus souvent...  
 
De toute façon, les 3/4 du temps, la solution la plus performante reste les tableaux.

Reply

Marsh Posté le 11-05-2004 à 11:41:38   

Reply

Marsh Posté le 11-05-2004 à 11:48:54    

seabee a écrit :



oui oui, je deconnais :o
(cela dit, il existe bcp de code ou le choix n'est pas fait entre LinkedList et ArrayList, et ArrayList est utilisé "à priori", du moins dans le code que je vois tous les jours c'est comme ça :D)

Reply

Marsh Posté le 11-05-2004 à 11:53:55    

the real moins moins a écrit :

oui oui, je deconnais :o
(cela dit, il existe bcp de code ou le choix n'est pas fait entre LinkedList et ArrayList, et ArrayList est utilisé "à priori", du moins dans le code que je vois tous les jours c'est comme ça :D)


 
pov meumeul, il risque de mal le prendre et on va plus jamais le revoir de par chez nous  :(  :D

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed