Logik/Mathe Problem

[Luk10]

Hallo,

Ich habe folgendes Problem:
Ich habe ein kleines Programm für eine Passwortabfrage geschrieben! Der Kompiler sagt alles ist okay bliblablub ...

Nur stimmt irgendwas nicht mit der runterzähl-funktion von y ...
Wenn man die main() methode ausführt und immer eine falsche benutzerzahl und key eingibt tritt die break funktin im LoginTest schon bei 1 try verbleibend ein ...

was stimmt da nicht?

Hier der code:

// Klasse
public class Login {
  //Attribute
  private int loginzahl;
  private int loginkey;
  private boolean x = false;
  private int y = 6;
   //Methoden
   public void loginPruefen () {
       if (loginzahl == 123 && loginkey == 987) {
           System.out.println ("Zugang gewährt");
           x = true;
        }
       else {
           System.out.println ("Zugang verwehrt, try again!");
           if (y >= 2) {
               System.out.println (y - 1 + " Versuche verbleibend");
               if ( y == 1) {
                   System.out.println ("Kein Versuch mehr verbleibend!");
                   System.out.println ("Starten Sie das Programm erneut, für weitere Versuche!");;
                }
               else {
                   y--;
                }
            }
        }
    }                
   public boolean getX () {
      return x;
    }
    public int gety () {
        return y;
    }
    public void setLoginzahl (int p1) {
      loginzahl = p1;
    }
    public void setLoginkey (int p2) {
        loginkey = p2;
    }
}

import java.util.Scanner;
 
public class TestLogin {
  public static void main() {
    Login user = new Login ();
    Scanner input = new Scanner(System.in);
      while (user.getX() == false) {
        System.out.println("Geben Sie die Anmeldenummer ein: ");
        user.setLoginzahl(input.nextInt());
        System.out.println("Geben Sie den Anmeldekey ein: ");
        user.setLoginkey(input.nextInt()); 
        user.loginPruefen ();
        if (user.gety () == 1) {
            break;
        }
    }
}
}

irgendwas stimmt entweder mit den gleichheits zeichen nicht oder mit der y-- funktion ...

danke

 

[SlaterB]

wenn y 2 war und falsch eingegeben wurde, dann erscheint die Ausgabe '1 Versuch verbleibend', y wird auf 1 reduziert
und in der Main-Methode kommt das Break in Zeile 54/55,
da musst du auf 0 prüfen oder dir sonst den Ablauf anpassen

 

[Luk10]

wenn ich das so mache:

if (user.gety () == 0) {
            break;
        }

gibt es kein break ...

 

[javimka]

Zeile 23: [c]y--;[/c] auf Zeile 10 verschieben.

 

[Luk10]

klappt auch nicht ... bei der eingabe wiederholt sich dann nur 5 versuche noch ... weder break, noch 5, 4, 3, 2, 1, keiner

 

[SlaterB]

wie lautet dein aktueller Code?
bei mir gehts (swohl mit == 0 als auch == 1 in der main, ist nur ne Frage der Schönheit ob ein Versuch mehr oder weniger,
das muss man sich alles zurechtbiegen wie man es möchte)

public class Test {

	public static void main(String[] args) {
		Login user = new Login();
		Scanner input = new Scanner(System.in);
		while (user.getX() == false) {
			System.out.println("Geben Sie die Anmeldenummer ein: ");
			user.setLoginzahl(input.nextInt());
			System.out.println("Geben Sie den Anmeldekey ein: ");
			user.setLoginkey(input.nextInt());
			user.loginPruefen();
			if (user.gety() == 1) {
				break;
			}
		}

	}

}

class Login {
	// Attribute
	private int loginzahl;
	private int loginkey;
	private boolean x = false;
	private int y = 6;

	// Methoden
	public void loginPruefen() {
		y--;
		if (loginzahl == 123 && loginkey == 987) {
			System.out.println("Zugang gewährt");
			x = true;
		} else {
			System.out.println("Zugang verwehrt, try again!");
			if (y >= 2) {
				System.out.println(y - 1 + " Versuche verbleibend");
				if (y == 1) {
					System.out.println("Kein Versuch mehr verbleibend!");
					System.out
							.println("Starten Sie das Programm erneut, für weitere Versuche!");
					;
				} else {

				}
			}
		}
	}

	public boolean getX() {
		return x;
	}

	public int gety() {
		return y;
	}

	public void setLoginzahl(int p1) {
		loginzahl = p1;
	}

	public void setLoginkey(int p2) {
		loginkey = p2;
	}
}

 

[javimka]

Nimm die if-Klausel in Zeile 18 raus und mache daraus ein "else if(...)" als Alternative für if (y>=2).

public void loginPruefen() {
	y--;
	if (loginzahl == 123 && loginkey == 987) {
		System.out.println("Zugang gewährt");
		x = true;
	} else {
		System.out.println("Zugang verwehrt, try again!");
		if (y >= 2) {
			System.out.println(y - 1 + " Versuche verbleibend");

		} else if (y == 1) {
			System.out.println("Kein Versuch mehr verbleibend!");
			System.out.println("Starten Sie das Programm erneut, für weitere Versuche!");
			;
		}
	}
}