Panels aktualisieren

[Scotty]

Servus!
Also, ich hab folgendes Problem: Ich hab versucht, ein Panel zu einem Frame zu adden, welches ein BufferedImage enthällt, dass sich bei Tastatureingaben ändert. Dementsprechend soll sich auch der auf dem Bildschirm angezeigte Inhalt des Panels ändern. Aber irgendwie funktioniert der KeyListener bei dem Panel nicht, beim Frame jedoch schon. Der MouseListener funktioniert auch beim Panel einwandfrei, allerdings wird trotz der repaint()-Anweisung der neue Inhalt nich auf dem Panel angezeigt. Wie wäre das Problem zu lösen? Könnte ich statt dem Panel vielleicht einen anderen Component nutzen, etwa InternalFrame oder Canvas.

Mal noch was anderes: seit einiger Zeit werden bei mir Brüche, die kleiner als 1 sind (z.B. 1/32) immer mit dem Wert 0 zurückgegeben. Ich konnte das Problem bisher umgehen, aber normal ist das doch auch nicht, oder?

MfG
Scotty

 

[Wildcard]

Aber irgendwie funktioniert der KeyListener bei dem Panel nicht, beim Frame jedoch schon.

Ich denke mal du hast dem Panel nicht den Focus gegeben?

Mal noch was anderes: seit einiger Zeit werden bei mir Brüche, die kleiner als 1 sind (z.B. 1/32) immer mit dem Wert 0 zurückgegeben. Ich konnte das Problem bisher umgehen, aber normal ist das doch auch nicht, oder?

Würde mich sehr überraschen wenn das jemals anders war :wink:

1.0/32.0

 

[Scotty]

Ke, also ich hab den Focus zum Panel und zum Frame geadded. Beim Frame funzt alles wunderbar, aber beim Panel wieder nicht. Ich versteh auch nicht so ganz, wie ich mit dem Focus die Tastatureingaben verarbeiten kann. Theoretisch müssten ja die Eingaben bearbeitet werden, wenn das Panel quasi anvisiert wurde.

Hehe, danke für den Tipp mit den Kommas! Man sieht oft den Wald vor Bäumen nicht. :roll:

 

[Wildcard]

Wenn du mir jetzt noch den relevanten Code zeigst kann ich dir bestimmt auch sagen was du falsch machst...

 

[Scotty]

class window extends JFrame{
                
                /*...*/

                public window(){
                                /*...*/
                                getContentPane().add(new myPanel());}}

class myPanel extends JPanel implements KeyListener,MouseListener,FocusListener{

                /*...*/

	public myPanel(){
		super(true);
		setVisible(true);
		addMouseListener(this);
		addKeyListener(this);
		addFocusListener(this);
		repaint();}
		
	public void focusGained(
		FocusEvent e){}
		
	public void focusLost(
		FocusEvent e){}

