Warum tut sich nix

[jona]

Warum tut sich hier nix,wenn ich auf den Knopf drücke ?
Danke

import java.awt.Color;
import java.awt.Component;
import java.awt.Container;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.SystemColor;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.geom.Arc2D;
import java.awt.geom.Ellipse2D;
import java.awt.geom.Line2D;
import java.awt.geom.Rectangle2D;
import java.awt.geom.Arc2D.Double;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JOptionPane;
import javax.swing.JPanel;


public class quadrate {
	public static void main (String[] args){
		MyFrame frame =new MyFrame();
		
		frame.show();
	}
}
		
		class MyFrame extends JFrame
		{
public MyFrame ()
{
	setTitle ("Main frame");
	setSize (500,500);
	MyPanel panel = new MyPanel();
	Container pane =getContentPane ();
	pane.add(panel);
	
	}
		

	class MyPanel extends JPanel
	{
		public MyPanel()
		{
			JButton yButton = new JButton("Yellow");
			add(yButton);
			MyAction kk = new MyAction();
			yButton.addActionListener(kk);
			
			

	    	 
		
		}
	public void paintComponent (Graphics g)
	{
		
		super.paintComponent (g);
		Graphics2D g2 = (Graphics2D) g;
		
		setBackground (Color.green);
	
	
		Rectangle2D rect1 = new Rectangle2D.Double(40,300,60,100);
		g2.draw(rect1);
		g2.setPaint(Color.red);
		g2.fill(rect1);
		g.setColor(Color.black);
		Line2D line1 =new Line2D.Double(40,300,60,200);
		g2.draw(line1);
		
		Line2D line2 =new Line2D.Double(60,200,100,300);
		g2.draw(line2);
	
		
		double x1 =200,y1=30,x2=100,y2=100;
		
		
	    

		for ( double i = y1,i1= y2; i <= 300; i+=20,i1+=20 ) 
			
		{
	   	Ellipse2D arc2 = new Ellipse2D.Double(x1,i,x2,i1);
	   	g2.setPaint(Color.yellow);
	   	g2.draw(arc2);
	   	
	   	
	   	
	   
		
	    	
	    	;
	}
	}}
				

	 class MyAction implements ActionListener
	{
		public void actionPerfomed (ActionEvent event)
		{
			
			JOptionPane.showMessageDialog(null, "Color" );
		}

		@Override
		public void actionPerformed(ActionEvent e) {
			// TODO Auto-generated method stub
			
		}

	}
	
	}

 

[icarus2]

Kann es sein, dass du vergessen hast in dieser Methode hier den Code zu schreiben:
[Java]
@Override
public void actionPerformed(ActionEvent e) {
// TODO Auto-generated method stub

}
[/Java]

Diese Methode wird bei einem Event aufgerufen. Wenn kein Code drin ist passiert auch nix. Schreie da mal den Code JOptionPane.show....(...) rein.

 

[AmunRa]

Du hast einmal in deiner ActionListener Klasse die Methode

public void actionPerfomed (ActionEvent event)

und daruter eine Lehre Methode actionPerformed( ActionEvent e)


lösch die Untere Weg und füge in der MEthode darüber das R an der richtigen Stelle ein.


WEiter solltest du in deiner Main Methode micht frame.show() aufrufen, sonder frame.setVisible(true);

die show Methode is Depected und sollte daher nicht mehr verwendet werden.

 

[jona]

Danke,ich habe verstanden.:applaus:

Ich habe noch eine Frage. Wie kann ich ein Element von Graphics2D bewegen.
So geht es nicht,weil die alte Elemente bleiben und ich weiß nicht wie ich die löschen kann.

for ( double i = y1,i1= y2; i <= 300; i+=20,i1+=20 ) 
			
		{
	   	Ellipse2D arc2 = new Ellipse2D.Double(x1,i,x2,i1);
	   	g2.setPaint(Color.yellow);
	   	g2.draw(arc2);
	   	
	   	
	   	
	   
		
	    	
	    	;
	}

 

[hdi]

Das Löschen der alten Grafiken vor dem Zeichnen der neuen erreicht man durch

super.paintComponent(g);

in der ersten Zeile deiner paintComponent-Methode. Alternativ:

g.setColor(getBackground());
g.fillRect(0,0,getWidth(), getHeight());

Genau das ist nämlich die Implementierung der paintComponent in der Vaterklasse.

 

[SlaterB]

wichtig ist, dass du in einer paintComponent-Methode keine Bewegung darstellen kannst,
die Methode sollte immer sofort enden und nur den aktuellen Stand malen

Tutorial:
http://www.ralf-bauer.org/java/tutorial/Tutorial.zip

 

[jona]

Damit die Programm langsame zeichnet ,will ich die Methode sleep() benutzen. Irgendwo zeigt es Fehler.

zahl 43 a.sleep(500) ;
A cannot be resolved

Was soll ich tun ?

P/S Ihr seit alle so nett :applaus:

