kriege kein paint

[zahil]

hallo zussamen,

ich habe ein gui der cardalyout besizt. auf ein card habe ich ein button panel mit 9 buttons. und auf derselbe card habe ich ein animation (analog uhr).

ich will dass beim klicken eines buttons, ein punkt auf dem button gemalt wird. dafür habe ich ein test code geschreiben, und die läuft :-

public class GlassPane extends JPanel implements ActionListener{
            Object s;
         
         public GlassPane(AbstractButton aButton, Container contentPane) {             
             setVisible(true);
             //addActionListener(this);
             //this.repaint();
         }
         
         public void actionPerformed(ActionEvent i){
            Object s = i.getSource();
            setPoint(s);
            //this.repaint();
         }  
         
         public void setPoint(Object s){ 
                    if (s == b5) {
                       if(status[0] == false){
                          int x = (b5.getX() + b5.getWidth() - 4);
                          int y = (b5.getY() + b5.getHeight() - 2);
                          System.out.println (x + ":" + y);
                       }//*else{ x = 0;
                       y = 0; }//*
                    //}
                    this.repaint();
         }
             
         public void paintComponent(Graphics g){
                if(x != 0){
                     if (y != 0){
                        g.setColor(Color.red);
                        g.fillOval(x, y, 20, 20);
                     }   
                }
         }
       }

dass problem ist dass ich über haupt kein punkt auf dem button kriege.....weisst jemand warum?
ich hab den glass pane mit frame.setGlassPane(glasspane) gesetzt und dem richtig instanciert....

 

[SlaterB]

zunächst mal solltest du ohne Button arbeiten und nur versuchen,
in paintComponent einen Kreis zu malen, mit vorgegebenen x und y, Testwerten,
klappt das?


--------

woher kommen eigentlich die x und y in paintComponent,
sind das Exemplarvariablen, kompiliert das korrekt?

in der Operation setPoint definierst du jedenfalls LOKALE x und y,
diese werden nicht als Exemplarvariablen gespeichert oder ähnliches,
haben auf paintComponent demnach keine Auswirkung

 

[zahil]

ich habe test werte angegeben statt variabln, dass geht auch nicht.....

ja, die waren als lokale variablen definiert, mein fehler, aber habs korrigiert, aber immer noch keine wirkung. die sind jetzt als emxemplar definiert, aber immer noch kein kries.

 

[SlaterB]

wie gesagt: der Button interessiert kein Stück,
male erstmal so einen Kreis, von Anfang an in paintComponent hart reincodiert

geht das?


------

wenn nicht oder am besten trotzdem:
poste ein vollständiges Programm, fehlt ja wohl nur noch ein JFrame durmherum, 20 Zeilen mehr..,
dann kann man erkennen, ob du das GlassPane vielleicht falsch einbaust

oder ist das ein ganz normales JPanel?
setze erstmal eine Hintergrundfarbe/ einen Rahmen um das JPanel,
ist dieses überhaupt zu sehen?

 

[zahil]

na gut, aber der button funktionirt, guck mal in dem method setPoint habe ich system.out.println geschrieben, dass gibt eint output wenn der button gedruckt ist., nur dass kries wird nicht gemalt.

ich hab jetzt dass ganze als ein thread geschrieben. und ein system.out.println in paint auch programiert, aber ich kriege kein output davon.

warum springt dass program zum paint() methode nicht?

