boolean型変数にsetterメソッドで値を入れようとするとNullPointerが発生する

Question

お世話になっております。Screenクラスの２６行目
player1.setOrder(false)
にjava.lang.NullPointerExceptionが発生し対処に困っております。Player1,2の宣言場所を場所を変えたりといろいろしてみたのですが、なぜ発生したのかわかりません。何が原因なのでしょうか？

package スタート画面;

import 省略

public class Screen extends JPanel {
	public static CardLayout layout;
	public static JPanel jp;
	public static Player player1,player2 = new Player();
	private Random rand  = new  Random();
	public Screen() throws IOException {
		//先攻後攻決める
		int order = rand.nextInt(2);
		if(order < 1){
			player1.setOrder(true);
			player2.setOrder(false);
		}else {
			player1.setOrder(false);
			player2.setOrder(true);
		}

		//親パネル（このクラスのインスタンス）
		jp = new JPanel();
		jp.setBounds(135, 120, 930, 700);
		layout = new CardLayout();
		jp.setLayout(layout);

		//子パネル作成(別クラス　JPanel継承) 
		EnterPlayer pane1 = new EnterPlayer();
		GameScreen pane2 = new GameScreen(15,15,1);
		ResultScreen pane3 = new ResultScreen();

		jp.add(pane1);
		jp.add(pane2);
		jp.add(pane3);

		jp.setOpaque(false);
	}
}

package enterPlayer;

import java.awt.Color;
import java.awt.image.BufferedImage;

public class Player {
	private Color c;
	private String name;
	private boolean order;
	private BufferedImage color;
	public BufferedImage getImage(){
		return color;
	}
	public Color getColor() {
		return c;
	}
	public void setColor(Color c,BufferedImage color) {
		this.c = c;
		this.color = color;
	}
	public String getName() {
		return name;
	}
	public void setName(String name) {
		this.name = name;
	}
	public boolean getOrder() {
		return order;
	}
	public void setOrder(boolean order) {
			this.order = order;
	}
}

Answer 1

player1はpublic static Player player1,player2 = new Player();として宣言されていますが、ここでplayer2にはnew Player()が代入されるのに対して、player1はnullのままです。

Comments

[tarutarupop]

そうだったのですね...。このような書き方をしていたサンプルコードを見たことがあるのですが、このように書くメリットはあるのでしょうか？

Answer 2

player1のインスタンスが生成されていないからです。

public static Player player1 = new Player();
public static Player player2 = new Player();

Answer 3

// Groovyのソースです。
// 次のように書くと、初期化されるのは後ろの変数だけ
int a, b, c = 2;
System.out.println(a + ", " + b + ", " + c);
// 0, 0, 2

Object obja, objb, objc = new Object();
println(obja.toString() + ", " + objb.toString() + ", " + objc.toString());
// null, null, java.lang.Object@hoge

// 宣言と同時に、全部同じ値で初期化したければ","じゃなくて"="を使う
int d = e = f = 2;
System.out.println(d + ", " + e + ", " + f);
// 2, 2, 2

Object objd = obje = objf = new Object();
println(objd.toString() + ", " + obje.toString() + ", " + objf.toString());
// java.lang.Object@huge, java.lang.Object@huge, java.lang.Object@huge

// これで出来るけど、、、

変数は個別に宣言して、個別に初期化してください。
（大体のコーディング規約で禁止されています。Googleとか）
Google Java Style Guide - 4.8.2 Variable declarations