2

LogInScreen クラスに統合された単純なアカウント作成ユーザー GUI を構築しています。単純な User を作成し、それをシリアル化し、ログイン プログラムでそのアクセスを使用できるようにします。問題は、パスワードを入力しても決して正しくないことです。私のプログラムが私のパスワードが同じではないことを教えてくれるという問題が常に発生します。これを修正する方法がわからないので、方法を知りたいです。以下にコードを投稿します（問題がここにある可能性があるため、シリアル化クラスとともにすべて）。

ユーザー クラス:

package passwordProgram;

import java.util.ArrayList;

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import java.io.Serializable;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JPasswordField;
import javax.swing.JTextField;
import javax.swing.UIManager;

public class User implements Serializable, ActionListener {

    public static ArrayList<String> allUsernames = new ArrayList<String>();

    String username;
    String password;

    public static void main(String[] args) {
        try {
            UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
        } catch (Exception e) {
            e.printStackTrace();
        }
        User user = new User();
        user.mainGUI();
    }



    JFrame frame;
    JPanel panel;
    JTextField createUsername;
    JPasswordField createPassword;
    JPasswordField confirmPassword;
    JButton createAccount;
    JLabel noValid;

    public void mainGUI() {
        noValid = new JLabel();
        frame = new JFrame("Create a new account!");
        panel = new JPanel();
        panel.setBackground(Color.ORANGE);
        createPassword = new JPasswordField(10);
        confirmPassword = new JPasswordField(10);
        createUsername = new JTextField(10);
        JLabel userTxt = new JLabel("New Username: ");
        JLabel userPass = new JLabel("New Password: ");
        JLabel confirmPass = new JLabel("Confirm Password: ");
        createAccount = new JButton("Create your account!");

        panel.setLayout(new GridBagLayout());
        GridBagConstraints left = new GridBagConstraints();
        left.anchor = GridBagConstraints.WEST;
        GridBagConstraints right = new GridBagConstraints();
        right.anchor = GridBagConstraints.EAST;
        right.weightx = 2.0;
        right.fill = GridBagConstraints.HORIZONTAL;
        right.gridwidth = GridBagConstraints.REMAINDER;

        frame.getContentPane().add(BorderLayout.NORTH, noValid);
        frame.getContentPane().add(BorderLayout.CENTER, panel);
        panel.add(userTxt, left);
        panel.add(createUsername, right);
        panel.add(userPass, left);
        panel.add(createPassword, right);
        panel.add(confirmPass, left);
        panel.add(confirmPassword, right);

        frame.getContentPane().add(BorderLayout.SOUTH, createAccount);
        frame.setVisible(true);
        frame.setSize(500, 300);

        createAccount.addActionListener(this);
    }

    public void actionPerformed(ActionEvent event) {
        if (createUsername.getText().length() <= 0 ) {
            noValid.setText("That is not a valid username. Please try again.");
            frame.getContentPane().add(BorderLayout.NORTH, noValid);
        }

        else if (allUsernames.contains(createUsername.getText())) {
            noValid.setText("That username is already taken. Please try again.");
            frame.getContentPane().add(BorderLayout.NORTH, noValid);
        }

            //THIS IS THE PART I'M CONFUSED ABOUT
        else if (!(createPassword.getPassword().equals(confirmPassword.getPassword()))) {
            noValid.setText("Your passwords do not match!");
            frame.getContentPane().add(BorderLayout.NORTH, noValid);
        } else {    
            SaveUser sUser = new SaveUser();
            sUser.createAccount(this);
            noValid.setText("Account created successfully");
            frame.getContentPane().add(BorderLayout.NORTH, noValid);
        }
    }
}

そしてシリアライズクラス：

package passwordProgram;

import java.io.FileOutputStream;
import java.io.ObjectOutputStream;

public class SaveUser {
    public void createAccount(User u) {
        try {
            FileOutputStream fileOS = new FileOutputStream("userInfo.txt");
            ObjectOutputStream objectOS = new ObjectOutputStream(fileOS);
            objectOS.writeObject(u);
            objectOS.close();
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
}
4

5 に答える 5

7

getPassword()返品char[]不可String。使用する

!(Arrays.equals(createPassword.getPassword(), confirmPassword.getPassword()))

代わりは

于 2013-04-20T02:37:55.733 に答える
5

これがおもちゃのプロジェクトでない限り、パスワードをプレーンテキストで保存しないでください。代わりに、パスワードのハッシュを保存します。ユーザーがログインするときに、パスワードをハッシュし、保存されているハッシュと比較します。 この質問の受け入れられた回答には、いくつかのサンプル ハッシュ コードがあります。

于 2013-04-20T02:38:34.007 に答える
3

JPasswordField.getPassword()を返しますchar[]。2 つの char 配列を使用して呼び出すarray1.equals(array2)と (あなたが行っているように)、それらが同じ内容を持っているかどうかではなく、同じオブジェクト参照であるかどうかがチェックされます。次のように`Array.equals(char[] array1, char[] array2)を使用します。

else if (!Array.equals(createPassword.getPassword(), confirmPassword.getPassword()))
于 2013-04-20T02:37:35.140 に答える
3

配列自体の内容ではなく、char配列( getPassword()Objectsによって返される) を比較しています。これら 2 つの配列を比較する最も安全な方法は、次を使用することです。Arrays#equals

else if (!(Arrays.equals(createPassword.getPassword(), 
                            confirmPassword.getPassword()))) {
于 2013-04-20T02:36:05.130 に答える
2

getPassword()を返しますchar[]。代わりに試してください：

if (!(new String(createPassword.getPassword()).equals(new String(confirmPassword.getPassword()))
于 2013-04-20T02:34:46.460 に答える