Kein Rechtzeitiger Abbruch bei Rekursiverfunktion

[eclipseworker]

Ich habe ein Programm mit zwei Klassen geschrieben das plump die Eingabe des PIN-Codes bei einen Bankautomaten simuliert dazu habe ich die Klasse Kartenbenutzer die nicht mehr tut als bei ihrer Initialisierung eine public int Variable mit dem Namen PIN zu erzeugen und eine Klasse pinueberpruefer die im Code so auszieht:

import java.util.Scanner;
public class PINUeberpruefen
{
    private int zaehler;
    private int maximum;
    private Kartenbenutzer Kartenbenutzer;
    /**
     * Konstruktor für Objekte der Klasse PINUeberpruefen
     */
    public PINUeberpruefen(Kartenbenutzer kar)
    {
        zaehler = 0;
        maximum = 3;
        Kartenbenutzer = kar;       
    }
    
    public void neuerBenutzer(Kartenbenutzer neuKar){
        Kartenbenutzer = neuKar;
    }
    
    public void pineingabe(){
      Scanner sc = new Scanner (System.in);
      while (sc.hasNext() && !sc.hasNextInt()) {
			sc.next();
			System.out.println("Geben Sie den PIN als Zahl ein!");
	  } 
      int PIN = sc.nextInt();
      if (PIN == Kartenbenutzer.PIN){
            System.out.println("Eingabe erfolgreich!");
      } else {
            if (zaehler < maximum){
            zaehler++;
            int noch=maximum-zaehler;
            if (noch==1){
                    System.out.println("Ungueltige Eingabe noch "+noch+" Versuch");
                    pineingabe();
            }else{
                    System.out.println("Ungueltige Eingabe noch "+noch+" Versuche");
                    pineingabe();
            }
        } else {
                System.out.println("Karte gesperrt");   
                zaehler=0;
           }
      } 
   }
}

Ich würde jetzt erwarten, das die Ein-Ausgabe auf der Kosule in etwa so aussieht:
(Der Kartenbenutzer hat den PIN 1234)
1239
Ungueltige Eingabe noch 2 Versuche
12354
Ungueltige Eingabe noch 1 Versuch
2855
Karte gesperrt
Tatsächlich sieht sie so aus:
1239
Ungueltige Eingabe noch 2 Versuche
12354
Ungueltige Eingabe noch 1 Versuch
2855
Ungueltige Eingabe noch 0 Versuche
5899
Karte gesperrt
Was kann ich tun?

 

[timbeau]

maximum = 2;

 

[Dekker]

1. Das was mein Vorposter geschrieben hat.
2. Müsstest du eigentlich erst die Bedingungen Prüfen, dann rechnen. Das Problem ist das man beim Methodenaufruf direkt ne Pin eingeben darf. Du musst aber vorher Prüfen ob der Nutzer das darf.
3. Sowas programmiert kein Mensch rekursiv. Ne Schleife drum herum würds wesentlich einfacher und sinniger machen.

 

[jgh]

ne, dann passt das nicht mit der anzahl seiner Versuche...

[java=31] if (zaehler < (maximum - 1)) {[/code]

so könnte man das machen...

 

[Dekker]

ne, dann passt das nicht mit der anzahl seiner Versuche...

[java=31] if (zaehler < (maximum - 1)) {[/code]

so könnte man das machen...

? Das steht doch da...

 

[jgh]

[java=31] if (zaehler < maximum){[/code]

vs

[java=31] if (zaehler < (maximum - 1)) {[/code]

edit:
wobei ich eine rekursion nicht wirklich erkennen kann! Das ist ne while-Schleife, nicht mehr oder weniger...

 

[timbeau]

Doch, die Funktion pineingabe() ruft sich selber auf...

 

[jgh]

wo du es jetzt sagst...Recht hast du 🙂

 

[langhaar!]

wobei ich eine rekursion nicht wirklich erkennen kann! Das ist ne while-Schleife, nicht mehr oder weniger...

Schau mal in Zeile 36 und 39.