Игра "Угадай число"

@Mikle90 · Регистрация: 15.03.2012

Author24 — интернет-сервис помощи студентам

Написал программу угадывания числа. Программа работает, но есть проблема повторного вызова игры. Когда мы угадываем число, программа нам предлагает повторно начать игру, введя 1. Но после ввода 1, приложение завершает работу. Помогите разобраться

C++

#include<iostream>
#include<stdlib.h>
#include<time.h>
using namespace std;
int zagadka (int);
int main()
{
    setlocale(0,"");
    int chislo;
    char otvetili;
 
    cout << "Мое число между 1 и 1000\nВы можете его отгадать ?\nПожалуйста, напечатайте вашу первую догадку" << endl;
    cin >> chislo;
    int a = zagadka(chislo);
    if (a == 1) // не срабатывает это условие, когда число отгадали
    {
        cout << "Мое число между 1 и 1000\nВы можете его отгадать ?\nПожалуйста, напечатайте вашу первую догадку" << endl;
        cin >> chislo;
        zagadka(chislo);
    }
 
}
int zagadka (int chisl)
{
    srand(time(NULL));
    int zagadanoe = 1 + rand()%1000;
    int otvet;
 
    while (chisl != zagadanoe)
    {
        if (chisl < zagadanoe)
        {
            cout << "Мало. Введите снова" << endl;
            cin >> chisl;
        }
        if (chisl > zagadanoe)
        {
            cout << "Много. Введите снова" << endl;
            cin >> chisl;
        }
    }
 
    if (chisl == zagadanoe)
    {
        cout << "Отлично! Вы отгадали число" << endl
            << "Хотели бы сыграть еще раз ? (Да (1) или Нет (0))" << endl;
        cin >> otvet;
        if (otvet == 1)
            return 1;
        else if (otvet == 0)
            return 0;
    }
}

@The_bolT · 13.02.2013, 17:10

В вас otvet типа char а вы сравниваете как int значение!

C++

#include<iostream>
#include<stdlib.h>
#include<time.h>
using namespace std;
int zagadka (int);
int main()
{
    setlocale(0,"");
    int chislo;
    char otvetili;
 
    cout << "Мое число между 1 и 10\nВы можете его отгадать ?\nПожалуйста, напечатайте вашу первую догадку" << endl;
    cin >> chislo;
    int a = zagadka(chislo);
    if (a == 1) // не срабатывает это условие, когда число отгадали
    {
        cout << "Мое число между 1 и 10\nВы можете его отгадать ?\nПожалуйста, напечатайте вашу первую догадку" << endl;
        cin >> chislo;
        zagadka(chislo);
    }
 
}
int zagadka (int chisl)
{
    srand(time(NULL));
    int zagadanoe = rand()%10+1;
    char otvet;
 
    while (chisl != zagadanoe)
    {
        if (chisl < zagadanoe)
        {
            cout << "Мало. Введите снова" << endl;
            cin >> chisl;
        }
        if (chisl > zagadanoe)
        {
            cout << "Много. Введите снова" << endl;
            cin >> chisl;
        }
    }
 
    if (chisl == zagadanoe)
    {
        cout << "Отлично! Вы отгадали число" << endl
            << "Хотели бы сыграть еще раз ? (Да (1) или Нет (0))" << endl;
        cin >> otvet;
        if (otvet == '1')
            return 1;
        else if (otvet == '0')
            return 0;
    }
}

@Mikle90 · 13.02.2013, 17:13 **[ТС]**

Да, глупая ошибка

@Alexxx7 · 13.02.2013, 17:34

Всё верно в третий раз функия не вызовется, так как желание продолжить игру вы проверяете 1 раз, для постоянного отслеживания можно поменять if на while

Добавлено через 11 минут
Я бы упростил до такого

C++

#include<iostream>
#include<stdlib.h>
#include<time.h>
using namespace std;
void zagadka (void);
int main()
{
    setlocale(0,"");
    char a = '1';
    while(a == '1'){
        zagadka();
       cout << "Отлично! Вы отгадали число" << endl
            << "Хотели бы сыграть еще раз ? (Да (1) или Нет (0))" << endl;
       cin >> a;
    }
}
void zagadka (void)
{
    srand(time(NULL));
    int zagadanoe = 1 + rand()%1000;
    int otvet;
    cout << "Мое число между 1 и 1000\nВы можете его отгадать ?\nПожалуйста, напечатайте вашу первую догадку" << endl;
    cin >> otvet;
    while (otvet != zagadanoe)
    {
        if (otvet < zagadanoe)
            cout << "Мало. Введите снова" << endl;
        if (otvet > zagadanoe)
            cout << "Много. Введите снова" << endl;
            cin >> otvet;
    }
}

@makShot · 13.02.2013, 18:33

А еще можно добавить проверку числа a на валидные данные (на случай, если введено не 1 и не 0):

C++

#include<iostream>
#include<stdlib.h>
#include<time.h>
 
using namespace std;
 
void zagadka (void);
int main()
{
    setlocale(0,"");
    char a = '1';
    while(a == '1'){
        zagadka();
       cout << "Отлично! Вы отгадали число" << endl;
 
       do
       {
           if ((a != '1') && (a != '0'))
               cout <<  "Неверное значение. Пожалуйста, повторите попытку"    <<  endl;
 
           cout << "Хотели бы сыграть еще раз ? (Да (1) или Нет (0))" << endl;
           cin >> a;
       } while ((a != '1') && (a != '0'));
       
    }
}
void zagadka (void)
{
    srand(time(NULL));
    int zagadanoe = 1 + rand()%1000;
    int otvet;
    cout << "Мое число между 1 и 1000\nВы можете его отгадать ?\nПожалуйста, напечатайте вашу первую догадку" << endl;
    cin >> otvet;
    while (otvet != zagadanoe)
    {
        if (otvet < zagadanoe)
            cout << "Мало. Введите снова" << endl;
        if (otvet > zagadanoe)
            cout << "Много. Введите снова" << endl;
            cin >> otvet;
    }
}

@Mikle90 1 / 1 / 0 Регистрация: 15.03.2012 Сообщений: 23
	13.02.2013, 17:13 [ТС]	3
	Да, глупая ошибка 0

	13.02.2013, 18:33

Опции темы