Object in Array nach Prüfung löschen

[BMWbest]

Hey,

Ich schreibe gerade eine GUI Anwendung mit Swing, mit der es möglich sein wird, Verschiedene Arten von Bankkonten zu erstellen, Ein-/Auszahlungen und Überweisungen zu tätigen und Umsätze anzuzeigen.
Grafische Oberfläche steht, Konten lassen sich mit automatischer Zuteilung von Kontonummern auch generieren und jetzt möchte ich die Funktion einbauen, dass ich ein Konto wieder löschen kann.

Die Anwendung arbeitet (noch) nicht mit Datenbankanbindung, sondern mit Array Lists.

Meine Methode zum Ausfindig machen des bestimmten Kontos sieht momentan so aus:

public int gKontoLoeschen(int kontonummer) {
        for (Girokonto obj : gKontoliste) {
            if (String.valueOf(obj.getKontonummer()).equals(String.valueOf(kontonummer))) {
                //gKontoliste.remove(gKontoliste.get(kontonummer));
            } 
        }

        return kontonummer;
    }

In der Oberklasse der Konten gibt es neben der Kontonummer noch Namen und Kontostand.
Wie kann ich jetzt also sagen, dass wenn er die Kontonummer in der Array List gefunden hat, diese wieder löschen soll und mit ihr alle dazugehörigen Daten auch? Weil ein Index hab ich jetzt nicht angelegt, wüsste gerade auch nicht wie das gehen soll. Ist ja ein wenig anders wie bei Arrays

Danke euch schonmal!

 

[AmunRa]

Ich verstehe nicht ganz was du meinst.

Wenn du ein Object vom Typ Konto hast (Class Konto z.B. so )

public class Konto{
long konotNr;
int balance;
String name;

}

und dieses Object nirgends mehr refernziert wird. Wird das gesammte Object, mit all seinen Variablen vom Garbage Collector gelöscht.

Kannst du nochmal genau beschreiben was dich stört, oder wo das Problem liegt?

Lg
AmunRa


[EDIT]

HAb glaub ich grad dein Problem verstanden




Mit
list.remove(Object obj) kannst du ein Object wieder aus der Liste entfernen.

for (Girokonto obj : gKontoliste) {
            if (String.valueOf(obj.getKontonummer()).equals(String.valueOf(kontonummer))) {
                gKontoliste.remove(obj);
                break;
            } 
        }

[/EDIT]

 

[Volvagia]

Entweder du läufst mit dem Iterator über die Schleife, oder du benutzt eine Zählschleife.

public int gKontoLoeschen(int kontonummer) {
	for (int i = gKontoliste.size - 1; i >= 0; i--) {
		Bankkonto obj = gKontoliste.get(i);
		if (String.valueOf(obj.getKontonummer()).equals(String.valueOf(kontonummer))) {
			gKontoliste.remove(i);
		}
	}
	return kontonummer;
}

public int gKontoLoeschen(int kontonummer) {
	for (Iterator<Bankkonto> iterator = gKontoliste.iterator(); iterator.hasNext();) {
		Bankkonto konto = iterator.next();
		int listKontonummer = konto.getKontonummer();
		if (String.valueOf(listKontonummer).equals(String.valueOf(kontonummer))) {
			iterator.remove();
		}
	}
	return kontonummer;
}

btw. warum vergleichst du 2 (offenbar) Zahlen als String? Und warum gibst du den primitiven Parameter wieder zurück? Ist nicht wichtig, nur aus eigener Interresse.

 

[BMWbest]

@AmunRa
das habe ich bereits versucht. Wirft mir nur en Haufen Fehlermeldungen um die Ohren.
mit System.out.prinln konnte ich schon feststellen, dass er die Kontonummer nach Eingabe in der GUI findet, aber beim Löschen mit remove bekomme ich dann sowas:
"Exception in thread "AWT-EventQueue-0" java.util.ConcurrentModificationException"

@Volvagia
Den Rückgabewert nehm ich, um das in der GUI wieder anzeigen zu können, welche KNR gelöscht wurde (ja man könnte auch einfach auslesen und übertragen, aber dann würde es ohne if-Abfrage auch angezeigt werden, wenn das Konto gar nicht besteht. Somit hab ich weniger code, wenn ich erst nach erfolgreichem Löschen an das Textfield wieder das zurückgebe)
Das mit der Stringumwandlung ist momentan nur ne Notlösung,weil sonst "equals" nicht möglich ist und mit == wollte er es nicht machen.

Sind Iterationen nicht out (=unschöner Code)?

