javaのまるばつゲームのリセット方法とFontの変換

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
javaで○×ゲームのコードを自分で改造するという課題で以下のように改造したいのですが、できません。
1，ゲーム終了後に盤面をもう一度クリックすると、再度ゲームが始まるようにする
2，"引き分け"などの文字を大きくする

発生している問題・エラーメッセージ

エラーメッセージ

該当のソースコード

クラス1

public class OXmodel {
	private int brd[][];
	private int turn;
	OXmodel() {
		brd = new int[3][3];
		for(int y=0 ; y<3 ; y++) {
			for(int x=0 ; x<3 ; x++) {
				brd[y][x] = 0;
			}
		}
		turn=1;
	}
	public int getCell(int x, int y) { return brd[y][x]; }
	public void put(int x, int y) {
		if (judge() != -1) return;
		if ((0<=x)&&(x<3)&&(0<=y)&&(y<3)&&(brd[y][x]==0)) {
			brd[y][x]=turn;
			turn=3-turn;
		}
	}
	public int judge() {
		for(int i=0 ; i<3 ; i++) {
			if ((brd[i][0]!=0)&&
				(brd[i][0]==brd[i][1])&&(brd[i][1]==brd[i][2])) {
				return brd[i][0];
			}
			if ((brd[0][i]!=0)&&
				(brd[0][i]==brd[1][i])&&(brd[1][i]==brd[2][i])) {
				return brd[0][i];
			}
		}
		if (brd[1][1]!=0) {
			if ((brd[0][0]==brd[1][1])&&(brd[2][2]==brd[1][1])) {
				return brd[1][1];
			}
			if ((brd[2][0]==brd[1][1])&&(brd[0][2]==brd[1][1])) {
				return brd[1][1];
			}
		}
		int c=0;
		for(int y=0 ; y<3 ; y++) {
			for(int x=0 ; x<3 ; x++) {
				if (brd[y][x]==0) { c++; }
			}
		}
		if (c == 0) return 0;
		return -1;
	}

}

クラス2

import java.awt.Canvas;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.Font;

public class OXview extends Canvas{
	private OXmodel model;
	OXview(OXmodel model) {
		setSize(400,400);
		this.model = model;
	}
	@Override
	public void paint(Graphics g) {
		g.setColor(Color.white);
		g.fillRect(0,0,400,400);
		g.setColor(Color.black);
		for(int i=0 ; i<2 ; i++) {
			g.drawLine(20,140+i*120,380,140+i*120);
			g.drawLine(140+i*120,20,140+i*120,380);
		}
		for(int y=0 ; y<3 ; y++) {
			for(int x=0 ; x<3 ; x++) {
				int bx=x*120+20, by=y*120+20;
				int c = model.getCell(x, y);
				if (c == 1) {
					g.setColor(Color.red);
					g.drawOval(bx+20,by+20,80,80);
				} else if (c == 2) {
					g.setColor(Color.blue);
					g.drawLine(bx+20,by+20,bx+100,by+100);
					g.drawLine(bx+20,by+100,bx+100,by+20);
				}
			}
		}
		int j = model.judge();
	
		if (j == 0) {
			g.setColor(Color.black);
			g.drawString("引き分け",100,100);
		} else if (j == 1) {
			g.setColor(Color.black);
			g.drawString("〇の勝ち",100,100);
		} else if (j == 2) {
			g.setColor(Color.black);
			g.drawString("×の勝ち",100,100);
		}
	}
}

クラス3

import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import javax.swing.JFrame;

