質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.12%

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

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 1,386

m_s

score 39

したいこと

先ほど質問させていただいたもの(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型の"配列”にしたことが悪かったのでしょうか?

解答お願いします

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • MIURA_Yasuyuki

    2016/06/19 02:09

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

    キャンセル

  • swordone

    2016/06/19 09:52

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

    キャンセル

  • m_s

    2016/06/19 10:09

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

    キャンセル

  • m_s

    2016/06/19 10:14

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

    キャンセル

回答 3

checkベストアンサー

+1

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);
// …

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/06/21 13:11

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

    キャンセル

  • 2016/06/22 00:42

    おめでとうございます!

    キャンセル

+1

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/06/21 13:10

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

    キャンセル

0

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);
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/06/21 13:09

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.12%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる