0

コードに問題があります。私のコードのロジックは、入力したユーザー名が見つからない場合、確認ボックスが表示されます。実際に起こることは、入力したユーザー名が行で見つかった場合、「大丈夫です」という確認ボックスを起動しませんが、すべての行で検索を続け、else {statement} を初期化します。ところで、私の else ステートメントは確認ボックスです。

私のコードが書くサンプル data.dat

shawn | qwerty1234
cloe | password1234
jones | shittybrix

「shawn」と入力すると、if ステートメント「that's alright」が起動しますが、文字列全体の検索が続行され、入力されたユーザー名「shawn」が cloe と jones と等しくなくなり、else ステートメントが起動され、それぞれ 2 つの確認ボックスが表示されます。そのエラーを回避したいのですが、方法がわかりません。 REGISTER.JAVA でエラーが発生する

これはログイン用の私のコードです:

package login;

import java.awt.HeadlessException;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.*;
import javax.swing.JOptionPane;

public class Login extends javax.swing.JFrame {

    public Login() {
        initComponents0();
    }

    @SuppressWarnings("unchecked")
    private void initComponents0() {

        jLabel1 = new javax.swing.JLabel();
        jLabel2 = new javax.swing.JLabel();
        uname = new javax.swing.JTextField();
        jLabel3 = new javax.swing.JLabel();
        login = new javax.swing.JButton();
        reset = new javax.swing.JButton();
        pwd = new javax.swing.JPasswordField();
        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);
        jLabel1.setFont(new java.awt.Font("Tahoma", 0, 18)); // NOI18N
        jLabel1.setText("Login Pane");
        jLabel2.setText("User Name:");
        jLabel3.setText("Password:");
        login.setText("Login");
        login.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                String un = uname.getText();
                try {
                    try (BufferedReader br = new BufferedReader(new Filereader("data.dat"))) {
                        String strLine;
                        while ((strLine = br.readLine()) != null) {
                            if (strLine.startsWith(un)) {
                                JOptionPane.showMessageDialog(null, "Hello: " + un, "Registration", JOptionPane.INFORMATION_MESSAGE);
                            } else {
                                int sel = JOptionPane.showConfirmDialog(null, "It seems that you haven't registered yet? \n Lunch Registration Pane?", "Admin", JOptionPane.INFORMATION_MESSAGE);
                                if (sel == JOptionPane.YES_OPTION) {
                                    java.awt.EventQueue.invokeLater(new Runnable() {
                                        @Override
                                        public void run() {
                                            new Register().setVisible(true);
                                        }
                                    });
                                }
                            }
                        }
                    }
                } catch (IOException | HeadlessException ez) {
                    JOptionPane.showMessageDialog(null, "A null file was created in order to \n avoid File Catch errors", "Admin", JOptionPane.INFORMATION_MESSAGE);
                    File file = new File("data.dat");
                    try {
                        try (FileWriter writer = new FileWriter(file, true)) {
                            String data0 = "null";
                            String data1 = "null";
                            writer.write(data0 + " | " + data1 + "\n");
                        }
                    } catch (IOException | HeadlessException z) {
                        JOptionPane.showMessageDialog(null, e);
                    }
                }
            }
        });
        reset.setText("Reset Field");

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(getContentPane());
        getContentPane().setLayout(layout);
        layout.setHorizontalGroup(
                layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                .addGroup(layout.createSequentialGroup()
                .addContainerGap()
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING, false)
                .addComponent(jLabel1)
                .addGroup(layout.createSequentialGroup()
                .addComponent(jLabel2)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addComponent(uname, javax.swing.GroupLayout.PREFERRED_SIZE, 300, javax.swing.GroupLayout.PREFERRED_SIZE))
                .addGroup(layout.createSequentialGroup()
                .addComponent(jLabel3)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                .addGroup(layout.createSequentialGroup()
                .addComponent(login, javax.swing.GroupLayout.PREFERRED_SIZE, 137, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                .addComponent(reset, javax.swing.GroupLayout.PREFERRED_SIZE, 137, javax.swing.GroupLayout.PREFERRED_SIZE))
                .addComponent(pwd))))
                .addContainerGap(30, Short.MAX_VALUE)));
        layout.setVerticalGroup(
                layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                .addGroup(layout.createSequentialGroup()
                .addContainerGap()
                .addComponent(jLabel1)
                .addGap(18, 18, 18)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                .addComponent(jLabel2)
                .addComponent(uname, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                .addGap(18, 18, 18)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                .addComponent(jLabel3)
                .addComponent(pwd, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                .addGap(18, 18, 18)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                .addComponent(login)
                .addComponent(reset))
                .addContainerGap(javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)));

        pack();
    }

    /**
     * @param args the command line arguments
     */
    public static void main(String args[]) {
        java.awt.EventQueue.invokeLater(new Runnable() {
            @Override
            public void run() {
                new Login().setVisible(true);
            }
        });
    }
    // Variables declaration - do not modify
    private javax.swing.JLabel jLabel1;
    private javax.swing.JLabel jLabel2;
    private javax.swing.JLabel jLabel3;
    private javax.swing.JButton login;
    private javax.swing.JPasswordField pwd;
    private javax.swing.JButton reset;
    private javax.swing.JTextField uname;
    // End of variables declaration
}

