keylistener kaldes for hurtigt

Har du leget med tråde?

For det du skal ha' fat i er nok, at det skal køres i en tråd. Det der sker med det du har nu er, at alle dine keyevents bliver queued og udført inden gui'en igen får lov til at repainte.

Ved nærmere eftertanke, så er det ikke sikkert du skal ha' fat i tråde alligevel. Det kommer lidt an på hvad det er du vil. Det er svært at sige ud fra de oplysninger + kode du har givet.

Jeg har lige lavet et lille hurtigt eksempel, hvor man kan flytte en firkant rundt i en frame:


import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class MoveGui extends JFrame
{
    private MyRec rec;
    private final int speed = 2;
   
    public MoveGui ()
    {
        setSize (400, 400);   
        this.addKeyListener(new MyKeyListener ());
        rec = new MyRec (50, 50, 15, 15);
    }
   
    public void paint (Graphics g)
    {
        g.clearRect(0, 0, this.getWidth(), this.getHeight());
        g.drawRect(rec.x, rec.y, rec.width, rec.height);
    }
   
    private class MyRec
    {
        int x, y, width, height;
       
        public MyRec (int x, int y, int width, int height)
        {
            this.x = x;
            this.y = y;
            this.width = width;
            this.height = height;
        }
    }
   
    private class MyKeyListener extends KeyAdapter
    {
        public void keyPressed (KeyEvent e)
        {
            switch (e.getKeyCode())
            {
                case KeyEvent.VK_UP: rec.y -= speed; break;
                case KeyEvent.VK_DOWN: rec.y += speed; break;
                case KeyEvent.VK_RIGHT: rec.x += speed; break;
                case KeyEvent.VK_LEFT: rec.x -= speed; break;
            }
            repaint ();
        }
    }
   
    public static void main (String[] args)
    {
        new MoveGui().setVisible(true);
    }
}

hmm jeg er ikke helt sikker på hvad der får dit eksempel til at virke fremfor mit... så jeg prøver at lave en tråd

Jeg kan som sagt ikke sige, om det er nødvendigt med en tråd eller ej ud fra den smule du har vist. Men prøv.

Kommet videre ?

ja nu er det lavet i en tråd... men det ændre ikke på hvor godt spillet virker...

du får lige hele koden:

import java.awt.*;
import java.awt.event.*;

public class program
{
  public static void main(String[] arg)
  {
    vindue skærm = new vindue();
    skærm.setSize(640, 480);
    skærm.setTitle("The Game!");
    skærm.setVisible(true);
  }
}


import java.awt.*;
import java.awt.event.*;

public class vindue extends Frame
{
  public spiller person = new spiller(this);
  public Thread pers = new Thread(person);
  public Thread tid;
  public boolean igang = false;
  public kasse k;

  public void paint(Graphics g)
  {
    if (!igang)
    {
      pers.start();
      igang = true;
    }
    tid = new Thread(new timer(this));
    tid.start();
    Insets in = getInsets();
    g.translate(in.left, in.top);
    person.k.render(g);
  }
}


import java.awt.*;
import java.awt.event.*;

public class spiller implements Runnable
{
  public kasse k = new kasse(new Point(100, 100), 1, "kasse 1");
  public vindue f;

  public spiller(vindue f)
  {
    this.f = f;
  }

  public void run()
  {
    f.addKeyListener(new tast());
  }

  public class tast extends KeyAdapter
  {
    public void keyPressed(KeyEvent e)
    {
      try
      {
        Thread.sleep(50);
      } catch (Exception ex)
      {}
      if (e.getKeyCode() == KeyEvent.VK_UP)
      {
        k.bevæg(1);
      } else if (e.getKeyCode() == KeyEvent.VK_DOWN)
      {
        k.bevæg(2);
      } else if (e.getKeyCode() == KeyEvent.VK_RIGHT)
      {
        k.bevæg(3);
      } else if (e.getKeyCode() == KeyEvent.VK_LEFT)
      {
        k.bevæg(4);
      }
    }
  }
}


import java.awt.*;

public class timer implements Runnable
{
  public vindue f;
 
  public timer(vindue f)
  {
    this.f = f;
  }
 
