«while» вопросы цикла для Java

Question

То, что я пытаюсь сделать, это получить пароль от пользователя, затем проверить его на 4 условия. Если это правильно. Затем просто распечатайте «pass word is correct». Если это не так, покажите пользователю, что не так, и вернитесь в меню. Проблема в том, что я не могу вернуться в меню. ЕСЛИ я делаю цикл while в строке //User interface //suppose to put a while loop here, but..... System.out.println(); System.out.println("Please create a password and a legal one". Он возвращается, но некоторые из операторов «if» не запускаются. Может ли кто-нибудь увидеть, где я застрял? Спасибо.

import java.util.Scanner; 

public class Text_processing 
{ 

    public static void main(String[] args) 
    { 
     Scanner kb = new Scanner(System.in);  //user input   
     String user_input;  // To hold input 
     char[] array;   //Array for user_input 
     int letters = 0;  // Number of letters 
     int digits = 0;   // Number of digits 
     boolean nu = true;  // 
     boolean nl = true; 
     boolean nd = true; 
     boolean n7 = true; 


     //User interface 
     //suppose to put a while loop here, but..... 
     System.out.println(); 
     System.out.println("Please create a password and a legal one" 
          + " should contain the following elements:"); 
     System.out.println("\tAt least 7 characters in length."); 
     System.out.println("\tAt least 1 upper case letter."); 
     System.out.println("\tAt least 1 lower case letter."); 
     System.out.println("\tAt least 1 number."); 

     //Get a string from the user 
     user_input = kb.nextLine(); 

     //Convert it to a char array 
     array = user_input.toCharArray(); 

     for(int i = 0; i < array.length; i++) 
     { 
      if(!nu && !nl && !nd && !n7) 
      {    
       break; 
      } 

      if (array.length >= 7)  //not < 7 cos breaks 
      {   
       n7 = false;   
      } 

      if(Character.isUpperCase(array[i])) 
      { 
       nu = false;    
      } 

      if (Character.isLowerCase(array[i])) 
      { 
       nl = false; 
      } 

      if (Character.isDigit(array[i])) 
      { 
       nd = false;  
      } 

     } 

     if(n7) 
     { 
      System.out.println("Password does not contain 7 or more letters."); 
     } 
     if(nu) 
     { 
      System.out.println("Password does not contain an upper case letter."); 
     } 
     if(nl) 
     { 
      System.out.println("Password does not contain an lower case letter."); 
     } 
     if(nd) 
     { 
      System.out.println("Password does not contain a number."); 
     } 

      System.out.println("Password is correct."); 


    } 
} 

Answer 1

В текущем коде, вам нужно будет убедиться, что nu, nl, nd, n7 сбрасываются на каждой итерации основного контура.

boolean badPassword = false; 
do { 
    boolean nu = true; 
    boolean nl = true; 
    boolean nd = true; 
    boolean n7 = true; 
    /* 
    * Add everything from your code that appears below `boolean n7 = true;` 
    * and above System.out.println("Password is correct."); here... 
    */ 

    badPassword = (n7 || nu || nl || nd); 
} while (badPassword); 
System.out.println("Password is correct."); 

Answer 2

Что плохого в начало цикла а с:

String user_input;  // To hold input 

и заканчиваться до:

 System.out.println("Password is correct."); 

Answer 3

Зачем вам следует по этой причине уже вы отображали содержимое Как пароль должен быть.

просто напечатать «пароль не соответствует указанным выше критериям, пожалуйста, попробуйте еще раз»

Я думаю, что его достаточно

Answer 4

Стандартный шаблон для этой ситуации:

Scanner sc = new Scanner(System.in); 
String userInput = null; 
while (true) { 
    System.out.println("please enter something:"); 
    userInput = sc.next(); 
    if (<input fails some test>) { 
     System.out.println("some reason"); 
     continue; // try again 
    } 
    // repeat other tests similarly 
    break; // will only get to here if input was OK 
} 

петля может» t до ввода действительного ввода, после чего userInput будет удерживать это значение.

Обратите внимание также, как, поместив подсказку внутри цикла, вы можете избежать дублирования кода при недопустимом вводе.

Answer 5

Напишите простой способ проверки пароля следующим образом.

private static boolean isValid(char[] array) { 
    if(array.length<7){ 
     return false; 
    } 
    else{ 
     boolean isUpper=false, isLower=false, isDigit=false; 
     for (int i = 0; i < array.length; i++) { 
      char c = array[i]; 
      if(Character.isUpperCase(c)){ 
       isUpper=true; 
      } 
      if(Character.isLowerCase(c)){ 
       isLower=true; 
      } 
      if(Character.isDigit(c)){ 
       isDigit=true; 
      } 
     } 
     return isUpper && isLower && isDigit; 
    } 
} 

Answer 6

Я лично считаю, что код, который вы написали, слишком длинный для такой простой проверки.

Но проблема заключается в том, что рука может быть исправлена ​​только в том случае, если вы можете поместить цикл while, который проверяет, является ли действительный флаг истинным, а также считывает входной сигнал клавиатуры в конце.

Что-то вроде кода ниже:

boolean valid = false; 
while (!valid) { 

     for (int i = 0; i < array.length; i++) { 
      valid = true; 
      if (!nu && !nl && !nd && !n7) { 
       break; 
      } 

      if (array.length >= 7) // not < 7 cos breaks 
      { 
       n7 = false; 
      } 

      if (Character.isUpperCase(array[i])) { 
       nu = false; 
      } 

      if (Character.isLowerCase(array[i])) { 
       nl = false; 
      } 

      if (Character.isDigit(array[i])) { 
       nd = false; 
      } 

     } 

     if (n7) { 
      valid = false; 
      System.out 
        .println("Password does not contain 7 or more letters."); 
     } 
     if (nu) { 
      valid = false; 
      System.out 
        .println("Password does not contain an upper case letter."); 
     } 
     if (nl) { 
      valid = false; 
      System.out 
        .println("Password does not contain an lower case letter."); 
     } 
     if (nd) { 
      valid = false; 
      System.out.println("Password does not contain a number."); 
     } 
     if (!valid) { 
      user_input = kb.nextLine(); 
      array = user_input.toCharArray(); 
     } 
    } 

Answer 7

Вместо того, даже есть цикл, вы могли бы просто проверить всю строку, используя String.matches(). Это позволит существенно упростить вашу логику

Подобно

//For uppercase 
user_input.matches(".*[A-Z].*"); // true if contians 

//For lowercase 
user_input.matches(".*[a-z].*"); // true if contians 

//For digit 
user_input.matches(".*\\d.*"); // true if contians