KeyText to KeyCode

[Tim Porritt]

I'm trying to change a string such as "Up", to the integer 38.
So far, i've gotten all the lines from a file and added them to an ArrayList called "controls".

The file that I get the strings from looks like this:

Up:
Up
Down:
Down
Left:
Left
Right:
Right

The code that i've tried using to convert it is as follows:

private int keycode(String i)
	{
		if(i == "Up")
		{
			return KeyEvent.VK_UP;
		}
		else if(i == "Down")
		{
			return KeyEvent.VK_DOWN;
		}
		else if(i == "Left")
		{
			return KeyEvent.VK_LEFT;
		}
		else if(i == "Right")
		{
			return KeyEvent.VK_RIGHT;
		}
		else return 0;
	}

public void keyPressed(KeyEvent e) {

        int key = e.getKeyCode();
        
        if(visible == true)
        {
	        if (key == Integer.parseInt((String) controls.get(1))) 
	        {

	        }
	        if (key == Integer.parseInt((String) controls.get(3))) 
	        {

	        }
	        if (key == Integer.parseInt((String) controls.get(5))) 
	        {

	        }
	        if (key == Integer.parseInt((String) controls.get(7))) 
	        {

	        }
        }
    }

When i try this, the ifs don't work (as in, the strings don't match), and they just return the 0, and i don't know why.

Please help...

PS. I'll find a way to add all keys to the keycode method later on.

[Paul Clapham]

if(i == "Up")

That isn't the right way to compare two Strings to see if their contents are correct. That just compares to see if they are actually the same String object, which is a different thing entirely. Two different String objects can certainly contain the same data, and that's what is happening here. You should use String's equals() method to see if two Strings contain the same sequence of characters.