  public void run()
  {
    try
    {
      Thread.sleep(50);
    } catch (Exception e)
    {}
    f.repaint();
  }
}

Jeg kigger lige på det, når jeg kommer hjem senere.

jes lyder herligt :D

Kan du ikke lige prøve kort at beskrive, hvad det er du vil ha' programmet til at gøre?

Altså:

- Vil du kun ha' noget til at bevæge sig, når der trykkes på knapperne
- Eller skal det hele tiden bevæge sig, og knapperne skal blot indikere retning

Osv...

Jeg kan heller ikke compile din kode uden klassen: kasse

det skal kun bevæge sig når man trykker på knapperne... sådan set det samme som det lille eksempel du lavede, men jeg vil bare gerne ha det her til at virke. :P

nej hov:

import java.awt.*;

public class kasse
{
  protected Rectangle kassen;
  protected int hastighed;
  protected String navn;
  protected Point startP;

  public kasse(Point punkt, int hastighed, String navn)
  {
    kassen = new Rectangle(punkt.x, punkt.y, 20, 20);
    this.hastighed = hastighed;
    this.navn = navn;
    startP = punkt;
  }

  public void bevæg()
  {
    if (kassen.x < (400 + startP.x))
    kassen.x += hastighed;
    else if (kassen.y < (300 + startP.y))
    kassen.y += hastighed;
    else
    kassen.x += hastighed;   
  }

  public void flyt(Point punkt)
  {
    kassen.x += punkt.x;
    kassen.y += punkt.y;
  }

  public void sæt(Point punkt)
  {
    kassen.x = punkt.x;
    kassen.y = punkt.y;
  }

  public void render(Graphics g)
  {
    g.fillRect(kassen.x, kassen.y, kassen.width, kassen.height);
  }
}

Du har en del overflødig kode - efter min mening.
Her kommer mit forslag med udgangspunkt i din egen kode:

import java.awt.*;
import java.awt.event.*;

public class program
{
  public static void main(String[] arg)
  {
    vindue skærm = new vindue();
    skærm.setSize(640, 480);
    skærm.setTitle("The Game!");
    skærm.setVisible(true);
  }
}

import java.awt.*;
import java.awt.event.*;

public class vindue extends Frame
{
    public kasse k;
   
    public vindue ()
    {
        k = new kasse (new Point(10, 10), 5, "Mikkel");
        this.addKeyListener(new KeyAdapter()
        {
            public void keyPressed (KeyEvent e)
            {
                handleMovement (e);
            }
        });
    }
   
    public void paint(Graphics g)
    {
        Insets in = getInsets();
        g.translate(in.left, in.top);
        k.render(g);
    }
 
    private void handleMovement (KeyEvent e)
    {
        switch (e.getKeyCode())
        {
            case KeyEvent.VK_UP: k.bevæg(kasse.UP); break;
            case KeyEvent.VK_DOWN: k.bevæg(kasse.DOWN); break;
            case KeyEvent.VK_RIGHT: k.bevæg(kasse.RIGHT); break;
            case KeyEvent.VK_LEFT: k.bevæg(kasse.LEFT); break;
        }
        repaint ();
    }
}


import java.awt.*;

public class kasse
{
    public static final int LEFT = 0, RIGHT = 1, UP = 2, DOWN = 3;
   
    private int size = 20;
    protected int hastighed;
    protected String navn;
    protected Point point;
   
    public kasse(Point punkt, int hastighed, String navn)
    {
        this.hastighed = hastighed;
        this.navn = navn;
        point = punkt;
    }

    public void bevæg(int direction)
    {
        switch (direction)
        {
            case LEFT: point.x -= hastighed;break;
            case RIGHT: point.x += hastighed;break;
            case UP: point.y -= hastighed;break;
            case DOWN: point.y += hastighed;break;
        }   
    }
   
    public void flyt(Point punkt)
    {
        point = punkt;
    }
   
    public void render(Graphics g)
    {
        g.fillRect(point.x, point.y, size, size);
    }
}

jes så virker det fint, mange tak :P

Jeg tillader mig at smide et svar :)

Så vil jeg da tillade mig at sige tak igen :P