Comment forcer un nombre à être dans une plage en C#?

Question

En C#, je dois souvent limiter une valeur entière à une plage de valeurs. Par exemple, si une application attend un pourcentage, un entier d'une entrée utilisateur ne doit pas être inférieur à zéro ou supérieur à cent. Un autre exemple: s'il y a cinq pages Web qui sont accessibles par Request.Params["p"], j'attends une valeur de 1 à 5, de 0 ou 256 ou 99999.

Je termine souvent en écrivant un code assez laid comme:

page = Math.Max(0, Math.Min(2, page)); 

ou encore plus laid:

percentage = 
    (inputPercentage < 0 || inputPercentage > 100) ? 
    0 : 
    inputPercentage; 

est pas là une façon plus intelligente de faire de telles choses dans .NET Framework?

Je sais que je peux écrire une méthode générale int LimitToRange(int value, int inclusiveMinimum, int inlusiveMaximum) et l'utiliser dans tous les projets, mais peut-être qu'il existe déjà une méthode magique dans le cadre?

Si je devais le faire manuellement, quel serait le «meilleur» moyen (c'est-à-dire moins laid et plus rapide) de faire ce que je fais dans le premier exemple? Quelque chose comme ça?

public int LimitToRange(int value, int inclusiveMinimum, int inlusiveMaximum) 
{ 
    if (value >= inclusiveMinimum) 
    { 
     if (value <= inlusiveMaximum) 
     { 
      return value; 
     } 

     return inlusiveMaximum; 
    } 

    return inclusiveMinimum; 
} 

Answer 1

Je vois la réponse de Mark et relance par un this:

public static class InputExtensions 
{ 
    public static int LimitToRange(
     this int value, int inclusiveMinimum, int inclusiveMaximum) 
    { 
     if (value < inclusiveMinimum) { return inclusiveMinimum; } 
     if (value > inclusiveMaximum) { return inclusiveMaximum; } 
     return value; 
    } 
} 

Utilisation:

int userInput = ...; 

int result = userInput.LimitToRange(1, 5) 

Voir: Extension Methods

Answer 2

Une autre façon d'écrire votre fonction LimitToRange est la suivante. Je pense que c'est un peu plus facile à comprendre tout en restant efficace.

Answer 3

Cette opération est appelée 'pince' et il est généralement écrit comme ceci:

public static int Clamp(int value, int min, int max) 
{ 
    return (value < min) ? min : (value > max) ? max : value; 
} 

Answer 4

Non, il n'y a pas de méthode pour celle construite dans le framework. Je suppose qu'il a été omis parce que vous avez déjà Min et Max, donc vous pouvez l'accomplir en les utilisant.

Si vous écrivez votre propre méthode, peu importe comment vous l'écrivez. Si vous utilisez des instructions if ou l'opérateur conditionnel ?, il compilera quand même à peu près le même code de toute façon.

Answer 5

J'aime la réponse de Guffa, mais je suis surpris que personne n'ait encore posté une solution utilisant Min/Max.

public int LimitInclusive(int value, int min, int max) 
{ 
    return Math.Min(max, Math.Max(value, min)); 
} 

Answer 6

Une grande méthode plus propre qui fonctionnera avec plus que des nombres entiers (pris de ma propre bibliothèque de code partagé):

public static T Clamp<T>(T value, T min, T max) where T : IComparable<T> 
{ 
    if (value.CompareTo(min) < 0) 
     return min; 
    if (value.CompareTo(max) > 0) 
     return max; 

    return value; 
} 

Answer 7

Pour serrer les valeurs sans donner aux utilisateurs tous les commentaires que la valeur entrée par ils ont tort, en général, peut-être pas une bonne idée (à mon humble avis). Cela peut conduire à des bogues subtils plus tard, qui sont difficiles à déboguer, en particulier lorsque les valeurs min/max sont déterminées au moment de l'exécution.

Pensez-y. Vous avez 100 $ dans votre compte bancaire, et vous voulez transférer 150 $ à votre ami. Souhaitez-vous que votre système bancaire lance un InsufficientFundsException ou discute avec votre ami (e) que vous avez transféré 150 $ mais qu'il n'a reçu que 100 $ (en supposant que la banque a viré le virement à 100 puisque vous n'aviez pas suffisamment de fonds)

étant dit, vous devriez également regarder les contrats de code.

public void MyFunction (Type input) 
{ 
    Contract.Requires(input > SomeReferenceValue); 
    Contract.Requires (input < SomeOtherReferencValue); 

} 

Cela forcera l'entrée de l'utilisateur à se situer dans la plage.

Answer 8

J'aime le nom Clamp. Je suggère la classe suivante

public class MathHelper 
{ 
    public static int Clamp (int value,int min,int max) 
    { 
      // todo - implementation 
    } 
    public static float Clamp (float value,float min,float max) 
    { 
      // todo - implementation 
    } 
) 

ou si vous voulez utiliser les médicaments génériques, alors

public class MathHelper 
{ 
    public static T Clamp<T> (T value, T min, T max) where T : IComparable 
    { 
     // todo - implementation 
     T output = value; 
     if (value.CompareTo(max) > 0) 
     { 
      return max; 
     } 
     if (value.CompareTo(min) < 0) 
     { 
      return min; 
     } 
     return output; 
    } 
}