edit: Deine erste Variante funktioniert aber einwandfrei (nach size muss noch "()" oder?)

 

[pL4Gu333]

die Lösungen wurden ja schon gepostet... aber eine andere Frage was genau soll denn

public int gKontoLoeschen(int kontonummer) {

    return kontonummer;

bewirken ?

du änderst ja deine Kontonummer nicht, da wird die gleiche zurückgegeben, die du als Parameter geschickt hast dann kannst du auch gleich void machen^^

 

[pL4Gu333]

@Volvagia
Den Rückgabewert nehm ich, um das in der GUI wieder anzeigen zu können, welche KNR gelöscht wurde (ja man könnte auch einfach auslesen und übertragen, aber dann würde es ohne if-Abfrage auch angezeigt werden, wenn das Konto gar nicht besteht. Somit hab ich weniger code, wenn ich erst nach erfolgreichem Löschen an das Textfield wieder das zurückgebe)
Das mit der Stringumwandlung ist momentan nur ne Notlösung,weil sonst "equals" nicht möglich ist und mit == wollte er es nicht machen.

aber du bekommst in jedem Fall deine Kontonummer zurück egal ob sie gelöscht wurde oder nicht ... dann musst du das eher so machen:

(ich missbrauch mal kurz Volvagia- Code dafür )

public int gKontoLoeschen(int kontonummer) {
	for (int i = gKontoliste.size - 1; i >= 0; i--) {
		Bankkonto obj = gKontoliste.get(i);
		if (String.valueOf(obj.getKontonummer()).equals(String.valueOf(kontonummer))) {
			gKontoliste.remove(i);
                        return kontonummer;
		}
	}
	return null oder sowas ;
}

 

[BMWbest]

wie schon gesagt..
Ich möchte nacher in einem Textfeld ausgeben, dass die Kontonummer XXXXXXX erfolgreich gelöscht wurde.
Kopier ich die Kontonummer nur aus dem anderen Textfeld muss ich ja wieder ne Abfrage machen, um sicher zustellen, dass die Ausgabe nur dann erscheint, wenn das Konto auch wirklich da war und erfolgreich gelöscht wurde...
So übergeb ich dem Textfeld einfach die Rückgabe. verpack ich das noch in ne Try/Catch kann ich auch null zurückgeben oder ein Fehlerwert... unschön?

 

[pL4Gu333]

natürlich nicht null bei int
aber ich glaub du weißt was ich meine ....

kann bitte nen Admin meine Posts zusammenfügen?

 

[Volvagia]

edit: Deine erste Variante funktioniert aber einwandfrei (nach size muss noch "()" oder?)

Ups, ja.
Denk daran, dass du immer rückwärts durchlaufen musst, da sich sonst der Index noch nicht geprüfter Elemente ändert.

Sind Iterationen nicht out (=unschöner Code)?

Finde ich nicht. (Auch for each benutzt die Iterierbarkeit)
Aber Schönheit liegt ja im Auge des Betrachters.

Den Rückgabewert nehm ich, um das in der GUI wieder anzeigen zu können, welche KNR gelöscht wurde (ja man könnte auch einfach auslesen und übertragen, aber dann würde es ohne if-Abfrage auch angezeigt werden, wenn das Konto gar nicht besteht. Somit hab ich weniger code, wenn ich erst nach erfolgreichem Löschen an das Textfield wieder das zurückgebe)

Verstehe ich nicht, aber ist ja für das Problem nicht wichtig.

int i = bekommeKontoNrZumLoeschen();
int j = gKontoLoeschen(i);
textfeld.setText(String.value(j));

int i = bekommeKontoNrZumLoeschen();
gKontoLoeschen(i);
textfeld.setText(String.value(i));

Eine Liste zählt die Anzahl der Änderungen. Der Iterator vergleicht diese mit der Anzahl die die Liste wärend seiner Initialisation hatte. Wenn sie sich ändert fliegt die ConcurrentModificationException, um zu verhindern dass der Iterator vor allem bei Multithreading "out of sync" mit der Liste ist.

So übergeb ich dem Textfeld einfach die Rückgabe. verpack ich das noch in ne Try/Catch kann ich auch null zurückgeben oder ein Fehlerwert... unschön?

Du könntest auch eine boolean zurückgeben. Das machen Listen die nach Objekten suchen die gelöscht werden sollen auch so. Aber dabei musst du aufpassen, oft fällt der Rückgabewert als Prüfungsmöglichkeit nicht so auf. Insbesonders im Vergleich zu einer Checked Exception.

 

[BMWbest]

int i = bekommeKontoNrZumLoeschen();
int j = gKontoLoeschen(i);
textfeld.setText(String.value(j));

int i = bekommeKontoNrZumLoeschen();
gKontoLoeschen(i);
textfeld.setText(String.value(i));

mhh.. das versteh ich glaub noch nicht..
sollte ich das zusätzlich noch implementieren?

Was ich gemacht ist, dass ich noch einen zweiten "normalen" Array für jeden Kontentyp erzeuge, dem ich sämtliche generierte Kontennummern zuweise. Die generierte wird dann mit dem aus dem Array verglichen, damit keine doppelten Kontennummern vorkommen...
Jetzt stellt sich allerdings die Frage, wie ich die Kontonummer da wieder rausbekomme...
Mal schauen ob mir was einfällt , ansonsten frag ich nochmal...

 

[Natac]

@AmunRa
das habe ich bereits versucht. Wirft mir nur en Haufen Fehlermeldungen um die Ohren.
mit System.out.prinln konnte ich schon feststellen, dass er die Kontonummer nach Eingabe in der GUI findet, aber beim Löschen mit remove bekomme ich dann sowas:
"Exception in thread "AWT-EventQueue-0" java.util.ConcurrentModificationException"

Eine ConcurrentModificationException bekommst du immer dann, wenn du gerade mit einem Iterator über eine Collection (zum Beispiel eine ArrayList) läufst und währenddessen etwas an der Collection änderst.

ArrayList<String> list = new ArrayList<String>(10); //ArrayList mit 10 Einträgen erzeugen
Collections.fill(list, "Hallo"); // In jeden Eintrag "Hallo" eintragen

for(String entry : list){ // Diese for-Schleife nutz einen Iterator, auch wenn man das nicht sieht
  list.remove(entry) // BUMM! ConcurrentModificationException
}

Lösung 1 wurde ja schon geposstet: Den Iterator explizit verwenden und die

Iterator.remove(Object)

Methode nutzen. Diese ist aber leider nicht bei jedem Iterator implementiert.

Lösung 2 wäre, die Liste mit einem Index zu durchlaufen. Und wenn man dann noch hinten anfängt, kann man einfach Elemente rauslöschen, ohne dass es jemanden stört:

ArrayList<String> list = new ArrayList<String>(10); //ArrayList mit 10 Einträgen erzeugen
Collections.fill(list, "Hallo"); // In jeden Eintrag "Hallo" eintragen

for(int i = list.size()-1; i >= 0; i--){ // Per index durch die Liste laufen
  list.remove(i); // Index i löschen
}

Wollte ich der Vollständigkeit halber nur mal ergänzen.

 

[BMWbest]

hey vielen Dank... funktioniert jetzt einwandfrei..

noch ne Frage: ich hab ein paar verschiedenen LayerPanes jetzt um verschiedene Menüoptionen realisieren zu könne.
Unterschiedlichste Dinge (wie Einzahlungsmenü, Überwesiungsmenü etc) rufe ich über eine Combobox auf..
Ich schreibe also dann immer:

switch (this.jComboBox1.getSelectedIndex()) {
            case 0:
                this.jLayeredPaneAnlegen.setVisible(false);
                this.jLayeredPaneTrans.setVisible(false);
                this.jLayeredPane1.setVisible(false);
                break;
            case 1:
                this.jLayeredPaneAnlegen.setVisible(true);
                this.jLayeredPane1.setVisible(false);
                this.jLayeredPaneTrans.setVisible(false);
                break;
            case 2:
                this.jLayeredPane1.setVisible(true);
                this.jLayeredPaneTrans.setVisible(false);
                this.jLayeredPaneAnlegen.setVisible(false);
                break;
            case 3:
                this.jLayeredPaneTrans.setVisible(true);
                this.jLayeredPaneAnlegen.setVisible(false);
                this.jLayeredPane1.setVisible(false);
                break;
        }

ISt recht umständlich
gibt es keinen Befehl mit dem ich sagen kann "Alle LayerdPanes.setVisible(false) außer das eine " ??
Wäre um einiges angenehmener

 

[Volvagia]

Falls du immer nur 1 sichtbar hast kannst du sie bequem auf das CardLayout adden (benötigt beim Hinzufügen einen String als Constrain, sonst Exception) und bei Bedarf jederzeit austauschen.

 

[BMWbest]

mhh.. sagt mir jetzt gar nichts^^ ab und zu werden aber auch zwei gleichzeitig angezeigt
Gibts Alternativen?