Per Button Kreis zeichnen

[Nepos]

Hallo!

Mit den ganzen Swing/AWT sachen bin ich blutiger Anfänger. Bisher habe ich nur Aus und Eingaben per Konsole erledigt.

Ich möchte in ein jPanel ein Kreis zeichnen per klick auf eine Schaltfläche (Ich habe das ganze im Designmodus gestaltet)

Der Quellcode der Klasse:

import java.awt.Graphics;
import javax.swing.JPanel;

public class DrawPanel extends JPanel
{
 
  @Override
  protected void paintComponent( Graphics g )
  {
    super.paintComponent( g );
    g.drawOval(30, 30, 20, 20);
  }
}

Der Aufruf:

private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) {
        // TODO add your handling code here:

       jPanel1.add(new DrawPanel());
       repaint();
       
       
    }

Kann mir jemand sagen was ich falsch mache oder was ich noch nicht begriffen habe??

Liebe Grüße

Nepos

 

[Kevin94]

Du musst DrawPanel eine mindest-Größe zuweisen, sonst hat es die Größe (0,0).

 

[SlaterB]

bei GUI-Änderungen ist das Layout wichtig, welches hat jPanel1 überhaupt? ein validate()-Befehl könnte noch helfen

zudem musst du dem JPanel mit setPreferredSize() wahrscheinlich eine Größe geben, Java schaut sich nicht die Zeichenbefehle an um selbst eine Größe auszusuchen,
nur bei bestimmten Layouts wie BorderLayout wird gleich maximiert

hilfreich ist in gewissen Grenzen, allen möglichen Komponenten unterschiedlich farbige Border/ Rahmen zu geben,
eine schlechte Size würdest du damit erkennen, vergessenes validate() aber kaum

 

[Nepos]

Danke!

Ich hatte vergessen das Bild anzuhängen. Größe und Farbe habe ich in den Properties festgelegt.
Ich habe aber auch noch die Größe manuell festgelegt:

jPanel1.add(new DrawPanel());
       jPanel1.setSize(100, 200);
       repaint();

Per Knopfdruck schrumpft nun die Größe.
Kommt ein Validate dahinter, schrumpft nichts mehr dh das setSize(100,200) wird nicht ausgeführt?!
Kommt nur ein Validate statt repaint() passiert ebenfalls nichts.

Die Farbe des Kreises habe ich auf Rot gesetzt - ich sehe aber immer noch nüscht.

Liebe Grüße

Nepos

 

[SlaterB]

offensichtlich ist erstmal von der Size des DrawPanel die Rede, doch nicht von jPanel1!,
jenes sollte natürlich vorher auch schon gut sichtbar sein, sonst könnte es dort haken,

alles muss vom JFrame an gut zusammenspielen, vollständiges Testprogramm wäre hilfreich

 

[Nepos]

Achso. Wie gebe ich dem denn eine Größe? über super.setSize() ? Das hatte auch keinen Erfolg.

(...)vollständiges Testprogramm wäre hilfreich

Soll ich mein Testprogramm hochladen? Oder was meinst Du damit?

 

[SlaterB]

du hast doch schon einen Aufruf, nur beim falschen Panel,
wie du den an das DrawPanel dranbekommst oder besser gleich dort in den Konstruktor setzt,
wird doch wohl eine machbare Aufgabe sein

übrigens habe ich von setPreferredSize() gesprochen, setSize() bringt nichts, wird von den Layouts wieder überschrieben,
die dafür die PreferredSize berücksichtigen, kompliziert,
bei null-Layout, also kein Layout wäre es wiederum anderesrum setPreferredSize() bringt nichts, weil niemand da ist der sich das anschaut, und setSize() wäre selber nötig

> Soll ich mein Testprogramm hochladen? Oder was meinst Du damit?

ich sprach offensichlich mehr vom Posten, so macht man das in Foren, bisschen Code hast du ja auch schon gepostet,
Hochladen wäre die Alternative, klingt dann aber ja nach bedenklich viel Code.., möglichst kurz wäre besser,

hoffentlich gar nicht nötig

 

[njans]

Also das wäre das Minimalbeispiel.

