부울에 따라 do/while 루프를 종료하는 데 문제가 있습니다. - C++

Question

수정 됨. 고맙습니다.

아래의 코드에서는 컴파일 및 실행되지만 do/while 루프가 작동하지 않습니다. 반환 값은 작동하지만 while 루프는이를 인식하지 않아야하며 무기한으로 이동합니다. 나는 진실하고 틀린 반환을받을 수있다. 둘 다 루프를 멈추지 않습니다. 정말로 여기에서 길을 잃었고 대답을 찾을 수 없습니다. 고맙습니다.

//helper function for inputGuess 
//checks if the user's guess is in the lowercase alphabet and that 
// it has not been guessed before 
bool Hangman::validGuess() { 
    //checks if the character guess is in the lowercase alphabet 
    if (guess >= 97 && guess <= 122) { 
    //checks if the user's guess has been guessed previously 
    if (guessed.size() > 0) { 
     for (int i = 0; i < guessed.size(); i++) { 
     cout << "enter for\n"; 
     cout << "guess[i]: " << guessed[i] << endl; 
     if (guess != guessed[i]) { 
      cout << "1st true: guess has not been guessed\n"; 
      return true; 
     } 
     else { 
      cout << "1st false: same letter\n"; 
      return false; 
     } 
     } 
    } 
    else { 
     cout << "2nd true: guessed size is 0\n"; 
     return true; 
    } 
    } 
    else { 
    cout << "2nd false: not alphabet\n"; 
    return false; 
    } 
} 

//gets input for guess, checks if guess is valid, adds guess to guessed 
void Hangman::inputGuess() { 
    bool valid = false; 
    do { 
    cout << "Please enter your guess: "; 
    cin >> guess; 
    cout << endl; 
    valid = validGuess(); 
    cout << "valid: " << valid << endl; 
    } while (valid == false); 
    guessed.push_back(guess); 
} 

Answer 1

guessed이 비어있을 때 정의되지 않은 반품 케이스가 있습니다. 기본값은 반환되지 않으므로 값이 비어 있습니다.

관계없이이 코드는 지나치게 복잡해 보입니다. 이 같은 것이 더 좋을 수도 있습니다.

bool HangMan::validGuess() { 
    //checks if the character guess is in the lowercase alphabet 
    if (isalpha(guess)) { 
     //checks if the user's guess has been guessed previously 
     if (find(guessed.begin(), guessed.end(), guess) != guessed.end()) { 
      cout << "1st false: same letter\n"; 
      return false; 
     } 
     else { 
      cout << "1st true: guess has not been guessed\n"; 
      return true; 
     } 
    } else { 
     cout << "2nd false: not alphabet\n"; 
     return false; 
    } 
} 

Answer 2

당신은 당신의 문제인 validGuess()에 대한 매개 변수로 추측을 전달해야합니다. 추가하려고 시도하는 대신

this->guess 

을 입력하십시오.