Implementazione classe elemento db

[linux_r]

Salve ho il seguente quesito : ho un database con una tabella 'cliente' che contiene tutte le informazioni come nome, cognome, tel, mail, id(che si genera automaticamente),data_appuntamento, ora_appuntamento. Secondo voi è un errore implementare una classe che rappresenti un oggetto cliente del db tipo cosi:

public final class ClienteDB{    
    private String nome;
    private String cognome;
    private String telefono;
    private String mail;
    private java.sql.Date dataAppuntamento;
    private java.sql.Time oraAppuntamento;
      .......
}

aggiungendo ai metodi set e get tutta una serie di controlli per rispettare i vincoli di integrità della tabella.

[andbin]

La classe in sé è assolutamente ok, con i campi che hai messo.
Quando però parli di "controlli per rispettare i vincoli di integrità" bisogna vedere un attimo. La tua classe ClienteDB non dovrebbe però agire direttamente su db, ovvero tutti i vari metodi di "add", "find", "delete" ecc.... non dovrebbero essere qui dentro.

[linux_r]

si si lo so per questo ho creato una classe GestoreDb con dei metodi di tipo static (anche se un po fuori dall oop) che permettono di effettuare operazioni sul db, creando un oggetto static di tipo connection al suo interno , senza dover riaprire ogni volta una connessione. Quello che intendo io in questo caso è che uso delle espressioni regolari ,all'interno dei metodi set e get di questa classe per verificare che quando imposto un campo questo rispetta i requisiti del corrispondente campo del db. Esempio:

  public void setNome(String nome) throws IllegalArgumentException{
        boolean error=!Pattern.matches(regexNome, nome);
        if(error)
            throw new IllegalArgumentException("Formato del campo Nome"
                    + " non valido");
        this.nome = nome;
    }

[andbin]

con dei metodi di tipo static (anche se un po fuori dall oop)

Ehm .... sì, non proprio OOP ....

Quello che intendo io in questo caso è che uso delle espressioni regolari ,all'interno dei metodi set e get

Perché anche get?
Comunque se si tratta di "validazioni" dei parametri, va bene, nel senso che è ok. Anche l'uso di IllegalArgumentException è ok.

Già che usi le regex, aggiungo 2 note:
- se il "input" al Matcher (es. il tuo nome) è null, viene lanciato NullPointerException.
- se la regex è abbastanza complessa, puoi anche pre-compilare il Pattern (tenendolo es. in un campo di classe).

[linux_r]

si infatti regexNome è un campo di classe.