如何在 Java 中很好地调整循环？

Question

我目前正在编写一个小程序，向用户询问 PinCode，如果 Pin 正确则返回“:)”，如果 Pin 错误则返回“:(”。 我的代码由一个 java 文件和一个文本文件组成。

这是代码：

import java.io.*;
import java.util.*;

public class Main {

static public boolean readPinsData(File dataFile, ArrayList<Integer> data) {
   boolean err = false;
   try {
      Scanner scanner = new Scanner(dataFile);
      String line;
      while (scanner.hasNext()) {
         line = scanner.nextLine();
         try {
            data.add(Integer.parseInt(line));
         } catch (NumberFormatException e) {
            e.printStackTrace();
            err = true;
         }
      }
      scanner.close();
   } catch (FileNotFoundException e) {
      e.printStackTrace();
      err = true;
   }

   return err;
}


public static void main(String[] args) {

    System.out.println("-----------------------");
    System.out.println("MY APP");
    System.out.println("-----------------------");
    Console console = System.console();
    int pinSize = 0;
    int nbTry = 0;

    do{
      do{
    char passwordArray[] = console.readPassword("Enter pin: ");
    pinSize = passwordArray.length;

    if(pinSize != 4){
            System.out.println("Pin must be 4 digits");
        } else {
            System.out.println("Checking...");
        }


   ArrayList<Integer> pins = new ArrayList<Integer>();
   readPinsData(new File("bdd.txt"), pins);
   //System.out.println(pins);
   //System.out.println(passwordArray);


   String[] thePins = new String[pins.size()];
   for (int i = 0; i < thePins.length; i++) {
    thePins[i] = pins.get(i).toString();
}

   String passEntered = String.valueOf(passwordArray);





   for(int i = 0 ; i < thePins.length ; i++){
      if(passEntered.equals(thePins[i]) && pinSize == 4){
          System.out.println(":)");
        } else if(!passEntered.equals(thePins[i]) && pinSize == 4){
            nbTry++;
        }

    }   

  }while(nbTry < 3);
   }while(pinSize != 4);

}
}

这是存储所有好的 Pin 图的 bdd.txt：

1111
2222
3333
4444
5555
6666
7777
8888
9999

实际上我的问题是将尝试次数限制为 3 次。我需要解释一下：

--> 用户必须输入密码

--> 要么他输入一个好的 4 位数密码，然后打印“:)”（应用程序就完成了）

--> 要么他输入了错误的 4 位密码，要么打印出“:(”，并且 nbTry 必须是 ++。 在这种情况下，他只剩下 2 次尝试了

--> 他也可以输入 1 位密码或 2 位密码或 3 位数密码...在这种情况下 nbTry 不受影响，他只需重新输入 4 位数密码。

我不知道如何处理 nbTry 左侧的部分。有什么想法吗？

Answer 1

您希望他只能输入 4 位密码还是希望他能够输入任意长度的密码？

编辑：

阅读你的main 我看到你有两个'do...while`。如果您更改它们的顺序，它应该可以工作。我无法在 atm 测试它，因为我在移动位上尝试这样：

do {
  do {
      ....
  } while (pinSize != 4);
} while (nbTry < 3);

编辑2：

boolean loginCorrdect = false;
for (int i = 0; i < thePins.length; i++) {
   if (passEntered.equals(thePins[i]) && pinSize == 4) {
      System.out.println(":)");
      booleanCorrect = true;
      break;
   } else if (!passEntered.equals(thePins[i]) && pinSize == 4) {
      System.out.println(":(");
   }

}
if(!booleanCorrect && pinSize == 4){
   nbTry++;
}

希望您能理解它，因为它很难在移动设备上输入。

完整的主要代码： public static void main(String[] args) {

System.out.println("-----------------------");
System.out.println("MY APP");
System.out.println("-----------------------");
Console console = System.console();
int pinSize = 0;
int nbTry = 0;
boolean authenticated = false;

do {
   do {
      char passwordArray[] = console.readPassword("Enter pin: ");
      pinSize = passwordArray.length();

      if (pinSize != 4) {
     System.out.println("Pin must be 4 digits");
      } else {
     System.out.println("Checking...");
      }

      ArrayList<Integer> pins = new ArrayList<Integer>();
      readPinsData(new File("bdd.txt"), pins);
      // System.out.println(pins);
      // System.out.println(passwordArray);

      String[] thePins = new String[pins.size()];
      for (int i = 0; i < thePins.length; i++) {
         thePins[i] = pins.get(i).toString();
      }

      String passEntered = String.valueOf(passwordArray);

      for (int i = 0; i < thePins.length; i++) {
         if (passEntered.equals(thePins[i]) && pinSize == 4) {
    System.out.println(":)");
    authenticated = true;
    break;
}
}

} while (pinSize != 4);
  if (!authenticated && pinSize == 4) {
System.out.println(":(");
nbTry++;
 }
} while (nbTry < 3 && !authenticated);
}

Answer 2

如果我理解您的问题，您可以这样做（完成后您应该关闭（）您的扫描仪）-

static public boolean readPinsData(File dataFile, ArrayList<Integer> data) {
    boolean err = false;
    Scanner scanner = null; // so we can close the Scanner.
    try {
        scanner = new Scanner(dataFile);
        String line;
        while (scanner.hasNext()) {
            line = scanner.nextLine();
            try {
                data.add(Integer.parseInt(line));
            } catch (NumberFormatException e) {
                e.printStackTrace();
                err = true;
            }
            // Limit it to 3 attempts. Set err on 3rd.
            if (data.size() >= 3) {
                err = true;
                break;
            }
        }

    } catch (FileNotFoundException e) {
        e.printStackTrace();
        err = true;
    } finally {
        if (scanner != null) { // Close the Scanner.
            scanner.close();
        }
    }

    return err;
}

Answer 3

对您的控制流/逻辑进行自上而下的细化会有所帮助。 由于这有点家庭作业的味道，所以只是一个想法：

Set<String> pins = readPINs();
boolean authenticated = false;
for (int attempt = 0; attempt < 3; ++attempt) {
    String pin = askForPIN();
    if (!isSyntacticalValidPIN(pin)) {
       giveError();
       break;
    } else if (pins.contains{pin)) {
       authenticated = true;
       break;
    }
}
if (authenticated) {
    offerMenu();
}