Come scrivo questo metodo per aderire al Principio di Responsabilità Singola?

Question

Come scrivo questo metodo per aderire al Principio di Responsabilità Singola?

#1 da (2 voti)

1

Sto scrivendo il codice che importa i valori da un file CSV. Come parte del processo di importazione, ho bisogno di scrivere avvisi su un log (visualizzato sullo schermo) quando i valori nel file CSV non sono validi.

Il metodo seguente accetta un valore stringa e un valore massimo da verificare.

La mia preoccupazione è che il metodo faccia due cose: analizza un valore e genera voci di registro da visualizzare. Ciò viola il principio della responsabilità unica e, in caso affermativo, cosa si può fare al riguardo senza duplicare la stessa logica di controllo del numero in due metodi?

private double? GetValidDoubleInMinMaxRange(string importVal, double? maxScore)
{
    if(importVal == null)
    {
        return null;
    }

    double val;
    if(double.TryParse(importVal, out val))
    {
        if(val < 0)
        {
            AppInstance.Log.Add(this, LogLevel.Warning, Resources.DataScreenImport_ValueMustBePositive);
            return null;
        }

        if(maxScore.HasValue && val > maxScore.Value)
        {
            AppInstance.Log.Add(this, LogLevel.Warning, Resources.DataScreenImport_ValueGreaterThanMax);
            return null;
        }

        return val;
    }
    else
    {
        AppInstance.Log.Add(this, LogLevel.Warning, Resources.DataScreenImport_ValueWasNotValidInt);
        return null;
    }
}

c# single-responsibility

posta willem 25.02.2016 - 12:05

fonte

1 risposta

Leggi altre domande sui tag c# single-responsibility

È un modello comune in Python avere le variabili di istanza assegnate in un costruttore che iniziano con un trattino basso? Progettazione software per la convalida del modulo lato client

score 2 · Accepted Answer

Hai qualche caso di ossessione primitiva qui.

Nel tuo caso, creerei una parser dedicato DoubleMinMaxParser class e avrò TryParse(string) metodo statico che restituisce come la stringa parsed non è valida con una stringa valida nel caso valido e "undefined" in tutte le altre.

Forse qualcosa del tipo:

class DoubleMinMaxParser
{
    private readonly double _min;
    private readonly double? _max;

    public enum Result
    {
        Valid,
        TextWasNull,
        InvalidNumber,
        LessThanMin,
        MoreThanMax,
    }

    public DoubleMinMaxParser(double min, double? max)
    {
        _min = min;
        _max = max;
    }

    public Result TryParse(string importVal, out double value)
    {
        if (importVal == null)
        {
            value = 0;
            return Result.TextWasNull;
        }

        if (double.TryParse(importVal, out value))
        {
            if (_min < 0)
            {
                return Result.LessThanMin;
            }

            if (_max.HasValue && value > _max.Value)
            {
                return Result.MoreThanMax;
            }

            return Result.Valid;
        }
        else
        {
            return Result.InvalidNumber;
        }
    }
}

internal class CsvParser
{
    public string GetMessageForMinMaxResult(DoubleMinMaxParser.Result result)
    {
        switch (result)
        {
            case DoubleMinMaxParser.Result.TextWasNull:
                return Resources.DataScreenImport_ValueWasEmpty;
            case DoubleMinMaxParser.Result.InvalidNumber:
                return Resources.DataScreenImport_ValueWasNotValidInt;
            case DoubleMinMaxParser.Result.LessThanMin:
                return Resources.DataScreenImport_ValueMustBePositive;
            case DoubleMinMaxParser.Result.MoreThanMax:
                return Resources.DataScreenImport_ValueGreaterThanMax;
            default:
                throw new ArgumentOutOfRangeException("result");
        }
    }

    private double? GetValidDoubleInMinMaxRange(string importVal, DoubleMinMaxParser parser)
    {
        double value;
        DoubleMinMaxParser.Result result = parser.TryParse(importVal, out value);

        if(result == DoubleMinMaxParser.Result.Valid)
        {
            return value;
        }
        else
        {
            AppInstance.Log.Add(this, LogLevel.Warning, GetMessageForMinMaxResult(result));
            return null;
        }
    }
}