for文の中で変数pは初期化されていない可能性がありますとなってしまう

Question

###実現したいこと
Boardの中でStoneの中のものを呼び出す。その際に中心の座標であるp.xとp.yを代入していく。

###問題点
下記のようなエラーが出てきます。for文の中で一回一回初期化しなければならないのでしょうか。またそのようであればどのように初期化すればよいのでしょうか？

Reversi.java:87: エラー: 変数pは初期化されていない可能性があります
                p.x=px; //中心座標を定義
                ^
エラー1個

###コード

import java.awt.*;
import javax.swing.*;

//石に対応するクラス
class Stone{
    public final static int black=1; //黒
    public final static int white=2; //白
    private int observe; //表面の色
    //コンストラクタ
    Stone(){
        observe=0; //非配置にする
    }

    //表面の色を設定する
    void setObserve(int color){
        if(color==black || color==white){
            observe=color;
        }else{
            System.out.println("黒か白でなければいけません");
        }
    }

    //表面の色で中心p,半径radの円を塗りつぶす
    void paint(Graphics g,Point p,int rad){
        /*省略*/

    }
}

class Board{
    Stone[][] stone=new Stone[8][];

    //コンストラクタ
    Board(){
    }

    //画面描画
    void paint(Graphics g,int unit_size){
        int rad=32;

        /*省略*/

        int px=120,py=120;
        for(int i=0;i<8;i++){
            for(int j=0;j<8;j++){
                stone[i][j].setObserve(0);
            }
        }
        stone[3][3].setObserve(1);
        stone[3][4].setObserve(2);
        stone[4][3].setObserve(2);
        stone[4][4].setObserve(1);
        for(int i=0;i<8;i++){
            for(int j=0;j<8;j++){
                Point p;
                p.x=px; //中心座標を定義
                p.y=py;
                stone[i][j].paint(g,p,rad);
                py=py+80;
            }
            px=px+80;
        }
    }
}


public class Reversi extends JPanel{
    public final static int UNIT_SIZE=80;
    Board board =new Board();

    //コンストラクタ
    public Reversi(){
        setPreferredSize(new Dimension(800,800));
    }

    //画面描画
    public void paintComponent(Graphics g){
        board.paint(g, UNIT_SIZE); //Boardクラスのpaintメソッドを呼び出す
    }

    //起動
    public static void main(String[] args){
        JFrame f=new JFrame();
        f.getContentPane().setLayout(new FlowLayout());
        f.getContentPane().add(new Reversi());
        f.pack();
        f.setResizable(false);
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        f.setVisible(true);
    }


}

Comments

[jimbe]

全然ご質問と関係無いですが、 UNIT_SIZE が全く使われてないですね…。

Answer 1

Point は、 JFrame や Reversi 、 Board 等と同じくクラスです。
ですので、それらと同じように new でインスタンスを設定しなければ NullPointerException が発生するでしょう。

Comments

[langhtorn]

ありがとうございます。

[maisumakun]

ローカル変数については初期化しないパスがあると「コンパイルエラー」です（nullのない数値型でもそうです）。

Answer 2

Pointクラスはjava.awt.Pointでしょうか？
そうだと仮定して回答します。
以下のようにする必要があります。

        for(int i=0;i<8;i++){
            for(int j=0;j<8;j++){
-               Point p;
+               Point p = new Point();
                p.x=px; //中心座標を定義
                p.y=py;
                stone[i][j].paint(g,p,rad);
                py=py+80;
            }
            px=px+80;
        }

もっとも、これならこうしたほうがいいですが。

        for(int i=0;i<8;i++){
            for(int j=0;j<8;j++){
-               Point p;
-               p.x=px; //中心座標を定義
-               p.y=py;
+               Point p = new Point(px, py);
                stone[i][j].paint(g,p,rad);
                py=py+80;
            }
            px=px+80;
        }

Comments

[langhtorn]

ありがとうございます。