DriveByZero Exception - BMI-Rechner

[haui95]

Guten Tag,

ich habe mir in Java ein Programm zur Berechnung des BMI geschrieben, und zwar mit GUI, Splash Screen und einen eigenen Info Dialog. Jedoch habe ich nun ein Problem. Ich möchte eine ArithmeticException abfangen, falls in der Methode berechneBMI() die Größe durch 0 geteilt wird.
Ich rufe die Methode über einen ActionListener meines 3. Buttons auf, wobei die Methode berechneBMI() im ActionListener mit einem try-catch-Block versehen ist, um die anderen Exceptions, die bei Fehleingaben entstehen können, abzufangen.

Hier einmal der Code:

button3.addActionListener(new ActionListener()
        {
            @Override
            public void actionPerformed(ActionEvent e) 
            {
                progressbar1.setValue(0);
                listbox1.removeAll();
                DecimalFormat format1 = new DecimalFormat();
                double bmi = 0;
               
                try
                {
                    bmi = berechneBMI(Float.parseFloat(edit1.getText()), Float.parseFloat(edit2.getText()));   //   <--  DriveByZeroException wird hier so nicht abgefangen.
                    format1 = new DecimalFormat("0.00");
                    edit3.setText(String.valueOf(format1.format(bmi)));
                }
                catch(Exception ex)
                {
                    JOptionPane.showMessageDialog(null, ex.getMessage(), "Error", JOptionPane.ERROR_MESSAGE);
                    edit1.setText("");
                    edit2.setText("");
                    return;
                }
                
                listbox1.add(vergleicheBMI(bmi));
                progressbar1.setValue(1);

public double berechneBMI(double groesse, double gewicht)
    {
        double result;
        result = 0;
        
        result = gewicht / power(groesse, 2);     //  <--  Hier soll eine DivideByZeroException abgefangen werden.
        
        return result;
    }

 

[njans]

Und die eigentliche Frage?

BTW: Warum willst du die Abfangen? Du musst doch nur prüfen, ob der Divisor 0 ist.
Simple if Abfrage. Ich würde da vermutlich, je nachdem, ob die Eingabe durch den Nutzer gemacht wird, eventuell eine IllegalArgumentException werfen

 

[haui95]

Ja, die eigentliche Frage ist nun, wie ich eine ArithmeticException entweder in der Funktion oder in dem Buttonhandler, also dem ActionListener, abfangen kann. Ich möchte also, dass mein Programm mir sowohl eine Exception bei Fehleingaben abfängt, als auch eine DivideByZero Exception bei der Division durch 0. Ich habe ja das Problem, dass das Programm mir im try-catch-Block des ActionListeners zwar Fehleingaben abfängt, jedoch bei der Division durch 0 keine Meldung anzeigt.

Vielen Dank im Voraus !

 

[Fant]

Du kannst auch mehrere catch-Blöcke für ein try definieren, falls dir das hilft. So kannst du auf unterschiedliche Exceptions unterschiedlich reagieren. ... aber dein eigentliches Problem ist mir immer noch nicht wirklich klar.

 

[haui95]

So habe es nun mit Hilfe von njans gelöst. Ich werfe einfach in der Funktion berechneBMI() eine Exception, wenn die Größe gleich null ist. Diese fange ich wieder ab und sage dem ActionListener, dass eine Exception ausgelöst wurde. Und ab dem Zeitpunkt werden die weiteren Methoden und Instruktionen nicht mehr ausgeführt.

Hier der Code:

    public double berechneBMI(double groesse, double gewicht)
    {
        double result;
        result = 0;
        
        if (groesse <= 0)
        {
            throw new IllegalArgumentException();
        }
        
        try
        {
            result = gewicht / power(groesse, 2);
        }
        catch(IllegalArgumentException ex)
        {
            JOptionPane.showMessageDialog(null, ex.getMessage(), "Error", JOptionPane.ERROR_MESSAGE);
        }
        
        return result;
    }

        button3.addActionListener(new ActionListener()
        {
            @Override
            public void actionPerformed(ActionEvent e) 
            {
                progressbar1.setValue(0);
                listbox1.removeAll();
                DecimalFormat format1 = new DecimalFormat();
                double bmi = 0;
               
                try
                {
                    bmi = berechneBMI(Float.parseFloat(edit1.getText()), Float.parseFloat(edit2.getText()));
                    format1 = new DecimalFormat("0.00");
                    edit3.setText(String.valueOf(format1.format(bmi)));
                }
                catch(Exception ex)
                {
                    JOptionPane.showMessageDialog(null, ex.getMessage(), "Error", JOptionPane.ERROR_MESSAGE);
                    edit1.setText("");
                    edit2.setText("");
                    edit3.setText("");
                    edit3.setBackground(null);
                    return;
                }
                
                listbox1.add(vergleicheBMI(bmi));
                progressbar1.setValue(1);
            }
        });

 

[Fant]

Was soll der try-catch-Block in der berechneBMI-Methode. Da wird doch niemals eine entsprechende Exception in dem try-Bock geworfen... und selbst wenn, dann fängst du die doch eh später zwingend ab.

Wenn deine Methode eine IllegalArgumentException wirft, dann sollte sie auch

public double berechneBMI(double groesse, double gewicht) throws IllegalArgumentException { ...

heißen.

 

[haui95]

Ja, es funktioniert zwar, aber wie könnte ich es denn dann besser lösen, also z.B. kürzer schreiben oder optimieren ? ;-)

 

[Fant]

Ja, es funktioniert zwar, aber wie könnte ich es denn dann besser lösen, also z.B. kürzer schreiben oder optimieren ? ;-)

Einfach nur:

public double berechneBMI(double groesse, double gewicht) throws IllegalArgumentException
    {        
        if (groesse <= 0)
            throw new IllegalArgumentException();
        return gewicht / power(groesse, 2);
    }

Gruß Fant

Edit: Jetzt aber ^.^

 

[haui95]

Danke Fant

Aber den try-Block im ActionListener kann ich so lassen !?
Was passiert denn jetzt in der berechneBMI()-Methode ? Wirft die Methode eine Exception und behandelt diese im try-catch-block des ActionListeners ?

MfG

Hauke

 

[Fant]

Wenn groesse kleiner oder gleich Null ist, dann wird eine Exception geworfen. Die Methode wird dann nicht mehr weiter ausgeführt.
Da im try-Block im ActionListener nun eine Exception ankommt wird auch dieser nicht mehr ausgeführt, sondern dein Programm springt in den catch-Block, wo du auf die Exception entsprechend deinen Vorstellungen reagieren kannst.

 

[xehpuk]

Ich möchte eine ArithmeticException abfangen, falls in der Methode berechneBMI() die Größe durch 0 geteilt wird.

Wo keine ArithmeticException ist, kann man auch keine abfangen. Bei Gleitkommazahlen gibts nie eine:

public class FloatDivideByZero {
	public static void main(String... args) {
		System.out.println(1 / 0f);
		System.out.println(0 / 0f);
		System.out.println(-1 / 0f);
	}
}

Wenn deine Methode eine IllegalArgumentException wirft, dann sollte sie auch

public double berechneBMI(double groesse, double gewicht) throws IllegalArgumentException { ...

heißen.

Nein. RuntimeExceptions sollte man eher nicht in die throws clause setzen.

 

[ssoul26]

Bau doch dein Try-Catch einfach flexibler. Da kannst alle Exceptions einzeln bearbeiten.

try{
.....
.....
}catch (Exception ex){

      if(ex instanceof ArithmeticException){
         //Hier abarbeiten was bei dieser Exception passieren soll
      }else{
      System.out.println("Exception : "+ ex.getMessage());
      }

}

 

[njans]

Du kannst auch mehrere catch-Blöcke für ein try definieren, falls dir das hilft. So kannst du auf unterschiedliche Exceptions unterschiedlich reagieren. ... aber dein eigentliches Problem ist mir immer noch nicht wirklich klar.

So sollte man das dann eher machen:

        try
		{
			// Irgendwas
		}
		catch (IOException exception) // Exception nach Genauigkeit ordnen!
		{
			// handle exception
		}
		catch (Exception exception) // Als letztes kommt dann der allgemeinste tTyp.
                { 
			// handle exception
		}