Normal
Diese Zeile sollte dir suspekt vorkommen. Es kann ja NIE EIN Konto einer Map entsprechen. Also geht er IMMER in den if-Zweig und nie in den else.Richtiger wäre[code=Java]String kontoName = (JOptionPane.showInputDialog(null, "Name des Kontoinhabers", "Kontoverwaltung", JOptionPane.PLAIN_MESSAGE)); Konto konto = alleKonten.get(kontoName); if (konto != null) { alleKonten.remove(kontoName); } else { JOptionPane.showMessageDialog(null, "Zu diesem Namen liegt kein Konto vor", "Abbruch", JOptionPane.WARNING_MESSAGE); } break;[/code]Das hört sich nach einem Plan an und zwar eine guten. Du fragst ja bereits an mehreren Stellen de Benutzer nach verschiedenen Dingen. Also muss du nun Kontoname UND Betrag fragen. Letzteres in der Tat in einer NEUEN Variable speichern. Ich würde dann eine Funktion in Konto schreiben, wo der diesen Betrag als Parameter reinkommt und dem Konto gutgeschrieben wird.
Diese Zeile sollte dir suspekt vorkommen. Es kann ja NIE EIN Konto einer Map entsprechen. Also geht er IMMER in den if-Zweig und nie in den else.
Richtiger wäre
[code=Java]String kontoName = (JOptionPane.showInputDialog(null, "Name des Kontoinhabers", "Kontoverwaltung", JOptionPane.PLAIN_MESSAGE));
Konto konto = alleKonten.get(kontoName);
if (konto != null) {
alleKonten.remove(kontoName);
} else {
JOptionPane.showMessageDialog(null, "Zu diesem Namen liegt kein Konto vor", "Abbruch", JOptionPane.WARNING_MESSAGE);
}
break;
[/code]
Das hört sich nach einem Plan an und zwar eine guten. Du fragst ja bereits an mehreren Stellen de Benutzer nach verschiedenen Dingen. Also muss du nun Kontoname UND Betrag fragen. Letzteres in der Tat in einer NEUEN Variable speichern. Ich würde dann eine Funktion in Konto schreiben, wo der diesen Betrag als Parameter reinkommt und dem Konto gutgeschrieben wird.