glasspane instansiert :

 PCGUI(JFrame f) {
             controllingFrame = f;
             pane = new JPanel(new BorderLayout());
             f.setContentPane(pane);
             pane.setOpaque(true);
             gp = new GlassPane(b5, c2);
             f.setGlassPane(gp);

c2 ist der card darauf der glasspane malen sollte.

und jetzt class GlassPane

public class GlassPane extends JPanel implements Runnable, ActionListener{
        int x, y;
        Graphics screen;   
             
         public GlassPane(AbstractButton aButton, JPanel t) {             
             setVisible(true);
         }
         
         public void actionPerformed(ActionEvent i){
           s = i.getSource();
            if (s == b5) {
               Thread laufer = new Thread(this);
               laufer.run();
            }
         }
         
         public void run(){
                setPoint(s);
                paint(screen);
         }  
         
         public void setPoint(Object s){ 
                    if (s == b5) {
                       if(status[0] == false){
                          x = (b5.getX() + b5.getWidth() - 4);
                          y = (b5.getY() + b5.getHeight() - 2);
                          System.out.println(x + ":" + y);
                       }else{ x = 0;
                       y = 0; }
                    }
         }
             
         public void paint(Graphics screen){
                if(x != 0){
                     if (y != 0){
                        screen.setColor(Color.red);
                        screen.fillRect(x, y, 20, 20);
                        System.out.println(x + ":" + y);
                     }   
                }
         }
       }

 

[SlaterB]

ich persönlich werde mir das schon aus Prinzip nicht anschauen,
poste ein einfaches übersichtliches Programm, das den Fehler enthält,

siehe letztes Post in diesem Thread hier für ein Beispiel
http://www.java-forum.org/de/viewtopic.php?t=37640

60 Zeilen, voll lauffähig,
alles andere an Layout und Funktionalität interessiert nicht

 

[Guest]

poste ein vollständiges Programm, fehlt ja wohl nur noch ein JFrame durmherum, 20 Zeilen mehr..,

*lacht* glatte Fehleinschätzung :wink:

 

[Azrahel NA]

ich persönlich werde mir das schon aus Prinzip nicht anschauen

Wieso. er hat doch absolut genau gemacht was du wolltest

@zahil Bau ein KSKB (Kurzes Selbständiges Kompilierbares Beispiel), erstens findest du den Fehler dabei vielleicht schon selbst, zum zweiten wird sich echt keiner, so lieb die Jungs hier auch sind, den Akt machen sich durch 693 Zeilen Code durchzubeissen, außer du läßt mal mindestens nen Kasten dafür springen, das ist nämlich auch für die JavaGötter hier echt Arbeit :wink:

 

[zahil]

ok, alles was für glasspane wichtig ist....

ich versuche auch ein kskb zu bauen, bis dann könnte ihr dass ansehen

 

[SlaterB]

so, habs auch mal selber versucht,
gar nicht leicht, ein GlassPane zum Laufen zu bekommen,

es darf erst nach dem JFrame auf visible gestellt werden..,

und du solltest nie thread.run() aufrufen
dann wird nur die Operation aufgerufen, kein eigener Thread gestartet, dafür gibts thread.start()


und allgemein lieber paintComponent statt paint verwenden

import java.awt.Color;
import java.awt.Graphics;

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

public class TestGUI extends JFrame {

	public TestGUI() throws Exception {
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

		MyGlassPane gp = new MyGlassPane();
		setGlassPane(gp);
		setSize(200, 200);
		setVisible(true);
		gp.setVisible(true);
	}

	public static void main(String[] args) throws Exception {
		new TestGUI();
	}

}

class MyGlassPane extends JPanel implements Runnable {
	int x, y;

	public MyGlassPane() {
		Thread laufer = new Thread(this);
		laufer.start();
	}

	public void run() {
		while (true) {
			try {
				Thread.sleep(1000);
			} catch (InterruptedException e) {
				e.printStackTrace();
			}
			x = (int) (Math.random() * 100 + 20);
			y = (int) (Math.random() * 100 + 20);
			repaint();
		}
	}

	public void paintComponent(Graphics g) {
		super.paintComponent(g);
		System.out.println(x + ":" + y + " p ");
		if ((x != 0) && (y != 0)) {
			g.setColor(Color.red);
			g.fillRect(x, y, 20, 20);

		}
	}
}

 

[Azrahel]

Cool Slaterb, ich wusst garnicht das man das auch auf der GlassPane machen kann, ich hab mir da immer einen auf der ContentPane abgebrochen. Werd ich direkt mal vermerken müssen :applaus:

 

[SlaterB]

ähm, was kann man denn da 'machen', malen?
also das hätte ich schon vermutet, obwohl ich das auch noch nie gemacht hatte

 

[zahil]

es tut mir leid, aber es geht dass nicht, sobald man gp.setvisible macht, blödeweise kann man nichts durch den glass pane sehen...keine andere componenten und containers

 

[SlaterB]

tztz, behaupte ich dass es geht?

ich behaupte nur dass man da einen Kasten malen kann wenn man das ganze ordentlich programmiert,

vielleicht fehlt setOpaque(true/ false)



vielleicht machst du es so wie es in einem TUTORIAL steht,
bevor du dir da selber was bastelst
http://java.sun.com/docs/books/tutorial/uiswing/components/rootpane.html

dort ging das glaube ich, da wird JComponent statt JPanel für das GlassPane genommen,
und dann muss wohl das
super.paintComponent(g);
aus
public void paintComponent(Graphics g)
raus

(ich schaue nur wie es dort geht, frage nicht mich nach den genauen Details)