Gestione codici sconti

[davidinho]

Dovrei fare un semplice programma per la gestione di alcuni codici, in pratica deve solo permettere la generazione di alcuni codici che una volta utilizzati vengono segnati appunto come "già utilizzati" per evitare che si usufruisca più volte di un singolo codice

Dato che è molto che non programma OO ho perso la mano e mi ritrovo quindi in difficoltà anche su scelte semplicissime:
fin'ora ho creato le classi Codice, GeneratoreCodici e GestoreCodici, in quest'ultima mantengo una lista con i Codici presenti, solo non saprei come segnare quali codici sono utilizzati e quali invece sono ancora disponibili... voi come consiglieresti di agire?
Inizialmente pensavo di dividere le 2 tipologie di codice in 2 liste (codiciUtilizzati e codiciInutilizzati) ma non so perché ma ho la sensazione che non sia una buona scelta... come già detto è molto che non programmo quindi anche le cose più stupide per ora mi sfuggono

[Alex'87]

Potresti spiegare la relazione tra le varie classi?

Il GestoreCodici e il GeneratoreCodici sono collegati? Nel senso, è il GestoreCodici che attraverso il GeneratoreCodici crea un nuovo Codice oppure sono classi utilizzate separatamente?

[davidinho]

Si scusate, il GeneratoreCodici si occupa di generare i vari codici ed è utilizzato dal GestoreCodici proprio come specificato da Alex

Comunque ho pensato di utilizzare all'interno del GestoreCodici una HashMap che ha come chiavi i vari codici e come valori un valore booleano che indica se il codice è già stato utilizzato o meno...

[LeleFT]

Può essere una soluzione.
Altre tre soluzioni potrebbero essere le seguenti:

1) Usare un semplice ArrayList<Codice>. Quando viene generato un codice, verifichi se questo è già presente nell'ArrayList ( metodo contains() ). Se non c'è, lo inserisci, altrimenti è già stato usato.

2) Usare HashSet<Codice>. Quando viene generato un codice, provi ad inserirlo. Se l'inserimento ritorna true, significa che il codice è buono, altrimenti era già stato usato.

3) Usare un TreeSet<Codice>. Quando viene generato un codice, provi ad inserirlo (metodo add() che ritorna un boolean). Se l'inserimento va a buon fine, ok, altrimenti significa che il codice è già stato inserito in precedenza, quindi è già stato usato.

Sinceramente, non userei TreeSet, poichè tale struttura dati mantiene anche un ordinamento degli elementi, che a te non interessa, quindi sarebbe solo un aggravio ingiustificato.

Non userei nemmeno ArrayList poichè avresti un'inefficienza nell'uso di contains() che deve spazzolarsi tutti gli elementi.

Io userei HashSet.

Ciao.

[davidinho]

Beh ma come dici te dovrei usare quindi 2 collection, una per i codici utilizzati e una per i codici ancora inutilizzati, o ho capito male?

[LeleFT]

Originariamente inviato da davidinho
Beh ma come dici te dovrei usare quindi 2 collection, una per i codici utilizzati e una per i codici ancora inutilizzati, o ho capito male?

Forse non ho ben capito come funziona il tuo programma... i codici li genera tutti subito o all'occorrenza?


Ciao.

[davidinho]

come preferisce il cliente, è possibile fare entrambe le cose, potrebbe per esempio generare 100 codici, poi più in là generare altri 100 codici e dovrei quindi preoccuparmi di non far generare codici già creati precedentemente...
in effetti non avevo chiarito questo punto...

[LeleFT]

Originariamente inviato da davidinho
come preferisce il cliente, è possibile fare entrambe le cose, potrebbe per esempio generare 100 codici, poi più in là generare altri 100 codici e dovrei quindi preoccuparmi di non far generare codici già creati precedentemente...
in effetti non avevo chiarito questo punto...

Ok.
Quindi, se non vado errato, il tuo è un doppio problema:

1) Non generare codici doppi
2) Tenere traccia, fra i codici corretti generati, di quali sono già utilizzati e quali no.

E' corretto?

Se è così, allora la HashMap va benissimo, mentre l'HashSet non ti serve a nulla.
Da quel che avevo capito io, quando un codice viene generato è automatico anche che venga usato subito. Mi pare di capire che non è così.

Ciao.