数値が指定された範囲内にあるかどうかを判断するための流暢な方法を探しています。私の現在のコードは次のようになります。
int x = 500; // Could be any number
if ( ( x > 4199 && x < 6800 ) ||
( x > 6999 && x < 8200 ) ||
( x > 9999 && x < 10100 ) ||
( x > 10999 && x < 11100 ) ||
( x > 11999 && x < 12100 ) )
{
// More awesome code
}
これを行うためのより良い方法はありますか?
拡張メソッド?
bool Between(this int value, int left, int right)
{
return value > left && value < right;
}
if(x.Between(4199, 6800) || x.Between(6999, 8200) || ...)
このひどいハックを行うこともできます:
bool Between(this int value, params int[] values)
{
// Should be even number of items
Debug.Assert(values.Length % 2 == 0);
for(int i = 0; i < values.Length; i += 2)
if(!value.Between(values[i], values[i + 1])
return false;
return true;
}
if(x.Between(4199, 6800, 6999, 8200, ...)
ひどいハック、改善:
class Range
{
int Left { get; set; }
int Right { get; set; }
// Constructors, etc.
}
Range R(int left, int right)
{
return new Range(left, right)
}
bool Between(this int value, params Range[] ranges)
{
for(int i = 0; i < ranges.Length; ++i)
if(value > ranges[i].Left && value < ranges[i].Right)
return true;
return false;
}
if(x.Between(R(4199, 6800), R(6999, 8200), ...))
または、さらに良いです(これは重複する下限を許可しません):
bool Between(this int value, Dictionary<int, int> ranges)
{
// Basically iterate over Key-Value pairs and check if value falls within that range
}
if(x.Between({ { 4199, 6800 }, { 6999, 8200 }, ... }
範囲タイプを定義してから、範囲のセットと拡張メソッドを作成して、値がいずれかの範囲にあるかどうかを確認します。次に、値をハードコーディングする代わりに、範囲のコレクションと、場合によってはいくつかの個別の範囲を作成して、理由を説明するのに役立つ名前を付けることができます。それらに興味がある:
static readonly Range InvalidUser = new Range(100, 200);
static readonly Range MilkTooHot = new Range (300, 400);
static readonly IEnumerable<Range> Errors =
new List<Range> { InvalidUser, MilkTooHot };
...
// Normal LINQ (where Range defines a Contains method)
if (Errors.Any(range => range.Contains(statusCode))
// or (extension method on int)
if (statusCode.InAny(Errors))
// or (extension methods on IEnumerable<Range>)
if (Errors.Any(statusCode))
MiscUtil の一部である一般的なRange
型に興味があるかもしれません。簡単な方法で反復することもできます。
foreach (DateTime date in 19.June(1976).To(25.December(2005)).Step(1.Days()))
{
// etc
}
(明らかに、それはいくつかのDateTime/TimeSpan関連の拡張メソッドも使用していますが、あなたはその考えを理解しています。)
私は個人的に@Antonによって提案された拡張方法を好みます-しかし、それができず、現在のコードに固執する場合は、次のように各行の最初の条件セットを逆にすることで、より読みやすくすることができると思います.。。
int x = 500; // Could be any number
if ( ( 4199 < x && x < 6800 ) ||
( 6999 < x && x < 8200 ) ||
( 9999 < x && x < 10100 ) ||
( 10999 < x && x < 11100 ) ||
( 11999 < x && x < 12100 ) )
{
// More awesome code
}
LINQアプローチ:
参照を追加します。
using System.Linq;
/// <summary>
/// Test to see if value is in specified range.
/// </summary>
/// <param name="aStart">int</param>
/// <param name="aEnd">int</param>
/// <param name="aValueToTest">int</param>
/// <returns>bool</returns>
public static bool CheckValueInRange(int aStart, int aEnd, int aValueToTest)
{
// check value in range...
bool ValueInRange = Enumerable.Range(aStart, aEnd).Contains(aValueToTest);
// return value...
return ValueInRange;
}
class Range {
public Range(int x, int y) {
X = x;
Y = y;
}
public int X { get; set; }
public int Y { get; set; }
}
var ranges = new List<Range>();
ranges.Add(new Range(4199,6800));
ranges.Add(new Range(6999,8200));
ranges.Add(new Range(9999,10100));
ranges.Add(new Range(10999,11100));
ranges.Add(new Range(11999,12100));
bool inRange = ranges.Count(r => x >= r.X && x <= r.Y) > 0;
//or -- Based on Jons recommendation
bool inRange = ranges.Any(r => x >= r.X && x <= r.Y);
ある時点で値のペアを反復処理する必要がある場合は、変数に入力して実行するときに、最大の下限値と最小の上限値をキャプチャすることをお勧めします。
if ( x>max_lower && x <min_upper)
{
// More awesome code
}
次のようなものを試してください:
struct Range
{
public readonly int LowerBound;
public readonly int UpperBound;
public Range( int lower, int upper )
{ LowerBound = lower; UpperBound = upper; }
public bool IsBetween( int value )
{ return value >= LowerBound && value <= UpperBound; }
}
public void YourMethod( int someValue )
{
List<Range> ranges = {new Range(4199,6800),new Range(6999,8200),
new Range(9999,10100),new Range(10999,11100),
new Range(11999,12100)};
if( ranges.Any( x => x.IsBetween( someValue ) )
{
// your awesome code...
}
}