public class DrawPanel extends JPanel
{
	/* (non-Javadoc)
	 * @see javax.swing.JComponent#paintComponent(java.awt.Graphics)
	 */
	@Override
	protected void paintComponent(Graphics g)
	{
		super.paintComponent(g);
		g.setColor(Color.RED);
		g.drawOval(0, 0, this.getWidth()-1, this.getHeight()-1);
	}
	
	public static void main(String[] args)
	{
		JFrame frame = new JFrame("Test");
		JPanel backgroundPanel = new JPanel(new BorderLayout());
		backgroundPanel.add(new DrawPanel(),BorderLayout.CENTER);
		frame.setContentPane(backgroundPanel);
		
		frame.setSize(500, 500);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setLocationRelativeTo(null);
		frame.setVisible(true);
	}
}

 

[Nepos]

Naja, von mir stammen nur die Zeilen 71-73. Den Rest hat NetBeans gemacht.

package Main;

import dijkstra.DrawPanel;

public class GraphForm extends javax.swing.JFrame {


    public GraphForm() {
        initComponents();
    }


    @SuppressWarnings("unchecked")

    private void initComponents() {

        jPanel1 = new javax.swing.JPanel();
        jButton1 = new javax.swing.JButton();

        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);

        jPanel1.setBackground(new java.awt.Color(255, 255, 255));

        javax.swing.GroupLayout jPanel1Layout = new javax.swing.GroupLayout(jPanel1);
        jPanel1.setLayout(jPanel1Layout);
        jPanel1Layout.setHorizontalGroup(
            jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGap(0, 0, Short.MAX_VALUE)
        );
        jPanel1Layout.setVerticalGroup(
            jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGap(0, 229, Short.MAX_VALUE)
        );

        jButton1.setText("jButton1");
        jButton1.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                jButton1ActionPerformed(evt);
            }
        });

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(getContentPane());
        getContentPane().setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addContainerGap()
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                    .addComponent(jPanel1, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                    .addGroup(javax.swing.GroupLayout.Alignment.TRAILING, layout.createSequentialGroup()
                        .addGap(0, 307, Short.MAX_VALUE)
                        .addComponent(jButton1)))
                .addContainerGap())
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addContainerGap()
                .addComponent(jPanel1, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED, 26, Short.MAX_VALUE)
                .addComponent(jButton1)
                .addContainerGap())
        );

        pack();
    }

    private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) {
        
       jPanel1.add(new DrawPanel());
       jPanel1.setSize(100, 200);
       repaint();

       
    }

    public static void main(String args[]) {

        try {
            for (javax.swing.UIManager.LookAndFeelInfo info : javax.swing.UIManager.getInstalledLookAndFeels()) {
                if ("Nimbus".equals(info.getName())) {
                    javax.swing.UIManager.setLookAndFeel(info.getClassName());
                    break;
                }
            }
        } catch (ClassNotFoundException ex) {
            java.util.logging.Logger.getLogger(GraphForm.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (InstantiationException ex) {
            java.util.logging.Logger.getLogger(GraphForm.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (IllegalAccessException ex) {
            java.util.logging.Logger.getLogger(GraphForm.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        } catch (javax.swing.UnsupportedLookAndFeelException ex) {
            java.util.logging.Logger.getLogger(GraphForm.class.getName()).log(java.util.logging.Level.SEVERE, null, ex);
        }
        //</editor-fold>

        java.awt.EventQueue.invokeLater(new Runnable() {
            @Override
            public void run() {
                new GraphForm().setVisible(true);
                
            }
        });
        

        
    }
    // Variables declaration - do not modify
    private javax.swing.JButton jButton1;
    private javax.swing.JPanel jPanel1;
    // End of variables declaration
}

Hier die Klasse:

package dijkstra;

import java.awt.Color;
import java.awt.Dimension;

import java.awt.Graphics;
import javax.swing.JPanel;

public class DrawPanel extends JPanel
{

  @Override
  protected void paintComponent( Graphics g )
  {
    super.paintComponent( g );
    super.setPreferredSize(new Dimension(300,300));
    g.setColor(Color.red);
    
    g.drawOval(30, 30, 20, 20);

  }
}

Wie mir gerade auffällt hat DrawPenel garkeinen Konstruktor oder irre ich ?

EDIT:

Das Beispiel funktioniert, aber damit öffnet sich ein neues Fenster beim klicken auf den Button - ich muss mal schauen ob ich das umschreiben kann das es auf das aktuelle Frame geht...