Warum wird nich gezeichnet?

[Kondition0]

Hallo liebes Forum,

ich versuche gerade ein Malprogramm zu erstellen und taste mich langsam vor.
Momentan soll an der Stelle, wo auf der Canvas (cv) geklickt wird, ein Punkt gemalt werden (komponente -> cvg)

Dazu habe ich eine eigene Methode gemacht ( zeichnen(cvg) ), die scheinbar aber fehlerhaft ausgeführt, bzw nicht erkannt wird.

Die Koordinaten werden ausgegeben (also bis zur mousePressed methode geht alles) doch die Zeichen Methode scheint nicht gestartet zu werden.

Wo liegt der Fehler? Ich seh ihn nichgt

//
// Freihand zeichnen
//

import java.awt.*;
import java.awt.event.*;
import java.math.*;
import java.awt.Color;
import java.awt.Graphics;

import javax.swing.JFrame;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;



//Interface
public class Malprogramm 
extends Frame 
implements MouseListener, Runnable, ActionListener 
{
	
int x;
int y;
Canvas cv;
Graphics cvg;

boolean grün=false , blau=false, rot = false, gelb = false;
Button b_grün,      b_blau     ,b_rot       , b_gelb , b_erase , b_save     ;

    public static void main(String[]args)
    {
    	Malprogramm	wgh =new Malprogramm ();
    	wgh.setSize(500,500);
    	wgh.setBackground(Color.gray);
    	wgh.show();	
    }
    
    public Malprogramm()
    {
    	super ("Mein Malprogramm");

    	//Fenster schließbar machen
        WindowListener wl = new WindowAdapter() {
            public void windowClosing(WindowEvent e) {
                System.exit(0);
            }
        };
        addWindowListener(wl);

        
    	// 1. Panel
    	Panel p1= new Panel();
    	p1.setBounds(10,45,40,40);

    	
    	//Grüner Button
		b_grün= new Button("grün");
		b_grün.setBackground (Color.green);

		
    	//Grüner Button
		b_gelb= new Button("gelb"); 
		b_gelb.setBackground (Color.yellow);

		
    	//Grüner Button
		b_blau= new Button("blau");
		b_blau.setBackground (Color.blue);

		
    	//Grüner Button
		b_rot= new Button("rot");
		b_rot.setBackground (Color.red);
		
    	//erase
		b_erase= new Button("erase");
		b_erase.setBackground (Color.white);

    	//save
		b_save= new Button("save");
		b_save.setBackground (Color.white);
		
		p1.add(b_grün);
		p1.add(b_gelb);
		p1.add(b_blau);
		p1.add(b_rot);
		p1.add(b_erase);
		p1.add(b_save);
		
		b_grün.addActionListener(this);
		b_gelb.addActionListener(this);
		b_rot.addActionListener(this);
		b_blau.addActionListener(this);
		
// Panel 2
		Panel p2 = new Panel ();
		p2.setLayout(null);

// CANVAS
	Canvas cv = new Canvas();
	cv.setBackground(Color.white);
	cv.setBounds(1,1,1000,1000);
	cv.addMouseListener(this);
	p2.add(cv);

	 
//Hauptfenster Layout
	setLayout(new GridLayout(2,1));
	add(p1);
	add(p2);
    }
    
//ActionPerformed Methode
	public void actionPerformed(ActionEvent e)
	{
		if(e.getSource()==b_blau)
		{
			blau=true;
			grün=false;
			gelb=false;
			rot=false;
		}
		
		if(e.getSource()==b_grün)
		{
			blau=false;
			grün=true;
			gelb=false;
			rot=false;
		}
		
		if(e.getSource()==b_gelb)
		{
			blau=false;
			grün=false;
			gelb=true;
			rot=false;
		}
		
		if(e.getSource()==b_rot)
		{
			blau=false;
			grün=false;
			gelb=false;
			rot=true;	
		}
		
	} 
	
	public void start(){
		Thread th = new Thread(this);
		th.start();
	}
	
	public void run(){
		
		while (true){	
		try {
				Thread.sleep (80);
			}
			catch (InterruptedException ex){}
			
		}

	}
   
    //Tastendruck und Aktion
    public  void mousePressed(MouseEvent event)
    {
    	
    	x = event.getX();
    	y = event.getY();
    	System.out.println(x + " " + y);
    	zeichnen(cvg);

 
    }
    public void mouseReleased(MouseEvent event)
    {

    }

    public void mouseClicked(MouseEvent event)
    {
    }
    
    public void mouseEntered(MouseEvent e) {}

    public void mouseExited(MouseEvent e) {}
    
 // ZeichenMethode   
    public void zeichnen(Graphics grafik)
    {
  //Grüne Farbe gewählt
    	if (grün==true)
    	{
    	cvg = cv.getGraphics();
    	cvg.setColor(Color.green);
  		cvg.fillOval(x, y, 20, 20);
    	}
   // Rote Farbe gewählt
    	if (rot==true)
    	{
    	cvg = cv.getGraphics();
    	cvg.setColor(Color.red);
  		cvg.fillOval(x, y, 20, 20);
    	}
    // blaue Farbe gewählt	
    	if (blau==true)
    	{
    	cvg = cv.getGraphics();
    	cvg.setColor(Color.blue);
  		cvg.fillOval(x, y, 20, 20);
    	}
    //Gelbe Farbe gewählt	
    	if (gelb==true)
    	{
    	cvg = cv.getGraphics();
    	cvg.setColor(Color.yellow);
  		cvg.fillOval(x, y, 20, 20);
    	}

    }
    
    
    



}

 

