dmazzoni / multisala Goto Github PK

Nel metodo register del GuestMA.
Bisognerebbe gestire il caso di un inserimento di un nome utente già presente nel metodo register del CentralServer, oppure lasciamo che appaia come una SQLException e leviamo la clausola sia da qui che dall'interfaccia.

Il metodo adminConnected() di AdminMS non viene mai invocato.

Manca invocazione di query.close() all'interno del metodo login() di AuthServer

Presente nell'interfaccia ICentralServer per insertReservation e editReservation ma mai lanciata nelle implementazioni.

Il problema è l'accesso al campo parent della classe AllReservationsPanel.

Il costruttore di AllReservationsPanel invoca prima di tutto il costruttore della superclasse ReservationPanel e poi inizializza il proprio campo parent.

Nel costruttore di ReservationPanel viene invocato il metodo updateView. Visto che in realtà stiamo costruendo un AllReservationsPanel, ad essere eseguita è la versione di updateView reimplementata proprio in AllReservationsPanel.

Ci troviamo quindi ora nella classe AllReservationsPanel. il metodo updateView accede al campo parent di questa classe, ma questo non è ancora stato inizializzato perchè l'invocazione a super non è ancora terminata.

Il costruttore di Reservation potrebbe lanciare una ReservationException se il numero di posti passato è negativo o nullo.

Nel dominio per i gruppi

show_id e free_seats scambiati

Si potrebbe impedire l'nserimento di una prenotazione per uno spettacolo già passato, ad esempio.

Si potrebbe, tramite le proprietà passate ai gruppi dal codice di Setup, dare la possibilità di impostare un valore personalizzato per il leaseValue dei server attivabili.