Изменение цикла while для размещения двух ситуаций

Question

Предположим, у меня есть цикл while, который зависит от двух отдельных входов. В ситуации 1 цикл while примет значение 1, а в ситуации два - Cin.eof(). Есть ли способ, которым я могу сделать это эффективно? Чтобы быть более кратким:

string hello; 
cin >> hello; 

if(hello == "one") 
{ 
    //make the while loop depend on value 1 
} 
else if(hello == "two") 
{ 
    //make the while loop depend on value !cin.eof() 
} 

while(/*depends on above conditional*/) 
{} 

Я не хочу, чтобы сделать что-то вроде:

if(hello == "one) 
{ 
    while(1){} 
} 
else if(hello == "two") 
{ 
    while(!cin.eof){} 
} 

, поскольку цикл в то время как по существу делает то же самое в каждой ситуации.

Answer 1

Для удобства чтения и в интересах cohesion, я думаю, вы должны переместить содержимое цикла в отдельную функцию:

void DoSomething() { /* ... */ } 

// ... 
if(hello == "one) 
{ 
    while(1){ DoSomething(); } 
} 
else if(hello == "two") 
{ 
    while(!cin.eof){ DoSomething(); } 
} 

Легче видеть, что разные петли while делают то же самое, но их условия разные.

Answer 2

Просто используйте или (||) как условие в цикле while. Установите первое условие if(hello == "one"). Теперь у вас есть цикл while, который будет зацикливаться, если одно из условий: true.

bool value = hello == "one"; 
while (value || !cin.eof) {} 

Answer 3

Я считаю, что вы ищете что-то вроде этого:

while((hello == "one") || (hello == "two" && !cin.eof)) { 
} 

Этот код будет делать то, что вы хотите, потому что он проверяет «переменная„один“? Если да, продолжайте выполнение. Если это не так, он проверяет: есть ли переменная «два»? Если это так, он проверяет наличие cin.eof.

Если это не так, цикл не будет выполняться. (&& 1 в первом состоянии было опущено, потому что это всегда «истина», сравнявшись и бесконечный цикл)

Edit:

Чтобы упростить вещи, вы можете рассмотреть этот код (как это было предложено в комментариях) :

bool HelloIsOne = (strcmp(hello, "one") == 0); 
bool HelloIsTwo = (strcmp(hello, "two") == 0); 

while(HelloIsOne || HelloIsTwo && !cin.eof) { 
} 

кронштейны, которые я размещены в предыдущем примере, на самом деле не нужны, потому что && связывает сильнее, чем ||, но они помогают общую ясность кода.

Answer 4

Вы можете сделать что-то вроде этого:

#include <iostream> 
#include <string> 

using namespace std; 
int main() 
{ 
    string hello; 
    cin >> hello; 
    while(hello=="one"?1:(!cin.eof())) 
    { 
     //do stuff 
    } 
    return 0; 
} 

Он проверяет, является ли строка hello это «один», и если это правда, условие while является 1, иначе это !cin.eof() как вы хотели.

Answer 5

Если вы используете C++ 11:

#include <functional> 

auto check = (hello == "one") ? []() bool -> { return 1; } : 
           []() bool -> { return !cin.eof(); }; 
while(check) { 
}; 

Answer 6

Как об этом:

switch(hello) 
    { 
     case 'one': 
     { 
      for(; 1;); 
      { 
      // your loop here 
      } 
      break; 
     } 
     case 'two': 
     { 
      for(;!cin.eof;) 
      { 
      // your other loop here 
      } 
      break; 
     } 
     default: 
     { 
      cout << " shouldnt get here unless bad user input" << endl; 
      break; 
     } 
    }