初期化しているにもかかわらずNullPointerExceptionが発生する

Question

お世話になっております。

javaのswingを学ぶため四目並べゲームのプログラムを作っていたのですが、java.lang.NullPointerExceptionのエラーが発生し解消に難航しています。

二つあり一つ目が、GameScreen コンストラクタ内のsetTelllimittext(system.getSecond());
Gamesystemにあるint secを参照するところ。

二つ目が、GameSystem　92行目screen.setTelllimittext(sec);
表示する秒数を更新するため、Gamescreenのtelllimitのテキストを変更する関数にsecを渡すところ。

どちらもsecを参照しているためこれがnullになっているのはわかるのですが、それがなぜなのかがわかりません。secの宣言時に値を入れてみたり宣言する場所を変えてみたりしたのですが解決できませんでした。初歩的な見落としのような気もしますが、どうかご指摘のほどお願いいたします。

public class GameScreen extends JPanel implements ActionListener{
	private int cn;		//升目の数　cn * cn
	private boolean press = true;
	private JPanel backpane;
	private JLabel telllimit;
	private GameSystem system;
	private JLabel tellcolor;
	private Color player_c;
	static  JLabel timelimit;
	static boolean play_turn;				//現在のターンを保持
	static boolean[] enter ;	//すべての入力を保持
	static ArrayList<ArrayList<JButton>> y_coordinate = new ArrayList<ArrayList<JButton>>();

	public static void main(String args[]){
		JFrame jf = new JFrame();
		GameScreen gs = new GameScreen(15);
		jf.add(gs);
		jf.pack();
		jf.setVisible(true);
		//ゲーム開始
		new GameSystem(15);
	}

	public GameScreen(int n){
		/*
		 * 画面生成
		 */
			//親パネル
		super();
		this.setPreferredSize(new Dimension(930,700));
		this.setLayout(null);
		play_turn = true;	//初期値（初期ターン）

			//ターン表示のためのパネル
		tellcolor = new JLabel();
		tellcolor.setBackground(getPlayer_c());
		tellcolor.setBounds(0, 0, 200, 200);
		this.add(tellcolor);

			//残り時間表示のためのパネル
		telllimit = new JLabel();
		telllimit.setBorder(new BevelBorder(BevelBorder.RAISED, Color.blue, Color.black));
		telllimit.setBounds(200, 0, 430, 200);
		telllimit.setForeground(Color.RED);
		setTelllimittext(system.getSecond());
		this.add(telllimit);

		cn = n;
		enter = new boolean[cn*cn];

			//升目パネル
		JPanel screen = new JPanel();
		screen.setBounds(100,220,720,450);
		screen.setLayout(new GridLayout(n,n,0,0));
		for(int i = 0;i<cn;i++){
			ArrayList<JButton> x_coordinate = new ArrayList<JButton>();
			for(int t = 0;t<cn;t++){
				JButton cell = CreateCell(i,t);
				cell.addActionListener(this);
				screen.add(cell);
				x_coordinate.add(cell);
			}
			y_coordinate.add(x_coordinate);
		}
		this.add(screen);
	}

	public JButton CreateCell(int x,int y){
		JButton btn = new JButton();
		btn.setSize(10, 10);
		btn.setActionCommand(x+"-"+y);
		return btn;
	}

	public void setTelllimittext(int second){
		String s= "残り時間　"+ second +" 秒";
		telllimit.setText(s);
	}

	@Override
	public void actionPerformed(ActionEvent e) {
		if(press == true){

			press = false;
		}

		String coordinate = e.getActionCommand();
		for(int k=0;k<cn;k++){
			for(int j = 0;j<cn;j++){
				String number = k+"-"+j;
				if(coordinate.equals(number)){
					int order = k*20 + j+1;	//入力の番号を作成
					if(play_turn == true){
						player_c = Color.RED;
						system.addA(order);
					}else{
						player_c = Color.BLUE;
						system.addB(order);
					}
					changeTurn();
					if(k == 19){					//入力が底の時
						processEnter(order,k,j,player_c);
					}else{							//入力が上部の時
						if(enter[order+20] == false){
							System.out.println("無効な入力です");
						}else{
							processEnter(order,k,j,player_c);
						}
					}
					system.createCombinations();
				}
			}
		}
		return;
	}

	public Color getPlayer_c(){
		return player_c;
	}

	public void changeTurn(){
		if(play_turn == true){
			player_c = Color.RED;
			play_turn = false;
		}else{
			player_c = Color.BLUE;
			play_turn = true;
		}
		tellcolor.setBackground(getPlayer_c());
	}

