Fehlerquelle leere ArrayList elegant abfangen

[nossek]

Ich habe eine Klasse CardFile, die unter anderem enthält:

- eine ArrayList "cards" mit Objekten der Klasse FileCard.
- eine Instanzvariable int currentCardIndex (welche Karte wird aktuell angezeigt und kann bearbeitet werden)
- Methoden zum Blättern nextCard() und previousCard(), die den currentCardIndex ändern
- Methoden die auf cards.get(currentCardIndex) zugreifen (löschen, ändern etc.)

Probleme gibt es logischerweise, wenn die ArrayList "card" leer ist.

Ich frage mich, wie man das am elegantesten abfängt.

Habe mir da folgendes überlegt:
In allen problematische Methoden cards.isEmpty() abfragen und in dem Fall
- bei Methoden ohne Rückgabewert nichts machen
- bei Methoden die ein Objekt zurückgeben sollen null zurückgeben

Wenn eine FileCard gelöscht wird ändert sich derzeit nichts am currentCardIndex, was bedeutet, daß die nächste FileCard zur Anzeige und Bearbeitung ausgewählt wird. Wenn die ArrayList nur noch eine FIleCard enthält und diese gelöscht wird ist daher currentCardIndex weiterhin = 0, obwohl ja gar keine FileCard mehr existiert, nicht schön.
Man könnte ja auch sagen, dass wenn das ListArray leer ist soll currentCardIndex = -1 sein. Das ließe sich erreichen, indem nach JEDEM löschen eines Elementes

if(cards.isEmpty){
   currentCardIndex = -1;
}

durchgeführt wird.

In dem Fall könnte man in den problematischen Methoden statt cards.isEmpty() eben currentCardIndex == -1 abfragen.

Wäre dies eine einigermaßen elegante Methode oder ist der ganze Entwurf Murks?

Ich hoffe daß die Frage nicht zu schwammig geraten ist!

 

[MySelV]

Hi,

meine Meinung ist zwar sicher subjektiv aber ich will sie trotzdem mal vorbringen:
Ich denke, dass es -an sich- egal ist, wie du es machst, weil der Aufrufer damit umgehen können muss. Das heißt, du legst fest, wie es sein wird und der der deinen Code aufruft darf sich darum kümmern, wie er mit Fehlern umgeht. Die Herausforderung für dich besteht also eigentlich darin:
- Dem Aufrufer mitzuteilen, wenn etwas schief ging (dafür würde ich eine Exception verwenden, die der Aufrufer entsprechend behandeln kann, z.B. Fehlermeldung ausgeben)
- Die Rückgaben konsistent zu halten (heißt, wenn du keine Exception im Fehlerfall wirfst, gib immer "null" o.ä. zurück)
- ~subjektiv~ Wenn etwas nicht durchgeführt werden kann, weil der Aufrufer einen Wert nicht gesetzt hat (z.B. deine Liste "cards" ist leer), dann ist dass ein Fehler -> Exception

Ob das alles so gut und richtig ist, weiß ich nicht, aber ich hoffe dass sich noch ein paar mehr beteiligen

Grüße

 

[fastjack]

Schau Dir doch an, wie das in Java bei Listen etc. gelöst wird. Du kannst z.B. bei ungültigen Indizes eine IndexOutOfBoundsException werfen. Ich würde dasselbe Verfahren anwenden, dann bist wenigstens mit den schon vorhandenen Sachen konform.

 

[nossek]

Danke für eure Antworten!

Ich bin mit Exceptions noch nicht vertraut, nutze sie bislang nur beim Speichern und Laden von Dateien. Vermutlich sind meine derzeitigen Übungsprojekte noch zu übersichtlich, um den Nutzen ganz zu verstehen. Kannmir schon vorstellen: eine einfache Plausibilitätsprüfung der Argumente zu machen und dann einfach nichts zu machen wenn die Argumente nicht stimmen wird bei komplexen Projekten dann zu schwer nachvollziehbaren Fehlern führen...

Es ist auch schwierig als Anfänger Übungsprojekte schon auf Wiederverwendbarkeit des Codes anzulegen, wenn eh klar ist, daß man den Code nicht wiederverwenden wird, weil man hoffentich bald viel besseren Code schreiben wird. Aber da ist wohl Konsens, daß man sich gleich angewöhnt, es amtlich zu machen.

Muß wohl noch einige Hausaufgaben machen und kann derzeit keine konkreten Fragen zu dem Thema stellen...:rtfm: