OOP/ Listen...etc...

[Supra]

Moin,
ich lerne Java und hab wieder eine Frage. :S Ich habe momentan eine Größere Aufgabe aber komme kein Stück voran :S. Also ein Teil ist ich muss einen Kunden erstellen mit Kundennummer, vorname und Name, soll aber prüfen das die Kundennumme nicht doppelt erstellt wird. Hänge da schon etwas länger drann. :bahnhof:
Hier mal mein Teil:

import java.util.*;
public class Bank
{
    List<Kunde> kunden = new ArrayList<Kunde>();

    public static void main(String[] args)
    {
        Bank bank = new Bank();
        bank.erstelleKunde(10000, "Michael", "Mustermann");
        bank.erstelleKunde(10000, "Max", "Mustermann");
        bank.erstelleKunde(10001, "Max", "Mustermann");

        for(Kunde kunde : kunden)
        {
            System.out.println(kunden);
        }

    }
    public void erstelleKunde(long kundenNummer, String vName, String name)
    {
        for(int i = 0; i < kunden.size(); i++)
        {
            if(kundenNummer == kunden.get(i).getKundenNummer()|| kundenNummer == 0)
            {
            System.out.print("Die Kundennummer ist bereits vergeben! Bitte wählen Sie eine andere aus!");
            }
            else
            {
                Kunde kunde = new Kunde();
                kunde.setKundenNummer(kundenNummer);
                kunde.setKundenVorname(vName);
                kunde.setKundenName(name);
                kunden.add(kunde);
            }
        }
    }
}

import java.util.*;
class Kunde
{
    private long kundenNummer;
    private String kundenName;
    private String kundenVorname;


    public void setKundenVorname(String vName)
    {
        kundenVorname = vName;
    }
    public String getKundenVorname()
    {
        return kundenVorname;
    }


    public void setKundenNummer(long nummer)
    {
        kundenNummer = nummer;
    }
    public long getKundenNummer()
    {
        return kundenNummer;
    }


    public void setKundenName(String name)
    {
        kundenName = name;
    }
    public String getKundenName()
    {
        return kundenName;
    }

    @Override
    public String toString()
    {
        return kundenNummer+ "" +" "+ "Kundenname: " + kundenVorname + " "+ kundenName;
    }
}

Naja ich hoffe mein versuch ist nicht all zu schlecht :S.
Fehlermeldung:
Bank.java:13:non-static variable kunden cannot be referenced from a static context
for(Kunde kunde : kunden)
^
Das gleiche bei System.out.println(kunden);.
^

 

[faetzminator]

Da die Variable nicht statisch ist, kannst du per [c]bank.kunden[/c] darauf zugreifen.

 

[Supra]

Achso.... ok jetzt läuft es, aber ausgeben tut er trotzdem nichts.

 

[faetzminator]

Nein, ich spreche von Z13.

for(Kunde kunde : bank.kunden) {
    System.out.println(kunde);
}

Ist aber nur sichtbar, da du dich in der gleichen Klasse befindest. Schöner wären Getter (und ggf. Setter):

for(Kunde kunde : bank.getKunden()) {
    System.out.println(kunde);
}

// ...

public List<Kunde> getKunden() {
    return kunden; // oder nach Wahl auch Collections.unmodifiableList(kunden)
}

 

[Supra]

Ja ich habe es gefunden. hmmm :/

 

[Supra]

Leider muss ich

public void erstelleKunde(long kundenNummer, String vName, String name)
{
}

so lassen, weil das so vorgegeben ist :S.

 

[faetzminator]

Hab ich irgendwas anderes behauptet!? Du sollst diese Methode auch nicht anpassen.

 

[Supra]

Nene mein ich ja auch nicht , ich habe halt nur das problem das ich jetzt keinen der kunden ausgegeben bekomme, liegt das an meiner Methode?:bahnhof:

 

[jgh]

das liegt imho daran, dass du über ein leeres Array iterierst und dort dann den Kunden -in Abhängigkeit von vergebener KdNr addest, oder nicht-! Da deine ArrayList aber eine size() von 0 hat, springt er garnicht in die Schleife...

 

[Supra]

Ok das habe ich mir gedacht, also müsste ich eine do-while schleife machen, damit er dies mindestens einmal ausführt? Wäre das eine Möglichkeit?

 

[faetzminator]

Nein. Gar nicht in einer Schleife, nur die Überprüfung willst du dort machen. So in etwa würd ich das machen:

public void erstelleKunde(long kundenNummer, String vName, String name) {
    if (checkKundenNummer(kundenNummer)) {
        Kunde kunde = new Kunde();
        kunde.setKundenNummer(kundenNummer);
        kunde.setKundenVorname(vName);
        kunde.setKundenName(name);
        kunden.add(kunde);
    } else {
        System.out.print("Die Kundennummer ist bereits vergeben! Bitte wählen Sie eine andere aus!");
    }
}

protected boolean checkKundenNummer(long kundenNummer) {
    if (kundenNummer <= 0) {
        return false;
    }
    for (Kunde kunde : kunden) {
        if (kundenNummer == kunde.getKundenNummer()) {
            return false;
        }
    }
    return true;
}

 

[jgh]

kannst du sicherlich auch so lösen, ich pers. würde eher den Sonderfall abdecken, dass kein Kunde in der ArrayList ist, also wenn

kunden.size()==0

dann adde einfach den Kunden, weil das überprüfen ob die KdNr schon vergeben ist, obsolet ist...ansonsten überprüfe ob es die KdNr schon gibt und handel dann entsprechend. Wobei ich meine zu wissen, dass du da noch ein wenig im Code nacharbeiten müsstest, ohne Gewähr

 

[Supra]

:bloed::idea:Achwas okey, ja danke euch das sieht ja wesentlich besser aus als meines, dann will ich mal gucken.

 

[faetzminator]

jgh, warum das denn? Wenns keine Einträge gibt, wird einfach 0 mal durch die Liste iteriert.
Die Implementation von Supra war so oder so falsch, sie hätte jeden Kunden 0 bis n mal hinzugefügt, wobei n die Grösse der bisherigen Liste war.

 

[jgh]

mein Beitrag war nicht auf deine Lösung bezogen, sondern auch die des TO´s
Deine Lösung sieht imho recht elegant und richtig aus...