NullPointerExceptionが発生する

Question

現在、ゲームを作っているのですが、NullPointerExceptionがクラスの配列で発生してしまいます。ソースは

import java.awt.event.*;

import src.yukkuri.lib.*;
import src.yukkuri.lib.Game.*;

enum Battle
{
	SETUP,		//	敵出現のメッセージ
	MAIN,		//	コマンド選択中
	P_ACTION,	//	プレイヤーの行動
	M_ACTION,	//	モンスターの行動
	RESET,		//	MAINへのリターン
	RESULT;		//	アイテム入手、経験値、お金入手
}

public class debug2 extends BaseGame
{
	static boolean debug = true;

	static MapImageChip	m_mic;
/*	newSprite m_player;
	newSprite player_up;		//	angle:1
	newSprite player_right;		//	angle:2
	newSprite player_left;		//	angle:3
	newSprite player_down;		//	angle:4*/
	newSprite[] m_player = new newSprite[ 5 ];		//	上右下左現在
	int angle = 4;
	boolean upMenu = false;
	boolean upMove = true;

	GameLib	gl;
	Sprite boss_one;
	int boss_one_hpmax = 1000;
	int boss_one_hpmin = 1000;

	Sprite boss_two;
	int boss_two_hpmax = 10000;
	int boss_two_hpmin = 10000;
	String message = "";

	static int l = 0;

	player p1;

	public static void main( String[] args )
	{
		new debug2();
	}

	public debug2()
	{
		super( "てすとげーむ", 1920, 1088, false );
	}

	@Override public void Create()
	{
		gl = new GameLib();

		setBackground( 0x000000 );
		try{
			//	マップ用
/*			player_up = new newSprite( ".\image\char_p_hero01.amg", 0, 9, 48, 64 );
			player_right = new newSprite( ".\image\char_p_hero01.amg", 0, 73, 48, 64 );
			player_down = new newSprite( ".\image\char_p_hero01.amg", 0, 137, 48, 64 );
			player_left = new newSprite( ".\image\char_p_hero01.amg", 0, 201, 48, 64 );
			m_player = new newSprite( ".\image\char_p_hero01.amg", 0, 137, 48, 64 );*/
			for( int l = 3; l > -1; l-- ){
				System.out.println( l );
				m_player[ l ] = new newSprite( ".\image\char_p_hero01.amg", 0, 9 + l * 64, 48, 64 );
			}
			m_player[ 0 ] = new newSprite( ".\image\char_p_hero01.amg", 0, 9 + 0 * 64, 48, 64 );
			System.out.println( "抜けたよ" );
			m_player[ 4 ] = m_player[ 2 ];	//	初期状態を下向きに設定
			m_mic = new MapImageChip( ".\image\chip02d_dungeon.amg", 64, 64 );

			//	戦闘用
			boss_one = new Sprite( ".\image\human232_l.amg", 0, 0, 695, 486 );
			boss_two = new Sprite( ".\image\monster171_l.amg", 0, 0, 683, 504 );
			p1 = new player( ".\bgm.se\nc98796.wav", 0 );
		}catch( java.io.IOException ie ){ ie.printStackTrace(); } 

		m_mic.m_map = new int[]{
			//			5			  10			 15				20			   25			  30
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,	//	5
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,	//	13
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,	//	15
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
			3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3,
		};
	}

	@Override public void Key_Press( int code )
	{
		if( debug ){ System.out.println( "キー" + code + "が押されたよ！" ); }
		if( code == KeyEvent.VK_D ){
			if( debug ){
				System.out.println( "デバッグをオフにしました" );
				debug = false;
			}else{
				System.out.println( "デバッグをオンにしました" );
				debug = true;
			}
		}
		if( code == KeyEvent.VK_X ){
			if( upMenu ){
				upMenu = false; upMove = true;
			}else{
				upMenu = true; upMove = false;
			}
		}if( code == KeyEvent.VK_ENTER )boss_one_hpmin -= 10;
		if( upMove ){
			m_player[ 4 ].move( code );
			if( code == KeyEvent.VK_UP )    angle = 1;
			if( code == KeyEvent.VK_RIGHT ) angle = 2;
			if( code == KeyEvent.VK_LEFT )  angle = 3;
			if( code == KeyEvent.VK_DOWN )  angle = 4;
		}
		if( debug ) System.out.println( "X:" + m_player[ 4 ].m_x + " Y:" + m_player[ 4 ].m_y );
		repaint();
	}

