Konstruktor wird nicht als erstes aufgerufen!

[CarloC]

Hallo zusammen,

mich würde interessieren warum bei meiner Subklasse (siehe unten) zuerst "btnLoeschen", anstatt der print im Konstruktor ausgegeben wird. Der print im Konstruktor wird auch erst aufgerufen wenn ich auf einen Button klicke. Müsste der normal nicht direkt beim erzeugen des Objekts aufgerufen werden?

Ich hatte zuerst das Problem, dass ich die Variablen "zeile", "daten" usw. nicht in der actionPerformed Methode zur Verfügung hatte (NullPointException). Ich denke das hängt auch damit zusammen, dass der Konstrutor das "this.zeile = zeile;" etc. erst am Ende macht.

Wo ist mein Denkfehler?

Danke für die Hilfe.

Superklasse: Loeschen

public class Loeschen extends JDialog implements ActionListener
{
	private Container c;
	private JFrame owner;
	private JLabel lblMeldung;
	private JPanel pnlMeldung, pnlButtons;
	protected JButton btnLoeschen, btnAbbrechen;
	private Stplweb stplweb;

	public Loeschen(JFrame owner)
	{
		super(owner, "OrgEinheit löschen", true);

		this.owner = owner;

		this.setSize(350, 120);
		this.setResizable(false);

		c = getContentPane();
		c.setLayout(new GridLayout(3,1));

		lblMeldung = new JLabel("Wollen sie die Organisationseinheit wirklich löschen?");
		lblMeldung.setSize(300,20);
		lblMeldung.setLocation(20,40);

		btnLoeschen = new JButton("OK");
		btnLoeschen.setSize(100,20);
		btnLoeschen.addActionListener(this);

		btnAbbrechen = new JButton("Abbrechen");
		btnAbbrechen.setSize(100,20);
		btnAbbrechen.addActionListener(this);

		pnlButtons = new JPanel();
		pnlButtons.setLocation(70,80);
		pnlButtons.add(btnLoeschen);
		pnlButtons.add(btnAbbrechen);

		c.add(lblMeldung);
		c.add(pnlButtons);

		this.setVisible(true);
	}

	public void actionPerformed(ActionEvent arg0)
	{
		//
	}
}

SubKlasse: GliederungsEinheitLoeschen

public class GliederungsEinheitLoeschen extends Loeschen
{
	private GliederungsplanModel model;
	private int zeile;
	private Vector daten;
	
	public GliederungsEinheitLoeschen(JFrame owner, GliederungsplanModel model, int zeile, Vector daten)
	{
		super(owner);

		this.model = model;
		this.zeile = zeile;
		this.daten = daten;
		
		System.out.println("Konstruktor");
	}

	public void actionPerformed(ActionEvent arg0)
	{
		Object aktion = arg0.getSource();

		if (aktion == btnLoeschen)
		{
			System.out.println("btnLoeschen");

			this.setVisible(false);
			this.dispose();
		}

		if (aktion == btnAbbrechen)
		{
			this.setVisible(false);
			this.dispose();
		}
	}
}

Ausgabe:
btnLoeschen
Konstruktor

 

[CarloC]

