二维数组中的 NullPointerException

Posted

技术标签:

【中文标题】二维数组中的 NullPointerException【英文标题】:NullPointerException in 2D array 【发布时间】:2013-06-26 23:46:17 【问题描述】:

我编写了这段代码(自包含),它引发了一个让我感到困惑的 NPE。它涉及模型中使用的Ray 对象的二维数组。 Ray 对象被初始化并设置为模型。当模型外需要它们时,代码调用model.getRays()。此时代码会迭代 2D 数组并记录每个对象,因为.. 在代码调用 getRays() 后仅 2-3 行,2D 数组的第一个元素是 null

没有发生可能干扰数组中的对象的线程,所以我很困惑,只是在测试它们包含有效对象之后,它们怎么可能是null

此代码有什么问题以及如何修复它?

import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.geom.*;
import java.awt.image.BufferedImage;
import java.util.logging.*;
import javax.swing.*;
import javax.swing.border.EmptyBorder;

public class TestNullOn2DArray 

    public static void main(String[] args) 
        Runnable r = new Runnable() 

            @Override
            public void run() 
                GUI gui = new GUI();

                JFrame f = new JFrame("Test Null On 2D Array");
                f.add(gui.getContainer());
                f.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
                f.setLocationByPlatform(true);

                f.pack();
                f.setMinimumSize(f.getSize());
                f.setVisible(true);
                gui.plot();
            
        ;
        SwingUtilities.invokeLater(r);
    


class GUI 

    Logger log = Logger.getAnonymousLogger();
    private JPanel container;
    JLabel output;
    private JMenuBar menuBar;
    private BufferedImage canvas;
    private CustomModel model =
            new CustomModel(5, 1, .66f);
    private int defaultPositionNumber = 5;
    private int defaultAngleNumber = 5;

    public void plot() 
        Ray[][] rays = model.getRays();
        log.log(Level.INFO, "Rays: " + rays);
        for (int ii = 0; ii < rays.length; ii++) 
            for (int jj = 0; jj < rays[ii].length; jj++) 
                Ray ray = rays[ii][jj];
                log.log(Level.INFO, "Ray: " + ray);
                ray.resolve();
            
        
    

    public JPanel getContainer() 
        if (container == null) 
            container = new JPanel(new BorderLayout(5, 5));
            container.setBorder(new EmptyBorder(4, 4, 4, 4));

            ImageIcon icon = new ImageIcon(getCanvas());
            output = new JLabel(icon);
            container.add(new JScrollPane(output), BorderLayout.CENTER);
        
        return container;
    

    public void refreshCanvas() 
        model = new CustomModel(5, 6, .66f);
        canvas = null;
        BufferedImage bi = getCanvas();
        output.setIcon(new ImageIcon(bi));
    

    public void initializeRays(Graphics2D g) 
        log.log(Level.INFO, "initializeRays()");

        int startAngle = 1;
        int numberAngles = defaultAngleNumber;
        int inc = 90 / numberAngles;
        int numberPositions = defaultPositionNumber;
        Ray[][] rays = new Ray[numberAngles][numberPositions];
        g.setTransform(AffineTransform.getTranslateInstance(0d, 0d));
        for (int ii = rays.length - 1; ii > 0; ii--) 
            for (int jj = 0; jj < rays[0].length; jj++) 
                GeneralPath gp = new GeneralPath();

                double rads = 2d * Math.PI * (startAngle + (inc * ii)) / 360d;
                double x = 400d;
                double y = 100d;
                double yStart = y - (Math.sin(rads) * 100d);
                double xStart = x + (Math.cos(rads) * 100d);
                float r = 2f * (float) ((rads / Math.PI));
                float b = (float) jj / (float) rays[0].length;
                Color color = new Color(r, 1 - r, b, .6f);
                g.setColor(color);
                g.drawLine((int) x, (int) y, (int) xStart, (int) yStart);
                gp.moveTo(xStart, yStart);
                gp.lineTo(x, y);
                gp.closePath();
                Ray ray = new Ray(gp, color);
                log.log(Level.INFO, "" + ray);
                rays[ii][jj] = ray;
            
        
        model.setRays(rays);
        if (output != null) 
            output.repaint();
        
    

    public BufferedImage getCanvas() 
        if (canvas == null) 
            Dimension size = model.getPreferredSize();
            int w = size.width;
            int h = size.height;
            canvas = new BufferedImage(w, h, BufferedImage.TYPE_INT_ARGB);
            Graphics2D g = canvas.createGraphics();

            g.setColor(Color.WHITE);
            g.fillRect(0, 0, w, h);
            g.setColor(new Color(125, 27, 155, 127));

            initializeRays(g);

            g.dispose();
        
        return canvas;
    


