Come dovrei segnalare più errori come risultato della convalida?

Perché non legare i messaggi di errore in una raccolta in un'eccezione personalizzata e lanciare quell'eccezione al completamento? Il codice ricevente può quindi rilevare l'eccezione e iterare la raccolta per segnalare gli errori. Registrerei gli errori separatamente invece di concatenarli (sembra un problema di presentazione)

Un'alternativa è usare qualcosa come Scala O struttura , che è possibile restituire dal metodo. La tua implementazione di Either (come suggerisce il nome) restituirà i risultati positivi di quel metodo o una raccolta di errori.

Costruire un oggetto "risultato di convalida" è una buona via di mezzo, se le eccezioni diventano troppo disordinate.

public class ValidationResult
{
    public ValidationResult()
    {
        errorMessages = new List<string>();
    }

    public bool IsValid { get; private set; }

    private List<string> errorMessages;

    public IEnumerable<string> ErrorMessages
    {
        get { return errorMessages; }
    }

    public void AddErrorMessage(string errorMessage)
    {
        IsValid = false;
        errorMessages.Add(errorMessage);
    }
}

Quindi puoi creare il tuo risultato di convalida quando convalidi diversi campi o proprietà:

ValidationResult result = new ValidationResult();

if (name == null)
    result.AddErrorMessage("The 'Name' field is required");

if (age < 18)
    result.AddErrorMessage("The 'Age' field must be greater than or equal to 18");

return result;

Quindi puoi elaborare quei risultati:

ValidationResult result = x.Validate();

if (result.IsValid)
{
    // Save to database
}
else
{
    // Show result.ErrorMessages to the user
}

Nota: il mio codice è in C #, poiché non hai specificato in quale lingua sono stati scritti gli esempi di codice.

Come hai detto, la gestione è "molto complicata e confusa".

Il mio suggerimento è di aggiungere un public String[] getErrorMessage() alla classe responsabile della registrazione dell'utente.

Prendi in considerazione però che alcuni errori non permetteranno di intraprendere ulteriori passi in modo che l'elenco degli errori sia incompleto in quanto non sarebbero stati tentati passi che avrebbero anche fallito.

Un'altra opzione consiste nel creare un'eccezione personalizzata, contenente tutti i messaggi di errore e lanciarla all'ultimo momento. In questo modo, il codice di chiamata farebbe e.getErrorMessages() .

Hai intenzione di fare qualcosa con gli errori diversi da visualizzarli all'utente? In caso contrario, quello che faccio di solito è semplicemente costruire una stringa contenente il testo di tutti gli errori. Come:

StringBuilder errors=new StringBuilder();
if (find_name(new_user_name)) {
  errors.append("User name already in use<br/>";
}
if (password.length==0) {
  errors.append("Password required<br/>");
}
if (prettiness(favorite_color)<20) {
  errors.append("Favorite color is not pretty enough<br/>");
}
... whatever other tests ...
if (errors.length>0) {
  showError(errors);
} else {
  processRegistration();
}

(Ho messo br's dopo ogni errore qui sull'idea che li mostreremo su una pagina web.)

Ho avuto alcuni casi in cui ho creato una raccolta di qualche tipo e aggiunto i messaggi di errore alla raccolta, ma in retrospettiva non guadagna nulla. Devo ancora aggiungere le stringhe insieme per la visualizzazione.

Se si restituiscono errori da una funzione e il chiamante può provare a fare qualcosa sugli errori piuttosto che visualizzarli, è tutta un'altra storia. Ho difficoltà a pensare a un momento in cui ho dovuto farlo, ma penso che avrei creato una raccolta di codici di errore e ho definito i codici di errore come enum.