Exception in Konstruktor - trotzdem wird Objekt erzeugt!

[Achtel]

Hallo zusammen,

ich habe eine Klasse (sagen wir A), in der ich über einen Button gesteuert eine Instanz einer anderen Klasse (sagen wir B) erzeugen kann. Für jene Instanz werden über Dialogboxen einige Werte abgefragt. Was ich erreichen will, ist, dass wenn in einer dieser Dialogboxen auf "Cancel" geklickt wird, der Konstruktor der Klasse B eine Exception wirft, der Konstruktor somit verlassen wird und im catch-Block der Klasse A über System.out.println dem User mitgeteilt wird, dass die Instanz nicht erzeugt wurde.

Theoretisch alles schön...praktisch wird das Objekt aber trotzdem erzeugt, was ich daran sehe, dass ich bestimmte Funktionen, die nur mit einer aktiven B-Instanz funktionieren, trotzdem ausführen kann?!

Hier mal Code:

1. Button Klasse A

private void addClassButtonActionPerformed(ActionEvent evt)
  {
    // create new box - dimension = (0,0) --> setSize in mouseReleased
	try
	{
	  activeInstance = new ClassBox(TickFrame.this,main_panel,doc,status_panel);
	}
	catch(Exception exc)
	{
	  System.out.println("Instance wasn't created.");
	}
	// add instance to main_panel
    main_panel.add(activeInstance);
  }

2. Dialogbox-Beispiel aus Konstruktor B

// dialog for class name
     try
     {
       c_name = (String) JOptionPane.showInputDialog(frame,
                                                   "Enter the class name",
                                                   "Name?",
                                                    JOptionPane.PLAIN_MESSAGE,
                                                    null,
                                                    null,
                                                    "Enter class name here...");
       if(c_name == null) 
       {
    	 Exception exc =  new Exception();
    	 throw exc;
       }
     }
     catch(Exception exc)
     {
       return;
     }

Der Konstruktor ist auch entsprechend mit "throws Exception" versehen. Irgendwelche Ideen, Vorschläge, Änderungswünsche?

Danke!

 

[Icewind]

wie wärs wenn du das hinzufügen der erzeugten instanz nur machst wenn der konstruktor keine exception geworfen hat??

 

[Achtel]

Ja, hab ich schon geändert... Problem war aber der try-catch-block irgendwie. Hab ne Runtime-Exception draus gemacht und jetzt funktionierts...

Bis auf eine Stelle, wo ich einen Button mit ActionListener erstelle, den ActionListener aber in einer anonymen Klasse definiere. Wenn ich hier die Exception auslöse, wird nicht der komplette Konstruktor frühzeitig beendet, sondern nur die innere Methode dieser anonymen Klasse. Irgendwelche Vorschläge, wie ich das nach außen propagiere?

Danke!

 

[Mag1c]

Moin,

also RuntimeExceptions sind dafür da, Fehlerzustände zu signalisieren, bei denen eine weitere Abarbeitung des Programms nicht möglich ist bzw. zu undefinierten Ergebnissen führen kann. Das Problem in deinem Fall war, daß du die Exception, die du erzeugt hast, gleich wieder mit einem try-catch aufgefangen hast und damit den Konstruktor ordnungsgemäß verlassen hast. Wenn du einfach mal das try-catch weglässt, sollte das schon funktionieren.

BTW: ich finde es sehr eigenartig, in einem Konstruktor per Dialog-Boxen nach irgendwelchen Daten zu fragen. Normalerweise würde man sowas vor der Objekt-Erstellung machen und die Werte dann dem Konstruktor übergeben oder per set-Methoden in das Objekt schreiben.

Gruß
Mag1c