Multiple Input.GetKey

Question

Hi,

I want a piece of code which says "if none of the following keys of pressed then..."

So this is what I think is most suitable. (I prefer Javascript).

 if (!(Input.GetKey(KeyCode.UpArrow) || Input.GetKey(KeyCode.W) || Input.GetKey(KeyCode.RightArrow) || Input.GetKey(KeyCode.D) || Input.GetKey(KeyCode.DownArrow) || Input.GetKey(KeyCode.S) || Input.GetKey(KeyCode.LeftArrow) || Input.GetKey(KeyCode.A) || Input.GetKey(KeyCode.UpArrow) || Input.GetKey(KeyCode.W))) {
 // Then do what I want.
 }

(I haven't tried it yet but) it's extremely long, and the length is quite annoying. Is there another way to do this, something like this:

 if (!(Input.GetKey(KeyCode.UpArrow, Keycode.W //etc...

So how I could do this?

PS: This is different to if any key is pressed. Thanks!

Answer 1

Best Answer

Answer by Owen-Reynolds · Jul 05, 2013 at 06:48 PM

A few weeks into a regular bring programming class you'd see two ways to handle "none of the above":

 if(Input.GeyKeyDown("a")) {   .... }
 else if(Input.GetKeyDown("b")) { ..... }
 else if ...
 else   // None of the above goes here

If you can't organize things that neatly, the flag trick also works:

 bool keyWasHandled=false;

 if(Input.getKetDown("y")) { ... keyWasHandled=true; }
 // everyone who processes a key should set: keyWasHandled=true

 if(Input.AnyKeyDown && keyWasHandled==false) {
   // we have a key that no one else wanted
 }

The array method (Bunny, above) is "better," but only if you know arrays and, everyone else who's going to mess with your code.

Add comment · Show 2 · Share

Bunny83 · Jul 06, 2013 at 10:04 AM 0

Share

Your first solution won't probably not work for his case ;) It looks like he has the classical WASD and the arrow-keys movement input. So when doing elseif chains only one of the ifs can be executed. The second one will of course work and is in most cases the best solution. It avoids redundancy of keynames. I just answered the question most straight-forward ;)

Actually UT could change the Input.Get$$anonymous$$ey function so it would take a params array since that wouldn't break any existing code

Yharooer · Jul 06, 2013 at 12:56 PM 0

Share

Well all I want the of statements to do is change a static variable, so this works for me.

Answer 2

Answer by Bunny83 · Jul 05, 2013 at 01:15 PM

Well, in C# you can create a function with a params parameter. This allows you to pass as much parameters as you like. I'm not sure if that's possible in UnityScript.

This would work in C#

 bool GetAnyKey(params KeyCode[] aKeys)
 {
     foreach(var key in aKeys)
         if (Input.GetKey(key))
             return true;
     return false;
 }

This function could be used like this:

 if (GetAnyKey(KeyCode.UpArrow, Keycode.W /* , ... */ ))

Or for your case

 if (!GetAnyKey(KeyCode.UpArrow, Keycode.W /* , ... */ ))

As general alternative you could define your keys in an array and pass the array to the function.

Add comment · Show 1 · Share

Bunny83 · Jul 05, 2013 at 01:18 PM 0

Share

Sorry, missed the array brackets ^^. fixed it.

Answer 3

Answer by mentaldepth · May 04, 2015 at 11:50 AM

Here it is working :D thanks for info guys. (very simple but it works all directions no sound cut off)

 var AudioFile : AudioClip;
 
 function Update() {
 
 if ( Input.GetButtonDown( "Horizontal" ) || Input.GetButtonDown( "Vertical" ) )
 {
      GetComponent.<AudioSource>().clip = AudioFile;
      GetComponent.<AudioSource>().Play();
 }   
  
     else if ( !Input.GetButton( "Horizontal" ) && !Input.GetButton( "Vertical" ) && GetComponent.<AudioSource>().isPlaying )
 {
      GetComponent.<AudioSource>().clip = AudioFile;
      GetComponent.<AudioSource>().Stop();
 }
 
 }

Add comment · Share

Answer 4

Answer by JohnnySunshine · Jul 05, 2013 at 01:02 PM

You could use input axes; by default, you have "Vertical" and "Horizontal" mapped to arrow keys and WASD, which gives you:

 if (Mathf.Approximately(0.0f, Input.GetAxis("Horizontal") && Mathf.Approximately(0.0f, Input.GetAxis("Vertical"))
 {
     // Then you do what you want
 }

Add comment · Share

Answer 5

Answer by calegxm999 · Jun 23, 2020 at 06:41 AM

if (Input.GetKey(KeyCode.LeftArrow) && Input.GetKey(KeyCode.RightArrow))

Add comment · Share

Multiple Input.GetKey

5 Replies

Your answer

Follow this Question

Related Questions