Discussion :

[flora806]

Bonjour,
j'ai un problème avec la syntaxe du return à la fin de ce code.Quand j'exécute mon programme j'ai une NullPointerException qui est dûe à la ligne du return.
j'ai l'impression que ça ne retourne pas vraimnt le prix de l'article en fonction des paramètres de la méthode

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
public int getPriceValue(int userID, int articleID){
      HashMap ArticlesUsers = getArticles(userID);
      if (ArticlesUsers != null && ArticlesUsers.get(articleID) != null){
  return ((dSet)ArticlesUsers.get(articleID)).getPriceValue(userID,articleID);
 
      }
      return ((dSet)ArticlesUsers.get(articleID)).getPriceValue(userID,articleID);
  }

Help!

[Duc Lebowski]

Je pense que ArticlesUsers.get(articleID) retourne null !

La question est : que retournes-tu quand ArticlesUsers.get(articleID) vaut null ?

Tu peux soit :

- retourner une valeur par défaut (ex: 0).
- lever une exception car ce n'est pas un comportement attendu.

[flora806]

j'avais une ligne aussi pour ce cas, je faisais "return 0".
pour, ArticlesUsers.get(articleID), j'ai fait un test en appelant cette méthode dans mon main et ça m'affiche bien des résultats

[Duc Lebowski]

Il y a quelque chose que je ne comprends pas :

Dans ((dSet)ArticlesUsers.get(articleID)).getPriceValue(userID,articleID), la méthode getPriceValue(userID,articleID) est la même que celle dont tu fournis le code source. La méthode s'appelle elle-même avec les mêmes paramètres ?

[flora806]

t'as bien raison, c'est la même méthode
ça paraît pas très cohérent, sinon je peux faire comment si je veux récupérer le PriceValue?

[Duc Lebowski]

En fait je ne vois pas trop bien ce que tu veux faire...

Pourquoi quand tu fais getArticles(userID) tu ne renvoies pas plutôt une Collection d'objets de type Article ayant un atribut price ?

Je suis peut-être à côté de la plaque mais tu pourrais dans ce cas faire quelque chose de simple comme :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
int total = 0;
Collection/*<Article>*/ articlesUser = getArticles(userID);
for (Iterator it=articlesUser.iterator(); it.hasNext();)
{
    Article article = (Article)it.next();
    total += article.getPrice();
}

Ou encore plus simple si tu es en Java 5 :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
int total = 0;
for (Article article : getArticles(userID))
    total += article.getPrice();

Quels types d'élements contient ta HashMap ArticlesUsers ? (clé/valeur)