Initialisieren von Fenstern über Methoden

[Fabulus]

Hallo Freunde!

Ich habe eine Klasse "Window", deren Instanzen, wie der Name sagt, Fenster sind. Da ich aber nicht für jedes Panel und Label den Kommando in den Konstruktor schreiben möchte, habe ich gedacht, ich schreibe eine Methode "create_entry", die die Panels und Labels uebergeben bekommt.
Um leichter auf diese zuzugreifen, speicher ich sie in einem Array. Mein Problem ist jetzt, dass ich die Methode create_entry im Konstruktor aufrufe, dort werden die Attribute initialisiert und im Konstruktor das soeben erstellte Panel im main-Frame packen will. Hierbei kommt aber eine NullPointerException. Ich habe die Vermutung, dass das daran liegt, dass die Panels und labels nur Methodenintern initialisiert werden, sprich, außerhalb der Methode nicht genutzt werden können. Wie kann ich das verhindern?

Hier mein Code:

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

import javax.swing.*;

public class Window extends JFrame implements ActionListener {


	private static final long serialVersionUID = 1L;
	protected JFrame main;
	protected String title;
	protected JLabel[] label;
	protected JLabel[] dots;
	protected JTextField[] text;
	protected JPanel[] panel;

	
	
	public Window(String t, int labels, int panels, int dot, int nr_text) {
		title=t;
		
		/*
		 * Initialize Window-Properties
		 */
		main=new JFrame(t);
		this.label=new JLabel[labels];
		this.dots=new JLabel[dot];
		this.text=new JTextField[nr_text];
		this.panel=new JPanel[panels];
		
		/*
		 * Initialize Entries
		 */
		create_entry(label[0], "Beginn: ", text[0], dots[0], text[1], panel[0]); // begin entry
		
		main.add(panel[0]);
		main.pack();
		main.setVisible(true);
	}
	
	/* 
	 * Diese Funktionen generieren einen Zeileneintrag. 
	 * Beispiel:
	 * Beginn: XXX : XXX 
	 */
	public void create_entry(JLabel name, String entry, JTextField hours, JLabel dots, JTextField mins, JPanel p) {
	
		name=new JLabel(entry);
		hours=new JTextField(4);
		dots=new JLabel(" : ");
		mins=new JTextField(4);
		p=new JPanel();
		p.add(name);
		p.add(hours);
		p.add(dots);
		p.add(mins);
	}
	
	
	@Override
	public void actionPerformed(ActionEvent arg0) {
		// TODO Auto-generated method stub
	}
	
	public static void main(String[] args) {
		String title="Testfenster 1 ";
		Window main=new Window(title, 20, 20, 5, 14);
	}
}

Ich hoffe doch und bin mir sicher, dass ihr mir weiterhelfen könnt

Viele Grüße,
Jan

 

[ymene]

Ich habe die Vermutung, dass das daran liegt, dass die Panels und labels nur Methodenintern initialisiert werden, sprich, außerhalb der Methode nicht genutzt werden können. Wie kann ich das verhindern?

Deine Vermutung ist nicht richtig. Du kannst sowohl Panels als auch Labels an beliebiger Stelle initialisieren lassen. Arbeitest du mit einer IDE wie Eclipse? Da würde dir der Debugmodus vermutlich weiter helfen. Das Problem ist gerade, dass du deiner create_entry Methode mehrere null-parameter übergibst, da die Arrays nicht gefüllt sind. Dort initialisiert du zwar die Objekte, aber du packst sie zu keiner Stelle in die Arrays rein. Du definierst nur ihre Größe. Es fehlt etwas ala:

panel[ 0 ] = p;

in deiner create_entry Methode, nachdem du p initialisiert hast. Damit sollte es dann (vorläufig) funktionieren.

Allerdings bringt deine Konstruktion noch einige andere Tücken mit sich, aber die wirst du sicher schnell selber bemerken .

 

[Fabulus]

Ach, die kleinsten Fehler übersieht man am schnellsten! Dass ich da nich selbst drauf gekommen bin, Mensch.
Danke dir Sofern es weitere Probleme gibt, wie du sagst (das ist momentan nur mein Grundgerüst, um zu testen, ob es überhaupt klappt), werd ich es hier schreiben.

 

[Fabulus]

Soweit so gut, ich entschuldige mich für den doppelpost, falls diese nicht erlaubt sind, aber ansonsten sieht man ja nicht die Aktualisierung, oder?

Ich bin jetzt soweit, dass das Panel mit den Labels und Entries erstellt wird, möchte diese jetzt nun mit einem GridBagLayout packen. Ist es sinnvoll, die Panels, die ich erstellt habe, mit dem GBL zu packen oder ist es sinnvoller, die einzelnen Komponenten des Panels in das GBL zu packen?