	@Override public void Paint()
	{
/*		setColor( 0xffff255 );
		if( boss_one_hpmin < 1 ){
			if( l == 0 ){
				System.out.println( "音楽再生" );
				p1.playMusic();
				l++;
			}
			message = "この程度で負けると思うなよ！敵が復活してしまった！";
			draw( boss_two, 625, 65 );
			fillRect( 10, 20, 19 * gl.per( boss_two_hpmax, boss_two_hpmin ), 25 );
		}else{
			message = "ボスが現れた！";
			draw( boss_one, 625, 65 );
			fillRect( 10, 20, 19 * gl.per( boss_one_hpmax, boss_one_hpmin ), 25 );
		}
		setColor( 0xffffff );
		draw( 0, 600, 1920, 600 );
		drawRect( 10, 20, 1900, 25 );
		setFont( 60 );
		draw( message, 50, 700 );
		String[] command = { "攻撃", "防御", "移動", "魔法", "アイテム", "逃げる" };
		for( int l = 0; l < 6; l++ ){
			draw( command[ l ], 1600, 700 + 60 * l );
		}*/
		m_mic.putMapCip( this, 0, 0, 30, 17 );
		
		if( angle == 1 ) draw( m_player[ 0 ], m_player[ 4 ].m_x, m_player[ 4 ].m_y );
		if( angle == 2 ) draw( m_player[ 1 ], m_player[ 4 ].m_x, m_player[ 4 ].m_y );
		if( angle == 3 ) draw( m_player[ 2 ], m_player[ 4 ].m_x, m_player[ 4 ].m_y );
		if( angle == 4 ) draw( m_player[ 3 ], m_player[ 4 ].m_x, m_player[ 4 ].m_y );
		if( upMenu ){
			setColor( 0x33000000, true );
			fillRect( 0, 0, 600, 1080 );
		}
	}
}

class newSprite extends Sprite
{
	public newSprite( String fname, int x, int y, int w, int h ) throws java.io.IOException
	{
		super( fname, x, y, w, h );
	}

	public void move( int code )
	{
		if( code == KeyEvent.VK_RIGHT ){
			m_x += 48;
		}if( code == KeyEvent.VK_LEFT ){
			m_x -= 48;
		}if( code == KeyEvent.VK_UP ){
			m_y -= 32;
		}if( code == KeyEvent.VK_DOWN ){
			m_y += 32;
		}
	}
}

この中の、エラー個所は

for( int l = 3; l > -1; l-- ){
				System.out.println( l );
				m_player[ l ] = new newSprite( ".\image\char_p_hero01.amg", 0, 9 + l * 64, 48, 64 );
			}

です。よろしくお願いします。

追記
スタックトレース
Exception in thread "main" java.lang.NullPointerException at debug2.Create(debug2.java:69) at src.yukkuri.lib.Game.BaseGame.<init>(BaseGame.java:29) at debug2.<init>(debug2.java:52) at debug2.main(debug2.java:47)

javaのバージョンは7です。

Comments

[umyu]

NullPointerExceptionが発生時のスタックトレースをcodeタグで囲ってそのまま貼ってくださいな。

Answer 1

スタックトレースの追記ありがとうございました。

スタックトレースのdebug2.Create(debug2.java:69)が質問文のコードを見る限り以下の行です、

m_player[ l ] = new newSprite( ".\image\char_p_hero01.amg", 0, 9 + l * 64, 48, 64 );

Spriteクラスの定義が質問文に無いので確実な事はいえませんが、
上記行でNullPointerExceptionが発生しえるのは変数:m_playerです、
以下のようにfinal宣言に変更して再代入を抑止しても上記問題が発生するか確認して頂けませんか？

