GUI, making main page lead into an instructions page

Here is the full code just in case.

 using UnityEngine;
 using System.Collections;
 [RequireComponent (typeof (AudioSource))]
 
 public class $$anonymous$$ain$$anonymous$$enuGUI : $$anonymous$$onoBehaviour 
 {
     public AudioClip beep;
     public GUISkin menuSkin;
     public Rect menuArea;
     public Rect playButton;
     public Rect quitButton;
     public Rect instructionsButton;
     public Rect menuAreaNormalized;
     public string menuPage = "main";
     public Rect instructions;
         
     void Start()
     {
         //following 3 lines are used to make the game work on all resolutions
         menuAreaNormalized = new Rect(menuArea.x * Screen.width - (menuArea.width * .5f),
             menuArea.y * Screen.height - (menuArea.height *.5f),
             menuArea.width, menuArea.height);
             
     }
     
     void OnGUI()
     {
         // makes all gui elements have this skin. $$anonymous$$akes it easier to edit all elements in one go
         GUI.skin = menuSkin;
         GUI.BeginGroup (menuAreaNormalized);
         //making button 1... we access GUI class, it's button component, pass in a rect value, and then assign it stuff.
         if(menuPage == "main")
         {
             if(GUI.Button (new Rect(playButton), "Play"))
             {
                 //we send "island" to our buttonaction function, to load the island level
                 StartCoroutine("ButtonAction", "Island");
             }
             //button 2...
             if(GUI.Button (new Rect(instructionsButton), "Instructions"))
             {
             audio.PlayOneShot(beep);
             //doesn't enable play or quit buttons, sets page to instructions page
             menuPage = "instructions";
             }
             //button 3...
             if(GUI.Button (new Rect(quitButton), "Quit"))
             {
                 //we send "quit" to our buttonaction function...
                 StartCoroutine("ButtonAction", "quit");
             }
             else if (menuPage == "instructions")
             {
                 Debug.Log ("does it work?");
                 GUI.Label(new Rect(instructions),
                     "You awake on a mysterious island... Find a way to signal for help or face certain doom!");
                 Debug.Log("doesitwork2?");
                 if(GUI.Button(new Rect(quitButton), "Back"))
                 {
                     Debug.Log ("doesitwork3");
                     audio.PlayOneShot(beep);
                     menuPage="main";
                 }
             }
         }
         
         GUI.EndGroup ();
     }
     
     IEnumerator ButtonAction(string levelName)
     {
         audio.PlayOneShot(beep);
         yield return new WaitForSeconds(.35f);
         
         if(levelName != "quit")
         {
             Application.LoadLevel(levelName);
         }
         else
         {
             Application.Quit();
             Debug.Log ("Have Quit");
         }
         
     }
     
     // Update is called once per frame
     void Update () {
     
     }
 }
 
 

generally, beginners should not use else-if it's just too easy to screwup.

t's just too confusing. i have a lifetime of experience in softwar and I can't understand your code , at all.

re-write using simple "breakaway" code. it's a technique you need to master and extremely simple.

so it's something like this ..

 if ( quit ..)
 {
 do that stuff
 do that stuff
 return;
 }
 
 
 if ( button down ..)
 {
 do that stuff
 do that stuff
 return;
 }
 
 
 if ( button up ..)
 {
 do that stuff
 do that stuff
 return;
 }
 
 
 if ( instructions ..)
 {
 do that stuff
 do that stuff
 return;
 }
 
 
 if ( quit button pressed ..)
 {
 do that stuff
 do that stuff
 return;
 }

notice each block ends with "return;" You get the idea right?

So re-write everything that way.

No program$$anonymous$$ technique is perfect in all situations, but this is what you need here and now.

Secondly. you see you have a huge amount of code under if(menuPage == "main")

You must change it like this

  if (menuPage == "main")
    {
    _hereIHandleA$$anonymous$$ain();
    }
 
 function _hereIHandleA$$anonymous$$ain()
   {
   lots of code here
   }

A person just can't exist with untidy code. You must tidy up first eh !

It's like having a dirty, unkept bed. If you can't keep the sheets on your bed spotless, instantly kill yourself to make more room on the planet for other people :)

Similarly you must tidy-up your code ok?

try writing "breakaway" code like above as that will probably help dramatically in the first instance.

don't use "else-if" if you're a beginner. it's just too easy to get confused.

Thank you for the response.

I understood my code up until he add us add the else if, so thank you for the alternative. I would have approached this differently normally, but I was trying to follow the books instructions. No offense to the books author though.. because I am pretty sure I did something wrong somewhere to make it look like that.

Anyways, I will try writing the code in the way that you've recommended. Thanks.