• Products
  • Solutions
  • Made with Unity
  • Learning
  • Support & Services
  • Community
  • Asset Store
  • Get Unity

UNITY ACCOUNT

You need a Unity Account to shop in the Online and Asset Stores, participate in the Unity Community and manage your license portfolio. Login Create account
  • Blog
  • Forums
  • Answers
  • Evangelists
  • User Groups
  • Beta Program
  • Advisory Panel

Navigation

  • Home
  • Products
  • Solutions
  • Made with Unity
  • Learning
  • Support & Services
  • Community
    • Blog
    • Forums
    • Answers
    • Evangelists
    • User Groups
    • Beta Program
    • Advisory Panel

Unity account

You need a Unity Account to shop in the Online and Asset Stores, participate in the Unity Community and manage your license portfolio. Login Create account

Language

  • Chinese
  • Spanish
  • Japanese
  • Korean
  • Portuguese
  • Ask a question
  • Spaces
    • Default
    • Help Room
    • META
    • Moderators
    • Topics
    • Questions
    • Users
    • Badges
  • Home /
  • Help Room /
avatar image
Question by skinchlea · Jan 30, 2019 at 02:02 AM · state-machineclickable

Ignoring button clicks for second action

Sorry for the title, I didn't know how to word it correctly.

Let's say I have a script:
This script has 2 states.

State 1: Idle
State 2: Unit Selected

State 1: if (left mouse button is clicked) {
Highlight friendly unit
switch to unit selected state
}

State 2: if (left mouse button is clicked) {
move unit to click position
}

Now the problem I'm having is that when I click to select a unit in the first state, the code accepts that click, but will switch states, get the location of the click and spawn the unit all within that same click of the button. Is there a way I can make the code take the first click then wait for the second click to initiate the second state?

I've tried switching the states only on mouse button up, but this does not seem to work.
Here's the actual code if my explanation doesn't make any sense.

 if (currentState == State.Empty) { //NOTHING SELECTED
 
             if (Input.GetMouseButtonDown (0)) { 
 
                 RaycastHit hit;
                 Ray ray = Camera.main.ScreenPointToRay (Input.mousePosition);
 
                 if (Physics.Raycast (ray, out hit, 100.0f)) {
                     Debug.Log ("You selected " + hit.transform.name); 
 
                     //IF PLAYER CLICKS ON A UNIT
                     if (hit.transform.tag == "Unit") { 
                         SelectedUnit = hit.transform.gameObject; 
                             currentState = State.Unit;
                     }
                 }
             }
         }
 
 
                 if (currentState == State.Unit) { //UNIT CURRENTLY SELECTED
 
                 if (Input.GetMouseButtonDown (0)) { 
                 RaycastHit hit;
                 Ray ray = Camera.main.ScreenPointToRay (Input.mousePosition);
                 SetTilePos ();
 
                 if (Physics.Raycast (ray, out hit, 100.0f)) {
 
                     //SELECT TILE WHILE UNIT IS SELECTED
                     if (hit.transform.tag == "Control") { 
                         SelectedUnit.GetComponent<AIMoveTo> ().target = hit.transform;
                         ClearUnit ();
                         currentState = State.Empty;
                     }
                 }
             }
         }










Comment

People who like this

0 Show 0
10 |3000 characters needed characters left characters exceeded
▼
  • Viewable by all users
  • Viewable by moderators
  • Viewable by moderators and the original poster
  • Advanced visibility
Viewable by all users

1 Reply

  • Sort: 
avatar image

Answer by tormentoarmagedoom · Jan 30, 2019 at 09:25 AM

Good day.

The solutios is siimple.

Look at your code, you have 2 general "if" parts, checking currentState.

BUT! the first part, changes this surrentstate, and it continues the script to find the next "if" part. You change the state right before that, so it enters this new if part.

The solution? MAke the code enters ONLY one of the "if" with a "else if"!

 if (currentState == State.Empty) { //NOTHING SELECTED
   {
     [things...]
    currentState = State.Unit;
    }
 
 else if (currentState == State.Unit) { //UNIT CURRENTLY SELECTED
  {
    [things...]
  }

Now it can only enters one of the part, even if one of this part change the condituion (the next iteration of the code will eneter to the other part)

Good luck!

Bye!

Comment
armagedoomgamesstudio
micropowermantenimiento
craftingtycoon

People who like this

3 Show 0 · Share
10 |3000 characters needed characters left characters exceeded
▼
  • Viewable by all users
  • Viewable by moderators
  • Viewable by moderators and the original poster
  • Advanced visibility
Viewable by all users

Unity Answers is in Read-Only mode

Unity Answers content will be migrated to a new Community platform and we are aiming to launch a public beta by June 9. Please note, Unity Answers is now in read-only so we can prepare for the final data migration.

For more information and updates, please read our full announcement thread in the Unity Forum.

Follow this Question

Answers Answers and Comments

164 People are following this question.

avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image avatar image

Related Questions

Can't click on any element on the scene 1 Answer

change animation to make character face a waypoint. 0 Answers

Getting Condition Value From Animator Transition in Script? 1 Answer

OnMouseDown not working on my Particle System 1 Answer

How would you blend a bool value in an Animator? (Specifically, change the spritemesh dynamically without both showing up) 0 Answers


Enterprise
Social Q&A

Social
Subscribe on YouTube social-youtube Follow on LinkedIn social-linkedin Follow on Twitter social-twitter Follow on Facebook social-facebook Follow on Instagram social-instagram

Footer

  • Purchase
    • Products
    • Subscription
    • Asset Store
    • Unity Gear
    • Resellers
  • Education
    • Students
    • Educators
    • Certification
    • Learn
    • Center of Excellence
  • Download
    • Unity
    • Beta Program
  • Unity Labs
    • Labs
    • Publications
  • Resources
    • Learn platform
    • Community
    • Documentation
    • Unity QA
    • FAQ
    • Services Status
    • Connect
  • About Unity
    • About Us
    • Blog
    • Events
    • Careers
    • Contact
    • Press
    • Partners
    • Affiliates
    • Security
Copyright © 2020 Unity Technologies
  • Legal
  • Privacy Policy
  • Cookies
  • Do Not Sell My Personal Information
  • Cookies Settings
"Unity", Unity logos, and other Unity trademarks are trademarks or registered trademarks of Unity Technologies or its affiliates in the U.S. and elsewhere (more info here). Other names or brands are trademarks of their respective owners.
  • Anonymous
  • Sign in
  • Create
  • Ask a question
  • Spaces
  • Default
  • Help Room
  • META
  • Moderators
  • Explore
  • Topics
  • Questions
  • Users
  • Badges