java - JTextField nextLine() を無視しますか? 【後編】

Question

更新され、より適切にフォーマットされたものについては、このリンクを参照してください。- 質問のバージョン。

これは最初の質問からのフォローアップです。私は、porfiriopartida からの提案を受けて、ロジックをユーザーインターフェイスから分離し、ロジックをという別のファイルに入れましたSimulator。したがって、GUI現在のプログラムは次のようになります。

import java.util.*;
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class GUI extends JPanel implements ActionListener {
    LinkedList<FoodItem> foodItemList = new LinkedList<FoodItem>();
    CategoryList<String> categoryList = new CategoryList<String>();
    Scanner in = new Scanner(System.in);
    protected JButton[] buttons = new JButton[6];
    String[] buttonNames = {"Add Item", "Remove Item", "Search for Item", "Update Item", "Check if List is Empty", "Print Items"};
    String[] buttonCommands = {"add", "remove", "search", "update", "isEmpty", "print"};
    protected JTextField textField;
    public GUI() {
        textField = new JTextField(20);
        textField.addActionListener(this);
        add(textField);
        for (int i = 0; i < 6; i++) {
            buttons[i] = new JButton(buttonNames[i]);
            buttons[i].setActionCommand(buttonCommands[i]);
            buttons[i].setVerticalTextPosition(AbstractButton.CENTER);
            buttons[i].setHorizontalTextPosition(AbstractButton.LEADING);
            buttons[i].addActionListener(this);
            add(buttons[i]);
        }
    }
    public void actionPerformed(ActionEvent e) {
        String command = e.getActionCommand();
        Simulator helper = new Simulator(textField);
        if (command.equals(buttonCommands[0]))
            helper.add(foodItemList);
        else if (command.equals(buttonCommands[1]))
            helper.remove(foodItemList);
        else if (command.equals(buttonCommands[2]))
            helper.search(foodItemList);
        else if (command.equals(buttonCommands[3]))
            helper.update(foodItemList);
        else if (command.equals(buttonCommands[4]))
            helper.isEmpty(foodItemList);
        else
            helper.print(foodItemList);
    }
    public static void createAndShowGUI() {
        JFrame frame = new JFrame("Vendor Interface");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        GUI newContentPane = new GUI();
        newContentPane.setOpaque(true);
        frame.setContentPane(newContentPane);
        frame.pack();
        frame.setVisible(true);
    }
}

そして、これはSimulatorプログラムです：

import javax.swing.JTextField;
import javax.swing.text.JTextComponent;

public class Simulator {
    private JTextComponent tf;
    public Simulator(JTextField textField) {
        this.tf = textField;
    }
    public void add(LinkedList<FoodItem> fi)    {
        System.out.println("Please enter the name of the product you want to add.");
        String text = tf.getText();
        try {
            if (fi.containsName(text))
                throw new DuplicateProductException();
            else
                fi.add(enterFoodItem(text));
        }
        catch(DuplicateProductException ex) {
            System.out.println("The product has already been added to the list.");
            System.out.println("Would you like to update or remove this item? (U for update, R for remove, N for neither)");
            String response = tf.getText();
            if (response.equals("U"))
                update(fi);
            if (response.equals("R"))
                remove(fi);
        }
    }
    public void remove(LinkedList<FoodItem> fi, String s)   {
        if (fi.size() == 1)
            fi.list = null;
        else
            fi.oneBeforeContainingName(s).setLink(fi.oneBeforeContainingName(s).getLink().getLink());
        System.out.println("The product has been removed.");
    }

    public void remove(LinkedList<FoodItem> fi) {
        System.out.println("Please enter the name of the product you want to remove.");
        String text = tf.getText();
        try {
            if (fi.containsName(text))
                remove(fi, text);
            else
                throw new NonexistentProductException();
        }
        catch(NonexistentProductException ex)   {
            System.out.println("The product does not exist in the list.");
            System.out.println("Would you like to add this product? (Y/N)");
            String response = tf.getText();
            if (response.equals("Y"))
                add(fi);
        }
    }
    public void search(LinkedList<FoodItem> fi) {
        System.out.println("Please enter the name of the product you want to search for.");
        String text = tf.getText();
        if (fi.containsName(text))  {
            System.out.println("The product is found.");
            System.out.println("Would you like to print it or update it? (P for print, U for update, N for neither)");
            String response = tf.getText();
            if (response.equals("P"))
                print(fi, text);
            if (response.equals("U"))
                update(fi, text);
        }
        else
            System.out.println("The product does not exist in the list.");
    }
    public void update(LinkedList<FoodItem> fi, String s){
        fi.nodeContainingName(s).setElement(enterFoodItem(s));
    }
    public void update(LinkedList<FoodItem> fi) {
        System.out.println("Please enter the name of the product you want to update.");
        String text = tf.getText();
        try {
            if (fi.containsName(text))
                update(fi, text);
            else
                throw new NonexistentProductException();
        }
        catch(NonexistentProductException ex)   {
            System.out.println("The product does not exist in the list.");
            System.out.println("Would you like to add this product? (Y/N)");
            String response = tf.getText();
            if (response.equals("Y"))
                add(fi);
        }
    }
    public void isEmpty(LinkedList<FoodItem> fi)    {
        if (fi.isEmpty())
            System.out.println("The list is empty.");
        else
            System.out.println("The list is not empty.");
    }
    public void print(LinkedList<FoodItem> fi, String s){
        LinkedListNode<FoodItem> temp = fi.list;
        while (!temp.getElement().name.equals(s))
            temp = temp.getLink();
        System.out.println(temp.getElement().toString());
    }
    public void print(LinkedList<FoodItem> fi)  {
        if (fi.isEmpty())
            System.out.println("The list is empty.");
        else
            System.out.println("Here is the list:" + fi.toString());
    }
    public FoodItem enterFoodItem(String input) {
        String[] requiredEntry = {"price", "quantity", "description", "size", "special order", "category"};
        double p = 0.0;
        int q = 0;
        String d = "";
        double s = 0.0;
        String so = "";
        String c = "";
        for (int i = 0; i < 6; i++) {
            System.out.println("Please enter the " + requiredEntry[i] + " of the product.");
            if (i == 0)
                p = Double.parseDouble(tf.getText());
            if (i == 1)
                q = Integer.parseInt(tf.getText());
            if (i == 2)
                for (int j = 0; j < 2; j++)
                    d = tf.getText();
            if (i == 3)
                s = Double.parseDouble(tf.getText());
            if (i == 4)
                for (int j = 0; j < 2; j++)
                    so = tf.getText();;
            if (i == 5)
                c = tf.getText();
        }
        return new FoodItem(input,p,q,d,s,so,c);
    }

