java - 类运行时的空指针

Question

当我运行这个项目时，我得到一个 NullPointer。

"Exception in thread "main" java.lang.NullPointerException
    at movieinfo.Swinggui.gui(Swinggui.java:71)
    at movieinfo.Swinggui.main(Swinggui.java:38)
"

这两行似乎没有，我正在访问尚未声明的内容。我尝试设置断点并调试自己修复它，但无济于事。感谢您帮助我，我真的不是要被喂食。

package movieinfo;

import java.awt.Color;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.List;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.File;
import java.io.IOException;
import java.nio.charset.Charset;
import java.util.ArrayList;
import java.util.Map;
import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JList;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
import javax.swing.JTextField;

import org.apache.commons.io.FileUtils;

import com.json.parsers.JSONParser;
import com.json.parsers.JsonParserFactory;


public class Swinggui {
    private static JButton enter;
    private static JTextField movietext;
    private static JTextArea movieinfo;
    private static JList listofmovies;//converts moviestowatch into gui element.
    private static File textfilemovie; //file which movies marked for watching are saved
    private static java.util.List<String> moviestowatch; //arraylist which is populated by textfilemovie than printed to GUI element.
    public static void main(String[] args) throws IOException
    {
yourMovies();
        gui();
        json();



    }
    public static void gui()
    {
        JFrame maingui = new JFrame("Gui");
        maingui.setLayout(new GridBagLayout());
        GridBagConstraints c = new GridBagConstraints();
        c.fill = GridBagConstraints.VERTICAL;
        enter = new JButton("Enter");
        c.gridx = 2;
        c.gridy = 1;
        maingui.add(enter, c);
        movieinfo = new JTextArea(5,20);
        movieinfo.setBorder(BorderFactory.createMatteBorder(2,2,2,2,Color.red));
        movietext = new JTextField(18);
        c.gridx = 1;
        c.gridy = 1;
        maingui.add(movietext, c);
        final JScrollPane scrolll = new JScrollPane(movieinfo);
        c.gridx = 1;
        c.gridy = 3;
        c.gridwidth = 2;
        maingui.add(scrolll, c);
        final JLabel titlee = new JLabel("Enter movie name below!");
        c.gridx = 1;
        c.gridy = 0;
        maingui.add(titlee, c);
        maingui.setResizable(false);
        maingui.setVisible(true);
        listofmovies = new JList(moviestowatch.toArray());
        c.gridx = 4;
        c.gridy = 3;
        maingui.add(new JScrollPane(listofmovies), c);
        movieinfo.setLineWrap(true);
        movieinfo.setWrapStyleWord(true);
        movieinfo.setEditable(false);
        scrolll.getPreferredSize();
        //pangui.setPreferredSize(new Dimension(300, 150));
        //pangui.add(scrolll, BorderLayout.CENTER);
        //movieinfo.add(scrolll);
        maingui.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        maingui.pack();

    }
    public static void json()
    {
        enter.addActionListener(new ActionListener(){

            private JsonParserFactory factory;
            private JSONParser parser;
            @SuppressWarnings("rawtypes")
            private Map jsonData;

            public void actionPerformed(ActionEvent e)  
            {
                System.out.println(apicall.getMovieInfo(movietext.getText()));
                factory = JsonParserFactory.getInstance();
                parser = factory.newJsonParser();
                jsonData = parser.parseJson(apicall.getMovieInfo(movietext.getText()));
                String Title = (String)jsonData.get("Title");
                String Year = (String)jsonData.get("Year");
                String Plot = (String)jsonData.get("Plot");
                movieinfo.setText("Title: "+Title+"\nYear: "+ Year +"\nPlot: "+Plot);
            }
        });
    }
    public static void yourMovies() throws IOException
    {
        textfilemovie = new File(org.apache.commons.io.FileUtils.getUserDirectory() + "/yourmovies.txt");

        textfilemovie.createNewFile();
        moviestowatch = FileUtils.readLines(textfilemovie);
        while (listofmovies.getSelectedValue().toString()!= null)
        {
            movietext.setText(listofmovies.getSelectedValue().toString());
            enter.doClick();
        }

    }
}

Answer 1

您在调用 yourMovies() 之前调用 gui()，因此在使用 ArrayList 之前不要初始化它。

但这不是这里重要的。不，重要的是你认识到如何自己调试一个NPE的过程:

• 检查抛出 NPE 的行
• 在该行中找到为空的变量
• 然后回头搜索代码，看看为什么当您认为它不应该为 null 时它为 null。

一旦您知道如何修复，通常这些错误很容易修复。

您的其他问题包括:

• 你过度使用了静态。您的代码没有“状态”，没有真正的 OOP 类，这将限制您扩展和改进代码的能力。唯一的静态方法应该是 main 方法，它应该只包含设置类和启动类的代码，仅此而已。
• 您的 json 代码应该在它自己的类中，与 GUI 类分开。这将允许您在与完全不相关的代码隔离的情况下调试和增强它。查找内聚和耦合。您希望类(class)的凝聚力高，耦合度低:将相关的东西放在一起，将不相关的东西分开。
• 您的 while 循环 while (listofmovies.getSelectedValue().toString()!= null)... 在其当前位置没有任何意义。在 GUI 按钮的创建过程中、构建过程中以及呈现之前，您不应该在 GUI 按钮上调用 doClick()。
• 您的代码未经计划就乱七八糟。在提交任何代码之前先在纸上写下程序的结构会好得多。