Как сделать этот код кода короче

-4

У меня есть код, который переключает значения в struct.Как сделать этот код кода короче

Например, я считаю себя писать код много, что выглядит следующим образом:

if(options.test == 1) 
{ 
    options.test = 2; 
} 
else if (options.test == 2) 
{ 
    options.test = 1; 
}

Есть ли более компактная версия, которую я могу использовать, чтобы сделать этот код короче?

источник

2014-03-10 user9993

Вы используете оператор компаньона '==' внутри скобок. Я предполагаю, что вы хотите присвоить '='? –

Упс, отредактирован. Благодарю. – user9993

Если у вас есть только 2 состояния, вы можете использовать int как bool вместо этого и просто использовать «значение» и «значение». Другим решением будет оператор switch. –

Если значения options.test либо 1 или 2 на протяжении выполнения программы, то вы можете просто сделать:

options.test = 3-options.test;

Если эта переменная может быть установлена на другие значения, то лучший способ справиться с это, как правило, с:

switch (options.test) 
{ 
    case 1: options.test = 2; break; 
    case 2: options.test = 1; break; 
    case ...: options.test = ...; break; 
    case ...: options.test = ...; break; 
    case ...: options.test = ...; break; 
    default: options.test = ...; break; 
}

Если значения от 0 до N (с относительно небольшим N), то вы можете также рассмотреть вопрос о хэширования.

Например, вместо:

switch (options.test) 
{ 
    case 0: options.test = 4; break; 
    case 1: options.test = 2; break; 
    case 2: options.test = 1; break; 
    case 3: options.test = 3; break; 
    case 4: options.test = 5; break; 
    case 5: options.test = 0; break; 
}

Вы можете сделать:

static int hash[] = {4,2,1,3,5,0}; 
options.test = hash[options.test];

источник

2014-03-10 11:58:57

Я тоже думал об этом, но мальчик сделает код более трудным для чтения и поддержки. –

Это всего лишь образец кода, и я сомневаюсь, что у него есть только два значения. –

@Joe White, хорошо - OP просит «более короткий» способ, не так ли? (хотя точно так же, как вы, я не совсем уверен, что бы это получилось) ... –

Похоже, это довольно общий вопрос, а не точный пример. Я считаю, что для этих простых утверждений одной линии, бросающих брекеты все еще отлично читается:

if(options.test == 1) 
    options.test = 2; 
else if (options.test == 2) 
    options.test = 1;

источник

2014-03-10 12:01:35 PeterJ

hahhaha .. он просит 1 код строки ... не удаление брекетов ... –

@AxitSoni, где OP запросил одну строку кода? Вопрос был для * короче * кода. Это полностью соответствует. –

жаль, что он отредактировал. может быть .. сначала, когда я увидел этот вопрос в то время, он есть :) –

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

options.test = setOptionTestValue(options.test);

метод

int setOptionTestValue(value) { 
    switch (value) { 
     case 1: return 2; 
     case 2: return 1; 
     default: return 0; 
    } 
}

источник

2014-03-10 12:01:42 KarelG

Хороший ответ, хотя это немного не соответствует критериям * короче *, которые ОП спрашивает – streppel

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

switch (options.test) { 
    case 1: 
     options.test = 2; 
     break; 
    case 2: 
     options.test = 1; 
     break; 
}

Если у вас есть только этот код, и ничего больше, используйте:

options.test = 3 - options.test;

источник

2014-03-10 12:02:27 ReeCube

Это выглядит это может быть state machine, тем более, что вы сказали, что повторяете этот код несколько раз (один раз для каждого типа перехода состояния?). Что делать с функцией, которая возвращает новое состояние?

int getNewStateAfterLogin(int oldState) { 
    switch (oldState) { 
     case 1: 
      return 2; 
     case 2: 
      return 1; 
     default: 
      return oldState; 
    } 
} 

options.test = getNewStateAfterLogin(options.test);

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

источник

2014-03-10 12:05:45

Для ввода {1,2}:

int map[3] = {0, 2, 1}; 

options.test = map[options.test];

Регулировка по мере необходимости для другого/ввода дополнительной.

источник

2014-03-10 12:06:53 alk

Как сделать этот код кода короче

ответ

Смежные вопросы