Java类[i]。子类异常无法读取字段“subclass ”,因为类[i]为空[重复]

Question

谢谢你的关注。让我们直奔问题。

这是我的课程的样子

`

class Moon{
        double distance;
        double angle;
        double diameter;
        String col;
        double centreOfRotationDistance;
        double centreOfRotationAngle;
    }
class Planet{
        double distance;
        double angle;
        double diameter;
        String col;
        Moon moon = new Moon();
    }

`

然而，当我试图访问< code >星球[i]时。像这样，java抛出< code > NullPointerException 。我的代码有什么问题吗？如果是，我该如何修复？

`

System.out.println("Creating planets...");
        String[] colArray = {"red", "orange", "yellow", "green", "blue", "indigo", "violet", "white", "red"};
        for(int i = 0; i < 8; i++){
            planets[i] = new Planet();
            planets[i].distance = 100 + (i * 100);
            planets[i].angle = 0 + (i * 20);
            planets[i].diameter = 20 + (i * 10);
            planets[i].col = colArray[i];
            System.out.println("Planet " + i + " created");
            System.out.println("Creating moon..." + i);
            planets[i].moon.distance = 10 + (i * 5);
            planets[i].moon.angle = 0 + (i * 20);
            planets[i].moon.diameter = i + 2;
            planets[i].moon.col = colArray[i++];
            planets[i].moon.centreOfRotationDistance = (100 + (i * 100))/10;
            planets[i].moon.centreOfRotationAngle = 0 - (i * 20);
        }
        System.out.println("Done creating planets.");
        System.out.println("Creating the sun...");

`

堆起来，以防有用

再次感谢您的阅读/回答

我认为我可能过于雄心勃勃，无法访问正在创建的类并从中获取值。因此，我尝试将代码更改为上面的片段，但没有成功？

问了几个朋友，没有人知道为什么会出错。因此发布

Answer 1

这些线是问题所在：

planets[i].moon.col = colArray[i++];
planets[i].moon.centreOfRotationDistance = (100 + (i * 100))/10;

在这些行中的第一行中，您将递增i-这意味着在下一行中，行星[i]引用元素仍然为空的数组索引。

我怀疑仅仅是这一简单的改变就能解决问题：

planets[i].moon.col = colArray[i];

顺便说一句，我还建议更改您的代码以创建一次有关Planet的所有内容，然后将其分配到数组中：

Planet planet = new Planet();
planet.distance = 100 + (i * 100);
planet.angle = 0 + (i * 20);
planet.diameter = 20 + (i * 10);
planet.col = colArray[i];
// etc
planets[i] = planet;

（我还建议使用私有字段，可能还有一个接受一堆值的构造函数，但那是另一回事。）