두 개의 값을 하나의 개체에 매핑하십시오.

Question

두 개의 작은 int 값 (범위 0-3)을 사용하고 해당 값을 기준으로 Color 개체를 반환하는 C#에서 함수를 작성하려고합니다. 문제는 프로그래밍 방식으로 두 값에서 색상을 결정할 수있는 방법이 없다는 것입니다. LED의 유형에 따라 다르며 하드 코딩해야합니다.

내가 생각할 수있는 가장 단순한 방법은 각 값을 검사하는 방대한 (16 가지 경우) if-else 문이지만 매우 훌륭한 해결책은 아닌 것 같습니다. 색상을 결정하는 더 좋은 방법이 있습니까?

Answer 1

Color 객체의 2 차원 배열은 어떻습니까?

Color[,] colors = new[,] { 
    { Color.FromArgb(1, 2, 3), Color.FromArgb(3, 4, 5), Color.FromArgb(6, 7, 8), Color.FromArgb(9, 10, 11) }, 
    { Color.FromArgb(21, 22, 23), Color.FromArgb(23, 24, 25), Color.FromArgb(26, 27, 28), Color.FromArgb(29, 30, 31) }, 
}; 

Color color = colors[index1, index2]; 

Answer 2

if/else을 사용하는 대신에 약간 더 깨끗한 방법으로 색상 값을 배열에 배치하고 두 개의 작은 int 값을 배열의 인덱스로 사용할 수 있습니다.

Answer 3

당신이 (좋든 나쁘 든)이 OO 접근 방식을 가고 싶은 경우에, 당신은 Tuple-> 색상 매핑을 사용할 수는없는 스파 스 될 수있는 2 차원 배열을 통해

Dictionary<Tuple<int, int>, Color> d = new Dictionary<Tuple<int, int>, Color>() 
{ 
    {Tuple.Create(2, 1), Color.Red}, 
    {Tuple.Create(1, 1), Color.Blue}, 
    {Tuple.Create(1, 2), Color.Green} 
}; 

장점을 nulls. 인덱스 순서로 초기화하지 않고 컬렉션 초기화 프로그램을 사용할 수도 있습니다. 두 색상에 대해 동일한 튜플을 사용하려고하면 초기화시에 명백한 ArgumentException이 표시됩니다.

Answer 4

if else 문의 무리를 방지의 선호하는 방법은 Switch를 사용하는 것입니다. 당신은 또한 enum를 사용하고 소규모의 int에 해당 INT 값을 지정할 수 있습니다 :

public enum ColorChoice 
{ 
    red = 01 //'01' instead of '1' to make clear your first digit is zero 
    blue = 02 
    green = 03 
    ...etc... 
] 

Answer 5

나는 16 색에 대한 열거 투표

enum Color 
{ 
    red = 0, 
    blue = 1 
} 

public Color GetColor(int v1, int v2) 
{ 
    int colorValue = v1 << 8; 
    colorValue |= v2; 
    return (Color)colorValue; 
} 

Answer 6

방법에 대해 :

static Color GetColor(int c1, int c2) 
{ 
    if (c1 > 3 || c1 < 0 || c2 > 3 || c2 < 0) 
    throw new ArgumentOutOfRangeException(); 

    var map = new Dictionary<string, Color> 
    { 
    {"00", Color.Black}, 
    {"10", Color.Red}, 
    {"20", Color.Blue}, 
    // etc, etc... 
    {"11", Color.DimGray}, 
    }; 

    var key = c1.ToString() + c2.ToString(); 
    return map[key]; 
} 

Answer 7

사실 두 개의 작은 정수 값을 결합하여 무언가 (적어도 하나의 색상)를 의미하게 만들었다는 사실은 나에게 당신이 그 구조체를 캡슐화하도록 정의 된 구조체를 갖는 것이 합리적 일 수 있음을 암시합니다. 엠.

이렇게하면 사전에 대한 키로 구조체를 사용할 수 있다는 이점이 있습니다. Equals 및 GetHashCode 메서드는 이미 적절하게 작동해야합니다.

다음은 예제입니다. 색상을 계산할 수 없다고 언급 했으므로 사전을 채워야하는 위치에 의견을 남겼습니다. 당신은 이것을 하드 코딩 할 수 있지만 파일이나 임베디드 리소스에서 값을 읽는 것이 이상적입니다.

EDIT : 내 예제 구조체를 변경할 수 없도록 업데이트했습니다.

struct MyStruct 
{ 
    private byte _X; 
    private byte _Y; 

    public MyStruct(byte x, byte y) 
    { 
     _X = x; 
     _Y = y; 
    } 

    public byte X { get { return _X; } } 
    public byte Y { get { return _Y; } } 

    private static Dictionary<MyStruct, Color> _ColorMap; 

    static MyStruct() 
    { 
     _ColorMap = new Dictionary<MyStruct, Color>(); 

     // read color mapping from somewhere...perhaps a file specific to the type of LED 
    } 

    public Color GetColor() 
    { 
     return _ColorMap[this]; 
    } 
}