Contesto
Sto lavorando al refactoring di un programma che salva i dati in un DB. Questo programma ha un'interfaccia grafica.
Il programmatore che ha fatto questo programma (quello per cui lavoro) vorrebbe che io rendessi questo progetto coerente con MVP.
Uno dei limiti del mio lavoro è provare a non modificare il codice aziendale originale, quindi anche se il codice è "MVPized" sarà facilmente comprensibile.
Informazioni sul codice
Abbiamo una funzione che salva un oggetto nel DB scritto nella vista. Quindi, invece di restituire errori, mostra direttamente il messaggio di errore.
Quei messaggi a volte sono frasi complete, a volte sono codici di errore.
Ho ~ 10 possibili errori nella funzione: Verifica se ogni campo non è vuoto, con un messaggio per ogni campo, Verifica se i valori non sono già usati, Verifica se il salvataggio nel database è andato a buon fine, controllo se una applciazione esterna è andata bene con la funzione nativa per dare un messaggio di errore (diverso dal controllare il campo per esempio). Ogni controllo come proprio messaggio di errore con a volte una speciale funzione nativa chiamata per dare il messaggio di errore ..
Quindi, ho spostato questa funzione nel modello e ho reso questa funzione restituita i codici di errore usando un tipo di enumerazione (o almeno provato), ma la conseguenza è che ora ho molti codici di errore diversi.
Ora ho quasi finito di refactoring questa funzione, ma non mi sento ho fatto la cosa giusta facendo questo.
Quindi la mia domanda è: creare molti codici di errore diversi è una buona idea? O più precisamente, è una buona idea avere un sacco di diversi codici di errore in una sola funzione?