    public class DuplicateProductException extends Exception    {
        public DuplicateProductException() {;}
    }
    public class NonexistentProductException extends Exception  {
        public NonexistentProductException() {;}
    }
}

プログラムを実行すると、ボタンを押した直後にAdd Item「追加する製品の名前を入力してください」と表示されます。（コンソール上で）しかし、テキストボックスに何かを入力する前であっても、すぐに次のエラーが表示されます。

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
    at Simulator.add(Simulator.java:12)
    at GUI.actionPerformed(GUI.java:31)
    at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
    at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
    at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
    at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
    at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(Unknown Source)
    at java.awt.Component.processMouseEvent(Unknown Source)
    at javax.swing.JComponent.processMouseEvent(Unknown Source)
    at java.awt.Component.processEvent(Unknown Source)
    at java.awt.Container.processEvent(Unknown Source)
    at java.awt.Component.dispatchEventImpl(Unknown Source)
    at java.awt.Container.dispatchEventImpl(Unknown Source)
    at java.awt.Component.dispatchEvent(Unknown Source)
    at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
    at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
    at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
    at java.awt.Container.dispatchEventImpl(Unknown Source)
    at java.awt.Window.dispatchEventImpl(Unknown Source)
    at java.awt.Component.dispatchEvent(Unknown Source)
    at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
    at java.awt.EventQueue.access$200(Unknown Source)
    at java.awt.EventQueue$3.run(Unknown Source)
    at java.awt.EventQueue$3.run(Unknown Source)
    at java.security.AccessController.doPrivileged(Native Method)
    at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
    at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
    at java.awt.EventQueue$4.run(Unknown Source)
    at java.awt.EventQueue$4.run(Unknown Source)
    at java.security.AccessController.doPrivileged(Native Method)
    at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
    at java.awt.EventQueue.dispatchEvent(Unknown Source)
    at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
    at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
    at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
    at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
    at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
    at java.awt.EventDispatchThread.run(Unknown Source)

今回は何が起こっているのでしょうか? どうすれば修正できますか?

score 2 · Accepted Answer

コードには次が含まれます。

JTextComponent tf = null;
String text = tf.getText();

null 値のオブジェクトでメソッドを呼び出すことはできません。それが原因NullPointerExceptionです。

たぶん、既存のからテキストを取得したいでしょうJTextField:

String text=tf.getText();

で新しいJTextField参照を作成するとJTextComponent tf = null;、インスタンス変数がローカル変数でシャドウされ、アクセスできなくなります。

編集：コンストラクターで、他の（同じくらい重要な）間違っていることについては、Reimusの回答を参照してください。

score 2 · Accepted Answer

textFieldのコンストラクターでシャドーイングしていますSimulator。交換

JTextField tf = textField;

と

tf = textField;

score 0 · Accepted Answer

これを使用した場合：

public void add(LinkedList<FoodItem> fi) {
    System.out.println("Please enter the name of the product you want to add.");
    JTextComponent tf = null; // tf points to NULL, you might remove this line.
    String text = tf.getText();

tf 参照が null に設定されているため、次のように直後に例外がスローされます。

String text = tf.getText(); //null.getText() !!

ここでも、コンストラクターでテキストフィールドをローカル変数に設定していますが、グローバル変数は null に設定されています。

プライベート JTextComponent tf; public Simulator(JTextField textField) { JTextField tf = textField; 次のように変更します。

  private JTextComponent tf;
  public Simulator(JTextField textField) {
   //you don't need "this" but that way you make sure you refeer to the class 
    this.tf = textField; variable.
  }

コードがそのままコンパイルされないため、それで十分かどうかはわかりません。

CategoryList クラスがありません。
FoodItem クラスがありません。
LinkedListNode クラスがありません。

...

public void print(LinkedList<FoodItem> fi, String s) {
    LinkedListNode<FoodItem> temp = fi.list;

LinkedList オブジェクトが常に LinkedListNode のインスタンスであるという保証はないと思います。ほとんどの場合、LinkedList の代わりに、cast または receive LinkedListNode を引数として使用する必要があります。

java - JTextField nextLine() を無視しますか? 【後編】

3 に答える 3

Related

Reference