Remplacement de l'instruction if else par un modèle de conception ou une meilleure approche

Devrait être plus approprié pour être réutilisé: Vous pouvez écrire une classe "Interval" avec la méthode "inRange" comme ceci:

public static readonly List<Interval<int>> range = new List<Interval<int>>
        {
                new Interval<int>(1, 0, 1),
                new Interval<int>(2, 1, 2),
                new Interval<int>(3, 2, 4),
                new Interval<int>(4, 4, 6),
                new Interval<int>(5, 6, 9),
                new Interval<int>(6, 9, 12),
                new Interval<int>(7, 12, 15),
                new Interval<int>(8, 15, 18),
                new Interval<int>(9, 18, 24),
                new Interval<int>(10, 24, 30),
                new Interval<int>(11, 30, 36),
                new Interval<int>(12, 36, 48),
                new Interval<int>(13, 48, 60),
                new Interval<int>(14, 60, int.MaxValue)
        };

var months = 5;
var visit = range.Where(x => x.InRange(months)).Select(x => x.Visit).FirstOrDefault();

Et puis utiliser comme ceci:

 public struct Interval<T>
       where T : IComparable
{
    public T Start { get; set; }
    public T End { get; set; }
    public T Visit { get; set; }

    public Interval(T visit, T start, T end)
    {
        Visit = visit;
        Start = start;
        End = end;
    }

    public bool InRange(T value)
    {
      return ((!Start.HasValue || value.CompareTo(Start.Value) > 0) &&
          (!End.HasValue || End.Value.CompareTo(value) >= 0));
    }
}

Cela ne fonctionnera que tant que la valeur de retour ( visite ) augmentera toujours linéairement pour chaque condition disponible (c'est-à-dire que visite augmente de 1 dans chaque si / else if block).

static void Main(string[] args)
{
    Console.WriteLine($"0: {VisitMonth(0)}");
    Console.WriteLine($"5: {VisitMonth(5)}");
    Console.WriteLine($"60: {VisitMonth(60)}");
    Console.WriteLine($"100: {VisitMonth(100)}");
    Console.ReadLine();
}

Cette approche a l'avantage de ne pas avoir à définir la valeur de retour ( visite code>) est. Ce qui rend cela extensible dans le sens où si une exigence arrive là où il y a une nouvelle limite quelque part au milieu (par exemple 22), vous n'aurez pas à remapper la valeur de visite pour chaque condition car il est simplement dérivé en fonction de sa position dans le tableau.

Voici un exemple de ce fonctionnement:

static readonly int[] _monthLimits = new int[] { 1, 2, 4, 6, 9, 12, 15, 18, 24, 30, 36, 48, 60 };

public static int VisitMonth(int months)
{
    int visit = 0;

    var maxMonths = _monthLimits[_monthLimits.Length - 1];
    if (months <= maxMonths)
    {
        // Only iterate through month limits if the given "months" is below the max available limit
        for (var i = 0; i < _monthLimits.Length; i++)
        {
            if (months <= _monthLimits[i])
            {
                visit = i + 1;
                break;
            }
        }
    }
    else
    {
        // The given "months" is over the max, default to the size of the array
        visit = _monthLimits.Length + 1;
    }

    return visit;
}

Une façon de procéder serait de stocker les valeurs que vous comparez dans une List , puis de renvoyer l ' index + 1 de premier élément qui remplit la condition mois , ou la liste Count + 1 si aucun ne remplit cette condition.

Nous utilisons int? pour nous permettre d'obtenir un résultat null lorsque nous appelons FirstOrDefault s'il n'y a pas de correspondance (nous ne utilisez int , car default (int) == 0 afin que nous ne sachions pas si nous correspondions au premier élément à l'index 0 ou s'il y en avait aucune correspondance et la valeur par défaut 0 a été renvoyée). De cette façon, nous pouvons tester un résultat null (ce qui signifie qu'il n'y a pas de correspondance), et retourner List.Count + 1 dans ce cas.

public static int Visit(int months)
{
    var values = new List<int?> {1, 2, 4, 6, 9, 12, 15, 18, 24, 30, 36, 48, 60};

    return (values.Select((v, i) => new {value = v, index = i})
        .FirstOrDefault(i => months <= i.value)?.index ?? values.Count) + 1;
}

void Main()
{
    var conditionsChain = new SimpleCondition(0, 1);
        conditionsChain.AddNext(new SimpleCondition(1, 1))
        .AddNext(new SimpleCondition(2, 2))
        .AddNext(new SimpleCondition(4, 3))
        .AddNext(new SimpleCondition(6, 4))
        .AddNext(new SimpleCondition(9, 5))
        .AddNext(new SimpleCondition(12, 6))
        .AddNext(new SimpleCondition(15, 7))
        .AddNext(new SimpleCondition(18, 8))
        .AddNext(new SimpleCondition(24, 9))
        .AddNext(new SimpleCondition(30, 10))
        .AddNext(new SimpleCondition(36, 11))
        .AddNext(new SimpleCondition(48, 12))
        .AddNext(new SimpleCondition(60, 13))
        .AddNext(new SimpleCondition(14));

    for (int i = 0; i < 62; i++)
    {
        Console.WriteLine($"{i}: {conditionsChain.Evaluate(i) - VisitMonth(i)}");
    }
}

class SimpleCondition
{
    private SimpleCondition _next;

    private int _key;
    private int _result;

    public SimpleCondition(int key, int result)
    {
        _key = key;
        _result = result;
    }

    public SimpleCondition(int result) : this(-1, result)
    {
    }

    public int Evaluate(int key)
    {
        if(_key == -1)
        {
            return _result; 
        }

        if(key <= _key)
        {
            return _result;
        }
        else
        {
            if(_next == null)
            {
                throw new Exception("Default condition has not been configured.");
            }
            return _next.Evaluate(key); 
        }
    }

    public SimpleCondition AddNext(SimpleCondition next)
    {
        return _next = next;
    }
}