try-catch

[schihab]

Hi @ all.

hab folgenden code:

public int abfragen(double muenze) throws Exception
  {
     int n=0;
     try
     {
       for(int i=0;i<=7;i++)
       {
         if(muenze==muenz[0][i])
         n=(int)muenz[1][i];
         else throw e;
       }
     }
     catch(Exception e)
     {
       System.out.println("Eingegebene Münze existiert nicht!!");
     }
     return n;
}

public void button2_ActionPerformed(ActionEvent evt) {
     double muenze = Double.parseDouble(textField2.getText());
     label3.setText("Es sind noch "+abfragen(muenze)+" münzen verfügbar");
  }

Es soll eine Münzsorte abgefragt werden. Wenn diese aber nicht vorhanden ist dann soll ein Exception ausgeworfen werden.
Klappt aber nicht.

Was hab ich falsch gemacht

Danke im voraus

Gruß
schihab

 

[maki]

Müsste heissen throw new Exception().

Aber Exceptions so zu verwenden ist nicht üblich/empfehlenswert.

 

[schihab]

Hi danke erstmal für deine antwort.

Wie meinst du das jetzt genau? hat leider nicht geklappt. Wie wärs denn am besten deiner meinung nach?

Danke nochmals

 

[maki]

Gib doch einfach 0 zurück wenn keine Münze gefunden wurde.

 

[Schmittp]

label3.setText("Es sind noch "+abfragen(muenze)+" münzen verfügbar");

erwartet auch noch einen catch+try da die Funktion abfragen(double muenze) mit throws Exception bestückt ist.

Ich vermute mal dass du in der forschleife überprüfen willst ob der Wert einem Wert in dem Array entspricht.

public int abfragen(double muenze) throws Exception 
{
   for (int i = 0; i <= 7; i++) 
   {
      if (muenze == muenz[0][i])
      {
      // falls der Wert im Array gefunden wurde => return
         return (int) muenz[1][i];
      }
   }
   // Falls das ganze Array den Wert nicht enthält => Exception
   throw new Exception("Eingegebene Münze existiert nicht!!");
}

 

[schihab]

HI

Vielen dank für die hilfreiche Antwort.
Es wird aber gerade folgender Fehler ausgegeben:

Muenzautomat.java:180:45: unreported exception java.lang.Exception; must be caught or declared to be thrown
label3.setText("Es sind noch "+abfragen(muenze)+" münzen verfügbar");

Woran liegt das?

Danke nochmal

 

[Ark]

Das wurde gerade einen Beitrag über deinem erklärt.

Ark

 

[Schmittp]

Wie oben schon geschrieben ist

public int abfragen(double muenze) throws Exception

throws Exception
der Grund.
Der Funktionskopf teilt dir mit dass sie eine Ausnahme werfen könnte. D.h. dass du diese Ausnahme beim Aufrufen dieserFunktion auch abfangen musst.

try
{
   double muenze = Double.parseDouble(textField2.getText());
   label3.setText("Es sind noch "+abfragen(muenze)+" münzen verfügbar");
}
catch(Exception e)
{
    e.printStackTrace();
}

Solltest dir aber nochmal genau überlegen was du mit dem Code wirklich erreichen willst und evtl. im Internet was über Exceptions durchlesen. Höchst wahrscheinlich ist es für dich wirklich einfacher einfach nur eine 0 zurückzugeben falls die Münze nicht gefunden wird und das ganze try und catch und exceptions zeugs zu entfernen.

 

[schihab]

UPS...Habs übersehen.

Danke

klappt jetzt perfekt.

VIELEN DANK NOCHMALS

 

[Marco13]

Um's nochmal zu betonen: "throws Exception" oder "catch (Exception e)" sind in den allermeisten Fällen Unfug (letzteres höchstes auf oberster Stufe, um bei einem fertigen Programm "zu verhinden, dass der Benutzer einen Stacktrace zu sehen bekommt"...). In diesem Fall wäre ein
catch/throws IllegalArgumentException // If the specified coin does not exist
vermutlich besser - wobei ich glaube, dass auch DAS nicht notwendig sein sollte....

 

[SchonWiederFred]

Wie genau ist dieses muenz-Array aufgebaut? Wahrscheinlich willst Du in echt eine Map.