NullPointerException エラーの対処方法

Question

コード
public class Part01 extends Frame{

	/**
	 * メインクラス
	 */
	public static void main(String args[]){
		new Part01();
	}

	/**
	 * コンストラクタ
	 */
	public Part01(){

		/**フレームの初期化*/
		super("Shooting Game");			//タイトル設定
		setBounds(0, 0, 500, 500);		//位置、サイズ設定
		setVisible(true);				//表示

		addWindowListener(new WindowAdapter(){
            public void windowClosing(WindowEvent evt){
                System.exit(0);	
            }
        });
		
		/**キャンバスの生成*/
		MyCanvas myCanvas = new MyCanvas();
		add(myCanvas);					//フレームにキャンバスを追加
		
		//スレッドを作成
		myCanvas.initThread();
	}
}



public class MyCanvas extends Canvas implements Runnable{

	Image imageBuffer;
	Graphics graphicsBuffer;

    /**
	 * コンストラクタ
	 */
	public MyCanvas(){
	}

	/**
	 * 描画処理
	 */
	@Override
	public void paint(Graphics g){
		g.drawImage(imageBuffer, 0, 0, this);
	}

	/**
	 * メインループ
	 */
	@Override
	public void run(){
		//オフスクリーンバッファ作成
		imageBuffer = createImage(500, 500);
		graphicsBuffer = imageBuffer.getGraphics();


		graphicsBuffer.setColor(Color.white);
		graphicsBuffer.fillRect(0, 0, 500, 500);
	}

	/**
	 * スレッドを初期化する。
	 */
	public void initThread()
	{
		Thread thread = new Thread(this);
		thread.start();
	}
}

コード
public class Part01 extends Frame{

	/**
	 * メインクラス
	 */
	public static void main(String args[]){
		new Part01();
	}

	/**
	 * コンストラクタ
	 */
	public Part01(){

		/**フレームの初期化*/
		super("Shooting Game");			//タイトル設定
		setBounds(0, 0, 500, 500);		//位置、サイズ設定
		setVisible(true);				//表示設定

		addWindowListener(new WindowAdapter(){
            public void windowClosing(WindowEvent evt){
                System.exit(0);	
            }
        });

		/**キャンバスの生成*/
		MyCanvas myCanvas = new MyCanvas();
		add(myCanvas);					//フレームにキャンバスを追加
	}

}



public class MyCanvas extends Canvas implements Runnable{


	Image imageBuffer;
	Graphics graphicsBuffer;



	/**
	 * コンストラクタ
	 */
	public MyCanvas(){
		Thread thread = new Thread(this);
		thread.start();
	}

	/**
	 * 描画処理
	 */
	@Override
	public void paint(Graphics g){
		g.drawImage(imageBuffer, 0, 0, this);
	}

	/**
	 * メインループ
	 */
	@Override
	public void run(){
		//オフスクリーンバッファ作成
		imageBuffer = createImage(500, 500);
		graphicsBuffer = imageBuffer.getGraphics();


		graphicsBuffer.setColor(Color.red);
		graphicsBuffer.fillRect(0, 0, 400, 500);	
	}

}

1つ目と2つ目はどちらもフレームにキャンバスを貼り付けて、そこに四角形を描いたプログラムになっています。
違いは1つ目はPart01クラスのコンストラクタでスレッドを呼び出しているのですが2つ目ではMyCanvasクラスのコンストラクタで呼び出している点です。
どっちでも特に変わりはないと思ったのですが、2つ目のほうでは実行するとエラーが出ました。
エラー内容
java.lang.NullPointerException
エラー箇所
graphicsBuffer = imageBuffer.getGraphics();

メソッドで呼び出しているかコンストラクタで呼び出しているかの違いでなぜ2つ目のほうはエラーが出てしますのでしょうか？

Answer 1

動作確認まで行っていませんが、

1つ目のコードではadd(myCanvas);が確実に実行された後にgraphicsBuffer = imageBuffer.getGraphics();が行われます。

2つ目のコードでは、add(myCanvas);とgraphicsBuffer = imageBuffer.getGraphics();はどちらが先に動くか、状況により変動します。

graphicsBuffer = imageBuffer.getGraphics();が、FrameにCanvasがaddされていることを必要条件とするならば、2つ目のコードにエラーが発生する可能性が生まれてきます。

Comments

[nakamura-]

回答ありがとうございます！
addを先にしとかないとダメでした！
ありがとうございました。

Answer 2

処理の順番が違います。
１つ目はaddしてからスレッドが動き、createImageが呼ばれますが、２つ目はスレッドが動き、creteImageを呼び、addしています。
ここからcreateImageはaddされていない場合はnullを返すように実装されていると推測できます。

Comments

[nakamura-]

回答ありがとうございます！
add先にしないとダメでした！
ありがとうございました。