[Marco13]

http://www.java-forum.org/plauderecke/79231-verbesserungsvorschlaege-33.html#post526853 (getGraphics auf Components ist böse)

Suche in der FAQ nach "Zeichnen in Swing"... bei konkreteren Fragen nochmal melden...

 

[Schandro]

Tut mir Leid das sagen zu müssen, aber: Du machst das zeichnen komplett falsch. Les dir mal ein Tutorial oder ein entsprechendes Kapitel aus einem Handbuch dazu durch.

Hier noch mein Standard Beispielcode:

package test;

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

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

public class XXX{

	public static void main(String[] args){
		new XXX();}


	private JFrame window = new JFrame();
	private int cnt; // wird zur Kommunikation ziwschen dem animationThread und paintComponent benutzt

	public XXX(){
		window.setBounds(100,100,400,200);
		window.add(panel);
		
		SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				window.setVisible(true); 
			}
		});
		
		Thread animationThread = new Thread(new Runnable(){
			public void run(){
				while(cnt < 300){
					++cnt;
					panel.repaint();
					
					try{
						Thread.sleep(10);
					}catch(InterruptedException e){
						e.printStackTrace();
					}
				}
			}
		});
		animationThread.start();
	}


	JPanel panel = new JPanel(){
		public void paintComponent(Graphics g){
			super.paintComponent(g);
			
			g.setColor(Color.BLACK);
			g.fillRect(cnt,50,50,50);
		}
	};

}

 

[Kondition0]

Ich weiß, dass ich einiges falsch mache, sonst würde es ja funktionieren...

ich habe bereits alles versucht, tuts gelesen etc, aber komm nich klar mit.
Könnt mir auch nicht weiterhelfen außer "les nochmal" oder ?

Schade

 

[Marco13]

Hmja, die Alternative wäre, deinen Code so umzuschreiben, dass er funktioniert. Vielleicht erklärt sich dazu noch jemand bereit. Bis dahin noch die eine, "goldene" Regel - wenn man die beachtet, kann man schon nicht mehr soooo viel falsch machen:

Alles, was man zeichnet, muss von der paint-Methode aus gezeichnet werden (und zwar in das Graphics-Objekt, das man in der paint-Methode übergeben bekommt)

 

[Schandro]

Könnt mir auch nicht weiterhelfen außer "les nochmal" oder ?

Naja, wie soll man das schon erklären?
Guck dir doch z.b. mal meinen Beispielcode an und schraub an ihm ein bisschen herum, um das Prinzip dahinter zu verstehen.

 

[Kondition0]

Naja, wie soll man das schon erklären?
Guck dir doch z.b. mal meinen Beispielcode an und schraub an ihm ein bisschen herum, um das Prinzip dahinter zu verstehen.

Das Programm hat leide nichts mit meinem zu tun.
Ich möchte auf das Canvas malen, du jedoch hast ein globales g als Variable.
Soweit war ich auch schon, da hat er dann gemalt.
Doch nun will ich auf der Canvas malen und brauch dazu eine eigene Zeichenmethode.

Ich glaub, ich hab einfach nur einen Flüchtigkeitsfehler gemacht und wollt schaun ob ihr erfahrenen Programmierer das schnell bemerkt.

Verweiße auf anderen SourceCode wie der bringt mir nix und hab ich selber zu Haufen.

 

[Kondition0]

Alles, was man zeichnet, muss von der paint-Methode aus gezeichnet werden (und zwar in das Graphics-Objekt, das man in der paint-Methode übergeben bekommt)

Das ist nicht war.
Ich hab ein Animationsprogramm mit eigener Zeichenmethode gemacht, das aus verschiedenen Gründen angebracht war (Doppel pufferung z.B)

 

[Schandro]

^^ Vergiss deinen Code erstmal. Du musst zuerst die Grundlagen von Zeichnen mit Swing/AWT verstehen. Deswegen hab ich meinen Code gepostet (der nix direkt mit deinem Code zu tun hat...), damit du mal ein Beispiel hast wie mans richtig macht.

Ich glaub, ich hab einfach nur einen Flüchtigkeitsfehler gemacht

Nein.

 

[Leroy42]

Das Programm hat leide nichts mit meinem zu tun.

Eben!

Ich möchte auf das Canvas malen, du jedoch hast ein globales g als Variable.

Schandro hat kein "globales g" sondern es ist als Übergabeparameter
in einer, durch das System aufgerufenen Methode, deklariert!

Soweit war ich auch schon, da hat er dann gemalt.

Toll!

Doch nun will ich auf der Canvas malen und brauch dazu eine eigene Zeichenmethode.

Nein! Du mußt nur die paintComponent-Methode deines Canvas überschreiben. :toll:

Ich glaub, ich hab einfach nur einen Flüchtigkeitsfehler gemacht und wollt schaun ob ihr erfahrenen Programmierer das schnell bemerkt.

Flüchtigkeitsfehler in einem ansonsten total wuseligen Programm
zu finden, liegt uns einfach nicht.

Verweiße auf anderen SourceCode wie der bringt mir nix und hab ich selber zu Haufen.

Dann ist dieses Forum für dich leider kaum hilfreich! ;(

 

[Marco13]

Das ist nicht war.
Ich hab ein Animationsprogramm mit eigener Zeichenmethode gemacht, das aus verschiedenen Gründen angebracht war (Doppel pufferung z.B)

Und in der paint-Methode hast du das, was du im Puffer gespeichert hattest, dann wirklich gezeichnet. Bei Swing ist Double Buffering aber eh schon eingebaut. Das nur nebenbei.