Aus endlosschleife kommen

[Jüngling]

hay
ich hab ein problem in folgenden applet:

import java.awt.*;
import java.awt.event.*;
import java.applet.Applet;

/**
  *
  * Beschreibung
  *
  * @version 1.0 vom 20.05.2009
  * Autor: Marius Obert
  */

public class ball extends Applet implements ActionListener{

  private int diameter1=200;               //durchmesser(diameter)1
  private int diameter2=50;                //durchmesser(diameter)2
  private int xP=100;                      //x.Koordinate
  private int yP=100;                      //y-Koordinate
  private int xChange=4,yChange=3;         // richtungs werte
  private int speed=50;                    //Drehgeschw.
  private Button btnDreh, btnStop;         //Buttons zum Drehen, Stoppen
  private TextField tfSpeed;               //Textfeld zur eingabe der drehgeschw.
  private boolean activ=false;             //Ob es sich drehen soll oder nicht
  private int anfStr=0;                    //zur Gradeingabe der Strahlen

  public void init() {
  
  btnDreh= new Button("Drehe!");           //Aktivierung Button Drehe
  add(btnDreh);
  btnDreh.addActionListener(this);

  btnStop= new Button("Stop!");            //Aktivierung Button Stop
  add(btnStop);
  btnStop.addActionListener(this);
  
  tfSpeed= new TextField(10);              //Aktivierung Textfeld Speed
  add(tfSpeed);
  tfSpeed.addActionListener(this);
  }
  
  public void actionPerformed(ActionEvent event){
    if(event.getSource()==btnDreh){             //Falls btnDreh, dann activ=true
      activ=true;
    }
    else if(event.getSource()==btnStop){       //Falls btnStop, dann activ=false
      activ=false;
    }
    else{
      speed=Integer.parseInt(tfSpeed.getText());   //Falls TfSpeed, dann speed=eingabe
    }

    repaint();                                       //erneuern
  }
  
  public void aussen(){                                 //zeichne äußeren kreis
    Graphics g=getGraphics();
    g.setColor(Color.red);
    g.drawOval(xP-1,yP-1,diameter1+2,diameter1+2);
  }

  public void loeaussen(){                                 //lösche äußeren kreis
    Graphics g=getGraphics();
    g.setColor(Color.white);
    g.drawOval(xP-1,yP-1,diameter1+2,diameter1+2);
  }

  public void innen(){                                     //zeichne inneren kreis
    Graphics g=getGraphics();
    g.setColor(Color.blue);
    g.fillOval(xP+diameter1/2-diameter2/2,yP+diameter1/2-diameter2/2,diameter2,diameter2);
  }
  
  public void loeinnen(){                                   //lösche inneren Kreis
    Graphics g=getGraphics();
    g.setColor(Color.white);
    g.fillOval(xP+diameter1/2-diameter2/2,yP+diameter1/2-diameter2/2,diameter2,diameter2);
  }
  
  public void striche(){                               //zeichne Strahlen
    for(int i=0;i<=10;i++){
      Graphics g=getGraphics();
      innen();                                        //zeichne erneut inneren kreis damit Strahlen ihn nicht überdecken
      g.setColor(Color.yellow);
      g.fillArc(xP,yP,diameter1,diameter1,anfStr+36*i,-5);
      innen();                                        //zeichne erneut inneren kreis damit Strahlen ihn nicht überdecken
    }
  }

  public void loestriche(){                        //lösche Strahlen
    for(int i=0;i<=10;i++){
      Graphics g=getGraphics();
      innen();                                     //zeichne erneut inneren kreis damit Strahlen ihn nicht überdecken
      g.setColor(Color.white);
      g.fillArc(xP,yP,diameter1,diameter1,anfStr+36*i,-5);
      innen();                                    //zeichne erneut inneren kreis damit Strahlen ihn nicht überdecken
    }
  }

  public void warten(int ms){                    //wartezeit
    try{
      Thread.sleep(ms);
    }
    catch(InterruptedException e){ }
  }

  
  public void paint(Graphics g){            //eigentlicher Vorgang

       aussen();
       striche();

       while(activ==true){

       aussen();
       striche();
       warten(speed);
       loestriche();
       anfStr=anfStr+5;
       loeaussen();
       loeinnen();
       if(xP<=0) xChange=-xChange;
       else if(xP>=getSize().width-diameter1) xChange=-xChange;
       else if(yP<=30) yChange=-yChange;
       else if(yP>=getSize().height-diameter1) yChange=-yChange;
       xP=xP+xChange;
       yP=yP+yChange;



       }

  }
}