class CustomModel 

    private int width;
    private int layers;
    private float offset;
    private Ray[][] rays;
    private Logger log = Logger.getAnonymousLogger();

    public CustomModel(int width, int layers, float offset) 
        this.width = width;
        this.layers = layers;
        this.offset = offset;
    

    public Line2D.Double getLineOfLasSegment() 
        Line2D.Double line = new Line2D.Double(0d, 0d, 1d, 1d);
        return line;
    

    public Dimension getPreferredSize() 
        int w = 600;
        int h = 300;
        Dimension prefSize = new Dimension(w, h);
        return prefSize;
    

    public int getWidth() 
        return width;
    

    public void setWidth(int width) 
        this.width = width;
    

    public int getLayers() 
        return layers;
    

    public void setLayers(int layers) 
        this.layers = layers;
    

    public Ray[][] getRays() 
        for (int ii = 0; ii < rays.length; ii++) 
            for (int jj = 0; jj < rays[ii].length; jj++) 
                log.log(Level.INFO, "Ray: " + this.rays[ii][jj]);
            
        
        return rays;
    

    public void setRays(Ray[][] rays) 
        this.rays = rays;
        for (int ii = 0; ii < rays.length; ii++) 
            for (int jj = 0; jj < rays[ii].length; jj++) 
                //Ray ray = rays[ii][jj];
                this.rays[ii][jj] = rays[ii][jj];
                log.log(Level.INFO, "Ray: " + this.rays[ii][jj]);
            
        
    

    public float getOffset() 
        return offset;
    

    public void setOffset(float offset) 
        this.offset = offset;
    


class Ray 

    private final Logger logger = Logger.getAnonymousLogger();
    private GeneralPath path;
    private boolean started = false;
    private Color color;

    Ray(GeneralPath path, Color color) 
        this.path = path;
        this.color = color;
    

    public void resolve() 
        logger.log(Level.INFO, "..resolving.");
    

    public GeneralPath getPath() 
        return path;
    

    public boolean isStarted() 
        return started;
    

    @Override
    public String toString() 
        String s = "Ray: "
                + "  started=" + started;
        return s;
    

输出

这显示了输出的最后一部分,检查 getRays() 将返回什么,然后是 NPE。

// ...
Jun 29, 2013 4:56:55 PM CustomModel getRays
INFO: Ray: Ray:   started=false
Jun 29, 2013 4:56:55 PM GUI plot
INFO: Rays: [[LRay;@1a9876e
Jun 29, 2013 4:56:55 PM GUI plot
INFO: Ray: null
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
        at GUI.plot(TestNullOn2DArray.java:52)
        at TestNullOn2DArray$1.run(TestNullOn2DArray.java:26)
        at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:251)
        at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:727)
        at java.awt.EventQueue.access$200(EventQueue.java:103)
        at java.awt.EventQueue$3.run(EventQueue.java:688)
        at java.awt.EventQueue$3.run(EventQueue.java:686)
        at java.security.AccessController.doPrivileged(Native Method)
        at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
        at java.awt.EventQueue.dispatchEvent(EventQueue.java:697)
        at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:242)
        at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:161)
        at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:150)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:146)
...

【问题讨论】:

你知道从一个问题中粘贴代码的感觉有多吗,让 IntelliJ 显示大量红色,并且实际使用运行?至少为此向你致敬。 应该这样问问题:) @Makoto 作为SSCCE 文档的作者,我可以向您保证,是的,我熟悉这种经历。 ;) 我犹豫是否称其为 SSSCE,因为在超过 200 LOC 时,许多人不会认为它“短”,但它始终是自给自足的。 【参考方案1】:

通过我的调试器查看它,看起来ray[0] 的所有内容都是空的。 其他的一切都存在。

所以,通过跟踪挖掘,我相信首先找到了数据放在那里的原因,以及为什么第一行是空的:

    我们打电话给f.add(gui.getContainer());。这本身似乎是无害的,但让我们看看 getContainer 在做什么......

    我们实例化ImageIcon icon = new ImageIcon(getCanvas());。再次,无害,但让我们再次进入兔子洞......

    然后我们(最终)在getCanvas() 中调用initializeRays(g);!这里出了什么问题? 这个: for (int ii = rays.length - 1; ii &gt; 0; ii--)

好吧,你倒数倒数。很奇怪,只有在极少数情况下才应该这样做,但我想你有你的理由。但是,倒数有两个问题:

你需要从长度 n-1 开始,你这样做;和 您的循环条件必须大于或等于 0。您确实具有该条件。

将其更改为包含大于或等于测试,您将不再有任何 NPE。

【讨论】:

冠军!修复是for (int ii=0; ii&lt;rays.length; ii++) 。至于为什么我要向后迭代数组。老实说,我不记得了!前进.. :) (自我注意)为自己的后部提供硬踢 a) 不计算屏幕上的光线数量并检查它们是否等于预期的数量。 & b) 不检查getRays() 输出的start清楚地 将第一个索引显示为null)。 c) 对循环使用倒数索引(没有明显原因)。

以上是关于二维数组中的 NullPointerException的主要内容,如果未能解决你的问题,请参考以下文章

如何将一个二维数组中的内容复制到另一个二维数组

Java 中的二维数组

二维数组1:java中的二维数组

Java中的“二维”数组

c++中的new()申请二维数组

二维数组中的查找