Перепишите, если еще в коммутаторе

Question

Я написал несколько if-то еще такие заявления:

if (workedDays > 0 && workedDays <= 180) 
    { 
    x= 14; 
    } 

    if (workedDays > 180 && workedDays <= 540) 
    { 
    x= 28; 
    } 

    if (workedDays > 540 && workedDays <= 1068) 
    { 
    x= 42; 
    } 
    else 
    { 
    x= 56; 
    } 

Я пытался переписать его как переключатель заявление, как это:

switch (workedDays) 
    { 
    case (workedDays > 0 && workedDays <= 180): 
     x=14; 
     break; 
    default: 
     break; 
    } 

Однако, Я получаю ошибку Cannot implicitly convert type 'bool' to 'int'.

Как это исправить?

Answer 1

Хотя switch(true) ответ @Jeroen может работать, я предлагаю следующее за readablity:

if (workedDays <= 180) 
    x = 14; 
else if (workedDays <= 540) 
    x = 28; 
else if (workedDays <= 1068) 
    x = 42; 
else 
    x = 56; 

С else if конструкции, нет необходимости перепроверить как минимальные и максимальные значения.

Answer 2

сделать:

switch (true) 
    { 
     case (workedDays > 0 && workedDays <= 180): 
      x=14; 
      break; 
     default: 
      break; 
    } 

Answer 3

Если вы считаете, что вы просите компилятор вы поймете, почему у вас есть проблемы.

Вы переключение на workedDays который предположительно INT:

switch(workedDays:int) 
{ 
    case (workedDays > 0 && workedDays <= 180): 

    break; 
} 

Где вы получили ваше дело, которое вычисляется в истинный или ложный результат (логическое значение), так что вы просите его выбирать между целыми числами, но дать ему bool как возможность!

Answer 4

Вы не можете. Операторы switch могут использоваться только с константами времени компиляции. See this question

Answer 5

Ну, мне лично не нравится иметь так много if-else-if's.

Обычно я придерживаюсь идеи «три или более, используйте для!».

Итак, я хотел бы сделать это: -

class Program 
{ 
    struct Data 
    { 
    public Data (int l, int u, int v) 
    { 
     lower = l; 
     upper = u; 
     value = v; 
    } 
    public int lower; 
    public int upper; 
    public int value; 
    } 

    static void Main (string [] args) 
    { 
    Data [] 
     // at the moment, this is coded into the source 
     // it can be read from a file at run time instead 
     data = { new Data (0, 180, 14), new Data (180, 540, 28), new Data (540, 1068, 42) }; 

    int 
     // again, the default case is hard coded, it too can be loaded from a file 
     x = 56, 
     workedDays = 100; 

    foreach (Data item in data) 
    { 
     if (workedDays > item.lower && workedDays <= item.upper) 
     { 
     x = item.value; 
     break; 
     } 
    } 
    } 
} 

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