Für das Packen mit dem GBL habe ich mir eine Methode addComponents geschrieben. In der Main-Methode werden die Panels erstellt und in das GBL hinzugefügt. Leider werden sie aber nicht beim Programmstart angezeigt. Mein Code sieht folgendermaßen aus:

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

import javax.swing.*;

public class Window extends JFrame implements ActionListener {


    private static final long serialVersionUID = 1L;
    protected JFrame main;
    protected String title;
    
    protected JLabel[] label;
    protected JLabel[] dots;

    protected GridBagLayout gbl;
    
    protected JTextField[] text;

    protected JPanel[] panel;

    
    
    public Window(String t, int labels, int panels, int dot, int nr_text, boolean createMenu, int sizeX, int sizeY) {
        
        /*
         * Initialize Window-Properties
         */
        title=t;
        main=new JFrame(t);
        main.setDefaultCloseOperation(EXIT_ON_CLOSE);
        main.setSize(sizeX,sizeY);
        

        gbl=new GridBagLayout();
        main.getContentPane().setLayout(gbl);

        
        this.label=new JLabel[labels];
        this.dots=new JLabel[dot];
        this.text=new JTextField[nr_text];
        this.panel=new JPanel[panels];
        
    }
    

    /* 
     * Diese Funktionen generieren einen Zeileneintrag. 
     * Beispiel:
     * Beginn: XXX : XXX 
     */
    public void createEntry(int index_label, String entry, int index_text,int index_text2, int index_dots, 
             int index_panel) {

        label[index_label]=new JLabel(entry);
        text[index_text]=new JTextField(2);
        dots[index_dots]=new JLabel(" : ");
        text[index_text2]=new JTextField(2);
        panel[index_panel]=new JPanel();
        panel[index_panel].add(label[index_label]);
        panel[index_panel].add(text[index_text]);
        panel[index_panel].add(dots[index_dots]);
        panel[index_panel].add(text[index_text2]);
    }
    
    
    public void createMenubar() {
        
        // noch zu erstellen
    }
    
    public void addComponent(    Container cont,
                                GridBagLayout gbl,
                                Component c,
                                int x, int y,
                                int width, int height,
                                double weightx, double weighty) {
        
        GridBagConstraints gbc = new GridBagConstraints(); 
        gbc.fill = GridBagConstraints.BOTH; 
        gbc.gridx = x; gbc.gridy = y; 
        gbc.gridwidth = width; gbc.gridheight = height; 
        gbc.weightx = weightx; gbc.weighty = weighty; 
        gbl.setConstraints( c, gbc ); 
        cont.add( c ); 
        
    }
    
    @Override
    public void actionPerformed(ActionEvent arg0) {
        // TODO Auto-generated method stub
        
    }

}

und die Main Klasse sieht so aus:

public class Main {

    public static void main(String[] args) {
        String title=""Testprogramm;
        Window main=new Window(title, 20, 20, 20, 14, true, 600,600);
        /*
         * Initialize main-Entries
         * createEntry(index_label, entry, index_text, index_text2, index_dots, index_panel)
         */
        main.createEntry(0, "Label 0", 0, 1, 0, 0); 
        main.createEntry(1, "Label1", 2, 3, 1, 1);
        main.createEntry2(2, 4, "Label 2", 4, 2, " h"); 
        main.createEntry2(3, 5, "Label 3", 5,3," h");
        
        main.createEntry(6, "Label 4", 6, 7, 2, 4);
        main.createEntry(7, "Label 5", 8, 9, 3, 5);
        main.createEntry(8, "Label 6", 10, 11, 4, 6);
        main.createEntry(9, "Label 7", 12, 13, 5, 7);
        
        main.addComponent(main.getContentPane(), main.gbl, main.panel[0], 0, 0, 1, 1, 0.0, 0.0);
        main.addComponent(main.getContentPane(), main.gbl, main.panel[1], 0, 1, 1, 1, 0.0, 0.0);
        main.addComponent(main.getContentPane(), main.gbl, main.panel[2], 1, 0, 1, 1, 0.0, 0.0);
        main.addComponent(main.getContentPane(), main.gbl, main.panel[3], 1, 1, 1, 1, 0.0, 0.0);
        
        
        main.setVisible(true);
    }
}

Wenn ich das Programm starte, ist die Fenstergröße minimal (leider nicht 600x600), und es ist nur das Panel zu sehen, welches "Label 7" und das dazugehörige Textfeld beinhaltet.

Was mach ich denn falsch?
Die createEntry-Methode sieht so ähnlich aus wie die erste, deshalb habe ich sie weggelassen
Achja, die AddComponents-Methode habe ich von "Java ist auch eine Insel" von Galileo Computings.




EDIT: Habe meinen Fehler gefunden. Ich sollte natürlich statt
[Java]main.setVisible(true);[/code]

main.main.setvisible(true);

schreiben. Da war wohl die Namensgebung etwas unglücklich.