Mostra per cortesia le due classi Prodotto/ProdottoScontato ... a "naso" credo ci sia qualche questione lì.
Questo tecnicamente è corretto. Ti dò solo 2 dritte:
- innanzitutto se hai pochi elementi non fa molta differenza ma se ne avessi tantissimi (migliaia ecc..), buttarli in un TreeSet solo per ordinarli è molto meno efficiente che clonare una lista ed ordinarla con il sort().
- se non vuoi fornire all'esterno del metodo la informazione che sotto-sotto hai una collezione ordinata, invece che Iterator puoi almeno indicare Iterable (i Set e i List sono Iterable)
public Iterable<Prodotto> getandOrderProdotti() {
In questo modo può essere il "target" del for-each e invece di usare l'iteratore esplicitamente, puoi fare:
for (Prodotto prodotto : fatt.getandOrderProdotti()) { .... }
bello no?![]()