第一次登录无效，第二次有效

Question

这是一个读取文件的程序，如果它不存在或为空，它调用一个方法将一个对象添加到ArrayList中，然后调用一个WriteFile方法将该对象写入文件。

try{ 
     FileReader fr = new FileReader(managerFile); 
     BufferedReader br = new BufferedReader(fr); 


    }catch (IOException e){ 
     System.out.println("New System. Register as Manager to start using it."); 
     manager.add(AddManager()); 
     WriteFile(managerFile, manager); 
    } 
    do{ 
     try{ 
      select = Personnel(); 

      switch(select){ 
       case 1: 
        if(BLogIN(beautician)){ 
         Action(select, Menu(select),customer); 
        } 
        else{ 
         Main.main(args); 
        } 
        break; 
       case 2: 
        if(MLogIN(manager)){ 
         Action(select, Menu(select), customer); 
        } 
        else{ 
         Main.main(args); 
        } 
        break; 
       case 3: 
        break; 
       default: 
        System.out.println("Invalid Selection."); 
        System.out.println("Please choose only from the options above"); 
        System.out.println(""); 
        break; 
      } 
     }catch (InputMismatchException e){ 

     } 
    }while(select < 1 || select > 3); 

写入文件后，它调用传递前面的ArrayList这个方法我输入正确的ID和密码，它说无效后处理登录

public static boolean MLogIN(ArrayList<Manager> a){ 

    boolean valid = true; 

    Scanner scan = new Scanner(System.in); 

    System.out.print("Enter Manager ID :"); 
    String id = scan.next(); 
    System.out.print("Enter Password :"); 
    String pass = scan.next(); 
    for(int i = 0; i < a.size(); i++){ 
     valid = id.equals(a.get(i).getID()); 
     valid = pass.equals(a.get(i).getPass()); 
    } 

    return valid; 
} 

一个登录的方法，但我第二次登录，它是有效的。为什么？

Answer 1

您的条件检查用户的输入是否正确，是不正确。 它应该像下面这样：

public static boolean MLogIN(ArrayList<Manager> a){ 

    boolean valid = false; 

    Scanner scan = new Scanner(System.in); 

    System.out.print("Enter Manager ID :"); 
    String id = scan.next(); 
    System.out.print("Enter Password :"); 
    String pass = scan.next(); 

    for(int i = 0; i < a.size(); i++) { 
     if (id.equals(a.get(i).getID()) && pass.equals(a.get(i).getPass())) { 
      valid = true; 
      break; 
     } 
    } 

    return valid; 
}