Каков наилучший способ написать несколько тегов, которые сравнивают две переменные.

Question

Я начинаю программировать на C#, и я пытался сравнить две переменные int (C#).

void CompareNumber() { 
    int oneNumber; 
    int secondNumber; 

    if (oneNumber > secondNumber) 
     { 
     DoSomething(); 
     } 

    else if (oneNumber < secondnumber) 
     { 
     DoSomethingElse(); 
     } 

    else if (oneNumber == secondnumber) 
     { 
     DoSomethingDifferent(); 
     } 
} 

В то время как это действительно работает, оно выглядит беспорядочным, особенно потому, что я сравниваю переменные таким образом много раз. Есть ли более сжатый способ сделать это, заставляя его выглядеть аккуратно? (За исключением исключения фигурных скобок)

Answer 1

То, как вы уже написали, это лучший способ :) В этом нет ничего плохого.

Однако, когда вы говорите «особенно потому, что я сравниваю переменные таким образом много раз», там может быть что-то исправить. В идеале вы должны иметь возможность повторно использовать то, что вы написали один раз.

Answer 2

Поскольку вы просите общие способы сделать, если утверждения выглядят более аккуратными, рассмотрите conditional operator «?:», Если вы назначаете значение. Это не работает для вызова функции недействительной, но если, например, вы хотите, чтобы вернуть большие из двух значений INT с CompareNumber(), вы можете сделать это:

int CompareNumber(int first, int second) 
{ 
    return first > second ? first : second; 
} 

не может сказать, что я вижу любые простые способы сделать ваш образец кода более элегантным. Если у вас есть три разных случая, вы ничего не можете пропустить. Конечно, я мог бы быть умный Алека и упростить функцию для вас (и это не будет иметь ошибки использования неинициализированных переменных):

void CompareNumber() 
{ 
    DoSomething(); 
} 

EDIT: Ранее указано, вы не можете вызывать функции с условным оператором, но Я имел в виду, что вы не можете вызвать функции void ... они должны вернуть значение или вы получите ошибку компилятора.

Answer 3

Вы можете сократить его, используя тройные операторы, но вы делаете это правильно.

void CompareNumber(int first, int second) { 

    (first > second) 
     ? DoSomething() 
     : (second > first) 
      ? DoSomethingElse() 
      : DoSomethingDifferent(); 
} 

Answer 4

Вы можете использовать switch на Signum дельты:

switch (Math.Sign(Alpha - Beta)) 
{ 
case -1: // Alpha < Beta. 
    // Do something. 
    break; 
case 0: // Alpha == Beta. 
    // Do something. 
    break; 
case 1: // Alpha > Beta. 
    // Do something.   
} 

Вы можете, конечно, использовать предложение default в switch обрабатывать окончательный случай.