Frage zu Exceptions

[dezzard]

Hi Leute,

ich schreibe ein Programm, welches Personen in einer ArrayList speichert.
Diese Personen haben verschiedene Eigenschaften (Name, Alter etc.)
Dann habe ich eine Methode, die schaut, ob ein Vorname in der Liste vorhanden ist.
Zb. "Klaus". Wenn der Name existiert, wird ein counter um 1 erhöht und am ende gibt er mir die Zahl aus, wie häufig der Name in der ArrayList ist.
Nun wollte ich mit try und catch die NullPointerException abfangen, falls ein Vorname halt eben auf null gesetzt ist.

public int countOfPersonsWithFirstName(String firstName2BeCounted)
    {

        int counter = 0;
       
        try
        {
            for (Person currentPerson : personen)
            {
                if (currentPerson.getVorname().equals(firstName2BeCounted))
                {
                    counter++;

                }
            }
            return counter;
        }catch(NullPointerException ex)
        {
            ex.printStackTrace();
        }
       

    }

Hier schimpft er mich natürlich, dass die Methode einen int-Wert zurückgeben muss und so funktioniert ex.printStackTrace(); natürlich nicht.
Was kann ich denn da machen?

Danke und liebe Grüße

 

[Thallius]

DAs macht man auch nicht. Wenn Du auf null testen willst, dann mach das auch und vergewaltige dafür nicht die armen Exceptions.

 

[mkdrive2]

1) Die try-catch-Anweisung sollte in der for-Schleife sein, da das Programm sonst mit der Vornamen-Zählung aufhört, wenn ein Vorname "null" ist.

2) Die return-Anweisung sollte außerhalb der try-catch-Anweisung sein, damit die Methode in jedem Fall etwas zurückgibt.

(3. Statt der Exceptions kannst du in der if-Abfrage natürlich auch einfach ein "currentPerson.getVorname() != null" hinzufügen.)

public int countOfPersonsWithFirstName(String firstName2BeCounted)
{
    int counter = 0;

    for (Person currentPerson : personen) {
        try {
            if (currentPerson.getVorname().equals(firstName2BeCounted)) {
                counter++;
            }
        } catch(NullPointerException ex) {
            ex.printStackTrace();
        }
    }
 
    return counter;
}

 

[mihe7]

@mkdrive2 Exceptions sind Ausnahmen. Wie der Name schon sagt, treten diese in Ausnahmefällen auf, d. h. wenn etwas Unvorhergesehenes eintritt. Hier ist es der Normalfall, dass ein Vorname null sein kann. Das ist ganz normal abzuprüfen und nicht über die Ausnahmebehandlung zu lösen.

public int countOfPersonsWithFirstName(String firstName2BeCounted) {
    int counter = 0;
    for (Person currentPerson : personen) {
        String firstName = currentPerson.getVoname();
        if (firstName != null && firstName.equals(firstName2BeCounted)) {
            counter++;
        }
    }
    return counter;
}

 

[mkdrive2]

Ich finde aber, dass solche Programme perfekt sind, um Exceptions zu üben und zu sehen, worauf man bei Exceptions achten muss. Das habe ich früher auch immer gemacht.

 

[dezzard]

Zunächst vielen Dank an alle

Ich finde aber, dass solche Programme perfekt sind, um Exceptions zu üben und zu sehen, worauf man bei Exceptions achten muss. Das habe ich früher auch immer gemacht.

Genau darum soll es hier auch gehen. Habe die Exceptions erst letzte Woche kennen gelernt und spiele jetzt etwas damit rum. Das eine einfach if-Anweisung hier genügt, ist mir bewusst. Allerdings soll es hier um Verständnis und Handling gehen. Einfach Exceptions benutzen, Syntax lernen etc.

 

[Thallius]

Dann nimm doch einfach einen Text String und versuche den in einen integer zu parsen. Da kriegste dann eine schöne Exception.

 

[dezzard]

Dann nimm doch einfach einen Text String und versuche den in einen integer zu parsen. Da kriegste dann eine schöne Exception.

Das habe ich zuerst gemacht. Dann habe ich gesehen, dass ich ja "throw new" exception machen kann. Damit habe ich die Exceptions, wie du anfangs erwähnt, wahrscheinlich völlig vergewaltigt aber zum Verständnis war das echt gut.

 

[mihe7]

public void setName(String name) {
    if (name == null) {
        throw new IllegalAgumentException("Name must not be null");
    }
    this.name = name;
}