broke31 ha scritto:
for(int i = 0;i<app.getPannello().getComponentCount();i++){
if(((JCheckBox)app.getPannello().getComponent(i)).isSelected()){
appoggio.mettiNelCarrello(offerteAttive.get(i));
// app.getPannello().remove(i);
}
}
Pensaci un attimo: se rimuovi il primo, il secondo diventa il primo ....
Comunque:
- Un accesso del genere con getComponent() ha senso solo se gli unici componenti sono appunto quei JCheckBox (immagino sia così, quindi ok). Comunque sarebbe meglio (e più pulito) tenersi un array/List di JCheckBox.
- Quando si fanno invocazioni in cascata del tipo x.getAaa().getBbb() ...... molte volte (non sempre, ovvio) è un sintomo che il design delle classi fa un po' "acqua".