-newSprite[] m_player = new newSprite[ 5 ];
+private final newSprite[] m_player = new newSprite[5];

---

無理でした。

at debug2.Create(debug2.java:69)
src.yukkuri.lib.Game.BaseGame.<init>(BaseGame.java:29) // ←　BaseGameのコンストラクタ<init>
at debug2.<init>(debug2.java:52)

スタックトレースを見るとBaseGameクラスのコンストラクタからBaseGameのCreateを呼び出していませんか。

---

BaseGameクラスが気になりますが。
継承したクラスでのcreateの呼び出しをprintlnしてみましたこのような出力になります。

■対策としては
案1,コンストラクタでオーバーライド対象のメソッドのcreate()の呼び出しをやめる。
コンストラクタの可視性の関係上、イベントリスナーの登録、スレッドの起動も避けたほうがよいです。
（私もサンプルコードではよく書きますが）

以下からは改造の手間がかかります。
案2, BaseGameが何を目的のクラスなのかで変わりますが。
BaseGameを継承するのをやめて、コンポジット(委譲）にする。
debug2クラスはBaseGameクラスのフィールドを持ち、baseGame.create();などを必要な都度呼び出す。
private BaseGame baseGame = new BaseGame();

案3, BaseGameはクラスではなくinterfaceで抽象メソッドの定義とdefalutメソッドにする。

個人的なオススメは案2でしょうか。
BaseGameというクラス名はいろいろな処理がありそうなクラス名に見えるので、
内部の処理をクラス分割することをお勧めします。

---

追記したBaseGameを見ました、率直な感想は継承する意味があまりないような。
クラスを導入する事で解消される問題もあまりなさそうに見えますし。
どちらかというと、PropertiesクラスもしくはJSON形式のファイルを使って
今、プログラムに埋め込んでいるMAP情報や画像のパスなどを外部化した方が管理しやすいかもです。

Comments

[yukkuri]

無理でした。
後、Sprite.javaのソースはこれです。

package src.yukkuri.lib.Game;

import java.awt.*;
import java.awt.image.*;
import java.io.*;
import javax.imageio.*;

public class Sprite
{
public BufferedImage m_bi;
public int m_x, m_y;

public Sprite( BufferedImage bi )
{
m_bi = bi;
}

public Sprite( BufferedImage bi, int x, int y, int w, int h )
{
m_bi = bi.getSubimage( x, y, w, h );
}

public Sprite( File file, int x, int y, int w, int h ) throws IOException
{
BufferedImage bi = ImageIO.read( file );
m_bi = bi.getSubimage( x, y, w, h );
}

public Sprite( String fname, int x, int y, int w, int h ) throws IOException
{
BufferedImage bi = ImageIO.read( new File( fname ) );
m_bi = bi.getSubimage( x, y, w, h );
}

public void draw( Graphics g )
{
g.drawImage( m_bi, m_x, m_y, null );
}

public void draw( Graphics g, int x, int y )
{
g.drawImage( m_bi, x, y, null );
}
}

(直貼りですいません、マークダウン方式がよくわからなくて・・・)

[yukkuri]

そうですね。
BaseGameクラスでFrameを作成して、
そこでCreateを呼び出して、Overrideでこっちが呼び出されて・・・
というイメージです。ただ、前も上の処理でできていたので、あまり関係ないかも・・・

[umyu]

@yukkuriさんへ
前の処理は配列じゃないのではないでしょうか？
player_upとか別の変数で宣言してますよね。
えっとですね、コンストラクタでインスタンスメソッドの呼び出しは避けたほうがよいというのがありまして。
http://irof.hateblo.jp/entry/2016/01/09/231631

[yukkuri]

いまリンクのサイトを拝見しまして、なんとなく理解できました。
umyuさんの言うとおり、前は配列ではなく、変数が5つでした。
それをまとめようとしたら、今のエラーというわけです。

[umyu]

あ、ちょっと貼り直します。
https://wandbox.org/permlink/E9iNepepBnaan4Nk

[umyu]

Subのコンストラクタにprint文を追加するとわかりやすいかもです。

[yukkuri]

そういうことですか！
そしたら、どうしたらいいんでしょうか？
ご教授お願いします。

[yukkuri]

一応BaseGameのソースを貼りますね。
package src.yukkuri.lib.Game;

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

/**
* JFrameのラッパークラスです。Gameの基本を作成します。
*
* @author yukkuriP
*/
public class BaseGame extends JFrame
{
public BaseGraphic m_bgra;

private BaseGame( String title, boolean resize )
{
m_bgra = new BaseGraphic( this );
setDefaultCloseOperation( EXIT_ON_CLOSE );
setTitle( title );
setResizable( resize );
}

public BaseGame( String title, int w, int h, boolean resize )
{
this( title, resize );
getContentPane().setPreferredSize( new Dimension( w, h ) );
pack();
getContentPane().add( m_bgra );
Create();
setVisible( true );
}

public BaseGame( String title, int x, int y, int w, int h, boolean resize )
{
this( title, resize );
setBounds( x, y, w, h );
getContentPane().setPreferredSize( new Dimension( w, h ) );
pack();
getContentPane().add( m_bgra );
Create();
setVisible( true );
}

public void clearRect( int x, int y, int width, int height )
{
m_bgra.clearRect( x, y, width, height );
}

public void draw( Sprite s )
{
m_bgra.draw( s );
}

public void draw( String str, int x, int y )
{
m_bgra.draw( str, x, y );
}

public void draw( Sprite s, int x, int y )
{
m_bgra.draw( s, x, y );
}

public void draw( int x1, int y1, int x2, int y2 )
{
m_bgra.draw( x1, y1, x2, y2 );
}

public void drawRect( int x, int y, int width, int height )
{
m_bgra.drawRect( x, y, width, height );
}

public void fillRect( int x, int y, int width, int height )
{
m_bgra.fillRect( x, y, width, height );
}

public void setBackground( int rgb )
{
m_bgra.setBackground( rgb );
}

public void setColor( int rgb )
{
m_bgra.setColor( rgb );
}

public void setColor( int argb, boolean hasalpha )
{
m_bgra.setColor( argb, hasalpha );
}

public void setFont( int size )
{
m_bgra.setFont( size );
}

public void setFontGC( Font f )
{
m_bgra.setFontGC( f );
}

public void Create(){}

public void Key_Press( int code ){}

public void Key_Release( int code ){}

public void Paint(){}
}

[yukkuri]

思ったのですが、BaseGameは継承元の予定で最初から作っていたので、内部処理は全くありません。汎用的なクラスを目指していたのですが、どうしたらいいでしょうか。

[umyu]

汎用的なクラスとはどういうニュアンスのクラスでしょうか。何かの目的のためにBaseGameと質問文のプログラムを作られたのですよね？

[m.ts10806]

横から失礼します。
コメント欄ではマークダウンが使えないので質問に追記されては？
文字制限にひっかかるようでしたら関係なさそうな部分を削るなど工夫が必要ですが。

[yukkuri]

汎用的なクラスというのは、継承して、その人が実装したいシステムなどを追加できる(というイメージの)クラスのことですね。

[umyu]

@mts10806さんへ
指摘ありがとうございます、そうですね。この回答欄に記載していただいても、他の回答者の方の目が届かないので、質問文を編集して頂けませんか？＞@yukkuriさんへ
@yukkuriさんへ
そういう感じだとUnityを参考にクラス設計をした方がよいのですが。
一般論として開発者一人だと技術力と時間の関係上作りきれないかと。
https://docs.unity3d.com/ja/current/Manual/ScriptingImportantClasses.html
個人的には画面周り(JFrame/JPanel)は実装するシステムによってかなり変わるので、ゲームで使用するリソースを管理しやすい形に纏める方がいいかもです。
で、質問文の本題であるNullPointerExceptionの件から少し横道にそれて来たので、話題を振っておいてなんですが、対策方法は既に回答に記載していますので、別の質問（ゲームの設計のあり方）みたいなのを立てて頂けるといいかもです。

[yukkuri]

わかりました。ありがとうございました。