Riddhi Panja Ответов: 2

Исправление ошибок. Printf показывает дважды, но scanf принимает только один раз в цикле while.


#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int RockPaperScissor(char you, char computer)
{
    // returns 1 if you win, -1 if you lose and 0 if it is a draw
    // Condition for draw
    // Cases covered:
    // rock-rock
    // paper-paper
    // scissor-scissor
    if (you == computer)
    {
        return 0;
    }

    // Non-draw conditions
    // Cases covered:
    // rock-paper
    // paper-rock
    // rock-scissor
    // scissor-rock
    // paper-scissor
    // scissor-paper

    if (you == 'r' && computer == 'p')
    {
        return -1;
    }
    else if (you == 'p' && computer == 'r')
    {
        return 1;
    }

    if (you == 'r' && computer == 's')
    {
        return 1;
    }
    else if (you == 's' && computer == 'r')
    {
        return -1;
    }

    if (you == 'p' && computer == 's')
    {
        return -1;
    }
    else if (you == 's' && computer == 'p')
    {
        return 1;
    }
}
int main()
{
    int i = 1, your_count = 0, comp_count = 0;
    char you, computer;

    printf("--------------------\t\tRock Papers Scissors\t\t--------------------\n\n");
    printf("--------------------\t\t(First to 3)\t\t--------------------\n\n\n\n");

    while (i != 0)
    {
        //taking user's choice
        printf("\n\nEnter 'r' for Rock,  'p' for Paper,  's' for Scissor: ");
        scanf("%c", &you);

        srand(time(0)); //generating a random choice from computer
        int number = rand() % 100 + 1;

        if (number < 33)
        {
            computer = 'r';
        }
        else if (number >= 33 && number < 66)
        {
            computer = 'p';
        }
        else
        {
            computer = 's';
        }

        //Calling the game function to decide winner
        int result = RockPaperScissor(you, computer);

        //Showing the decisions
        printf("\nYou chose %c and Computer chose %c. ", you, computer);

        //Winner showcase
        if (result == 0)
        {
            printf("\n\nThe round Tied.");
        }
        else if (result == 1)
        {
            your_count++;
            printf("\nYou won %d times.", your_count);
        }
        else if (result == -1)
        {
            comp_count++;
            printf("\nComputer won %d times.", comp_count);
        }
        if (your_count == 3 || comp_count == 3)
        {
            i = 0;
        }
        else
        {
            i = 1;
        }
    }
    if (your_count > comp_count)
    {
        printf("\n\nYou won the game!!");
    }
    else
    {
        printf("\n\nComputer won the game!!");
    }
    return 0;
}


Что я уже пробовал:

Я попробовал использовать for loop, но не смог понять, как его использовать здесь.

2 Ответов

Рейтинг:
2

Patrice T

Цитата:
Я попробовал использовать for loop, но не смог понять, как его использовать здесь.

Ваш код ведет себя не так, как вы ожидаете, или вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш код, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]

1.11 — отладка программы (пошаговое выполнение и останова) | выучить C++[^]

Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.


Рейтинг:
0

CPallini

Проблема возникает из-за наличия дополнительных символов во входном буфере. Обходной путь может заключаться в их отбрасывании, например
вместо

Цитата:
scanf("%c", &you);
использовать
do
{
  scanf("%c", &you);
} while ( ! is_allowed(you) );
с
int is_allowed(char choice)
{
  switch (choice)
  {
    case 'r':
    case 'p':
    case 's':
    return 1;
  }
  return 0;
}