[JAVA] Ordinare ArrayList di oggetti diversi.

[andbin]

Rispondo in ordine sparso.

5)Tutto ciò però non mi spiega perchè mi ordina i ProdottoScontato in ordine crescente ed i Prodotto in ordine decrescente, sempre per prezzo..

Certo che lo spiega!! È proprio quella formula (sbagliata) che causa questo comportamento particolare ma improprio.

sconto * 100 / super.getPrezzo()

Essendo il prezzo originale al denominatore di una divisione, a parità di sconto, più il prezzo è grande, più il risultato è piccolo. Ed ecco il senso inverso rispetto al concetto espresso (giustamente) dal Comparatore.

2) si hai ragione non avevo notato che eliminasse quelli con lo stesso prezzo, teoricamente doveva eilimanre quelli con lo stesso codice, non avevo pensato che lui eilima quei elementi per i quali il comparator ritorna 0.

Sì, con TreeSet il senso del "senza duplicati" è sempre relativo al criterio di comparazione con Comparable oppure Comparator.

1) Non risponderò alle tue segnalazioni non perchè non siano giuste ma perchè fuorvianti rispetto alla domanda principale a cui non siamo ancora riusciti a dare una risposta. Non voglio giustificare le mie scelte, se è per questo ci sono troppe eccezioni, l'input doveva essere da un'altra parte ecc ecc, ma per ora fa nulla.

Per carità ... non ho nulla da obiettare se non accetti/gradisci dei consigli che potrebbero aiutarti a migliorare molto il tuo codice.
Presumo che alla fine riceverai un "voto" per l'esercizio. Il voto è tuo ... non mio. Se l'esercizio fosse per me, il codice sarebbe ragionevolmente impeccabile e il voto piuttosto alto.

Posso solo augurarti di ricevere il voto da qualcuno o non troppo competente o di manica molto larga ..

[Dario95]

Rispondo in ordine sparso.


Certo che lo spiega!! È proprio quella formula (sbagliata) che causa questo comportamento particolare ma improprio.

sconto * 100 / super.getPrezzo()

Essendo il prezzo originale al denominatore di una divisione, a parità di sconto, più il prezzo è grande, più il risultato è piccolo. Ed ecco il senso inverso rispetto al concetto espresso (giustamente) dal Comparatore.



Sì, con TreeSet il senso del "senza duplicati" è sempre relativo al criterio di comparazione con Comparable oppure Comparator.


Per carità ... non ho nulla da obiettare se non accetti/gradisci dei consigli che potrebbero aiutarti a migliorare molto il tuo codice.
Presumo che alla fine riceverai un "voto" per l'esercizio. Il voto è tuo ... non mio. Se l'esercizio fosse per me, il codice sarebbe ragionevolmente impeccabile e il voto piuttosto alto.

Posso solo augurarti di ricevere il voto da qualcuno o non troppo competente o di manica molto larga ..

1)Ho solo detto che i tuoi commenti se pur giusti erano off-topic, tutto qui.
2)Non ho mai detto di essere un genio della programmazione ma non capisco perchè devi offendere, sopratutto visto che neanche tu sai rispondere alla domanda e risolvere il problema.
3)Dopo 16 anni di programmazione sarebbe decisamente strano se tu non scrivessi codice migliore del mio..
3)Voglio proprio vedere se i tuoi codici sono impeccabili...
4)Se ti va di darmi una mano fai pure, se devi continuare invece solo a fare il 'professore' astieniti...
5)

Essendo il prezzo originale al denominatore di una divisione, a parità di sconto, più il prezzo è grande, più il risultato è piccolo. Ed ecco il senso inverso rispetto al concetto espresso (giustamente) dal Comparatore.

Scusami non ne capisco il senso, cioè si la formula è sbagliata e quello che dici tu è giusto, ma quello che conta è il risultato, cioè anche se a parità di sconto il risultato era più piccolo rispetto a quanto grande fosse il prezzo, alla fien cmq quella formula restituiva un double che il comparatore doveva confrontare con gli altri, quindi perchè me li ordinava in senso opposto? Attenzione non sto dicendo che quello che dici è sbagliato, sto chiedendo di spiegarmelo.. e cmq ho corretto il calcolo della percentuale e come vedi continua a non confrontare i Prodotto con i ProdottoScontato.

3  Prod 10.0
1  Prod 4.0
14  Prod 3.0
27  Prod 2.0
18  ProdScontato 9.0
21  ProdScontato 7.0
15  ProdScontato 7.0
22  ProdScontato 5.0
30  ProdScontato 4.0
12  ProdScontato 2.0

[andbin]

ma non capisco perchè devi offendere

Scusa ma di quale offesa parli? Ti auguro (e ribadisco, in maniera buona e sincera) di ottenere un (buon!) voto da qualcuno che non sia così attento e "pignolo" come invece lo sono io.
Se ti avessi augurato il contrario ... allora sì, sicuramente sarebbe stato scortese.

3)Voglio proprio vedere se i tuoi codici sono impeccabili...

Infatti ho detto ragionevolmente impeccabile. Io non ho alcuna presunzione/pretesa di sapere tutto o fare tutto perfettamente.
Ma sono molto preciso, accurato e scrivo codice meglio di tanti altri, ad esempio di certi tizi che ho conosciuto per lavoro che si "vantavano" di lavorare su progetti di alto livello internazionale ma poi quando si trovavano davanti ad una tastiera... eh ... beh, te lo lascio immaginare.

Attenzione non sto dicendo che quello che dici è sbagliato, sto chiedendo di spiegarmelo.. e cmq ho corretto il calcolo della percentuale e come vedi continua a non confrontare i Prodotto con i ProdottoScontato.

Allora ti dò un ultimo indizio ... poi non ti "scoccio" più.
Il toString() in Prodotto stampa il 'prezzo', quello originale, NON scontato, perché è la variabile di istanza. Ma il tuo comparatore si basa sul getPrezzo(), che per ProdottoScontato calcola il prezzo scontato al volo e lo restituisce.

Pertanto, come minimo, metti nel toString() anche il getPrezzo() e vedrai che questo prezzo è quello in ordine decrescente!
E magari in ProdottoScontato metti un override di toString(), invoca la versione "super" e appendi dietro lo sconto (che ora non fai mai vedere) in una forma es. "sconto XX%" (poi scegli tu).