これは、確認ボックスで [はい] をクリックしたときに起動する登録コードです

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package login;

import java.awt.HeadlessException;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.File;
import java.io.FileWriter;
import java.io.IOException;
import javax.swing.JOptionPane;

/**
 *
 * @author Jfetizanan
 */
class Register extends javax.swing.JFrame {

    /**
     * Creates new form GUIREG
     */
    public Register() {
        initComponents();
    }

    /**
     * This method is called from within the constructor to initialize the form.
     * WARNING: Do NOT modify this code. The content of this method is always
     * regenerated by the Form Editor.
     */
    @SuppressWarnings("unchecked")
    private void initComponents() {
        jLabel1 = new javax.swing.JLabel();
        jLabel2 = new javax.swing.JLabel();
        unamereg = new javax.swing.JTextField();
        jLabel3 = new javax.swing.JLabel();
        pwdreg = new javax.swing.JTextField();
        submit = new javax.swing.JButton();
        resetreg = new javax.swing.JButton();
        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);
        jLabel1.setFont(new java.awt.Font("Tahoma", 0, 18)); // NOI18N
        jLabel1.setText("Registration Pane");
        jLabel2.setText("User Name:");
        jLabel3.setText("Password:");
        submit.setText("Submit");
        submit.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                System.out.println("Runned");
                File file = new File("data.dat");
                try {
                    try (FileWriter writer = new FileWriter(file, true)) {
                        String data0 = unamereg.getText();
                        String data1 = pwdreg.getText();
                        writer.write(data0 + " | " + data1 + "\n");
                    }
                    JOptionPane.showMessageDialog(null, "Registered", "Registration", JOptionPane.INFORMATION_MESSAGE);
                } catch (IOException | HeadlessException z) {
                    JOptionPane.showMessageDialog(null, e);
                }
            }
        });
        resetreg.setText("Reset Field");

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(getContentPane());
        getContentPane().setLayout(layout);
        layout.setHorizontalGroup(
                layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                .addGroup(layout.createSequentialGroup()
                .addContainerGap()
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING, false)
                .addComponent(jLabel1)
                .addGroup(layout.createSequentialGroup()
                .addComponent(jLabel2)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addComponent(unamereg))
                .addGroup(layout.createSequentialGroup()
                .addComponent(jLabel3)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                .addComponent(pwdreg)
                .addGroup(layout.createSequentialGroup()
                .addComponent(submit, javax.swing.GroupLayout.PREFERRED_SIZE, 139, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED)
                .addComponent(resetreg, javax.swing.GroupLayout.PREFERRED_SIZE, 139, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addGap(0, 0, Short.MAX_VALUE)))))
                .addContainerGap(42, Short.MAX_VALUE)));
        layout.setVerticalGroup(
                layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                .addGroup(layout.createSequentialGroup()
                .addContainerGap()
                .addComponent(jLabel1)
                .addGap(18, 18, 18)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                .addComponent(jLabel2)
                .addComponent(unamereg, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                .addGap(18, 18, 18)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                .addComponent(jLabel3)
                .addComponent(pwdreg, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                .addGap(18, 18, 18)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE)
                .addComponent(submit)
                .addComponent(resetreg))
                .addContainerGap(javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)));

        pack();
    }
    /**
     * @param args the command line arguments
     */
    // Variables declaration - do not modify
    private javax.swing.JLabel jLabel1;
    private javax.swing.JLabel jLabel2;
    private javax.swing.JLabel jLabel3;
    private javax.swing.JTextField pwdreg;
    private javax.swing.JButton resetreg;
    private javax.swing.JButton submit;
    private javax.swing.JTextField unamereg;
    // End of variables declaration
}
4

1 に答える 1

3

その大きな混乱を通して問題を見つけることができました。こんにちは「何とか」と言ってポップアップするifステートメントでは、whileループから抜け出し、elseステートメントを移動する必要があります。while ループのすぐ外側にある else ステートメントの内容をコピーします (最初の人と一致しないからといって、後の人と一致しないという意味ではないので、最後まで怒鳴らないでください)。次のようにします。

BufferedReader br = new BufferedReader(
        new InputStreamReader(fstream));
String strLine;
boolean registered = false;
while ((strLine = br.readLine()) != null) {
    if (strLine.startsWith(un)) {
        registered = true;
        break;
    }
}
if(registered) {
    //blah
} else {
    //blah
}

また、Lunch 登録を Launch 登録に変更する必要があります ;)

あなたのコメントに応えて、それはこの行によるものです：

    setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);

両方のフレームにそれがあり、JVMを閉じる​​とすぐに停止するように指示されています。これは、ログイン フレーム (ログイン ウィンドウを閉じた場合はプログラムを停止する) には必要ですが、登録ウィンドウには必要ありません。そのためには、必要なjavax.swing.WindowConstants.DISPOSE_ON_CLOSEのは、フレームを閉じるときにフレームを破棄することを示しています (適切なクリーンアップ)。すべてのフレームが破棄された場合、JVM は終了します (他のスレッドが実行されていないと仮定します)。

于 2012-08-26T15:30:26.153 に答える