!!Variabile d'istanza SET!!

[Klibion]

ho provato così, ma ho un problema con il metodo GetRipetizioni, non riesco a ritornare un valore int...


import java.util.ArrayList;


public class MultiInsieme {


private ArrayList<MultiElemento>set;



public MultiInsieme (){
new ArrayList<MultiInsieme>();
}

public int getRipetizioni (double valore){

for(MultiElemento elemento:set){
if(valore!=elemento.getValore()){
return 0;
} else {
return elemento.getRipetizioni();
}
}
return
}

public void addElemento (double valore,int ripetizioni){
for (MultiElemento elemento:set){
if (valore==elemento.getValore()){
elemento.setRipetizioni(ripetizioni+1);
}
set.add(elemento);
}
}
}

[andbin]

ho provato così, ma ho un problema con il metodo GetRipetizioni, non riesco a ritornare un valore int...

Ora il problema è una questione di "concetto". Quando è che puoi restituire 0 per indicare che NON hai trovato il valore? Pensa: puoi farlo DENTRO il for oppure solo DOPO una volta terminato il for?


addElemento è ancora sbagliato, per almeno 3 punti:
- Quando trovi quel valore, se l'oggetto MultiElemento contiene 5 ripetizioni e il ripetizioni passato in argomento vale 6, tu dovresti in quello STESSO oggetto MultiElemento mettere il valore di 11 ripetizioni (5+6). È questo che stai facendo? No.
- Se trovi quel valore, a parte aggiornare le ripetizioni nel MultiElemento, NON devi fare altro. Non devi continuare con i cicli, non devi aggiungere nulla. In sostanza dovresti far ritornare SUBITO il metodo.
- Se NON trovi quel valore (e qui ripeto come all'inizio: puoi farlo DENTRO il for oppure solo DOPO terminato il for?), dovresti inserire un NUOVO oggetto MultiElemento.


Ribadisco quanto avevo già detto in precedenza: bisogna RAGIONARE, è questo che serve nella programmazione