Расширенный оператор switch в цикле while? - PullRequest
8 голосов
/ 07 мая 2010

Я только начал C ++, но у меня есть некоторые предварительные знания по другим языкам (vb некоторое время назад, к сожалению), но у меня странное затруднение. Мне не нравилось использовать так много операторов IF, и я хотел использовать switch / case так, как это казалось более чистым, и я хотел получить на практике ... Но ..

Допустим, у меня есть следующий сценарий (теоретический код):

while(1) {

  //Loop can be conditional or 1, I use it alot, for example in my game
  char something;
  std::cout << "Enter something\n -->";
  std::cin  >> something;

  //Switch to read "something"
  switch(something) {
    case 'a':
      cout << "You entered A, which is correct";
      break;
    case 'b':
      cout << "...";
      break;
  }
}

И это моя проблема. Допустим, я хотел выйти из цикла WHILE, для этого потребовалось бы два оператора break?

Это явно выглядит неправильно:

case 'a':
  cout << "You entered A, which is correct";
  break;
  break;

Так что я могу сделать только оператор IF для 'a', чтобы использовать break ;? Я что-то упускаю очень просто?

Это решило бы многие мои проблемы, которые у меня сейчас есть.

Ответы [ 9 ]

32 голосов
/ 07 мая 2010

Я бы преобразовал проверку в другую функцию.

bool is_correct_answer(char input)
{
    switch(input)
    {
    case 'a':
        cout << "You entered A, which is correct";
        return true;
    case 'b':
        cout << "...";
        return false;
    }
    return false;
}

int main()
{
    char input;
    do
    {
        std::cout << "Enter something\n -->";
        std::cin  >> input;
    } while (!is_correct_answer(input));
}
13 голосов
/ 07 мая 2010

Можно просто проверить цикл while для значения bool, установленного в одном из ваших операторов case.

bool done = false;    
while(!done)
{
 char something;
  std::cout << "Enter something\n -->";
  std::cin  >> something;

  //Switch to read "something"
  switch(something) {
    case 'a':
      cout << "You entered A, which is correct";
      done = true; // exit condition here
      break;
    case 'b':
      cout << "...";
      break;
  }
}
5 голосов
/ 07 мая 2010

Два break оператора не выведут вас из цикла while. Первый break только выводит вас из оператора switch, а второй никогда не достигается.

Вам нужно сделать условие цикла while ложным, предполагая, что в цикле нет ничего после оператора switch. Если после переключателя есть другой код, вы должны проверить состояние после switch и break там.


bool done = false;

while(! done)
{
  // do stuff
  switch(something)
  {
    case 'a':
    done = true;  // exit the loop 
    break;
  }

  //  do this if you have other code besides the switch
  if(done)
   break;  // gets you out of the while loop

  // do whatever needs to be done after the switch

}

5 голосов
/ 07 мая 2010

Да, C и C ++ не могут сказать «выход из нескольких разрушаемых блоков» (где «разрушаемый блок» - это любой цикл или переключатель). Обходные пути включают goto s и использование логических переменных для записи того, должен ли также сломаться внешний «разрушаемый блок» (ни элегантно, но, это жизнь).

3 голосов
/ 07 мая 2010

Вы можете попробовать:

  • Использование флагов
  • Использование Goto
  • Наличие внутреннего разрушаемого блока в функции
  • Использование исключений
  • Использование longjump и setjmp

Тема, очень похожая на этот вопрос

http://www.gamedev.net/community/forums/topic.asp?topic_id=385116

1 голос
/ 09 мая 2010

Вас может заинтересовать именованная идиома цикла в C ++.

#define named(blockname) goto blockname; \
                         blockname##_skip: if (0) \
                         blockname:

#define break(blockname) goto blockname##_skip;

named(outer)
while(1) {

  //Loop can be conditional or 1, I use it alot, for example in my game
  char something;
  std::cout << "Enter something\n -->";
  std::cin  >> something;

  //Switch to read "something"
  switch(something) {
    case 'a':
      cout << "You entered A, which is correct";
      break(outer);
    case 'b':
      cout << "...";
      break(outer);
  }
}
1 голос
/ 07 мая 2010

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

0 голосов
/ 07 мая 2010

Вы можете заменить коммутатор на слегка переработанное решение OO ...

#include <iostream>
#include <map>
#include <set>

class input_responder
{
    std::set<char> correct_inputs;
    std::map<char, const char*> wrong_inputs;

public:

    input_responder()
    {
        correct_inputs.insert('a');
        wrong_inputs['b'] = "...";
    }

    bool respond(char input) const
    {
        if (correct_inputs.find(input) != correct_inputs.end())
        {
            std::cout << "You entered " << input << ", which is correct\n";
            return true;
        }
        else
        {
            std::map<char, const char*>::const_iterator it = wrong_inputs.find(input);
            if (it != wrong_inputs.end())
            {
                std::cout << it->second << '\n';
            }
            else
            {
                std::cout << "You entered " << input << ", which is wrong\n";
            }
            return false;
        }
    }
};

int main()
{
    const input_responder responder;
    char input;
    do
    {
        std::cout << "Enter something\n -->";
        std::cin  >> input;
    } while (responder.respond(input) == false);
}
0 голосов
/ 07 мая 2010

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

...