public class quadrate {
	public static void main (String[] args){
		MyFrame frame =new MyFrame();
		
		frame.show();				
	}
}		
		class MyFrame extends JFrame		
		{
			public class Stopjj extends Thread {			
				public void run()
				{ System.out.println( "soooooooooo  " );				
				 }													
			}
public MyFrame ()
{
	setTitle ("Main frame");
	setSize (500,500);
	MyPanel panel = new MyPanel();
	Container pane =getContentPane ();
	pane.add(panel);
	Stopjj a = new Stopjj();
    a.start();
     
	}
	class MyPanel extends JPanel
	{	
		public MyPanel()
		{
			JButton yButton = new JButton("Yellow");
			add(yButton);
			MyAction kk = new MyAction();
			yButton.addActionListener(kk);	    	 	
		}
	public void paintComponent (Graphics g)
	{
		
		super.paintComponent (g);
		Graphics2D g2 = (Graphics2D) g;
		double x1 =200,y1=30,x2=100,y2=100;
for ( double i = y1,i1= y2; i <= 300; i+=1,i1+=1 ) 
{
	a.sleep(500) ;
	super.paintComponent (g);	
		{
		setBackground (Color.green);
		
		Rectangle2D rect1 = new Rectangle2D.Double(40,300,60,100);
		g2.draw(rect1);
		g2.setPaint(Color.red);
		g2.fill(rect1);
		g.setColor(Color.black);
		Line2D line1 =new Line2D.Double(40,300,60,200);
		g2.draw(line1);
		
		Line2D line2 =new Line2D.Double(60,200,100,300);
		g2.draw(line2);
	   	Ellipse2D arc2 = new Ellipse2D.Double(x1,i,x2,i1);
	   	
	   	g2.setPaint(Color.yellow);
	   	g2.fill(arc2);		 		    	
	}}	
}
	}};
	 class MyAction implements ActionListener
	{
		public void actionPerformed(ActionEvent e) {
			JOptionPane.showMessageDialog(null, "Colorgggggg" );			
		}
	}

 

[SlaterB]

a ist nur eine lokale Variable im Konstruktor, in anderen Methoden nicht bekannt (ein Klassenattribut wäre es)

macht aber nix, denn du brauchst a gar nicht, sleep ist sowieso eine statische Methode von Thread, also Thread.sleep(..) schreiben

macht aber auch nix, denn so geht es sowieso nicht, was ich um 20:42 geschrieben habe ignorierst du?
das paint darf nur paar ms dauern und ist dann fertig,
eine längere Schleife mit sleep kannst du in die run-Methode des Threads einbauen wenn du willst, nach jedem Zyklus repaint() aufrufen,
siehe Tutorial

 

[jona]

Ich habe versucht aber es klappt nicht Warum ?
Der Kreis soll sich von oben nach unten bewegen. Und es soll sichtbar sein.
Tutorial habe ich ein wenig gelesen.

public class quadrate{
	public static void main (String[] args){
		MyFrame frame =new MyFrame();
		frame.show();	
	}
}		
class MyFrame extends JFrame				
		{
			Color col = Color.green;
			
public MyFrame ()
{
	setTitle ("Main frame");
	setSize (500,500);
	setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE ); 
	MyPanel panel = new MyPanel();
	Container pane =getContentPane ();
	pane.add(panel);	 
	}
	class MyPanel extends JPanel
	{	
			public MyPanel()
		{
		JButton yButton = new JButton("Night");
			add(yButton);
			MyAction kk = new MyAction();
			yButton.addActionListener(kk);	    	 	
		}
	public void paintComponent (Graphics g)
	{
		
		super.paintComponent (g);
		Graphics2D g2 = (Graphics2D) g;
		
		double x1 =200,y1=30,x2=100,y2=100;
for ( double i = y1,i1= y2; i <= 300; i+=10,i1+=10 ) 
{
	class Stopjj extends Thread {	
		
	public void run()
	{ System.out.println( "thread go  " );
	
	try {
		for (int f=1;f<=10;f++){
		Thread.sleep(500);
		repaint();
		}
		
	} catch (InterruptedException e) {
		e.printStackTrace();
	}	 }		
}	
{	
	super.paintComponent (g);	
		{
		setBackground (col);
		Rectangle2D rect1 = new Rectangle2D.Double(40,300,60,100);
		g2.draw(rect1);
		g2.setPaint(Color.red);
		g2.fill(rect1);
		g.setColor(Color.black);
		Line2D line1 =new Line2D.Double(40,300,60,200);
		g2.draw(line1);
		
		Line2D line2 =new Line2D.Double(60,200,100,300);
		g2.draw(line2);
	   	Ellipse2D arc2 = new Ellipse2D.Double(x1,i,x2,i1);
	   	repaint();
	   	g2.setPaint(Color.yellow);
	   	g2.fill(arc2);	
	   	Stopjj a = new Stopjj();
	    a.start();
	    
	} }	
}
	}};
	 class MyAction implements ActionListener
	 
	{		
		public void actionPerformed(ActionEvent e) {
			col = Color.BLACK;
			repaint();
			JOptionPane.showMessageDialog(null, "night" );				
		}	
	}	}

 

[ARadauer]

} }
}
}};

das alles ist wesentlich einfacher, wenn man sich um saubere einrückungen bemüht ;-)

Benutz du Eclipse? Strg+A und Strg+I

was soll die Klasse Stopjj mitten im Code? so macht man das nicht :noe: :shock:

 

[SlaterB]

zum dritten Mal: keine Schleife oder sonstige Bewegung in der paintComponent,
nein es hilft auch nicht, sinnlos irgendwelche neuen Klassen darin zu definieren

das ist ein kompliziertes Thema, was du einfach durch ausprobieren nicht erlernen kannst,
zum dritten Mal: siehe Tutorial

 

[jona]

Benutz du Eclipse? Strg+A und Strg+I

:toll:

 

[jona]

Ich habe es endlich einigermase geschafft .