                public void keyTyped(
		KeyEvent e){
			char c=e.getKeyChar();
			if(c=='x'){repaint();}
	
	/*...
                restliche key- & mouseevents
                ...*/

                public void paint(
                                Graphics g){
                                                /*neues BI erzeugen und zeichnen*/}}

 

[Wildcard]

1. Klassennamen werden IMMER groß geschrieben.
2. für was der FocusListener?
3. Bei Swing in der paintComponent zeichnen
4. diesem Code nach hat das Panel mit Sicherheit nicht den Fokus!
5. ist vieleicht besser wenn du mir mehr zeigst, da sind bestimmt noch ein paar Fehler drin :wink:

 

[Scotty]

der Compiler hat sich bei mir noch nie beschwert, warum auch?
der focuslistener erfüllt sozusagen keinen zweck. ich hab das eben mit dem 'Focus' verwechselt. was uns zu 4. bringt: wie zentriere ich den focus auf einen component?
paintComponent also, naja, ich hab mich schon gewundert, warum ich immer erst die repaint aufrufen musste, bis da was passiert ist.
hehe, so lange der compiler nicht meckert geht das schon in ordnung. aber was soll man machen, ich hab mir alles mühsam aus irgendwelchen tutorials zusammenstückeln müssen, da kann das schon vorkommen.

 

[Wildcard]

der Compiler hat sich bei mir noch nie beschwert, warum auch?
der focuslistener erfüllt sozusagen keinen zweck. ich hab das eben mit dem 'Focus' verwechselt. was uns zu 4. bringt: wie zentriere ich den focus auf einen component?
paintComponent also, naja, ich hab mich schon gewundert, warum ich immer erst die repaint aufrufen musste, bis da was passiert ist.
hehe, so lange der compiler nicht meckert geht das schon in ordnung. aber was soll man machen, ich hab mir alles mühsam aus irgendwelchen tutorials zusammenstückeln müssen, da kann das schon vorkommen.

Nicht alles was syntaktisch erlaub ist ist auch richtig! Es gibt Java-Coding-Conventions(in den FAQ) aus die man
sich aus gutem Grund halten soll.
den Fokus bekommt man mit requestFocus()
auch wenn du die paintComponent benutzt wirst du repaint brauchen!

 

[Guest]

ok, da die tastatureingaben nun immer noch nicht bearbeitet werden, hier noch mal der komplette code des panels.
bei mouseclicked funktioniert's, aber bei keytyped nicht.

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

class Panel3D
extends JPanel
implements KeyListener,MouseListener{
	Image dbImage;
	Graphics dbGraphics;

	public Panel3D(){
		super(true);
		setVisible(true);
		addMouseListener(this);
		addKeyListener(this);
		requestFocus();}
		
	public void mousePressed(
		MouseEvent e){}
		
	public void mouseTyped(
		MouseEvent e){}
		
	public void mouseReleased(
		MouseEvent e){}
	
	public void mouseExited(
		MouseEvent e){}
		
	public void mouseEntered(
		MouseEvent e){}
		
	public void mouseClicked(
		MouseEvent e){
			m=m.rotate(new vector(1,0,0));repaint();System.out.println("click");}
	
	public void keyTyped(
		KeyEvent e){
			char c=e.getKeyChar();
			if(c=='w'){m=m.rotate(new vector(1,0,0));
			if(c=='s'){m=m.rotate(new vector(-1,0,0));}
			if(c=='a'){m=m.rotate(new vector(0,-1,0));}
			if(c=='d'){m=m.rotate(new vector(0,1,0));}
			if(c=='q'){m=m.rotate(new vector(0,0,-1));}
			if(c=='e'){m=m.rotate(new vector(0,0,1));}
			if(c=='ö'){m=m.add(new vector(0,1,0));}
			if(c=='p'){m=m.add(new vector(0,-1,0));}
			if(c=='l'){m=m.add(new vector(-1,0,0));}
			if(c=='ä'){m=m.add(new vector(1,0,0));}
			if(c=='o'){m=m.add(new vector(0,0,-1));}
			if(c=='ü'){m=m.add(new vector(0,0,1));}
			if(c=='1'){mode=0;}
			if(c=='2'){mode=1;}
         repaint();}
		
	public void keyPressed(
		KeyEvent e){}
		
	public void keyReleased(
		KeyEvent e){}		
		
	image map=new image(System.getProperty("user.dir")+new String("/map1.bmp"));
	matrix m=new matrix();
	int mode=0;
			
	public void update(
		Graphics graph){
			if(dbImage==null){
				dbImage=createImage(this.getSize().width,this.getSize().height);
				dbGraphics=dbImage.getGraphics();}
			dbGraphics.setColor(new Color(255,255,255));
			dbGraphics.fillRect(0,0,this.getSize().width,this.getSize().height);
			dbGraphics.setColor(getForeground());
			paint(dbGraphics);
			graph.drawImage(dbImage,0,0,this);}
		
	public void paintComponent(
		Graphics g){
  		surface s=new surface(m);
  		s=s.rgbscale(map);
  		g.drawImage(s.toBufferedImage(
  			new BufferedImage(
  				getSize().width,
  				getSize().height,
  					BufferedImage.TYPE_INT_RGB),
  			map,
  			mode),0,0,this);}}

 

[Wildcard]

da dein code absolut unlesbar ist (deshalb in Zukunft die conventions beachten) hab ich dir ein Beispiel gemacht:

import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;

import javax.swing.JFrame;
import javax.swing.JPanel;


/**
 * TODO short description for BeispielFrame.
 * 


 * Long description for BeispielFrame.
 * 
 * @author Wildcard
 * @version $Revision: $
 */
public class BeispielFrame extends JFrame
{
    public BeispielFrame()
    {
        setSize(500,500);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        BeispielPanel panel = new BeispielPanel();
        panel.setSize(300,300);
        add(panel);
        repaint();
    }
    
    public static void main(String args[])
    {
        BeispielFrame frame = new BeispielFrame();
        frame.setVisible(true);
    }
}

class BeispielPanel extends JPanel
{
    
    private boolean blau;
    
    public BeispielPanel()
    {
        setFocusable(true);
        requestFocus();
        addKeyListener(new KeyListener()
        {
            public void keyTyped(KeyEvent e)
            {
                // TODO Auto-generated method stub
                blau=!blau;
                repaint();
            }

            public void keyPressed(KeyEvent e)
            {
                // TODO Auto-generated method stub

            }

            public void keyReleased(KeyEvent e)
            {
                // TODO Auto-generated method stub

            }
        });
        
        //wenn man nicht alle MouseListener MEthoden braucht bietet sich
        //ein MouseAdapter an
        addMouseListener(new MouseAdapter()
        {
            public void mousePressed(MouseEvent e)
            {
                // TODO Auto-generated method stub

            }
        });
    }
    
    protected void paintComponent(Graphics g)
    {
        if(blau)
            g.setColor(Color.BLUE);
        else
            g.setColor(Color.RED);
        g.fillRect(0,0,300,300);
    }
}

 

[Scotty]

ok, es lag an setFocusable(true). danke.
hmm, ich komm mit meinem code ganz gut klar, aber trotzdem, wo kann ich denn diese conventions mal einsehen?

 

[Wildcard]

hmm, ich komm mit meinem code ganz gut klar, aber trotzdem, wo kann ich denn diese conventions mal einsehen?

Das glaub ich dir sogar, aber eben niemand ausser dir ... :wink:
http://java.sun.com/docs/codeconv/html/CodeConvTOC.doc.html