Keiner eine Ahnung? :-(

 

[Wildcard]

Hört sich seltsam an. Setz mal einen Breakpoint in die actionPerformed, und schau nach wer die Methode aufgerufen hat.

 

[AlArenal]

Der print im Konstruktor wird auch erst aufgerufen wenn ich auf einen Button klicke. Müsste der normal nicht direkt beim erzeugen des Objekts aufgerufen werden?

Aus deinem geposteten Code ist nicht ersichtlich wo du denn eine Instanz von GliederungsEinheitLoeschen erstellst. Wenn du das erst in Loeschen#actionPerformed machst, werden (so vermute ich mal) der Konstruktor von GliederungsEinheitLoeschen und dessen #actionPerformed unmittelbar hintereinander aufgerufen. Ich kann dir nicht sagen warum, aber manchmal kommts vor, dass bei Ausgabe über System.out Sachen vertauscht werden.

Ich selbst habs aber schon länger nicht mehr beobachtet, da ich meist noch mit Loggern arbeite.

ALlein der Gedanke, der Konstruktor würde NICHT als erstes aufgerufen, ist aber völlig abwegig

 

[SnooP]

ich würde das auch nochmal genauer debuggen - mit einigen Breakpoints an den relevanten stellen, dann kann man den Verlauf evtl. besser verfolgen...

 

[CarloC]

Ich werde das jetzt mal mit dem Debugger unter die Lupe nehmen.

Aber nochmal eine Verständnisfrage: Wenn ich von der Klasse ein Objekt erzeuge, muss dann normalerweise nicht der System.out.println("Konstruktor") ausgeführt bzw. ausgegeben werden?

Das passiert nämlich erst wenn ich auf einen Button klicke und nicht in dem Moment wo ich das Objekt erzeuge.

EDIT: Also wenn ich mal die ganzen anderen Sachen aus der Klasse rausnehme und auch nicht mehr von der Superklasse erbe, dann gibt er "Konstruktor" direkt beim erzeugen aus.

 

[AlArenal]

Ich wiederhole mich, aber:

In obigem Code ist nicht ersichtlich, wo du eine Instanz erzeugst. Was sollen wir also antworten?

 

[CarloC]

Sorry, also die Instanz erzeuge ich in der Klasse, in der sich mein JTable befindet:

	public void mouseClicked(MouseEvent arg0)
	{

		Point point = arg0.getPoint();

		int zeile = table.rowAtPoint(point);
		int spalte = table.columnAtPoint(point);

		if (spalte == 0)
		{
			GliederungsEinheitLoeschen gliederungsEinheitLoeschen = new GliederungsEinheitLoeschen(frame, model, zeile, daten);
		}
	}

Wenn jemand auf die erste Spalte der Tabelle klickt (dort befindet sich ein Lösch-Icon) geht der Dialog auf (GliederungsEinheitLoeschen). Dieser erbt bekanntlich von der Klasse Loeschen.

Ich habe jetzt nochmal was getestet: Wenn ich in meiner Subklasse nicht mehr von der Superklasse erbe und somit auch den super() Aufruf rausnehme, wird "Kontruktor" ausgegeben sobald jemand auf die erste Spalte klickt bzw. das Objekt erzeugt wird. Ich denke es hängt irgendwie mit dem super() Aufruf zusammen. Anscheinend führt er danach ja nicht mehr den Rest des Konstruktors aus. ???:L

 

[Murray]

Und wenn Du solche Ausgaben verwendest, um festzustellen, wann das Programm welchen Code durchläuft, dann ist es ratsam, hinter das System.out.println noch ein System.out.flush zu setzen.

 

[CarloC]

Okay, danke.

Ich habe jetzt gefunden woran es liegt. Im Konstruktor der Superklasse schreibe ich ja nochmal:

super(owner, "OrgEinheit löschen", true);

Anscheinend geht das nicht. Wenn das dort steht führt er nämlich nicht mehr den Konstruktor der Subklasse weiter aus.

Kann mir einer sagen warum man das so nicht machen kann? Ich weiß jetzt nicht wie ich sonst den owner angeben soll...

 

[SlaterB]

wieso denkst du, dass es was mit dem super-Konstruktor zu tun hat?

durch setzten von weiteren System.out.println siehts bei mir jedenfalls sehr deutlich danach aus, dass alles bis zum
'this.setVisible(true); '
ausgeführt wird, alles danach aber nicht mehr,

das Problem ist damit zwar auch weiterhin mysterös, aber relativ einfach zu umgehen,

this.setVisible(true); muss aus dem Konstruktor raus und separat gerufen werden, oder eben am Ende des Konstruktors der Sub-Klasse

this.setVisible(true); sollte sowieso generell nicht im Konstruktor gerufen werden,
vielleicht wird die Anzeige der Subklasse ja erst noch im folgenden Konstruktor der Sub-Klasse festgelegt?

dass es zu so einem Fehler kommt ist mir unverständlich, aber es ist doch offensichtlich generell nicht ratsam in der Oberklasse schon auf visible zu stellen

 

[CarloC]

Du hast recht. Vielen Dank für die Hilfe!