[Java] Progettazione classi astratte e non, costruttori e metodi da rivedere

[VYCanisMajoris]

Salve ragazzi, devo fare un progetto in java utilizzando l'ereditarietà. Il problema è che sto riscontrando delle difficoltà nella progettazione dei metodi.

Vi dico prima di tutto dove ho problemi:
1. Le specifiche del problema(scritte sotto) mi impongono che un oggetto 'Disco33Giri' abbia una sequenza(quindi un array) massima di 14 eoggetti 'Brano'. Quindi al costruttore di Disco33Giri dovrò passare un array di massimo 14 oggetti. Se volessi fare un controllo proprio su questa cosa mi basterebbe mettere un if che controlla la lunghezza dell'array, se maggiore di MAX=14 il parametro passato al costruttore non verrà associato poichè esso ha piu di 14 elementi. E proprio qui viene il problema. La classe 'Disco33Giri' in realtà è una sottoclasse e nel suo costruttore viene invocato il supercostruttore della superclasse, e come sappiamo benissimo questa invocazione deve essere fatta alla prima riga, quindi per come sto progettando la questione l'if non può esserci...
Nel mio codice potete trovare questa situazione da correggere in:
Disco33Giri: righe 4-7
SupportoPerGiradischi: righe 3-5
Supporto: righe 4-7

2. Il metodo next() che trovate implementato nella classe 'Giradischi'. Che cosa fa? non fa altro che far avanzare di 1 la variabile selBrano che è quella che si occupa di selezionare un brano. I brani dell'Array sono 14, quindi, quando selBrano=14 e chiamo il metodo next() dovrebbe tornare a 0. Quindi se arrivo a MAX_BRANI devo riazzerare. Ogni SupportoPerGiradischi ha il suo MAX_BRANI. Quindi devo fare un if in cui identifico se ho a che fare con un 'Disco33Giri' o un Disco45Giri di cui conosco ORA il MAX_BRANI . Se lo faccio il programma funzionerà correttamente ma se in futuro un programmatore volesse inventare una nuova classe ad es.Disco90Giri che ha un MAX_BRANI diverso da Disco33-45Giri dovrà per forza modificare il metodo next() inmplementato in 'Giradischi' aggiungendo un if. Ci deve essere un altro modo per progettare questo metodo in maniera universale.
Nel mio codice potete trovare questa situazione da correggere in:
Giradischi: righe 42-44
Disco33Giri: riga 2

Le specifiche del problema sono queste:

Progettare una classe RiproduttoreMusicale che rappresenti un generico riproduttore
• La classe deve realizzare i seguenti metodi:
– inserisciSupporto(): permette di inserire un supporto musicale(es. CD, nastro, ecc.)
– espelliSupporto(): espelle il supporto
– getBrano(): restituisce l’attuale brano in esecuzione (null se non sta eseguendo)
– play(): esegue il brano attualmente selezionato
– stop(): interrompe l’esecuzione
– next(): seleziona il prossimo brano
– prev(): seleziona il brano precedente
– toString(): visualizza le informazioni del brano attualmente in esecuzione

• Realizzare quindi i seguenti riproduttori
– Giradischi
– Mangianastri
– Lettore CD
– Lettore Mp3

• Realizzare inoltre diversi tipi di supporto:
– Disco a 33 giri (14 brani)
– Disco a 45 giri (2 brani)
– Compact Disc (20 brani)
– Nastro (con un numero specificato di minuti, numero di brani pari al numero di minuti / 5)
– Memoria USB (1024 brani)
• Gli ultimi due supporti permettono di registrare/inserire brani nella posizione specificata

• Ciascun supporto può contenere il numero massimo di
brani specificati tra parentesi nella precedente
diapositiva e viene costruito con una data sequenza
(eventualmente vuota) di brani
• Modellare inoltre la classe Brano che contenga
l’informazione sul nome del brano e il cantante

Detto questo per semplicità io vi posto soltanto queste classi che sono estese cosi(accanto il link al codice):
-RiproduttoreMusicale http://paste.ubuntu.com/941755/
---Giradischi http://paste.ubuntu.com/941756/

-Supporto http://paste.ubuntu.com/941711/
----SupportoPerGiradischi http://paste.ubuntu.com/941713/
--------Disco33Giri http://paste.ubuntu.com/941716/

-Brano http://paste.ubuntu.com/941727/

[PGI-Bis]

Nel metodo next puoi usare la lunghezza dell'array sup.brani per evitare lo sforamento (sup.brani.length).

Per quanto riguarda il super, l'invocazione di supercostruttore Java genera un contesto statico. Significa che tra le parentesi del super puoi fare tutto quello che vuoi, purchè non dipenda dall'esistenza di "this". Puoi ad esempio dire:

super(array.length == MAX ? array : new Brani[MAX]);

ma anche:

static Brani[] metodo(Brani[] in) {
    if(in.length < MAX) return new Brani[14]
    else return in;
}
super(Disco33Giri.metodo(array))

L'unico intoppo è che ciò che è static in Java non è ridefinibile (è "static" con riferimento al collegamento statico e in contrasto con quello dinamico). Sarebbe meglio non usarlo ma io non vedo grandi alternative a meno che tu non possa usare un tuo tipo di struttura dati diverso dall'array.

[franksisca]

infatti secondo me è lì il problema...sicuro che devi usare un array statico???


se ci sono solo 10 titoli come ti comporti?

[=KaTaKliSm4=]

Ehm scusami ma il controllo della lunghezza dell'array non puoi farlo nella superclasse?Esempio (C#) :

public class Disco33Giri : SupportoPerGiradischi
{
public Disco33Giri(Brano[] Brani)
:base(Brani){ ...; }
}

public class SupportoPerGiradischi : Supporto
{
public SupportoPerGiradischi(Brano[] Brani)
:base(Brani){ ...; }
}

public class Supporto
{
public Supporto(Brano[] Brani)
{
if(Brani.Lenght > 14)
throw new Exception("Superato il limite di elementi.");
}
}

[PGI-Bis]

Nella superclasse gli mancherebbe il "14". Non so se possa cambiare il costruttore della superclasse in modo che accetti anche un intero, sul genere:

SuperClasse(array, max) { ... }

Disco33(array) { super(array, 14); }

Ma credo di no, nel senso che SuperClasse gli è stata data così com'è.

[=KaTaKliSm4=]

Originariamente inviato da PGI-Bis

Nella superclasse gli mancherebbe il "14". Non so se possa cambiare il costruttore della superclasse in modo che accetti anche un intero, sul genere:

SuperClasse(array, max) { ... }

Disco33(array) { super(array, 14); }

Ma credo di no, nel senso che SuperClasse gli è stata data così com'è.

Nell'ipotesi che la superclasse gli sia stata data, può sempre effettuare il controllo nella sottoclasse e generare un'eccezione, continuo a non vedere il problema , in caso contrario basterebbe modificare il costruttore per aggiungere un argomento di tipo Int, come da te accennato...