On mouse down switch on / off

Question

Hi guys.

i have ran into yet a problem.

i want to make a object button that changes a bool when its clicked. problem is that i only get the log: Buildmode = true.

here is my current code for it:

 public bool buildmode;  void OnMouseDown()
 {
     //if this is a buildmodeSwitch, im having different switches later.
     if (buildmodeSwitch)
     {    
         //then, if buildmode = true
         if (buildmode == true)
         {
             //then set buildmode to false
             buildmode = false;
             Debug.Log("Buildmode = false");
         }
         //if buildmode = false
         
         
         if (buildmode == false)
         {
             //then set buildmode to true this is the only debug.log i get ;/
             buildmode = true;
             Debug.Log("Buildmode = true");
         }
     }
     else
     {
         Debug.Log("Not a Buildmodeswitch");
     }

ive also tried else and else if. didnt work either ;/ i need it to be public as i need to access it from another script too.
I have tried stating its state in "void start" too ,so that the the unity editor cant mess with it.

Any ideas or examples. Couldnt find any help that didn't include GUI.

Thanks -Gravn

Answer 1

Best Answer

Answer by aldonaletto · Apr 09, 2012 at 12:45 AM

If you just want to toggle the boolean, assign the negated (!) variable to itself:

public bool buildmode;

void OnMouseDown(){

 if (buildmodeSwitch)
 {  
    buildmode = !buildmode; // toggle the boolean
 }
 else
 {
    Debug.Log("Not a Buildmodeswitch");
 }

}

1 Show 1 · Share

Gravn · Apr 09, 2012 at 01:09 AM 0

Share

thanks man, keep doing what your doing :D Really helps a lot of people reaching their goals :D

Answer 2

Answer by kolban · Apr 09, 2012 at 01:09 AM

As for your original problem, it contains a coding error ... you have coded:

 if buildMode == true then
    set buildMode = false
 
 // At this point, buildMode will *ALWAYS* be false
 
 if buildMode == false then
    set buildMode = true

What I think you want is:

 if buildMode == true then
    buildMode = false
 else
    buildMode = true

Also, Aldo's suggestions is bang on ... but I wanted to explain what was wrong with the original code.

1 Show 1 · Share

Gravn · Apr 09, 2012 at 01:14 AM 0

Share

hmm, i did start out with else, but that didnt work either, so i changed it. Thanks though :)

On mouse down switch on / off

2 Replies

Unity Answers is in Read-Only mode

Follow this Question

Related Questions