public class OX implements MouseListener {
	private OXmodel model;
	private OXview view;
	OX() {
		model = new OXmodel();
		view = new OXview(model);
		JFrame frame = new JFrame();
		frame.add(view);
		view.addMouseListener(this);
		frame.pack();
		frame.setVisible(true);
		view.repaint();
	}
	public void mouseClicked(MouseEvent e) {
		int bx=e.getX(),by=e.getY();
		if ((20<=bx)&&(bx<380)&&(20<=by)&&(by<380)) {
			int x=(bx-20)/120, y=(by-20)/120;
			model.put(x,y);
			view.repaint();
		}
	}
	public void mousePressed(MouseEvent e) {}
	public void mouseReleased(MouseEvent e) {}
	public void mouseExited(MouseEvent e) {}
	public void mouseEntered(MouseEvent e) {}

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

試したこと

1についてOXmodelクラスのgetCellメソッドにおいて、
if (judge() != -1) return;
を
if (judge() != -1)
for(int y1=0 ; y1<3 ; y1++) {
for(int x1=0 ; x1<3 ; x1++) {
brd[y1][x1] = 0;
と書いたところ、盤面がりセットされると同時に次のマスが書かれてしまうので、１ターン開けてから始まるようにしたいです。

２についてjava.awt.Font クラスと、GraphicsクラスのsetFontメソッドを使って書きたいのですが、
g.setColor(Color.black);
g.setFont(Font ~);
g.drawString("引き分け",100,100);
と書いたところエラーが出ました。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[jimbe]

コードのマークダウンはファイル毎に分けて頂けますか。コピペの際に修正が必要になり、修正で間違えてしまうと現象が再現できなくなる可能性があります。

> ～ １ターン開けてから ～
「ターン」とは何のことでしょう。

> ～ エラーが出ました。
エラーの詳細をご提示ください。

[jimbe]

Swing(JFrame) と AWT(Canvas) のコンポーネントを混在しての使用は危険ですので避けた方が良いと思います。

[TN8001]

> g.setFont(Font ~);

肝心のところを省略しないでください。

> と書いたところエラーが出ました。

エラーが出たのならエラー文を記載してください。

[BeatStar]

エラーメッセージを読みましょう。
エラーメッセージは罵倒ではなく、「コンパイラ等からのメッセージ」です。

Answer 1

いろいろ弄っていたらこんななりました。

package teratail_java.q370179;

import java.awt.BorderLayout;

import javax.swing.*;

public class OX extends JFrame {
  public static void main(String[] args) {
    new OX().setVisible(true);
  }

  OX() {
    super("OX");
    setDefaultCloseOperation(EXIT_ON_CLOSE);

    JPanel backpanel = new JPanel(new BorderLayout());
    backpanel.setBorder(BorderFactory.createEmptyBorder(20,20,20,20));
    add(backpanel);

    backpanel.add(new OXview(new OXmodel()));

    pack();
  }
}

package teratail_java.q370179;

import java.awt.Color;
import java.awt.Graphics;

enum Cell {
  SPACE("引き分け") {
    void draw(Graphics g, int x, int y, int size) {}
  },
  CIRCLE("〇の勝ち") {
    void draw(Graphics g, int x, int y, int size) {
      g.setColor(Color.RED);
      g.drawOval(x,y,size,size);
    }
  },
  CROSS("×の勝ち"){
    void draw(Graphics g, int x, int y, int size) {
      g.setColor(Color.BLUE);
      g.drawLine(x,y,x+size,y+size);
      g.drawLine(x,y+size,x+size,y);
    }
  };

  private String result;
  Cell(String result) {
    this.result = result;
  }
  abstract void draw(Graphics g, int x, int y, int size);
  Cell next() { return this == CIRCLE ? CROSS : CIRCLE; }
  String getResult() { return result; }
}

package teratail_java.q370179;

import java.awt.*;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;

import javax.swing.*;

public class OXview extends JPanel {
  private static final int SIZE = 120;

  private class OXMouseListener implements MouseListener {
    @Override
    public void mouseClicked(MouseEvent e) {
      //System.out.println("mouseClicked: "+e);
      if(startLabel.isVisible()) {
        start();
        return;
      }

      int x = e.getX() / SIZE;
      int y = e.getY() / SIZE;
      if(model.get(x,y) != Cell.SPACE) return;
      Cell winner = model.set(x,y);
      repaint();

      if(winner != null) end(winner);
    }

    @Override
    public void mousePressed(MouseEvent e) {}
    @Override
    public void mouseReleased(MouseEvent e) {}
    @Override
    public void mouseExited(MouseEvent e) {}
    @Override
    public void mouseEntered(MouseEvent e) {}
  }

  private JLabel startLabel, resultLabel;
  private OXmodel model;

  OXview(OXmodel model) {
    super(null);
    setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));
    setPreferredSize(new Dimension(SIZE*3,SIZE*3));
    setMinimumSize(getPreferredSize());
    setBackground(Color.white);
    addMouseListener(new OXMouseListener());

    this.model = model;

    startLabel = createLabel("CLICK TO (RE)START !", Color.RED, true);
    resultLabel = createLabel("", Color.BLACK, false);

    add(Box.createVerticalGlue());
    add(startLabel);
    add(Box.createVerticalGlue());
    add(resultLabel);
    add(Box.createVerticalGlue());
  }

  private JLabel createLabel(String text, Color color, boolean visible) {
    JLabel label = new JLabel(text);
    label.setMaximumSize(new Dimension(Short.MAX_VALUE, (int)label.getMaximumSize().getHeight()));
    label.setHorizontalAlignment(SwingConstants.CENTER);
    label.setFont(label.getFont().deriveFont(30f));
    label.setForeground(color);
    label.setVisible(visible);
    return label;
  }

  private void start() {
    startLabel.setVisible(false);
    resultLabel.setVisible(false);
    model.clear();
    repaint();
  }

  private void end(Cell winner) {
    startLabel.setVisible(true);
    resultLabel.setText(winner.getResult());
    resultLabel.setVisible(true);
    repaint();
  }

  @Override
  public void paintComponent(Graphics g) {
    super.paintComponent(g);

    g.setColor(Color.black);
    for(int i=0 ; i<2 ; i++) {
      g.drawLine(0,SIZE+i*SIZE,SIZE*3,SIZE+i*SIZE);
      g.drawLine(SIZE+i*SIZE,0,SIZE+i*SIZE,SIZE*3);
    }

    for(int y=0 ; y<3 ; y++) {
      for(int x=0 ; x<3 ; x++) {
        int bx = x*SIZE, by = y*SIZE;
        model.get(x, y).draw(g, bx+10, by+10, SIZE-20);
      }
    }
  }
}

package teratail_java.q370179;

public class OXmodel {
  private Cell brd[][] = new Cell[3][3];
  private Cell turn;

