Probleme mit Gridlayout

[koiti]

Hallo Leute,
ich möchte in ein Gridlayout 4 Elemente einfügen, die gleich groß sind.
Leider wird nur das erste Objekt angezeigt

        int xmax=this.Faxprop.getAnzHorizont();
        int ymax=this.Faxprop.getAnzVertikal();
       
        this.setLayout(new GridLayout(ymax, xmax));
        this.circles= new MyCircle[ymax][xmax];
   
        System.out.println(xmax+ " "+ymax);
        for(int i=0;i<xmax;i++){
            for(int j=0;j<ymax;j++){
                
                MyCircle m= new MyCircle();
                m.setX(i);
                m.setY(j);    
                this.circles[j][i]=m;
                this.add(m); 
            }
        }

ymax und xmax würden überprüft und betragen 2
ich habe leider keine ahnung warum nur ein element angezeigt wird.

besten dank für eure hilfe
koiti

 

[njans]

Hmm einfach mal ins Blaue geraten:
Hast du bei Circles equals und hashCode überschrieben, so dass new Circle == new Circle, wahr ergibt?
Oder versuchst du das gerade auf einen Frame oder JFrame direkt zu packen?

 

[koiti]

Danke für deine antwort.
ich glaube, dass ich nichts davon mache.
hier sind beide vollständigen klassen

import Property.FaxProperties;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.GridLayout;
import java.awt.Insets;

/**
 *
 * @author markus
 */
public class InputPanel extends javax.swing.JPanel {

    /** Creates new form InputPanel */
    public InputPanel() {
        initComponents();
       
        this.setBackground(Color.red);
    }
    private MyCircle[][] circles;
    protected FaxProperties Faxprop;

    /**
     * Get the value of Faxprop
     *
     * @return the value of Faxprop
     */
    public FaxProperties getFaxprop() {
        return Faxprop;
    }

    /**
     * Set the value of Faxprop
     *
     * @param Faxprop new value of Faxprop
     */
    public void setFaxprop(FaxProperties Faxprop) {
        this.Faxprop = Faxprop;
        

    }
    public void paintCircle(){
        int xmax=this.Faxprop.getAnzHorizont();
        int ymax=this.Faxprop.getAnzVertikal();
       
        this.setLayout(new GridLayout(ymax, xmax));
        this.circles= new MyCircle[ymax][xmax];
   
        System.out.println(xmax+ " "+ymax);
        for(int i=0;i<xmax;i++){
            for(int j=0;j<ymax;j++){
                
                MyCircle m= new MyCircle();
                m.setX(i);
                m.setY(j);
                m.setSize(10, 10);
                m.setPreferredSize(new Dimension(10, 10));
                m.setMaximumSize(new Dimension(10, 10));
                m.setMinimumSize(new Dimension(10, 10));

                this.circles[j][i]=m;
                this.add(m);
              
               

            }
        }
    
        


    }

    /** This method is called from within the constructor to
     * initialize the form.
     * WARNING: Do NOT modify this code. The content of this method is
     * always regenerated by the Form Editor.
     */
    @SuppressWarnings("unchecked")
    // <editor-fold defaultstate="collapsed" desc="Generated Code">                          
    private void initComponents() {

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(this);
        this.setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGap(0, 400, Short.MAX_VALUE)
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGap(0, 300, Short.MAX_VALUE)
        );
    }// </editor-fold>                        


    // Variables declaration - do not modify                     
    // End of variables declaration                   

}

MyCircle.java

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */

/*
 * MyCircle.java
 *
 * Created on 26.11.2011, 16:48:31
 */

package GUI;

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

/**
 *
 * @author markus
 */
public class MyCircle extends javax.swing.JPanel {

    /** Creates new form MyCircle */
    public MyCircle() {
        initComponents();
        this.setBackground(Color.GREEN);
    }
   @Override
  protected void paintComponent( Graphics g ) {
        super.paintComponent(g);
        g.drawOval(0,0,this.getSize().height, this.getSize().height);
        g.drawString("X"+this.X+ "y"+this.Y, 10, 10);


    }
    protected int X;

    /**
     * Get the value of X
     *
     * @return the value of X
     */
    public int getX() {
        return X;
    }

    /**
     * Set the value of X
     *
     * @param X new value of X
     */
    public void setX(int X) {
        this.X = X;
        this.repaint();
    }
    protected int Y;

    /**
     * Get the value of Y
     *
     * @return the value of Y
     */
    public int getY() {
        return Y;
    }

    /**
     * Set the value of Y
     *
     * @param Y new value of Y
     */
    public void setY(int Y) {
        this.Y = Y;
        this.repaint();
    }


    /** This method is called from within the constructor to
     * initialize the form.
     * WARNING: Do NOT modify this code. The content of this method is
     * always regenerated by the Form Editor.
     */
    @SuppressWarnings("unchecked")
    // <editor-fold defaultstate="collapsed" desc="Generated Code">                          
    private void initComponents() {

        setPreferredSize(new java.awt.Dimension(20, 20));

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(this);
        this.setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGap(0, 400, Short.MAX_VALUE)
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGap(0, 300, Short.MAX_VALUE)
        );
    }// </editor-fold>                        


    // Variables declaration - do not modify                     
    // End of variables declaration                   

}

 

[njans]

paintCircle() wird ja auch nie aufgerufen^^
Nebenbei fügt dein GuiBuilder (Netbeans?) da auch schon ein Grouplayout ein, das würdest du dann überschreiben. Das kann dann lustige Phänomene werfen. Also an dieser Stelle würde ich empfehlen, die Gui von Hand zu schreiben.

 

[bERt0r]

Ausserdem überschreibst du die getX() bzw getY() Methoden von Component. Das verursacht auch manchmal komische Verhaltensweisen.