登录顺序的逻辑

我有一个代码，打开一个文件来检查用户给出的用户名或密码是否在文件中，但是代码显示登录成功时没有登录。我是编程初学者，所以我需要一些帮助来修复错误，并且使用基本的初学者编码，而不需要复杂的库或方法。登录顺序的逻辑

我得到下面的输出，这说明我的程序并不表现我怎么想它：

USERNAME OR PASSWORD INCORRECT! TRY AGAIN 
SUCCESSFUL, YOU ARE LOGGED IN!

下面是代码：

package login; 

import java.io.File; 
import java.io.FileNotFoundException; 

public class Main { 


    public static void main(String[] args) throws FileNotFoundException { 
     // TODO code application logic here. 
     File file = new File("users.txt"); 
     Login newLogin = new Login(file); 

     newLogin.checkLogin("word", "wordet"); 
    } 

} 


package login; 

import java.io.File; 
//library for file exception 
import java.io.FileNotFoundException; 
import java.util.ArrayList; 
//library for opening file 
import java.util.Scanner; 

public class Login { 

    private File file; 
    private ArrayList<String> usernames; 
    private ArrayList<String> passwords; 
    private int tries = 3; 

    public Login(File file) { 
     this.file = file; 
     this.usernames = new ArrayList<String>(); 
     this.passwords = new ArrayList<String>(); 
     this.tries = 3; 

    } 

    public void readLines() throws FileNotFoundException{ 

     ArrayList<String> lines = new ArrayList<String>(); 
     Scanner input = new Scanner(this.file); 
     while (input.hasNextLine()){ 
      lines.add(input.nextLine()); 
     } 
     input.close(); 

     usernames.clear(); 
     passwords.clear(); 
     for(int i = 0; i < lines.size(); i++) { 
      if (i %2 == 0) { 
       usernames.add(lines.get(i)); 
      } else { 
       passwords.add(lines.get(i)); 
      } 
     } 
    } 

    public void checkLogin(String user, String pass) throws FileNotFoundException{ 
     this.readLines(); 


     if (tries == 0){ 
      System.out.println("YOU HAVE TRIED TOO MANY TIMES!"); 
     } else { 
      for(int i = 0; i < this.usernames.size(); i++) { 
       if (this.usernames.contains(user)) { 
        if (passwords.get(i).equals(pass)) { 
         System.out.println("SUCCESSFUL, YOU ARE LOGGED IN!"); 
        } else { 
         System.out.println("USERNAME OR PASSWORD INCORRECT! TRY AGAIN"); 
         tries--; 
        } 
       } 
      } 
     } 
    } 
}

来源

2016-04-28 HAMZAH AHMED

你得到的错误是什么。 – tesnik03

你能显示文件的内容吗？ – raven

请使用标题更具描述性！ – Vucko

想象一下，您有两个"Darth Vader"用户，一个密码为"Luke"，另一个密码为"Leia"。此外，假设您只想接受密码为"Luke"的密码，并且密码为"Leia"的"Darth Vader"在您的文本文件中首先出现。

注意，你的代码，你会在你的for循环，进入if (this.usernames.contains(user)) 为"Darth Vader"错误的密码，因而，因为他不具备良好的密码，它会显示"USERNAME OR PASSWORD INCORRECT! TRY AGAIN"（你进入其他）！现在

，遵循同样的逻辑，在for的迭代，我们有"Darth Vader"与良好的密码，您将显示"SUCCESSFUL, YOU ARE LOGGED IN!"

这是一个解决办法：我允许

public void checkLogin(String user, String pass) throws FileNotFoundException{ 
     this.readLines(); 


     if (tries == 0){ 
      System.out.println("YOU HAVE TRIED TOO MANY TIMES!"); 
      return; 
     } 
     int i = 0; 
     for(; i < this.usernames.size(); i++) { 
       if (!this.usernames.contains(user)) 
        continue; 

       if (passwords.get(i).equals(pass)) { 
        System.out.println("SUCCESSFUL, YOU ARE LOGGED IN!"); 
        break; 
       }    
     } 
     if (i == this.usernames.size()) 
      System.out.println("USERNAME OR PASSWORD INCORRECT! TRY AGAIN"); 
    }

通知我自己在代码中改变了一些东西，比如删除了else，跟在if (tries == 0)之后，它使代码看起来更清晰（并且更易于调试）。

来源

2016-04-28 21:25:08 Simonlbc

它看起来像问题在于这个代码块。

  for(int i = 0; i < this.usernames.size(); i++) { 
       if (this.usernames.contains(user)) { 
        if (passwords.get(i).equals(pass)) { 
         System.out.println("SUCCESSFUL, YOU ARE LOGGED IN!"); 
        } else { 
         System.out.println("USERNAME OR PASSWORD INCORRECT! TRY AGAIN"); 
         tries--; 
        } 
       } 
      }

检查用户名时，您只是看到用户是否存在，而不是当前用户“i”是否与输入内容匹配。这应改为：

if (usernames.get(i).equals(user)) {

的逻辑则是：

枚举所有用户。如果当前用户名与测试用户名匹配，请检查密码。如果提供的密码与记录中的内容相匹配，请将用户登录。否则，登录失败。

来源

2016-04-28 21:09:07 CConard96

登录顺序的逻辑

回答

相关问题