Weder ActionListener noch Graphics wollen so recht

[SebiG]

Hallo zusammen,

ich bin ein Java-Neuling und möchte ein Programm haben, dass auf Klick eines Buttons ein Diagramm auf ein Frame zeichnet. Das ganze sollte via AWT laufen, da ich von Swing noch weniger Ahnung habe.
Ich schaffe es inzwischen sogar ein Frame passender Größe mit einem Button zu erzeugen, aber ich verzweifle daran dem Button beizubringen etwas zu tun und auch die Methode paint() reagiert bei mir überhaupt nicht, bzw. tut nichts.

Ich pack euch mal den Quelltext hier rein und wäre richtig begeistert, wenn ihr mir helfen könntet. Ich finde nämlich weder mit dem Javaheft vor mir, noch mit den zahllosen Quellen im Netz eine halbwegs funktionierende Lösung (mal klappt das Zeichnen, dafür gehen die EVents nicht, etc.) und das seit Stunden.

import java.awt.*;
import java.awt.event.*;
import java.awt.Graphics;
public class zufall extends Frame implements ActionListener{
	   static Frame frame=new Frame("zufall");
	   static Button berechne = new Button("Berechnen");
	   public static void main(String [] args){
		 //Frame erzeugen und für das korrekte Schließen sorgen:
	      //Frame frame = new Frame("zufall");
	      frame.addWindowListener(new WindowAdapter() {
	    	   public void windowClosing(WindowEvent e) {
	    	      System.exit(0);
	    	   }
	    	});
	      frame.setSize(360,450);
	      frame.setVisible(true);
	      //frame.show();	      
	      frame.setLayout( new FlowLayout());
	      frame.add(berechne);
	      berechne.addActionListener(null);
	   }
	   public void paint( Graphics g )
	   {
	     g.drawString( "Hallo Welt", 100, 60 );
	   }
	   public void actionPerformed( ActionEvent e )
	        {
	        	frame.setSize(500,500);
	        	berechne.setLabel("Toll");
	        
	    }
	}
}

Vielen Dank !

 

[Marco13]

Ich würde dir von Anfang an zu Swing raten: Es ist nicht "schwieriger" als AWT - eher im Gegenteil, wenn du z.B. später verzweifelt versuchst, das Flackern durch double-buffering wegzukriegen :wink:

Und sonst... Klassennamen schreibt man groß.
berechne.addActionListener(null);
ist schonmal offensichtlich falsch. Außerdem sollte man nicht irgendwelche statischen Fields verwenden. Und im Moment wird ja NIE irgendwo ein "Zufall"-Objekt erstellt. Nur ein Frame. Aber es empfiehlt sich, NICHT die paint-Methode von einem Frame zu überschreiben.

Hier mal wit wenigen Änderungen "lauffähig gemurkst" Das sollte aber nicht als Ausgangspunkt dienen.

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

public class Zufall extends Frame implements ActionListener
{
      Button berechne = new Button("Berechnen");

      public static void main(String [] args)
      {
         Zufall frame = new Zufall();
         frame.addWindowListener(new WindowAdapter() {
             public void windowClosing(WindowEvent e) {
                System.exit(0);
             }
          });
         frame.setSize(360,450);
         frame.setVisible(true);
         //frame.show();
         frame.setLayout( new FlowLayout());
         frame.add(frame.berechne);
         frame.berechne.addActionListener(frame);
      }
      public void paint( Graphics g )
      {
          super.paint(g);
           g.drawString( "Hallo Welt", 100, 60 );
      }
      public void actionPerformed( ActionEvent e )
      {
          berechne.setLabel("Toll");
      }
}

Hier mal ein bißchen aufgeräumt, mit Swing

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

public class Zufall extends JFrame implements ActionListener
{
    //--->--- Ignorieren ;-)
    public static void main(String [] args)
    {
        SwingUtilities.invokeLater(new Runnable()
        {
            public void run()
            {
                new Zufall().setVisible(true);
            }
        });
    }
    //---<--- Ignorieren ;-)



    //------------ Variablen (Instanzvariablen, Fields)

    private JButton berechne;
    private PaintPanel paintPanel;


    //------------ Konstruktor
    public Zufall()
    {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(360,450);
        setLayout(new BorderLayout());

        // Button oben hin
        berechne = new JButton("Berechnen");
        berechne.addActionListener(this);
        add(berechne, BorderLayout.NORTH);

        // PaintPanel in die mitte
        paintPanel = new PaintPanel();
        add(paintPanel, BorderLayout.CENTER);
    }


    //------------- für "implements ActionListener"
    public void actionPerformed( ActionEvent e )
    {
        berechne.setText("Toll");
        paintPanel.togglePaintSomething();
    }
}


// Panel das zeichnet
class PaintPanel extends JPanel
{
    private boolean paintSomething = false;

    public void togglePaintSomething()
    {
        paintSomething = !paintSomething;
        repaint();
    }

    public void paintComponent(Graphics g)
    {
        super.paintComponent(g);

        //---Zeichencode: ---------------------

        if (paintSomething)
        {
            g.drawString("Hallo", 20,20);
        }

        //-------------------------------------
    }
}

Dem "paintPanel" kannst du jetzt die Daten übergeben, die du brauchst, um das zu zeichnen, was du willst...

 

[SebiG]

Hallo Marco,

vielen Dank für die schnelle und umfassende Hilfe.
Ich komm leider jetzt nicht mehr dazu das zu versuchen, aber es sieht schonmal sehr vielversprechend aus !

Vielen Dank von einem Java-Neuling, der bisher gewohnt war IDEs zu haben und sich daher um Formularerstellung keine Gedanken machen musste ;-)

 

[Marco13]

Für Java gibt es auch (jede Menge) IDEs mit GUI-Buildern. Empfehlen würde ich aber "aus Prinzip" keinen davon :wink: