!!Variabile d'istanza SET!!

[Klibion]

Ho provato con questa soluzione:

public class MultiInsieme {

ArrayList<MultiElemento>set;
MultiElemento x;

public MultiInsieme (ArrayList<MultiElemento>set){

this.set=set;
set = new ArrayList<MultiElemento>();
}

public void getRipetizioni (double valore){

for (int i=0; i<set.size(); i++){
if (i==x.valore){
x.ripetizioni++;
}
}
}


public void addElemento ( double valore, int ripetizioni ){

for (int i=0; i<set.size(); i++){
if (i==x.valore){
x.ripetizioni++;
}
set.add(x);
}
}
}

[andbin]

public class MultiElemento {
       
    final double valore=3.14;
    int ripetizioni=5;

Il testo iniziale dice espressamente "un costruttore che crea un MultiElemento dati valore e ripetizioni". Quindi quale è il senso di questi dati schiantati così a caso?

Ho provato con questa soluzione:

Innanzitutto:

private ArrayList<MultiElemento> set;

Poi quel MultiElemento x; ti assicuro che NON ti serve.

    public MultiInsieme (ArrayList<MultiElemento>set){
        
        this.set=set;
        set = new ArrayList<MultiElemento>();
    }

Il testo dice "La classe deve avere: · un costruttore che crea un MultiInsieme vuoto".
Che senso ha ricevere come argomento una lista, assegnarla a set e poi subito dopo assegnare di nuovo a set un ArrayList vuoto? NON ha senso.

     public void getRipetizioni (double valore){
         
         for (int i=0; i<set.size(); i++){
             if (i==x.valore){
                x.ripetizioni++;
             }
         }
     }

getRipetizioni che cosa ti fa pensare? Che dovrebbe RESTITUIRE il numero di ripetizioni. E allora che senso ha il void come tipo di ritorno? E poi anche l'incremento di ripetizioni su un oggetto??

E poi: perché confronti un indice i con un valore double? Sono tutte cose che NON hanno alcun senso.

    public void addElemento ( double valore, int ripetizioni ){
        
        for (int i=0; i<set.size(); i++){
             if (i==x.valore){
                x.ripetizioni++;
             }
             set.add(x);
         }
    }

Il testo dice:

un metodo addElemento (double valore, int ripetizioni) che aggiunge al multiInsieme l'elemento valore con il numero di ripetizioni indicato. Attenzione: se nel multinsieme c'è già un elemento con lo stesso valore, viene solo aggiornato il suo campo ripetizioni, altrimenti viene aggiunto all'arraylist un nuovo MultiElemento

Questo vuol dire una cosa abbastanza semplice: ricevo un valore, vado a cercare questo valore in tutti gli oggetti MultiElemento presenti nel ArrayList (che l'oggetto MultiInsieme mantiene al suo interno). Se lo trovo, gli aggiungo semplicemente il numero di ripetizioni. Se alla FINE di tutto il ciclo non lo trovo, allora inserisco un nuovo oggetto MultiElemento.
Ti sembra più chiaro?



P.S. dovresti sforzarti di fare qualche ragionamento in più.

[Klibion]

Quando il testo mi chiedeva di creare un MultiInsieme vuoto, pensavo che mi dicesse di creare un ArrayList MultiInsieme senza elementi al suo interno. Avevo capito che questo ArrayList rappresentasse il MultiInsieme e che dovessi inserire i MultiElementi al suo interno. Inoltre non ho capito se le variabili sono le stesse per entrambe le classi e se le dovrei collegare...