1

当我运行这个项目时，我得到了一个 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();
        }

    }
}
4

5 回答 5

9

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

但这不是这里重要的。不，重要的是您了解如何自行调试 NPE 的过程：

  • 检查抛出 NPE 的行
  • 查找该行上为空的变量
  • 然后重新搜索代码以查看为什么当您认为它不应该为空时它为空。

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

您的其他问题包括：

  • 您过度使用静态。您的代码没有“状态”，没有真正的 OOP 类，这将限制您扩展和改进代码的能力。唯一的静态方法应该是 main 方法，它应该只包含设置类和启动它们的代码，仅此而已。
  • 您的 json 代码应该在自己的类中，与 GUI 类分开。这将允许您在完全不相关的代码中进行调试和增强。查找内聚和耦合。你希望你的类凝聚力高，耦合度低：把相关的东西放在一起，把不相关的东西分开。
  • 您的 while 循环while (listofmovies.getSelectedValue().toString()!= null)...在其当前位置没有意义。doClick()在创建、构建和渲染之前，您不应该调用GUI 的按钮。
  • 您的代码在没有计划的情况下被拼凑在一起。更好的是在提交任何代码之前先在纸上写出程序的结构。
于 2013-10-18T12:42:46.377 回答
3

这两行似乎没有，我正在访问尚未声明的内容。

相信 JVM。如果它说在那条线上有一个 NPE，那就是真的。您的“似乎”是一个错误的假设。

我尝试设置断点和调试来自己修复它，但无济于事。

听起来您需要学习如何在 IDE 中设置断点。如果你做得好，它会很快解决。你用的是哪一个？

谢谢你帮助我，我真的不是要求被喂食。

不幸的是，你要求被喂食。

于 2013-10-18T12:43:19.000 回答
2

你moviestowatch的没有初始化。

您正在调用yourMovies()初始化moviestowatch。

但是您正在尝试访问moviestowatch.gui()

于 2013-10-18T12:41:55.327 回答
2

你在第 35 行声明了 moviestowatch

private static java.util.List<String> moviestowatch; //arraylist which is populated by textfilemovie than printed to GUI element.

并在函数 gui() 的第 71 行使用它

    listofmovies = new JList(moviestowatch.toArray());

在使用它之前对其进行初始化。

调用函数的顺序是

  gui();
  json();
  yourMovies();

您的列表正在 yourMovies() 方法中初始化。您正在 gui() 方法中使用它。所以在调用实际使用它的 gui() 方法之前调用 yourMovies() 方法。

编辑：

您的下一个错误是在以下 while 循环中获得空指针@第 115 行。

要解决上述错误，您所做的是更改序列并在 gui() 之前调用 yourMovies()。但是在这里，您已经在 gui 中初始化了 listofmovies 列表并在 yourMovies() 方法中使用:)。

一般的编程实践是，对于所有列表/对象，您必须在每次使用前手动检查。它应该在使用前进行初始化。否则会抛出 NPE。

于 2013-10-18T12:42:28.713 回答
-2

如果你得到 NullPointerException 它总是意味着你的类字段之一没有被初始化。你不可能得到带有局部引用变量的 NPE，因为编译器会警告你。我自己尝试在我声明它的行上初始化不需要额外输入的所有内容，这种做法为我节省了很多 NPE。这尤其适用于容器。

于 2013-10-18T13:06:38.000 回答