java.awt.geom.Point2D.DoubleのDouble型配列がnullになってしまう

Question

###したいこと
先ほど質問させていただいたもの(java.lang.NullPointerException の発生原因が分からないです)の続きで質問させていただきます。
java.awt.geom.Point2D.DoubleのDouble型をもちいてdouble型の数値を座標x,yに代入したいです
###発生している問題・エラーメッセージ

• 該当のソースコード

import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.geom.Point2D.Double;

import javax.swing.JFrame;
import javax.swing.JPanel;;

public class Main extends JPanel{

	Mouse  mouse = new Mouse();
	public static Double[] point;
	public static int point_num = 0;
	public static final int SIZE = 300;

	public Main(){
		final int POINT_SIZE = 10;
		point = new Double[POINT_SIZE];//該当部分:pointがnullかも？

		addMouseListener(mouse);
	}



	public static void main(String[] args) {
		new Main();
		// TODO 自動生成されたメソッド・スタブ
		JFrame f = new JFrame();
		f.setSize(600,400);
		f.add(new Main());
		f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		f.setResizable(true);
		f.setVisible(true);


	}
//(以下省略)
}

デバックした所、コンストラクタ内の変数pointがnullになっていました。mainメソッドでは、Mainコンストラクタが呼び出されているのでなぜnullになっているのか自分では分からない状況です。Double型の"配列”にしたことが悪かったのでしょうか？

解答お願いします

Comments

[MIURA_Yasuyuki]

変数pointがnullだったのはどこのタイミングでしょうか?また、何をもってnullだと判断していましたか?「new Main();」の後だったらnullにはならないと思います。

[swordone]

問題は、pointがnullだったのか、配列pointの中身がnullだったのか、ですが、pointのがnullというのは間違いないのですか？

[m_s]

pointがnullだったのは
point = new Double[POINT_SIZE];
をMainコンストラクタで呼び出した直後です
eclipseのデバックのウォッチ機能でpointを確認した所配列pointの各要素が全てnullになっていました。
きちんと配列pointの宣言を行っているので、なぜpointがnullになっているのか分からないです。

[m_s]

swordoneさん
デバックで確認すると配列pointの各要素がnullになっていました。
ということは、pointにはいっている配列の先頭アドレスもnullになっているのかなぁと思ったのですがやっぱりそういうものなのでしょうか？

Answer 1

"pointがnull"と"pointの配列の中身がnull"は意味が違います。
前者はいわば「配列の箱がない状態」、後者は「箱はあるが、中身が空の状態」です。
それぞれ、先頭アドレスがnullと、先頭アドレスはあるがそのアドレスが持つアドレスがnullという状態に該当します。ここを混同しないようにしてください。

Comments

[m_s]

解決のきっかけになりました！
ありがとうございました

Answer 2

pointがnullだったのは point = new Double[POINT_SIZE]; をMainコンストラクタで呼び出した直後です eclipseのデバックのウォッチ機能でpointを確認した所配列pointの各要素が全てnullになっていました。 きちんと配列pointの宣言を行っているので、なぜpointがnullになっているのか分からないです。

配列pointはできたが、配列の中身が無い状態です。配列pointの各要素がnullになるのは正しいです。

point[0] = new Double(x0, y0);
point[1] = new Double(x1, y1);
// …

みたいに各要素に代入していく必要があります。

Comments

[m_s]

pointの各要素にnew Doubleしたらエラー回避できました！
ありがとうございます

[MIURA_Yasuyuki]

おめでとうございます！

Answer 3

NullPointerExceptionが発生する原因としては、
pointの配列がnull、または、point[0]のDouble型が未定義のいずれかが考えられます。
おそらく、後者ではないでしょうか。

pointの変数を初期化するタイミングで、配列の参照先も初期化してはいかがでしょうか。
具体的には、以下の様に0で初期化してみてください。
point = new Double[POINT_SIZE];
Arrays.fill(point, 0d);

また、コンストラクタ内でstaticの変数を初期化しています。

おそらく、クラスの意図としてはインスタンスごとに使用すると推測しますので、
staticは外して、コンストラクタ呼出し後に変数に格納するのがよいです。
Main main = new Main();

staticに使用したい場合、以下の様に記載して、
コンストラクタでは初期化しないのがよいです。
コンストラクタはインスタンスを生成するために使用する機能です。

final int POINT_SIZE = 10;
public static final Double[] point = new Double[POINT_ZIZE];
static {
Arrays.fill(point, 0d);
}

Comments

[m_s]

解決しました！ありがとうございました！