Grosse If / Else verschachtelung übersichlitcher gestlten

[Ovoxo]

Hallo zusammen,

Ich habe ein kleines Problem. Ich habe mir eine Applikation basierend auf dem MVC Muster gemacht. Nun muss ich die Benutzereingaben von etwa 15 Textfeldern kontrollieren. Dabei ist eine sehr grosse If / Else Verschachtelung entstanden, welche die Lesbarkeit des Quellcodes beeinträchtigt. Nun wollte ich Fragen, ob es eine Möglichkeit gibt, dies etwas Leserfreundlicher zu gestalten.

if(student != null){
   if(student.name != null){
      if(Student.vorname != null){
          ......
      }else{
         //Fehlermeldung
      }
   }else{
      //Fehlermeldung
   }
}else{
   //Fehlermeldung
}

 

[Jardcore]

Erstmal kannst du Dinge in Methoden auslagern. Dann könntest du switch case benutzten.
In deinem Fall könntest du auch gucken ob du wirklich die ganzen != null Prüfungen brauchst. Das sieht schon ein bisschen wusselig aus. Versuche zu vermeiden, dass Dinge überhaupt null werden können, zum Beispiel vorname und name mit einem leeren String vorbelegen. ("")

Dein Student könnte auch eine isValid() Methode bekommen die prüft ob alle Eingaben gültig sind.

public boolean isValid() {
    return !student.name.equals("") && !student.vorname.equals("");
}

Daraus könnte man dann sowas erstellen.

public boolean istGueltigerStudent(Student student) {
    if(student != null) {
        // Fehlermeldung
        return false;
    }
    if(!student.isValid()) {
        // Fehlermeldung
        return false;
    }
  return true;
}

public void doSomething() {
    if(istGueltigerStudent()) {
        // speichere Student oder was auch immer
    }
}

Ps.: Sind alles nur Anregungen, bin kein Validierungsexperte

 

[Jardcore]

Wobei ich gerade gemerkt habe das du die Fehlermeldung nicht in die Methode istGueltigerStudent() packen solltest. Das wären zu viele Aufgaben für eine Methode. Hier sollte nur geschaut werden ob der Student gültig ist. Die Fehlermeldungen sollte an einer anderen Stelle abgearbeitet werden.

 

[JStein52]

Aber warum brauchst du zwei Methoden: isValid(..) und istGueltigerStudent(..) ? Allein der Name lässt schon erahnen dass sie wohl dasselbe tun ?

 

[Jardcore]

Das eine ist eine Methode des Student Objekts, dort wird davon ausgegangen das der Student != null ist. Wie soll man auch eine Methode auf null aufrufen
Wenn man nun prüfen möchte ob der Student != null ist und ob er gültig ist gibt es die istGueligerStudent() Methode, die erstens keine Methode des Student-Objekts ist und zweitens die Null-Prüfung durchführt.

Finde sowas auch nicht immer schön, besser wäre wie oben schon erwähnt dafür zu sorgen das ein Student niemals Null sein kann.

 

[JStein52]

Ok. Sollte dann die isValid() aber nicht so aussehen:

public boolean isValid() {
    return !name.equals("") && !vorname.equals("");
}

Oder bestenfalls noch ein this davor ?

 

[Jardcore]

Stimmt, du hast recht. Da ist mir wohl ein Fehler unterlaufen. Danke für den Hinweis