Thread: KeyListener won't listen

Why won't my "player1" move???
Class with main

import java.awt.Color;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import javax.swing.JFrame;
import javax.swing.JPanel;
 
 
public class Spelplan extends JPanel implements KeyListener
{
 
    Gubbe player1;
 
    int x = 100;
    int y = 100;
 
    public Spelplan()
    {
        setLayout(null);
        player1 = new Gubbe(30, 50, x, y);    
        this.setLocation(20, 30);
        add(player1);
        player1.addKeyListener(this);
        this.setBackground(Color.black);
 
    }
 
    public static void main(String[] args) 
    {
        JFrame f = new JFrame();
        f.setSize(300,300);
        f.setLocation(200,200);
        f.setTitle("RogueTest");
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        Spelplan p = new Spelplan();
        f.add(p);
        f.setVisible(true);
 
    }
 
    public void keyTyped(KeyEvent e)
    {
 
    }
 
    public void keyPressed(KeyEvent e)
    {
 
        if(e.getKeyCode() == 38)
        {
            System.out.println("Pressed");
            y = y + 10;
        }
 
        repaint();
    }
 
    public void keyReleased(KeyEvent e)
    {
 
    }
 
}//class

Other class

import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import javax.swing.JPanel;
 
 
public class Gubbe extends JPanel
{
    private int bas, hojd, xpos, ypos;
 
 
 
    public Gubbe(int b, int h)
    {
     bas = b;
     hojd = h;
     this.setBackground(null);
     this.setPreferredSize(new Dimension(bas, hojd));
    }
 
 
 
    public Gubbe(int bas, int hojd, int xpos, int ypos)
    {
        this.bas = bas;
        this.hojd = hojd;
        this.xpos = xpos;
        this.ypos = ypos;
        this.setSize(bas, hojd);
        this.setLocation(xpos - bas/2, ypos);
 
 
    }
 
 
 
    @Override
    public void paintComponent(Graphics g)
    {
        int x[] = { bas/2, 0, bas};
        int y[] = { 0, hojd, hojd };
 
        g.setColor(Color.red);
        g.fillOval(10,20, 20, 20);
 
 
 
    }
 
 
 
}

I tried using the .setFocusable(true) in "public Spelplan". I also kept the printline "Pressed" inside the if-statement. Now it says "Pressed", but the component does not move.

but the component does not move.

What will make it move? What statement is supposed to make it move?

Originally Posted by Norm

What will make it move? What statement is supposed to make it move?

Because I change the value of y:

y = y + 10;

Or?? wouldn't that work?

What difference does changing the value of y make to where the component is?

Originally Posted by Norm

What difference does changing the value of y make to where the component is?

Because the Y value is the components Y - value. When I change the Y value

int y = 100;

the component is changing it's position. Then I just thought it sounded reasonable that it would also change position if I made the value change by pressing the "up" - key. It did not do that

Yeah, I made it work!

if(e.getKeyCode() == 38)
        {
            y = player1.getY();
            System.out.println("Pressed");
//            y = 200;
            player1.setLocation(x, y -= 10);
            repaint();
 
        }