EventListeners, SwingTimer - Find whats slowing down this 20 lines of code!

Question

Hi, the code below is a very simple way to detect user input, and also have multiple keys pressed at the same time. However, what I implemented this, my program began to slow down quite a bit.
I'm wondering if anyone sees any problems in this code which would slow my program down.
Thanks,
Nate
     // Detect user input
     class userinput implements ActionListener, KeyListener{
          public userinput(){
               Timer whew = new Timer(50, this);
               whew.start();
               addKeyListener(this);
               setFocusable(true);
     public void keyPressed(KeyEvent e) {
          int keyCode = e.getKeyCode();
        if (keyCode == KeyEvent.VK_DOWN){
             //y = y + speed;
             goingbackwards = true;
        if (keyCode == KeyEvent.VK_UP){
             //y = y - speed;
             //goingforward = true;
             gof = true;
        if (keyCode == KeyEvent.VK_LEFT){
             //x = x - speed;
             goingleft = true;
        if (keyCode == KeyEvent.VK_RIGHT){
             //x = x + speed;
             goingright = true;
        if (keyCode == KeyEvent.VK_SPACE){
             spacedown = true;
     public void keyReleased(KeyEvent e) {
          int keyCode = e.getKeyCode();
          if (keyCode == KeyEvent.VK_DOWN){
              goingbackwards = false;
          if (keyCode == KeyEvent.VK_UP){
               gof = false;
          if (keyCode == KeyEvent.VK_LEFT){
         goingleft = false;
          if (keyCode == KeyEvent.VK_RIGHT){
         goingright = false;
          if (keyCode == KeyEvent.VK_SPACE){
              spacedown = false;
     public void keyTyped(KeyEvent e) {     
     @Override
     public void actionPerformed(ActionEvent ac) {
        if (gof && y >= 15){
             y = y - speed;
             //break;
             if (goingleft && x >= 15){
             x = x - speed;
             //break;
             if (goingright && x <= 920){
                  x = x + speed;
                 //break;
             if (goingbackwards && y <= 665){
                  y = y + speed;
                 //break;
             if (spacedown && health >= 1){
                sx = x + 36;            
                sy = y - 19;
                fire();
                 //break;
     Thanks for reading,
Nate

Answer 1

MeMo_HI wrote:
WOW jeje i just notice your code looks like a game
since i'm not an expert in games dont take this as an advice this is only a question: why would you need to slow down the x=x-speed? cant you use a smaller value for speed?
all the events run inside a single Thread called ThreadDispatcher so if you are calling events a lot then your calling fire() a lot and if the fire() method is
a long task the other events have to wait until the fire method finished to get their time.
Edited by: MeMo_HI on Feb 1, 2009 12:30 AM
dispatch thread yes that is
Edited by: MeMo_HI on Feb 1, 2009 12:30 AM
sorry i confuse the words sometimesMuch to learn he has. Hmmmm.
~~ Yoda