застрял в бесконечном do while loop

Question

Я пытаюсь сделать простую догадки моей номерной игры в C++, но компьютер должен угадать мой номер. Но проблема в том, что я застрял в этом бесконечном цикле. Я просто начинающий, поэтому это действительно базовая программа.

Это мой код:

int secretNumber = rand() %100 + 1; // random number between 1-100 
int tries=0; 
int input; 

cout <<"typ your number\n"; 
cin >> input; 

do 
{ 
    cout <<secretNumber <<endl; 
    ++tries; 

    if (secretNumber > input) 
    { 
     cout <<"To high i guess?\n"; 
    } 
    else if (secretNumber < input) 
    { 
     cout <<"To low I guess?\n"; 
    } 
    else 
    { 
     cout <<"Yes, i got it in " <<tries <<" tries!"; 
    } 
}while (input != secretNumber); 

return 0; 

} 

Answer 1

компьютер только гадать (случайным) один раз при запуске программы, но должен угадать каждый раз цикл повторяется. вы должны переместить генерацию случайных чисел внутри цикла:

int secretNumber = 0; 
int tries=0; 
int input; 

cout <<"typ your number\n"; 
cin >> input; 

do 
{ 
    /* the fix is here */ 
    secretNumber = rand() %100 + 1; // random number between 1-100 
    cout <<secretNumber <<endl; 
    ++tries; 

    if (secretNumber > input) 
    { 
     cout <<"To high i guess?\n"; 
    } 
    else if (secretNumber < input) 
    { 
     cout <<"To low I guess?\n"; 
    } 
    else 
    { 
     cout <<"Yes, i got it in " <<tries <<" tries!"; 
    } 
}while (input != secretNumber); 

return 0; 

Answer 2

место cin >> input в теле цикла

Answer 3

Значение переменной input никогда не изменяется в цикле, поэтому условие терминатор input != secretNumber никогда не встречались.

Вы должны взять входной сигнал внутри цикла. Поэтому напишите cin >> input в начале цикла.

Edit:

Если компьютер должен угадать, то до сих пор значение input должно быть изменено в цикле, который не присутствует в вашем коде. Цикл работает с тем же значением в input каждый раз.

Чтобы заставить вас сделать предположение, вы должны следовать какой-либо схеме. Компьютер может набирать цифры случайным образом - вы можете пройти через перемещение secretNumber = rand()%100 + 1 внутри цикла. Но этот подход может не работать хорошо, цикл может работать очень долго. Это показано в ответе @ Kaii.

Более эффективный подход - Binary Search. В этом случае вы должны следить за догадками, которые делает компьютер. Сохраняйте две переменные high и low, которые должны хранить догадки выше и ниже input соответственно. Всякий раз, когда угадывайтесь выше номера, храните его в high и сохраняйте любую догадки ниже input в low. Затем компьютер должен попробовать свое новое предположение между high и low. Случайное предположение должно быть secretNumber = low + rand() % (high - low). В худшем случае это займет целых 100 итераций. Для достижения наилучших результатов каждая догадка должна быть (high + low)/2. В соответствии с условиями один из high и low будет обновляться на каждой итерации. Такой подход обеспечит, что компьютер угадает правильный номер в пределах 7 догадок.

В коде это должно быть так:

int secretNumber = rand() % 100 + 1; // random number between 1-100 
int tries=0; 
int input; 
int low = 1, high = 100; 

cout <<"typ your number\n"; 
cin >> input; 

do 
{ 
    secretNumber = (high + low)/2; 
    cout << secretNumber <<endl; 
    ++tries; 

    if (secretNumber > input) 
    { 
     cout << "Too high I guess?\n"; 
     high = secretNumber; 
    } 

    else if (secretNumber < input) 
    { 
     cout << "Too low I guess?\n"; 
     low = secretNumber; 
    } 

    else 
    { 
     cout << "Yes, i got it in " << tries << " tries!"; 
    }   
} while (input != secretNumber); 

return 0; 

Answer 4

Вы должны сделать вход в цикле. Прямо сейчас вы запрашиваете значение ПЕРЕД началом цикла, а затем никогда не запрашиваете у пользователя другого номера. Если догадка не может быть изменена, цикл никогда не сможет выйти.

изменяется на

do { 
    cin >> input; 

будет решать бесконечную петлю.

Answer 5

Думаю, вы должны переместить генерацию случайных чисел внутри цикла.

Answer 6

бесконечный цикл в вашем коде, потому что аргумент while всегда верен, пока не получите истинное число.

вам нужно добавить еще один аргумент. Например, вы хотите установить максимальную попытку в 5 раз.то это будет как этот

do{ 
    //do your stuff here 
}while((input != secretNumber)&&(tries<=5)) 

при вводе неправильный номер в 5 раз, приложение завершит