Продолжение цикла до хорошего ввода?

Question

Я пытаюсь создать программу, в которой пользователю нужна учетная запись для доступа к другим ее частям. Я хочу, чтобы это было настроено так, чтобы, если пароль подтверждения пользователя 2 не совпадает, они должны повторно ввести информацию. Кроме того, если пользователь оставляет что-либо пустое, они должны повторно ввести всю информацию. Как я могу это сделать, чтобы он продолжал цикл до тех пор, пока пользователь не попадет в хорошую информацию?

if (e.getSource() == okButton) { 
     if(!passString.equals(passStringConfirm) || userName.equals(null) || passString.equals(null) || passStringConfirm.equals(null)){ 
       enterUsername.setText(""); 
       enterPassword.setText(""); 
       enterConfirmPassword.setText(""); 
       } 
      } 

Это то, что у меня есть до сих пор, и работает только на одну итерацию. Я пытался сделать это, и он постоянно распечатывал предупреждающее сообщение, которое я пытался распечатать на JOptionPane.

import java.awt.Color; 
import java.awt.FlowLayout; 
import java.awt.event.ActionEvent; 
import java.awt.event.ActionListener; 

import javax.swing.BorderFactory; 
import javax.swing.JButton; 
import javax.swing.JFrame; 
import javax.swing.JLabel; 
import javax.swing.JOptionPane; 
import javax.swing.JPasswordField; 
import javax.swing.JTextField; 
import javax.swing.border.Border; 

public class CreateAccount extends JFrame implements ActionListener { 

    JLabel username = new JLabel("Enter your username"); 
    JTextField enterUsername = new JTextField(null, 15); 
    JLabel password = new JLabel("Enter your password"); 
    JPasswordField enterPassword = new JPasswordField(null, 15); 
    JLabel passwordConfirm = new JLabel("Confirm your password."); 
    JPasswordField enterConfirmPassword = new JPasswordField(null, 15); 
    JButton okButton = new JButton("OK"); 

    String userName; 
    double initialDeposit; 

    public CreateAccount() { 

     add(username); 
     add(enterUsername); 
     add(password); 
     add(enterPassword); 
     add(passwordConfirm); 
     add(enterConfirmPassword); 
     add(okButton); 

     okButton.addActionListener(this); 

     setTitle("New Bank Account Creation"); 
     setVisible(true); 
     setLocationRelativeTo(null); 
     setSize(270, 300); 
     setDefaultCloseOperation(EXIT_ON_CLOSE); 
     setLayout(new FlowLayout()); 

    } 

    @Override 
    public void actionPerformed(ActionEvent e) { 

     char[] pass = enterPassword.getPassword(); 
     String passString = new String(pass); 
     char[] passConfirm = enterConfirmPassword.getPassword(); 
     String passStringConfirm = new String(passConfirm); 

     userName = enterUsername.getText(); 

     if (e.getSource() == okButton) { 
      if(userName == null || userName.isEmpty() || passString == null || passString.isEmpty() || !passString.equals(passStringConfirm)) { 
        enterUsername.setText(""); 
        enterPassword.setText(""); 
        enterConfirmPassword.setText(""); 
        Border redLine = BorderFactory.createLineBorder(Color.red); 
        enterUsername.setBorder(redLine); 
        enterPassword.setBorder(redLine); 
        enterConfirmPassword.setBorder(redLine); 
        repaint(); 
        } 
       } 
      super.dispose(); 

      int response = 0; 
      String firstDesposit = JOptionPane.showInputDialog("Welcome " + userName + ". Enter your initial deposit."); 
      initialDeposit = Double.parseDouble(firstDesposit); 
      if (response == JOptionPane.OK_OPTION) { 
       new Menu(); 
      } 
     } 
    } 

Answer 1

Неверный результат теста if. Если у вас есть String (s) null, вы получите NullPointerException. Я думаю, вы хотели

if (userName == null || userName.isEmpty() || passString == null 
      || passString.isEmpty() 
      || !passString.equals(passStringConfirm)) { 
    enterUsername.setText(""); 
    enterPassword.setText(""); 
    enterConfirmPassword.setText(""); 
} 

Тогда ваш код пользовательского интерфейса должен проверить, пусты ли они, прежде чем разрешить пользователю действовать. Наконец, в приведенном выше коде я считаю, что вы можете использовать setBorder(), чтобы дать этим полям красную границу.

if (userName == null || userName.isEmpty() || passString == null 
      || passString.isEmpty() 
      || !passString.equals(passStringConfirm)) { 
    Border redLine = BorderFactory.createLineBorder(Color.red); 
    enterUsername.setText(""); 
    enterPassword.setText(""); 
    enterConfirmPassword.setText(""); 
    enterUsername.setBorder(redLine); 
    enterPassword.setBorder(redLine); 
    enterConfirmPassword.setBorder(redLine); 
} 

Редактировать

на основе кода вы предоставили, но вам нужно это в другом месте!

if(userName == null || userName.isEmpty() || passString == null 
     || passString.isEmpty() || !passString.equals(passStringConfirm)) { 
    enterUsername.setText(""); 
    enterPassword.setText(""); 
    enterConfirmPassword.setText(""); 
    Border redLine = BorderFactory.createLineBorder(Color.red); 
    enterUsername.setBorder(redLine); 
    enterPassword.setBorder(redLine); 
    enterConfirmPassword.setBorder(redLine); 
    repaint(); 
} else { // <-- add this 
    super.dispose(); 
    int response = 0; 
    String firstDesposit = JOptionPane.showInputDialog(
      "Welcome " + userName + ". Enter your initial deposit."); 
    initialDeposit = Double.parseDouble(firstDesposit); 
    if (response == JOptionPane.OK_OPTION) { 
     new Menu(); 
    } 
} 

Answer 2

Предполагая, что e.getSource() является блокирующим вызовом.

while(true){ 
if (e.getSource() == okButton) { 
     if(!passString.equals(passStringConfirm) || userName.equals(null) || passString.equals(null) || passStringConfirm.equals(null)){ 
       enterUsername.setText(""); 
       enterPassword.setText(""); 
       enterConfirmPassword.setText(""); 
       } 
      } 
else 
    break; //correct password 

}