GUI menu how to make a list in a list. C#

Question

I'm still learning Unity and C# so I need help with this. I'm creating a main menu which I want it to say play game, you click on it then host or refresh and if you click host I want another page of settings to start hosting the server, but instead when I click Play Game it skips the page with Host and Refresh and goes straight to the Host menu. I was wondering how I would change my code or add something to it to allow me to do what I want. All the codes are in bits because I'm only including what I think is needed. C#

 private string clicked = "";
 
 private void OnGUI()



         if (GUI.Button(new Rect(Screen.width * (1.3f/18.55f),Screen.height * (50.08f/100.3f),Screen.width * (1.05f/6.55f), Screen.height * (0.45f/7.0f)),"Play Game"))
         {
             //code on what to do after clicked play
             clicked = "Play Game";
         }




     else if (clicked == "Play Game")
     {
         GUI.Button(new Rect(Screen.width * (1.3f/18.55f),Screen.height * (50.08f/100.3f),Screen.width * (1.05f/6.55f), Screen.height * (0.45f/7.0f)),"Host");
         {
             //What to do when Host is clicked
             clicked = "Host";
         }

Answer 1

Answer by fafase · Feb 14, 2013 at 01:36 PM

My first advice, avoid string for states but maybe an enum or some booleans. First because it is slower than enum and second because you are more prone to error since an enum will not let you use anything else than what is in.

Now for your issue:

 using UnityEngine;
 using System.Collections;

 public class NewBehaviourScript : MonoBehaviour {
 enum State{
    Play, HostRef, HostInfo
 }
 
 State current;
 void Start(){
     current = State.Play;
 }
  
 void OnGUI(){
     if(current ==State.Play){
         if (GUI.Button(new Rect(Screen.width * (1.3f/18.55f),Screen.height * (50.08f/100.3f),Screen.width * (1.05f/6.55f), Screen.height * (0.45f/7.0f)),"Play Game"))
         {
         //code on what to do after clicked play
         current = State.HostRef;
         }
     }else if(current ==State.HostRef){
         if (GUI.Button(new Rect(Screen.width * (1.3f/18.55f),Screen.height * (50.08f/100.3f),Screen.width * (1.05f/6.55f), Screen.height * (0.45f/7.0f)),"Hosting"))
         {
         //code on what to do after clicked play
             current = State.HostInfo;
         }
         if (GUI.Button(new Rect(Screen.width * (1.3f/18.55f),Screen.height * (50.08f/100.3f)*1.5f,Screen.width * (1.05f/6.55f), Screen.height * (0.45f/7.0f)),"Refresh"))
         {
         //code on what to do after clicked play
                 current = State.Play;
         }
     }else if(current == State.HostInfo){
     // Host info page
     }
 }

}

I assume Refresh is for going back but you put your own logic there

Add comment · Show 5 · Share

ItsLowee · Feb 17, 2013 at 01:45 AM 0

Share

I get an error, (7,7): error CS0116: A namespace can only contain types of namespace declarations.

fafase · Feb 17, 2013 at 08:37 AM 0

Share

I edited the answer, I was missing a } but your error does not pop up for me. You might have something missing like the namespace at the top.

ItsLowee · Feb 17, 2013 at 09:12 AM 0

Share

That fixed one error but i have another, CS(35,1): error CS8025: Parsing error, Even when i just copy and pasted the code.

fafase · Feb 17, 2013 at 11:15 AM 0

Share

I guess you missed the last } which is out of the code. One good thing when you code is to try and understand the error. here it is telling parsing error and probably something about missing }. That means you are missing a }. Parsing means you do not have the matching number of {}.

ItsLowee · Feb 17, 2013 at 11:33 AM 0

Share

I've guessed that and have tried it, thanks for confirming it but no matter what I do with {} it always has that error, i'll probably go back to the old code of the main menu. Thanks

GUI menu how to make a list in a list. C#

1 Reply

Your answer

Welcome to Unity Answers

Follow this Question

Related Questions