6
\$\begingroup\$

Im making a game in LWJGL 2 and Slick2d. Im trying to add an item to my inventory by releasing a key (the inventory system works properly, the part where I have to see if the button was RELEASED is difficult). Right now when I do...

if (Keyboard.getEventKey() == Keyboard.KEY_L) {
    if(Keyboard.getEventKeyState() == false){
        Inventory.addItem(1);
    }
}

... In my update method, Instead of it only calling the 2nd if statement once, ( because you can only release a key once) It just keeps going because the getEventKeyState variable stays false, but if Ive already released it once it should switch back to the opposite. Although I can do it by making it add an item when i hold the key, I cant exactly make it add only the 1 item, it keeps adding items while I hold it.

public static void addItem(int id){
    for(InventorySlot s : inventoryslots){
        if(s.getItem() == -1){
            s.setItem(id);
            System.out.println("Added item " + id + "!");
            break;
        }
    }
}

Thats the addItem method, and if I would approach this by making it add the item while holding, I would have to make it so that it adds it only once. Any suggestions please? Thank you

Improve this question
\$\endgroup\$
4
  • \$\begingroup\$ Your problem lies in your keyreleased event. When you hold a key, it calls the keypressed event every frame update as long as the key is held down. When you call keyreleased, your key state will be false as long as you aren't pressing the key. I believe Slick2D recommends using @Vokail 's answer. See slick.ninjacave.com/wiki/index.php?title=Input \$\endgroup\$
    – Nathaniel D. Hoffman
    Aug 16, 2016 at 12:28
  • \$\begingroup\$ Also, if you do insist on using keystates, which are recommended by LWJGL and technically more useful, I would recommend reading (or rereading) wiki.lwjgl.org/wiki/LWJGL_Basics_2_(Input). You really need to be bounding your input with "while(Keyboard.next()) {...}" and I would recommend switching the order of the if-statements in the code you posted just for clarity. \$\endgroup\$
    – Nathaniel D. Hoffman
    Aug 16, 2016 at 12:33
  • \$\begingroup\$ @NathanielD.Hoffman Looks like you know enough that you could create a decent answer out of this! \$\endgroup\$
    – Vaillancourt ♦
    Jan 24, 2020 at 2:51
  • \$\begingroup\$ @Vaillancourt I've posted an answer, after looking over this question and trying to refresh my memory. As it turns out, all of OP's code posted above works fine by itself, but there is a loop that needs to go around it in order to remove events from the key register. \$\endgroup\$
    – Nathaniel D. Hoffman
    Jan 24, 2020 at 4:30

2 Answers 2

Reset to default
1
\$\begingroup\$

The problem here is (likely) really simple. You need to call Keyboard.next() in order to pop the "L key was released" item out of the Keyboard queue, otherwise it'll just stick around in there.

Improve this answer
\$\endgroup\$
0
\$\begingroup\$

You can use also container reference, getInput and check if a key is pressed, down, etc..

For example:

container.getInput().isKeyPressed(Input.KEY_SPACE)

see more details on javadoc

Improve this answer
\$\endgroup\$

You must log in to answer this question.

Not the answer you're looking for? Browse other questions tagged .