Comportement étrange de ArrayList

Question

J'écris un site Web en utilisant JSP, JSTL, Servlets et JavaBeans. À un point de mon code, j'essaie d'utiliser une ArrayList d'objets, et une chose étrange se produit: quand j'ajoute le premier objet, c'est bien, et quand j'ajoute un second objet, il l'ajoute dans le deuxième place, mais l'objet à l'index (0) obtient les mêmes valeurs que l'objet à l'index (1).

Peut-être un problème est dans le

ArrayList<Article> articleList = new ArrayList<Article>(); 
Article newArticle = new Article(); 

Depuis articlelist est ArrayList de l'article classe. Est-ce que quelqu'un peut me signaler ce que je fais mal?

est mon code ci-dessous:

public ArrayList<Article> getArticles() 
{ 
    baseIO mySql = new baseIO(); 
    ArrayList<Article> articleList = new ArrayList<Article>(); 
    int articleId = 0; 

    try 
    { 
     String sql = 
      "select * from jsp_blog_article order by article_id Desc Limit 3"; 
     con = (Connection)mySql.getConnection(); 
     pstmt = (PreparedStatement) con.prepareStatement(sql); 
     ResultSet rs = pstmt.executeQuery(); 
     while (rs.next()) { 
      Article newArticle = new Article(); 
      newArticle.setArticleAuthor(rs.getString("article_name")); 
      newArticle.setArticleBody(rs.getString("article_body")); 
      newArticle.setArticleAuthor(rs.getString("article_author")); 
      newArticle.setArticleDate(rs.getString("article_date")); 
      articleId = Integer.parseInt(rs.getString("article_id")); 
      newArticle.setArticleId(String.valueOf(articleId)); 
      newArticle.setArticleComments(this.getCommentsNum(articleId)); 
      articleList.add(newArticle); 
     } 
     con.close(); 
     pstmt.close(); 
    } 
    catch(Exception e) 
    { 
     return null; 
    } 

    return articleList; 
} 

Et l'article classe

package objects; 

import java.io.Serializable; 

public class Article implements Serializable{ 
    private String articleName; 
    private String articleBody; 
    private String articleAuthor; 
    private String articleComments; 
    private String articleDate; 
    private String articleId; 

    public Article() 
    { 

    } 

    // all the getters and setters in place, but it is too long 
    // so i am not going to post them in forum 

} 

Answer 1

Je voudrais l'essayer de cette façon et voir ce que cela fait.

int x = 0; 
while (rs.next()) { 
    articleList.add(new Article()); 
    articleList.get(x).setArticleName(rs.getString("article_name")); 
    articleList.get(x).setArticleBody(rs.getString("article_body")); 
    articleList.get(x).setArticleAuthor(rs.getString("article_author")); 
    articleList.get(x).setArticleDate(rs.getString("article_date")); 
    articleList.get(x).setArticleId(rs.getString("article_id")); 
    articleList.get(x).setArticleComments(this.getCommentsNum(articleId)); 
    x++; 
} 

Answer 2

Vous appelez newArticle.setArticleAuthor deux fois ... Je sais que ce ne fait pas partie de votre problème de liste, mais est une remarque.

Answer 3

Le code a l'air bien, comment affichez-vous la liste qui vous fait penser que la même valeur est aux deux index? Votre problème est peut-être lié à ce code.

Answer 4

Ajoutez-vous des articles à la base de données en même temps que vous les lisez? Je pense qu'il est possible, en fonction de votre moteur de stockage, que vous ayez des problèmes de lecture pendant la mise à jour.

Answer 5

Le code doit être nettoyé par les autres commentaires, mais fonctionnellement, cela devrait fonctionner.

Voici ce que je pense qui se passe.

Votre code a les deux lignes suivantes dans ce:

newArticle.setArticleAuthor(rs.getString("article_name")); 
newArticle.setArticleAuthor(rs.getString("article_author")); 

et il n'y a pas d'appel correspondant à:

newArticle.setArticleName(rs.getString("article_name")); 

cela signifie que votre objet n'a pas de nom de l'article spécifié (même si le auteur est spécifié). Je parie que vous êtes en train de faire une sorte de traitement avant de afficher la liste qui fusionne en quelque sorte les articles avec le même nom. Comme approche générale du débogage, je vous recommande de simuler votre code pour pouvoir l'exécuter dans un débogueur et voir ce qui se passe réellement (en ce moment, votre système a tellement de pièces mobiles que ça va être difficile pour que vous puissiez vous concentrer sur le problème réel).

Dans le cas présent, ce serait aussi simple que d'exécuter la méthode one en dehors de votre conteneur Web et d'utiliser un débogueur pour examiner les objets de la liste qui sont retournés. Vous constaterez que les objets dans la liste sont, en effet, des objets séparés - ayant juste la même propriété articleName.