# Array of keys. Movement

**URL:** https://discourse.processing.org/t/array-of-keys-movement/2518
**Category:** Coding Questions
**Created:** [August 9, 2018, 9:36pm UTC](https://discourse.processing.org/t/array-of-keys-movement/2518 "2018-08-09T21:36:37Z")
**Posts on this page:** 3
**Page:** 1

<div class="post-metadata">

### Author: ![Tymek](https://avatars.discourse-cdn.com/v4/letter/t/b487fb/32.png) [@Tymek](https://discourse.processing.org/u/Tymek)
#### Post date: [August 9, 2018, 9:36pm UTC](https://discourse.processing.org/t/array-of-keys-movement/2518/1 "2018-08-09T21:36:37Z")

</div>

Hello, what is wrong with this code? Why player (black square) wont move in 4 directions, but only in 2.

```auto
Postac postac = new Postac();

boolean[] chodzenie = new boolean[4]; //UP DOWN LEFT RIGHT
/* 0 1 2 3 */

void setup() {
  size(600, 600, P2D);

  for (int i = 0; i < chodzenie.length; i++) {
    chodzenie[i] = false;
  }
 
  postac.location = new PVector(width/2, height/2);
  postac.velocity = new PVector(0, 0);

}

void draw() {
  background(190);

  postac.display();
  postac.chodzenie();
}

void keyPressed() {

  if (key == CODED && keyCode == UP) chodzenie[0] = true;
  if (key == CODED && keyCode == DOWN) chodzenie[1] = true;
  if (key == CODED && keyCode == LEFT) chodzenie[2] = true;
  if (key == CODED && keyCode == RIGHT) chodzenie[3] = true;
}

void keyReleased() {

  if (key == CODED && keyCode == UP) chodzenie[0] = false;
  if (key == CODED && keyCode == DOWN) chodzenie[1] = false;
  if (key == CODED && keyCode == LEFT) chodzenie[2] = false;
  if (key == CODED && keyCode == RIGHT) chodzenie[3] = false;
}

void mousePressed() {
}

class Postac {

  PVector location, velocity;
  int a = 20;

  void display() {
    fill(0);
    rect(location.x, location.y, a, a);
    location.add(velocity);

    println("0" + chodzenie[0] + " 1" + chodzenie[1] + " 2" + chodzenie[2] + " 3" + chodzenie[3]);
    println(velocity);
  }

  void chodzenie() {

    if (chodzenie[0] == true) {
      velocity.y = -1;
    } else if (chodzenie[0] == false) {
      velocity.y = 0;
    }

    if (chodzenie[1]) { 
      velocity.y = 1;
    } else if (!chodzenie[1]) {
      velocity.y = 0;
    }

    if (chodzenie[2]) { 
      velocity.x = -1;
    } else if (!chodzenie[2]) {
      velocity.x = 0;
    }

    if (chodzenie[3]) { 
      velocity.x = 1;
    } else if (!chodzenie[3]) {
      velocity.x = 0;
    }
  }
}

```

---

<div class="post-metadata">

### Author: ![kfrajer](https://yyz2.discourse-cdn.com/flex036/user_avatar/discourse.processing.org/kfrajer/32/196_2.png) [@kfrajer](https://discourse.processing.org/u/kfrajer)
#### Post date: [August 9, 2018, 10:44pm UTC](https://discourse.processing.org/t/array-of-keys-movement/2518/2 "2018-08-09T22:44:27Z")

</div>

It is because of your logic in your `chodzenie()` function. Think about the following lines of code:

```java
    if (chodzenie[0] == true) {
      velocity.y = -1;
    } else if (chodzenie[0] == false) {
      velocity.y = 0;
    }

    if (chodzenie[1]) { 
      velocity.y = 1;
    } else if (!chodzenie[1]) {
      velocity.y = 0;
    }

```

Can you spt the problem? Imagine yu press UP and then DOWN. Then imagine when you do the opposite: your press DOWN and then you press UP.

I am certain you will catch the bug by looking at these lines.

Kf

---

<div class="post-metadata">

### Author: ![Tymek](https://avatars.discourse-cdn.com/v4/letter/t/b487fb/32.png) [@Tymek](https://discourse.processing.org/u/Tymek)
#### Post date: [August 9, 2018, 11:12pm UTC](https://discourse.processing.org/t/array-of-keys-movement/2518/3 "2018-08-09T23:12:35Z")

</div>

Of course!! Thanks a lot.

````auto
    if (chodzenie[0]) 
      velocity.y = -1;
    if (chodzenie[1]) 
      velocity.y = 1;
    if (chodzenie[2]) 
      velocity.x = -1;
    if (chodzenie[3]) 
      velocity.x = 1;

    if (!chodzenie[0] && !chodzenie[1]) velocity.y = 0;
    if (chodzenie[0] && chodzenie[1]) velocity.y = 0;
    if (!chodzenie[2] && !chodzenie[3]) velocity.x = 0;
    if (chodzenie[2] && chodzenie[3]) velocity.x = 0;```
````