Salut

Je cherche à ajouter dans mon hashset, un objet en l'occurrence de type "penny" ?
Pour cela j'ai taper dans mon constructeur ces deux lignes :

  HashSet hashSetPenny = new HashSet();
  hashSetPenny.add(new Penny());

qui devraient initialiser un objet penny à mon hashset

Je rencontre aussi une erreur dans ma méthode pennyCount qui me dit qu'il n'y a pas d'objet initialisé (ce que j'ai initialisé dans mon constructeur)...

Merci

 
import java.util.HashSet;
import java.util.ArrayList;
import java.util.Iterator;

public class Pocket
{
    int sizeOfPocket;
    private HashSet<Penny> hashSetPenny;
    
    public Pocket(int sizeOfPocket)
    {   
        for(int counter = 0; counter < sizeOfPocket ; counter++ )
        {
            this.sizeOfPocket = sizeOfPocket;
            HashSet hashSetPenny = new HashSet();
            hashSetPenny.add(new Penny());
        } 
    }
    

    public int pennyCount()
    {
        return hashSetPenny.size();
    }
    
    public Penny removePenny(Penny penny)
    {
        Iterator<Penny> it = new Iterator<Penny>();
        Iterator <Penny> it = hashSetPenny.iterator();
        if (it.hasNext())
        {
          hashSetPenny.remove(penny);
        }
        else
        {
          return null;
        }
        return penny;
}
}

J'y connais pas grand chose en Java, mais à ce que j'ai vu, ta classe a un attribut privé
private HashSet<Penny> hashSetPenny;
Alors que dans le constructeur, tu as
HashSet hashSetPenny = new HashSet();
Que j'aurais à ta place essayé de remplacer en
hashSetPenny = new HashSet();
ou
this.hashSetPenny = new HashSet();

Il y a ce qu'a dit elmohe, et il faut aussi initialiser ton hashset en dehors de ta boucle for

Oui, pourquoi initialiser sizeOfPocket fois ?

Merci de vos réponses

J'ai remplacé mon code pour le constructeur par cela mais lorsque j'appelle la méthode pennyCount, ce qui est retourné est seulement 1 (lorsque je lui demande pour sizeOfPocket = 10 dans les paramètres)...
En l'occurrence je devrai avoir 10...

merci

mon code :

import java.util.HashSet;
import java.util.ArrayList;
import java.util.Iterator;

public class Pocket
{
    int sizeOfPocket;
    private HashSet<Penny> hashSetPenny;
    
    public Pocket(int sizeOfPocket)
    {   
        hashSetPenny = new HashSet();
        
        for(int counter = 0; counter < sizeOfPocket ; counter++ )
        {
            this.sizeOfPocket = sizeOfPocket;
            hashSetPenny = new HashSet();
            hashSetPenny.add(new Penny());
        } 
    }
    

    public int pennyCount()
    {
        return hashSetPenny.size();
    }
    

import java.util.HashSet;
import java.util.ArrayList;
import java.util.Iterator;

public class Pocket
{
    int sizeOfPocket;
    private HashSet<Penny> hashSetPenny;
    
    public Pocket(int sizeOfPocket)
    {   
        hashSetPenny = new HashSet();
        
        for(int counter = 0; counter < sizeOfPocket ; counter++ )
        {
            this.sizeOfPocket = sizeOfPocket;
            hashSetPenny = new HashSet(); <-------- Vire cette ligne
            hashSetPenny.add(new Penny());
        } 
    }
    

    public int pennyCount()
    {
        return hashSetPenny.size();
    }

En fait cette ligne fait que tu prend hashSetPenny, et tu le remet à zéro, à chaque tour de boucle.
En dehors de ta for, juste au-dessus, tu l'as déjà initialisé. Donc en enlevant cette ligne, tout devrait marcher

merci l'ami

Sur ma méthode removePenny() , j'ai essayer de faire fonctionner mon iterator mais je suis sur que je ne suis pas loin du tout !

    public Penny removePenny(Penny penny)
    {
        Iterator<Penny> iterator = hashSetPenny.iterator();
        if (iterator.hasNext()) {
            
            hashSetPenny.remove(penny);
            pennyCount(); 
            //return hashSetPenny.size();            
        }
        else if (!iterator.hasNext())
        {
            return null;
        }
        
        return penny;
    }

Lorsque j'appelle cette méthode et que je fais passer comme paramètre un penny objet, cela me renvoie seulement le penny objet et aucun penny objet est enlevé...

Merci

Je comprends pas trop ce que tu veux faire... Si c'est juste une fonction pour enlever un objet de la liste tu peux pas juste hashSetPenny.remove(penny) quand tu en as besoin en testant si penny est dans la liste ?

Oui c'est juste une méthode qui sert à enlever un objet de mon hashSetPenny, mais en l'occurrence c'est ce que je fais (mis à part si je ne comprends pas ce que fait un interator) avec mon iterator non ?

Oui mais justement je me demandais (après je connais pas trop le java), si un iterator sert vraiment à quelque chose quoi.
Si c'est ça la cause du problème alors que tu pouvais faire plus simple... (Je me trompe peut-être)

On m'a conseillé un iterator pour faire cela...
Si ce n'est pas un iterator comment aurai tu procéder ?
merci

Bah je sais pas, tout simplement utiliser la fonction pour retirer l'objet de la liste ?

Oui je cherche à créer une méthode qui retire cet objet de la liste, elle n'est pas créer

Dans la class Penny redéfinie les méthodes hashCode() et equals()

Je cherche à revoir la méthode removePenny() en utilisant un iterator (je veux absolument utiliser un iterator)

public Penny removePenny(Penny penny) {
        Iterator<Penny> iterator = hashSetPenny.iterator();

        if (iterator.hasNext()) {
            Penny iterPenny = iterator.next();

            if (iterPenny.equals(penny) { // méthode equals() à redéfinir
                iterator.remove(); // Suppression de la dernière valeur retournée par next()
                pennyCount();
                return iterPenny;         
            }
            return null;
        }
}

Tout ça peut être réduit à :
hashSetPenny.removeIf(item -> item.equals(penny));