	/*	色塗り処理	*/
	public void processEnter(int order,int k,int j,Color player_c){
		/* 過去に入力があればtrueなければfalse */
		if( enter[order] == true){
			System.out.println("既に入力されています");
			return;
		}else{
			y_coordinate.get(k).get(j).setBackground(player_c);
			enter[order] = true;
		}
	}
}

public class GameSystem implements ActionListener{
	private int limit;
	private Timer timer;
	private int sec = 0;
	private Combinations<Integer> c ;
	private GameScreen screen ;
	//private Screen sc;
	private ArrayList<Integer> number_a = new ArrayList<Integer>();	//player1の入力
	private ArrayList<Integer> number_b = new ArrayList<Integer>();	//player2の入力
	private boolean finish = false;


	public GameSystem(int l){
		limit = l;
		timer = new Timer(limit*1000,this);
		timer.start();
	}

	public void createCombinations(){
		if(GameScreen.play_turn == !true){
			if(number_a.size() > 3){
				c = new Combinations<Integer>(number_a,4);
				judgeWinner();
			}else{
				return;
			}
		}else{
			if(number_b.size() > 3){
				c = new Combinations<Integer>(number_b,4);
				judgeWinner();
			}else{
				return;
			}
		}

	}
	/*	勝者判定	*/
	public void judgeWinner(){
		Iterator<List<Integer>> i = c.iterator();
		while(i.hasNext()){
            List<Integer> t = i.next();
            Collections.sort(t);			//小さい順に並び替え
            Collections.reverse(t);		//大きい順へ
            if(t.get(0)-t.get(3)==3 && t.get(0)-t.get(1)==1	||
               t.get(0)-t.get(1)==20 && t.get(1)-t.get(2)==20 && t.get(2)-t.get(3)==20 ||
               t.get(0)-t.get(1)==21 && t.get(1)-t.get(2)==21 && t.get(2)-t.get(3)==21 ||
               t.get(0)-t.get(1)==19 && t.get(1)-t.get(2)==19 && t.get(2)-t.get(3)==19 ){
            	finish = true;
            	System.out.println("ゲーム終了");
            }
		}
	}
	public void addA(int n){
		number_a.add(n);
	}

	public void addB(int n){
		number_b.add(n);
	}

	public int getSecond(){
		return sec;
	}

	@Override
	public void actionPerformed(ActionEvent e) {
		if(finish == true){
			timer.stop();
			/*後々
			e.setSource(this);
			sc.actionPerformed(e);
			*/
		}
	    if (sec == limit){
	         sec = 0;
	        screen.changeTurn();
	    }else{
	 		screen.setTelllimittext(sec);
	        sec++;
	    }
	}
}

Answer 1

public class GameScreen extends JPanel implements ActionListener{
    private int cn;        //升目の数　cn * cn
    private boolean press = true;
    private JPanel backpane;
    private JLabel telllimit;
    private GameSystem system;

public class GameSystem implements ActionListener{
    private int limit;
    private Timer timer;
    private int sec = 0;
    private Combinations<Integer> c ;
    private GameScreen screen ;

「GameScreen has GameSystem」「GameSystem has GameScreen」の関係で循環参照してしまっているので、まず構造を変えたほうがいいのでは？

Comments

[tkturbo]

たぶん循環参照による再帰的なコンストラクタ呼び出しがStackOverflowErrorの原因と思われ。

Answer 2

どちらもsecを参照しているためこれがnullになっているのはわかるのですが

残念ながら、この認識自体が違います。

1つ目は、フィールドとしてprivate GameSystem system;となっていますが、コンストラクタ内で代入がまったくないため、setTelllimittext(system.getSecond());のところでもsystem == nullとなっています。

2つ目の方でも同じくprivate GameScreen screen;へのオブジェクト代入がないため、これがnullのままです。

Comments

[tarutarupop]

一つ目はGameScreenのコンストラクタでSystemを初期化することで無事解決できました。
しかし、二つ目の解決法がわかりません。具体的にはscreenをインスタンス化するタイミングがわかりません。GameSystemのほうでGameScreenのインスタンス化を行おうとするとjava.lang.StackOverflowErrorというエラーが発生しました。GameScreenから呼び出されたGameSystemのほうで新たにGameScreenをインスタンス化したため無限ループが発生したのかなと予想しているのですが...これはどのように解決すればよいのでしょう？