Making a 2 player rock paper scissors game in c. I. Am a beginner.

Question

1.00/5 (2 votes)

See more:

C

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

/*coded by Roman R*/


    char P1_Name[20], P2_Name[20];
    char rock, paper, scissors;
    char P1Choice, P2Choice, Greaterchoice;

    char rock_paper_scissors(P1_Name, P2_Name) {
        char Greaterchoice;
    if (P1Choice==rock && P2Choice==scissors || P1Choice==scissors && P2Choice==paper || P1Choice==paper && P2Choice==rock) {
        Greaterchoice = 0;
    } else if (P1Choice==rock && P2Choice==paper || P1Choice==paper && P2Choice==scissors || P1Choice==scissors && P2Choice==rock) {
        Greaterchoice = 1;
    } else {
        Greaterchoice = 2;
    }
   
    printf("Player1 Enter your name.:");
        scanf ("%s", P1_Name);
    printf("Player2 Enter your name.:");
        scanf ("%s", P2_Name);
       
char get_p1_input() {
    char P1Choice
    printf("Enter rock paper or scissors:\n")
    scanf("%c", &P1Choice);
    return P1Choice;
    }
char get_p2_input() {
    char P2Choice
    printf("Enter rock paper or scissors:\n")
    scanf("%c", &P2Choice);
    return P2Choice;
    }
           
int main() {
        if (Greaterchoice==0) {
        printf("%s", Won, P1_Name);
        }
       
        if (Greaterchoice==1) {
        printf("%s", won, P2_Name)
        }
       
        if (Greaterchoice==2) {
        printf("It,s a draw");
        }
    return 0;
}

What I have tried:

I am a beginner and would please like some advice how to complete this 2 player rock paper scissors game. I am studying but I am having a hard time to find the right answers. Please come at this assuming i have little knowledge on the terminology. Thank you!

Posted 3-Dec-20 6:54am

Member 14999245

Updated 4-Dec-20 5:46am

CPallini

v2

Add a Solution

Comments

Rick York 3-Dec-20 16:17pm

You should indent your code better. It is much easier to follow and understand when you do.

W∴ Balboos, GHB 4-Dec-20 7:43am

You say you had a "hard time to find the right answer". I am hoping you're trying to learn to program. You shouldn't be looking for "the right answer". You should be looking on how to create an answer. How to do some specific thing - which is learning how to do something you can use again and again.

5 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

CPallini · Answer 1 · 2020-12-03T20:50:00

I'm giving you a working version, albeit minimalistic, of your code. Note there aren't global variables (they are evil).

C

#include <stdio.h>

const char Rock = 'r';
const char Scissors = 's';
const char Paper = 'p';

char get_input(int player_no)
{
  while (1)
  {
    printf("Player %d, please enter Rock (r) Paper (p) or Scissors (s):\n", player_no);
    char choice = getchar();
    getchar(); // rempve the newline in the input buffer
    if ( (choice == Rock) || (choice == Scissors) || (choice == Paper))
      return choice;
  }
}

int winner_number( char p1_choice, char p2_choice)
{
  if ( (p1_choice==Rock && p2_choice==Scissors) || (p1_choice==Scissors && p2_choice==Paper) || (p1_choice==Paper && p2_choice==Rock))
  {
    return 1;
  }
  else if ( (p1_choice==Rock && p2_choice==Paper) || (p1_choice==Paper && p2_choice==Scissors) || (p1_choice==Scissors && p2_choice==Rock) )
  {
    return 2;
  }
  return 0;
}

int main()
{
  char answer;
  do
  {
    char p1_choice = get_input(1);
    char p2_choice = get_input(2);
    int winner = winner_number( p1_choice, p2_choice);
    if ( winner == 0)
      printf("it is a draw\n");
    else
      printf("the winner is player %d\n", winner);

    printf("do you wish to continue?\n");
    answer = getchar();
    getchar(); // remove the newline in the input buffer
  } while ( answer == 'y');
  return 0;
}

OriginalGriff · Answer 2 · 2020-12-03T07:46:00

That will not compile.

You cannot execute code unless it is inside a function in C - the system wouldn't know what code you want to starts with!
So you need a main function so that the system knows where to start executing code - it calls that when your app starts, and it can call any other functions you need to.

Patrice T · Answer 3 · 2020-12-03T08:41:00

Quote:
I am studying but I am having a hard time to find the right answers.

Your code do not behave the way you expect, or you don't understand why !

There is an almost universal solution: Run your code on debugger step by step, inspect variables.
The debugger is here to show you what your code is doing and your task is to compare with what it should do.
There is no magic in the debugger, it don't know what your code is supposed to do, it don't find bugs, it just help you to by showing you what is going on. When the code don't do what is expected, you are close to a bug.
To see what your code is doing: Just set a breakpoint and see your code performing, the debugger allow you to execute lines 1 by 1 and to inspect variables as it execute.

Debugger - Wikipedia, the free encyclopedia[^]

Mastering Debugging in Visual Studio 2010 - A Beginner's Guide[^]
Basic Debugging with Visual Studio 2010 - YouTube[^]

1.11 — Debugging your program (stepping and breakpoints) | Learn C++[^]

The debugger is here to only show you what your code is doing and your task is to compare with what it should do.

Member 14999245 · Answer 4 · 2020-12-04T04:25:00

Solution 4

Wow Thank you for the code. It helps me to understand how code works. I am going to see if I can modify it. I will post back with what I have.

Posted 4-Dec-20 4:25am

Member 14999245

Comments

Richard MacCutchan 4-Dec-20 11:10am

This is not a solution. You should use the Have a Question or Comment? link below CPallini's message, so he gets notified of your comment.

Member 14999245 · Answer 5 · 2020-12-04T05:46:00

Solution 5

Can i ask questions CPallini? If there is a better way you would prefer please let me know. So on line 9 you state that while parameter is 1 printf. What is 1 referring to? I assume while 1 means while true. True to what or in what aspect?

Posted 4-Dec-20 5:46am

Member 14999245

Comments

CHill60 4-Dec-20 12:35pm

If you want to comment on a solution use the "Have a Question or Comment?" link next to it. Do not post comments as solutions - the member will not be notified of your response.
You have already been told this!

CPallini 4-Dec-20 14:32pm

Yes you may well ask. It would have been better asking directly (At the bottom of my solution). Anyway, according to C programming language:
https://en.cppreference.com/w/c/language/while
So, yes, 1 (like any non-zero value) is interpreted as 'true' that is the loop is executed.

Making a 2 player rock paper scissors game in c. I. Am a beginner.

5 solutions

Solution 3

Solution 1

Solution 2

Solution 4

Solution 5

Add your solution here

Preview 0