mein prob ist das ich nicht mehr aus der endlosschleife rauskomme.....
den mechanismus habe ich ja eig. mit dem Stop-Button geplant. Aber während der schleife überprüft es nicht ob der button gedrückt wurde......kann man i-wie ActionPerformed merhfach aufrufen

 

[SlaterB]

niemals getGraphics(); aufrufen,
in paint() hast du das Graphics-Objekt, das kannst du als Parameter an andere Methoden verteilen

kein Schleife in paint(), solange die läuft, kommt kein ActionListener zum Zuge,
lasse separat einen Thread laufen, der alle 30 ms repaint() aufruft

 

[Schandro]

1. Du darfst NIEMALS in einer paint Methode was andere als malen machen, schon garnicht ne Unendlichschleife oder Thread.sleep.

2. Du darfst NIEMALS getGraphics(); aufrufen.


Hier mal ein Beispielcode:

package snippetbin;

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
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;
    private JPanel panel;
    private int cnt; // wird zur Kommunikation ziwschen dem animationThread und paintComponent benutzt
    private javax.swing.Timer paintTimer;
    
    public XXX(){
    	window = new JFrame();
        window.setBounds(100,100,400,200);
        
        panel = new JPanel();
        window.add(panel);
        
        
        panel.setLayout(new BorderLayout());
        
        JButton button = new JButton("Stop/Start");
        button.addActionListener(new ActionListener(){
			@Override
			public void actionPerformed(ActionEvent e) {
				if(paintTimer.isRunning()){
					paintTimer.stop();
				}
				else{
					paintTimer.start();
				}
        
			}
        });
        panel.add(button,BorderLayout.NORTH);
      
        panel.add(paintPanel);

        SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                window.setVisible(true);
            }
        });


        paintTimer = new javax.swing.Timer(10,new ActionListener(){
			public void actionPerformed(ActionEvent event){
				++cnt;
				paintPanel.repaint();
			}
		});
        
        paintTimer.start();
        
    }


    private JPanel paintPanel = new JPanel(){
        public void paintComponent(Graphics g){
            super.paintComponent(g);

            g.setColor(Color.BLACK);
            g.fillRect(cnt,50,50,50);

            paintThings(g);
        }
    };

    private void paintThings(Graphics g){ // anstatt hier getGraphics() aufzurufen kannst du dir auch einfach das Graphics-Object als Parameter mitgeben lassen. Wichtig: Diese Methode darf und kann nur von der paintComponent-Methode aus aufgerufen werden!
    	g.drawLine(0,45,1000,45);
    	g.drawLine(0,105,1000,105);
    }

}

 

[Jüngling]

wieso darf man getGraphics(); nicht aufrufen? unser IT lehrer hat uns so dieses applet gezeigt.....repaint() kenne ich zwar aber dann flimmert es doch?

danke für deinen verbesserungsvorschlag schandro...jedoch kenne ich die hälfte der import klassen nichteinmal geschweige den kann ich sie benutzen....

 

[Paddelpirat]

Um das flimmern weg zu bekommen solltest du mal nach dem Stichwort "DoubleBuffered" suchen. Ich habe es mal auf die schnelle folgendermaßen implementiert, wobei es sicherlich noch besser geht:

public void paint(Graphics g){            //eigentlicher Vorgang
	  paintDoubleBuffered(g);
  }
  
  public void paintDoubleBuffered(Graphics g) {
	  paintDirect(g);
	  malen(g);
  }
  
  public void paintDirect(Graphics g) {
	  g.setColor(Color.white);
	  g.fillRect(xP-10, yP-10, diameter1+36, diameter1+36); //Löschen des alten Rades
	  malen(g);
  }
  
  private void malen(Graphics h) {
	  Graphics g = getGraphics();
	  aussen(g);
      striche(g);
      
      aussen(g);
      striche(g);
      anfStr=anfStr+5;
      if(second) { //Die position wird nur bei jedem zweiten Zeichnen verändert, siehe DoubleBuffered.
    	  if(xP<=0) xChange=-xChange;
    	  else if(xP>=getSize().width-diameter1) xChange=-xChange;
    	  else if(yP<=30) yChange=-yChange;
    	  else if(yP>=getSize().height-diameter1) yChange=-yChange;
    	  
    	  xP=xP+xChange;
    	  yP=yP+yChange;
      }
      second = !second; //Boolean der angibt, ob es sich um das erste oder zweite Bild handelt.
  }

Die ganzen Lösch-Methoden habe ich mal aus Performance-Gründen durch das Zeichnen eines einfachen weißen Rechtecks ersetzt.

 

[Jüngling]

also danke für eure antworten ich habe es jetzt mit repaint() und ohne filmmern hinbekommen.....sogar meine lehrer waren beeindruckt dass es ohne einen thread geht

ganz großes DANKE!!!