repaint oder update Methode?

[dark_angel]

Ich versuche gerade ein Frogger Spiel zu programmieren und bin soweit, dass ich den Frosch am unteren Bildrand "sitzen" habe. Jetzt möchte ich, dass er sich, wenn ich bspw. die Pfeiltaste nach oben drücke, nach oben "bewegt". Ich habe bisher 2 Klassen, einen Frame und einen Canvas... Ich dachte mir nun es wäre sinnvoll einfach die Y-Position des Bildes (des Frosches) mit der entsprechenden set-Methode zu verändern wenn die Taste gedrückt wird und es dann neu darstellen zu lassen... Aber welche Methode brauche ich dazu, repaint oder update und wo genau müsste ich die hinschreiben?

 

[Beni]

Canvas froschZeichnung = ...
froschZeichnung.repaint();

 

[dark_angel]

Hm, so in etwa hatte ich mir das auch gedacht, aber dann muss der Fehler wohl wo anders liegen ... Also compilieren tut er es ohne Fehlermeldung, der Frosch bleibt halt nur da wo er ist...

Vielleicht ist hier ja irgendein elementarer Fehler ...?:

...
public void keyTyped(KeyEvent e)
  {
          if(e.getKeyCode() == e.VK_KP_UP)
          {
                  up();
                  bild1.repaint();
          }
  }

  
  public static void up()
 {
           ImageCanvas.setYPos1(20);
           
  }
...

...
public static void setYPos1(int value)
  {
            yPos1 = yPos1+value;
         
  }
...

 

[Beni]

Öhm, wieso verwendest du "static"? Erscheint mir nicht sehr sinnvoll (um nicht zu sagen, total unsinnvoll).

Aber dann versuch doch mal: frame.repaint();
Und schreib in der paint-Methode ein System.out.println ( yPos1 ) (wenn sich die Ausgabe nicht verändert...)
Falls das Progi nicht allzuviel Code hat, kannst du hier auch ein bisschen mehr posten.

mfg Beni

 

[dark_angel]

Ich hab static genommen, weil er mir ohne sonst immer Fehlermeldungen angezeigt hat, so hats jetzt zumindest beim Compilieren erstmal geklappt...

Ok, ich poste mal den gesamten Code wenn das ok ist! Und danke für die Hilfe!

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


public class ImageFrame extends Frame implements KeyListener
{
        ImageCanvas bild1;

        class MeinWindowListener extends WindowAdapter {

                public void windowClosing(WindowEvent e) {
                        exitProgram();
                }
        }

        public void keyPressed(KeyEvent e) {}

        public void keyReleased(KeyEvent e) {}

        public void keyTyped(KeyEvent e) {
                if (e.getKeyCode() == e.VK_KP_UP) {
                        up();
                        bild1.repaint();
                }
        }

        public static void up() {
                ImageCanvas.setYPos1(20);
        }

        ImageFrame() {
      
                super();

                setTitle("Frogger!");

                setBackground(Color.blue);

                setSize(700, 500);

                initLayout();

                addWindowListener(new MeinWindowListener());

                setVisible(true);
        }


        protected void initLayout() {

                setLayout(new BorderLayout());
        
                ImageCanvas bild1 = new ImageCanvas(50, 50, "Frosch.gif",
                        Color.black);
                add("South", bild1);               
       }

        void exitProgram() {
                dispose();
                System.exit(0);
        }
 
        public static void main(String args[]) {
                 new ImageFrame();
        }
  }

import java.awt.Canvas;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Image;
import java.awt.MediaTracker;
import java.awt.Toolkit;

public class ImageCanvas extends Canvas
{

        protected String bilddatei;
        protected Image bild1;
        protected static int xPos1, yPos1;
        protected int width, height;
       
        public ImageCanvas(int w, int h, String datei, Color bc) {
                width = w;
                height = h;
                bilddatei = datei;
                
                setBackground(bc);
               
                xPos1 = 300;
                yPos1 = -10;

                loadImage();
        }
       
        private void loadImage() {
           
                Toolkit tk = getToolkit();
               
                bild1 = tk.getImage(bilddatei);

                MediaTracker mt = new MediaTracker(this);
                
                mt.addImage(bild1, 0);

               
                try {
                        mt.waitForAll();
                } catch (InterruptedException e) {}
        }
        
        public void paint(Graphics g) {
                
               g.drawImage(bild1, xPos1, yPos1, this);
        }
       
        
        public int getYPos1() {
                return yPos1;
        }

        public static void setYPos1(int value) {
                yPos1 = yPos1 + value;

        }
        
       public Dimension getPreferredSize() {
                return new Dimension(width, height);
        }

        public Dimension getMinimumSize() {
                return new Dimension(width, height);
        }

        public Dimension getMaximumSize() {
                return new Dimension(width, height);
        }
      
}

 

[Beni]

Huii, da sind aber noch ein paar Dinge komisch...

1. KeyListener implementieren ist schon gut, aber addKeyListener hast Du vergessen. (gleich beim Frame und bei dem ImageCanvas)
2. keyTyped wird nicht bei jedem Key aufgerufen (nur Buchstaben, Zahlen, ...). Also muss der Aufruf von up() entweder in keyPressed oder keyReleased erfolgen (pressed fände ich besser).
3. Je grösser y wird, desto tiefer kommt der Frosch zu liegen (wegen dem Koordinatensystem). Also nenn die Methode down und nicht up :wink:
4. VK_KP_UP ist für das Nummerfeld, VK_UP für die Pfeiltasten. War das Absicht?
5. nicht ImageCanvas bild1 = new ImageCanvas(50, 50, "Frosch.gif",Color.black); sondern nur bild1 = new ImageCanvas(50, 50, "Frosch.gif",Color.black);
Ansonsten ist die globale Variable bild1 == null, und das wird Probleme geben.
6. nicht ImageCanvas.setYPos1(20); sondnern bild1.setYPos1(20); (hier darf bild1 nicht null sein :wink: ), und dann kannst du alle "static" löschen.

mfg Beni

 

[dark_angel]

Ohja, jetzt wo du es sagst wird da einiges logischer :wink:!! Habs jetzt so versucht und es klappt, DANKE!!!

1.Hatte ich natürlich völlig vergessen *SchandeübermeinHaupt*
2.Ok, das wusste ich jetzt nicht, hatte mir nur kurz die API durchgelesen und es einfach damit versucht...
3.Hatte völlig verdrängt, dass oben links ja 0/0 ist und nicht unten, ups 8)...
4.Ja, wusste ich, war mir aber fürs erste egal, Hauptsache es klappt erstmal...
und 5. und 6., da ging mir jetzt endlich mal ein Licht auf warum das alles so komisch war!