codice ridondante ok? [chiuso]

4

Usando C # /. Net, ho una pagina in cui l'utente può inserire dati e salvare questi dati facendo clic su un pulsante.

Per salvare i dati, l'utente deve inserire una data valida, ovvero una data con un determinato formato.

La pagina contiene tra l'altro due moduli.

Il ValidateDate-Event ha la responsabilità di controllare il formato del campo data. Quindi controlla il formato e visualizza un messaggio se la data non è ben strutturata.

L'evento di salvataggio deve salvare le voci dell'utente. Per completare correttamente l'attività, è necessaria una data ben formata. Così ho progettato il modulo per verificare il formato della data. Questo ha il vantaggio che il modulo non può mai bloccarsi e non dipende dal modulo di convalida. Ma ho capito che in effetti faccio la stessa cosa due volte, in due moduli.

Sono solo curioso, come gestisci questa faccenda? Quale pensi sia il modo migliore per farlo?

1) ValidateDate

Responsabilità: verifica il formato del campo data (, informa l'utente)
Sotto-attività: verifica il formato del campo data

2) Salva

Responsabilità: salva i dati
Sotto-attività: verifica il formato del campo data

    
posta AGuyCalledGerald 18.10.2011 - 12:40
fonte

1 risposta

10

Fare la stessa cosa due volte è quasi sempre una cattiva idea. (vedi DRY ) Devi cambiarlo in due punti se la tua convalida cambia aumentando la tua possibilità di bug.

Il mio consiglio sarebbe avere sia Salva che Convalida chiama un altro metodo che convalida l'input. È comunque una buona idea controllare il salvataggio e convalidare come non dovresti mai fidarti dell'input dell'utente. Ad esempio potrebbero essere dannosi o potrebbero esserci alcuni bug bizzanti del browser che causano / consentendo loro di inviare dati non validi.

    
risposta data 18.10.2011 - 12:51
fonte

Leggi altre domande sui tag