layout in einer anderen klasse

[gammeltier]

hallo
ich habe mein layout da es recht komplex war in eine andere klasse verschoben:

main klasse:

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

public class mono extends JFrame{
	
	JPanel p1    = new JPanel();
	JLabel[] label = new JLabel[41];
	JTextField tf  = new JTextField();
	Container cp = getContentPane();
	layout lay;
	
	public mono(){
	  setSize(1000,1000);
                  lay.lay(this);
	}
	
	public static void main(String args[])	{
		mono f = new mono();
		f.setDefaultCloseOperation(EXIT_ON_CLOSE);
		f.show();
	}
}

und meine layout.java:

import java.awt.*;
import javax.swing.*;
import java.io.*;
import javax.imageio.*;

public class layout{

  public void lay(mono m){
	
	m.cp.setLayout( new FlowLayout() );
	m.p1.setLayout( new GridBagLayout() );
	GridBagConstraints c = new GridBagConstraints();
	c.fill = GridBagConstraints.HORIZONTAL;
	  
	Image image = null; 
	File file;
	for( int i=0; i<11; i++){
	  try{  
		file = new File( "images/"+(i+1)+".jpg" ); 
		image = ImageIO.read( file ); 
	  }catch( IOException e ){} 
	  m.label[i] = new JLabel( new ImageIcon(image) );
	    
	  c.ipadx   = 2;
	  c.ipady   = 2;
	  c.gridx   = i;
	  c.gridy   = 0;
	    
	  m.p1.add( m.label[i], c ); 
	}
	
                [...]

                m.cp.add( m.p1 );
  }
}

compilen is kein problem aber dann erhalte ich beim starten einen NullPointerException!!! wenn ich die layout.java mit extends mono mache und das mono m weglasse passiert genau das gleiche! was kann ich tun??

 

[Beni]

Du versuchst im Konstruktor von mono die Methode "lay" aufzurufen. Du habst aber die Variable "lay" niemals initialisiert, deshalb ist lay null, deshalb gibt es eine Exception.

Es gibt zwei Möglichkeiten:
Du initialisierst zuerst ein neues layout: "lay = new lay();", oder du machst die Methode "lay" statisch:

public class layout{
  public static void lay(mono m){

Und verwendest gar keine Variable mehr.

   public mono(){
     setSize(1000,1000);
        layout.lay(this);
   }

Wie auch immer, wenn es nur um das reine Code-Auslagern geht, würde ich die Klasse Mono nicht so auseinanderrupfen, was stören schon ein paar Zeilen Code mehr.

P.S. Klassennamen schreibt man normalerweise grosse "Layout" und "Mono".

 

[SebiB90]

du musst ja auch eine instance von layout erstellen
ersetzt das:

layout lay;

durch das:

layout lay = new layout();

 

[gammeltier]

sowohl bei dem static als aich dem erzeugen einer instance kommt immer noch nen nullpointer!!

 

[gammeltier]

ok hab den fehler gefunden. hab ein element hinzufügen wollen das noch nicht definiert war
also danke für die hilfe