  OXmodel() {
    clear();
  }

  void clear() {
    for(int y=0 ; y<3 ; y++) {
      for(int x=0 ; x<3 ; x++) {
        brd[y][x] = Cell.SPACE;
      }
    }
    turn = Cell.CIRCLE;
  }

  Cell get(int x, int y) {
    return brd[y][x];
  }

  Cell set(int x, int y) {
    brd[y][x] = turn;
    turn = turn.next();
    return judge();
  }

  private Cell judge() {
    for(int i=0 ; i<3 ; i++) {
      if(brd[i][0]!=Cell.SPACE && brd[i][0]==brd[i][1] && brd[i][1]==brd[i][2]) {
        return brd[i][0];
      }
      if(brd[0][i]!=Cell.SPACE && brd[0][i]==brd[1][i] && brd[1][i]==brd[2][i]) {
        return brd[0][i];
      }
    }

    if(brd[1][1]!=Cell.SPACE) {
      if(brd[0][0]==brd[1][1] && brd[2][2]==brd[1][1]) {
        return brd[1][1];
      }
      if(brd[2][0]==brd[1][1] && brd[0][2]==brd[1][1]) {
        return brd[1][1];
      }
    }

    for(int y=0 ; y<3 ; y++) {
      for(int x=0 ; x<3 ; x++) {
        if(brd[y][x] == Cell.SPACE) return null; //勝負はまだついていない
      }
    }
    return Cell.SPACE; //引き分け
  }
}

Comments

[syachi]

とても分かりやすかったです。
ありがとうございました！

[syachi]

すいません。
盤面に表示される○と☓を太くするにはどのように書いたら良いですか？

[jimbe]

マルバツは Cell の各 draw メソッドで書いていますので、 g を使って太くしてください。

[【Java】図形等の線幅や線の形を変更する](https://nompor.com/2017/12/04/post-1532/ )

Answer 2

ゲーム終了後に盤面をもう一度クリックすると、再度ゲームが始まるようにする

「マウスクリック時にゲームが終了していたら初期化する」ように動けば良いのだと思うので、
修正すべきは、クリックしたときの処理である OX.mouseClicked() です。

盤面 (OXmodel) の初期化処理を追加することになると思いますが、
これは、OXmodel のコンストラクタの中身と一緒ですよね？

そこで、初期化処理メソッドを新たに作成してコンストラクタの中身を移し、
コンストラクタと mouseClicked() の両方から呼ぶようにします。

mouseClicked() の方は、冒頭に if 文で処理を追加します。
ゲームが終了しているかどうかを判定できるメソッドは、既にどこかに作ってありますよね。
初期化後は、"〇の勝ち"とか表示されないように、return 文でメソッドを終了させます。

以上をまとめると、こうなります：

public class OXmodel {

    OXmodel() {
        init();
    }

    public void init() {
        // コンストラクタの中身
    }
}

public class OX implements MouseListener {

    public void mouseClicked(MouseEvent e) {

        if (ゲームが終了している) {
            model.init();
            view.repaint();
            return;
        }

        // 元々あった処理
        ・・・・・・
        ・・・・・・
    }
}

"引き分け" などの文字を大きくする

Graphics.getFont() でデフォルトフォントを取得し、deriveFont(float size) するのが良いと思います。

g.setFont(g.getFont().deriveFont(20.0f));

蛇足

質問の内容とは関係ありませんが、現在の作りでは、ウィンドウを×ボタンで閉じても、アプリケーションが終了しないと思います。OX クラスのコンストラクタに、以下の１行を追加したほうが良いと思います。

    OX() {
        ・・・・・・
        JFrame frame = new JFrame();
        ・・・・・・
        frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);  // <- この行
        frame.setVisible(true);
        ・・・・・・
    }