if und else problem...

[Supra]

Moin ich lerne java und habe wieder mal ein Problem-.-.
Mein Code:

public void zeigeKundeAn(long kundennummer)//throws BankException
    {
        for(int i = 0; i < kunden.size(); i++)
        {
            if(kunden.get(i).getKundenNummer()==(long)kundennummer)
            {
                Kunde kunde = null;
                kunde = kunden.get(i);
                System.out.println("Kundennummer: " +kunde);
                break;
            }
            else
            {
                System.out.print("Kein Kunden unter dieser Kundennummer: "+ kundennummer +  " gefunden.");
            }
        }
    }

Mein aufruf:

 bank.zeigeKundeAn((long)10000);

Ich hoffe das reicht euch, mein Problem ist das ich beides ausgegeben bekomm die Fehlermeldung und mein Kunden. :S ich habe den Kunden in einer Arraylist gespeichert.
Wäre nett wenn ihr mir helft.

 

[Marcinek]

Er geht für jeden Kunden durch.

Wenn du 3 Kunden hast, 2x kein Treffer und der dritte ein Treffer, dann ist klar, dass du beides siehst.

Warum ein Cast von long nach long? ;D

 

[Supra]

Weil er mir jedes mal sagt das die "10000" ein int ist und dann nicht Compatible ist.

Stimmt du hast recht wie kann ich das umgehen? Wenn ich die Fehlermeldung raus nehme aus der for- schleife wird er mir die ja auch anzeigen.

 

[TKausL]

public void zeigeKundeAn(long kundennummer)//throws BankException
    {
        for(int i = 0; i < kunden.size(); i++)
        {
            if(kunden.get(i).getKundenNummer()==(long)kundennummer)
            {
                Kunde kunde = null;
                kunde = kunden.get(i);
                System.out.println("Kundennummer: " +kunde);
                return;
            }
        }

	System.out.print("Kein Kunden unter dieser Kundennummer: "+ kundennummer +  " gefunden.");
    }

Was passiert dort:

Die Schleife iteriert über alle Kunden. WENN er einen findet druckt er diesen aus und Beendet sofort die ganze Methode (returnt aus dieser). Fals er KEINEN findet passiert natürlich garnichts in der Schleife, er kommt aus der Schleife raus und printet die Fehlermeldung.

BTW:

                Kunde kunde = null;
                kunde = kunden.get(i);
                System.out.println("Kundennummer: " +kunde);

Ist unnötige Schreibarbeit...

                Kunde kunde = kunden.get(i);
                System.out.println("Kundennummer: " +kunde);

oder gleich so:

                System.out.println("Kundennummer: " +kunden.get(i));

BTW2:

Nutze nicht (long)10000 sondern 10000L

 

[pappawinni]

Du könntest vor der Schleife eine boolean auf auf false setzen.
Wenn der Kunde beim Durchlauf der Schleife gefunden wird, dann setzt du die auf true.
Nach der Schleife gibst du dann eine Fehlermeldung aus, wenn die boolean immernoch auf false steht.

 

[Supra]

Ja ok so eine Sache mit der Schreibarbeit . Ich habe halt gedacht wenn er einen kunden mit der Kundennummer gefunden hat soll er ein break; machen. Marcinek hat mir ja eben schon gesagt woran es liegt wuste nicht wie ich das umgehe. :S

 

[Supra]

Ok danke euch .

 

[TKausL]

Ja ok so eine Sache mit der Schreibarbeit . Ich habe halt gedacht wenn er einen kunden mit der Kundennummer gefunden hat soll er ein break; machen. Marcinek hat mir ja eben schon gesagt woran es liegt wuste nicht wie ich das umgehe. :S

Natürlich, das Break bricht den Schleifendurchlauf ab. Allerdings bekommst du Trotzdem die Fehlermeldung 2 mal, wenn z.B. erst der dritte Kunde ein Treffer ist da Durchlauf 1 und 2 ja in den else-Zweig kommen.
Merk dir, wenn du nach etwas in einer Schleife "suchst", dann stecke die Fehlermeldung NIEMALS in die Schleife sondern erst danach.

 

[stone71]

Warum gibst Du nicht den Kunden zurück und bekommst entweder null oder den Kunden!?
dann kannst Du mit dem vorhanden Kunden alles mögliche machen.

public Kunde zeigeKundeAn(long kundennummer)//throws BankException
    {
        Kunde kunde = null;
        for(int i = 0; i < kunden.size(); i++)
        {
            if(kunden.get(i).getKundenNummer()==(long)kundennummer)
            {
                kunde = kunden.get(i);
                break;
            }
        } 
        return kunde;
    }

 

[TKausL]

Warum gibst Du nicht den Kunden zurück und bekommst entweder null oder den Kunden!?
dann kannst Du mit dem vorhanden Kunden alles mögliche machen.

public Kunde zeigeKundeAn(long kundennummer)//throws BankException
    {
        Kunde kunde = null;
        for(int i = 0; i < kunden.size(); i++)
        {
            if(kunden.get(i).getKundenNummer()==(long)kundennummer)
            {
                kunde = kunden.get(i);
                break;
            }
        } 
        return kunde;
    }

Was er letztendlich mit den Kunden macht ist ja seine Sache... und ein long braucht immernoch nicht zu einem long gecastet werden.

public Kunde zeigeKundeAn(long kundennummer)
    {
        for(int i = 0; i < kunden.size(); i++)
            if(kunden.get(i).getKundenNummer()==kundennummer)
                return kunden.get(i);
        return null;
    }

 

[Supra]

Ok TKausL das werde ich mir merken, ja ist einiges komisch ich hatte die Fehlermeldung nur einmal bekommen und den Kunden, obwohl ich 3 Kunden habe. Aber jetzt funktionierts. Ich habe ja eigentlich schon eine funktion mit findeKunde und darüber wollte ich den kunden auch suchen aber da habe ich momentan noch meine Probleme das er mir bei der funktion "findeKunde" ein kundenobjekt zurück gibt. Und bei "zeigeKundeAn" ein long übergeben. Da muss ich noch bischen zu lernen, deswegen löse ich das erstmal so.

 

[Supra]

das mit long hat sich jetzt auch geklärt, scheint als ob ich was davor falsch gemacht hätte. Weil ich woanders "Long" habe

 

[TKausL]

Ok TKausL das werde ich mir merken, ja ist einiges komisch ich hatte die Fehlermeldung nur einmal bekommen und den Kunden, obwohl ich 3 Kunden habe.

Ja, genau dieser Fall tritt auf, wenn Kunde #2 der ist, den du suchst.

1. Schleifendurchgang: Fehlermeldung
2. Schleifendurchgang: TREFFER!!! + Abbruch der Schleife

Du bekommst mit deinem Code immer X Fehlermeldungen, wobei X die Zahl ist, wieviele Kunden "gecheckt" werden bis der richtige gefunden wurde minus 1.

 

[Supra]

Alles klar