What is wrong with my RockPaperSissors script?

Question

I am a complete noob at programming and eger to lern, but i cant see to find out what the problem with this script it, after i chose my options nothing happens, it used to work, but then i changed "computer" from a random number to another player

 var ccc : GameObject;
 
 function OnGUI()
 {
     ccc = GameObject.Find("Camera");
 
     // check to see if the human has pressed a button
     var human = -1;
 
     // draw and check each button
     if (GUI.Button(Rect(40, 10, 80, 60), "Attack"))
         human = 0;
     
     if (GUI.Button(Rect(140, 10, 80, 60), "Range"))
         human = 1;
     
     if (GUI.Button(Rect(240, 10, 80, 60), "Defence"))
         human = 2;
     
     // if the human has pressed something, do the game logic.
     if(human > -1)
     {
         // choose a value for the computer
         var computer = -1;
         if (GUI.Button(Rect(40, 10, 80, 60), "Attack"))
             computer = 0;
         if (GUI.Button(Rect(140, 10, 80, 60), "Range"))
             computer=1;
         if (GUI.Button(Rect(240, 10, 80, 60), "Defence"))
             computer=2;
         
         if(computer > -1)
         {
             // set the results to equal to start
             var results=0;
             ccc.animation.Play("Intro");
         
             // did we win?
             if(human == 0 && computer == 2) results = 1;
             if(human == 1 && computer == 0) results = 1;
             if(human == 2 && computer == 1) results = 1;
             
             // did we lose?
             if(human == 0 && computer == 1) results = -1;
             if(human == 1 && computer == 2) results = -1;
             if(human == 2 && computer == 0) results = -1;
         }
     }
 }

Answer 1

Answer by Wolfram · Jan 12, 2013 at 12:28 AM

I haven't really used UnityGUI, but keep in mind that the GUI is build "on the fly" while executing OnGUI() each frame. So there will be three buttons. If you click one of them, "human" will be set, and only in this case three new buttons for the computer will be rendered. However, a) the button click has already been evaluated, b) these buttons overlay the "human" buttons at identical position. So none of the thre computer-"if"-conditions can ever be true.

If that is not the solution, you'd have to explain the problem some more. What are these computer buttons supposed to do, how do you intend to choose a value for computer, what do you mean by "i changed "computer" from a random number to another player", what is the problem you are seeing?

0 Show 1 · Share

AndyMartin458 · Jan 12, 2013 at 04:43 AM 0

Share

Yes, that's what is strange. I think that by "changed computer to player", this really means that buttons have been added for the computer. That is definitely why it is important to set the values of a class scope variable.

What is wrong with my RockPaperSissors script?

1 Reply

Your answer

Welcome to Unity Answers

Follow this Question

Related Questions