Methoden try / catch exception

[julschgi]

hey, ich habe ein Problem mit meiner try - catch()

undzwar möchte ich mein Programm auf Sonderzeichen etc. untersuchen
das klappt auch, allerdings möchte ich danach
die methode neu aufrufen, diese öffnet sich allerdings unendliche male.

was kann ich dagegen tun?

 public void anlass()
    {
        try {

            if (anlass.equalsIgnoreCase("Geb"))
            {
                System.out.println("Alter?");
               
                jahr = scan.nextInt();
                
              anlass = "Alles Gute zum " + jahr + ". Geburtstag!!!!!";
            } else if (anlass.equalsIgnoreCase("Freeday"))
            {
                anlass = "Frohe Feiertage!!";
            } else if (anlass.equalsIgnoreCase("Valentin"))
            {
                anlass = "Herzliche Gruesse und Kuesse xoxoxo";
            } 
            
            else
            {

                anlass();
            }
            }
        catch (Exception ex){
            
            anlass();
            
        }
    }

 

[Gast2]

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

na, wenn die Methode nicht funktioniert rufst du sie einfach nochmal auf, ohne dass sich etwas geändert hat

 

[fisch100]

Wie mein Vorsprechen schon bemerkt:
damit umgehst du den Sinn von try-catch wenn du im catch ebenso deine Methode ausführst

und dir ist klar das du die Methode in einer Schleife aufrufst?
wenn ich deinen Code verkürze schaut das etwa so aus:

public void anlass()
{
try {
anlass();
}
catch(Exception e)
{
anlass();
}
}

also er geht die methode durch und führt sie nochmals aus.
Ich weiß jedoch nicht was der Sinn oder Zweck davon ist, bzw was geplant wäre.

edit: außer eben du triffst einen der zwei Fälle dann hört es auf

ansich nützt man catch entweder um eben Fehler auszulassen - zb wenn deine Eingabe null wäre - dann kann dieses nicht verglichen werden was zu einer Exception führt. Du kannst dir bei solchen Fällen zb etwas über System.out.println() ausgeben lassen, zb: "Fehler in anlass: " + e

und ein kleiner Tipp: ich würde Methoden und Variablen entweder unterschiedliche Namen geben (bei dir anlass ) oder zb Methoden mit Großbuchstaben anfangen.

lg

 

[Pentalon]

Ich würde da eher eine while drübermachen mit einer Abbruchbedingung.
So wie das jetzt ist, produziert das irgendwann einen Stack overflow.
Ok, Du machst eine Tastatureingabe, also wird der Stack overflow nicht so schnell passieren, aber der Weg führt in diese Richtung. Natürlich kannst Du auch bei der Rekursion bleiben, aber dann halt mit einer Abbruchbedingung.

Deine Methode sollte Anlass (mit grossem A heissen). Die Variable vom Typ String heisst anlass.
Wenn Anlass keine Deiner 3 Bedingungen erfüllt geht das ganze über die else in einen Endlosloop, denn an der Variable anlass ändert sich ja nichts mehr.

Pentalon

 

[julschgi]

Ich würde da eher eine while drübermachen mit einer Abbruchbedingung.
So wie das jetzt ist, produziert das irgendwann einen Stack overflow.
Ok, Du machst eine Tastatureingabe, also wird der Stack overflow nicht so schnell passieren, aber der Weg führt in diese Richtung. Natürlich kannst Du auch bei der Rekursion bleiben, aber dann halt mit einer Abbruchbedingung.

Deine Methode sollte Anlass (mit grossem A heissen). Die Variable vom Typ String heisst anlass.
Wenn Anlass keine Deiner 3 Bedingungen erfüllt geht das ganze über die else in einen Endlosloop, denn an der Variable anlass ändert sich ja nichts mehr.

Pentalon

und dafür dann das try - catch weglassen? :applaus:

 

[Gast2]

und dafür dann das try - catch weglassen? :applaus:

ich versteh die Notwendigkeit aber auch nicht.
Ich würde, wenn ich eine Methode erneut aufrufen möchte, nie eine try-catch Anweisung nehmen, du könntest deine Methode dann höchstens abbrechen lassen, ansonsten wird sie ja bei

else anlass();

sowieso erneut aufgerufen (was ebenfalls zu einer Endlos-Schleife führen dürfte)