編集履歴

質問編集履歴

BoardクラスのSystem.out.println(d+1);をd+1→dに変更

2019/11/28 07:16

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -392,7 +392,7 @@
 		cy += direction[d].y;
-		System.out.println(d+1);
+		System.out.println(d);
 		ste[cx][cy].doReverse(ste[cx][cy].obverse);

手番、パス関連のコードを削除しました。

2019/11/28 07:16

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -6,7 +6,9 @@
 java環境で8×8盤面のオセロを作成した。
-- 石を置けないマスには置けない。
+- 石が置かれている、ひっくり返せないマスには置けない。
+- 手番は考慮しない。
 - 石を点(x,y)に置き、ひっくり返せる周囲の石を反転させたい。
@@ -14,18 +16,6 @@
-### 注釈
-```
-手番やパスの処理がおかしい点は無視して下さい。
-```
 ### 発生している問題
 ```

StoneクラスとBoardクラスを分割しました。

2019/11/27 04:46

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -145,3 +145,463 @@
 - このことから方角を取得するgetLine(int x, int y, Point d){}に問題があると考えた。
 ```
+### Stoneクラス
+```java
+class Stone {
+    public final static int black = 1;
+    public final static int white = 2;
+    public int obverse = 0;
+    Stone() {
+	obverse = 0;
+    }
+    //表面の色を設定
+    void setObverse(int color) {
+	if ( color == black || color == white)
+	    obverse = color;
+	else
+	    System.out.println("黒か白でなければいけません");
+    }
+    //白黒を入れ替える
+    void doReverse(int color) {
+	if ( color == black )
+	    obverse = white;
+	else if ( color == white )
+	    obverse = black;
+	else
+	    System.out.println("黒か白でなければいけません");
+    }
+    //表面の色で中心p, 半径radの円を塗り潰す
+    void paint(Graphics g, Point p, int rad) {
+	if (obverse == black) {
+	    g.setColor(Color.black);
+	    g.fillOval(p.x - 32, p.y - 32, 2*rad, 2*rad);
+	}
+	else if (obverse == white) {
+	    g.setColor(Color.white);
+	    g.fillOval(p.x - 32, p.y - 32, 2*rad, 2*rad);
+	}
+    }
+}
+```
+### Boardクラス
+```java
+class Board {
+    Stone[][] ste = new Stone[8][8];
+    public int num_grid_black;
+    public int num_grid_white;
+    private Point[] direction = new Point[8];
+    public int[][] eval_black = new int[8][8];
+    public int[][] eval_white = new int[8][8];
+    //コンストラクタ初期化
+    Board() {
+	for(int i=0; i<8; i++) {
+	    for(int j=0; j<8; j++) {
+		ste[i][j] = new Stone();
+	    }
+	}
+	ste[3][3].setObverse(1);
+	ste[3][4].setObverse(2);
+	ste[4][3].setObverse(2);
+	ste[4][4].setObverse(1);
+	//方向ベクトル
+	direction[0] = new Point(1,0);
+	direction[1] = new Point(1,1);
+	direction[2] = new Point(0,1);
+	direction[3] = new Point(-1,1);
+	direction[4] = new Point(-1,0);
+	direction[5] = new Point(-1,-1);
+	direction[6] = new Point(0,-1);
+	direction[7] = new Point(1,-1);
+    }
+    //盤面の中か？
+    boolean isOnBoard(int x, int y) {
+	if ( x<0 || 7<x || y<0 || 7<y)
+	    return false;
+	else
+	    return true;
+    }
+    //盤面(x,y)から方向dに向かって石を順番に取得
+    ArrayList<Integer> getLine(int x, int y, Point d) {
+	ArrayList<Integer> line = new ArrayList<Integer>();
+	int cx = x + d.x;
+	int cy = y + d.y;
+	while ( isOnBoard(cx, cy) && ste[cx][cy].obverse > 0 ) {
+	    line.add(ste[cx][cy].obverse);
+	    cx += d.x;
+	    cy += d.y;
+	}
+	return line;
+    }
+    //盤面(x,y)に石sを置いた場合に反転できる石の数を数える
+    int countReverseStone(int x, int y, int s) {
+	//既に石が置かれていたら、置けない
+	if (ste[x][y].obverse != 0) return -1;
+	//8方向をチェック
+	int cnt = 0;
+	for (int d=0; d<8; d++) {
+	    ArrayList<Integer> line = new ArrayList<Integer>();
+	    line = getLine(x, y, direction[d]);
+	    int n = 0;
+	    while ( n < line.size() && line.get(n) != s) n++;
+	    if (1 <= n && n < line.size()) cnt += n;
+	}
+	return cnt;
+    }
+    //石を置き他を反転させる
+    void setStoneAndReverse(int x, int y, int s) {
+	//8方向をチェック
+	for (int d=0; d<8; d++) {
+	    ArrayList<Integer> p = new ArrayList<Integer>();
+	    p = getLine(x, y, direction[d]);
+	    int cx = x;
+	    int cy = y;
+	    int n = 0;
+	    while ( n < p.size()-1 && p.get(n) != s) {
+		cx += direction[d].x;
+		cy += direction[d].y;
+		System.out.println(d+1);
+		ste[cx][cy].doReverse(ste[cx][cy].obverse);
+		n++;
+	    }
+	}
+    }
+    //盤面を見る
+    void evaluateBoard()
+    {
+	num_grid_black = 0;
+	num_grid_white = 0;
+	for ( int i=0; i<8; i++) {
+	    for (int j=0; j<8; j++) {
+		eval_black[i][j] = countReverseStone(i, j, 1);
+		if (eval_black[i][j] > 0) num_grid_black++;
+		eval_white[i][j] = countReverseStone(i, j, 2);
+		if (eval_white[i][j] > 0) num_grid_white++;
+	    }
+	}
+    }
+    //盤面をコンソールに表示する
+    void printBoard() {
+	for ( int i=0; i<8; i++) {
+	    for (int j=0; j<8; j++) {
+		System.out.printf("%2d ", ste[j][i].obverse);
+	    }
+	    System.out.println("");
+	}
+    }
+    //盤面の評価結果をコンソールに表示する
+    void printEval() {
+	System.out.println("Black(1):");
+	for ( int i=0; i<8; i++) {
+	    for ( int j=0; j<8; j++) {
+		System.out.printf("%2d " , eval_black[j][i]);
+	    }
+	    System.out.println("");
+	}
+	System.out.println("White(2):");
+	for ( int i=0; i<8; i++) {
+	    for ( int j=0; j<8; j++) {
+		System.out.printf("%2d " , eval_white[j][i]);
+	    }
+	    System.out.println("");
+	}
+    }
+    //画面描画
+    void paint(Graphics g, int unit_size) {
+	//背景
+	g.setColor(Color.black);
+	g.fillRect(0, 0, unit_size*10, unit_size*10);
+	//盤面
+	g.setColor(new Color(0, 85, 0));
+	g.fillRect(unit_size, unit_size, unit_size*8, unit_size*8);
+	//横線
+	g.setColor(Color.black);
+	for (int i = 0; i<9 ; i++) {
+	    g.drawLine(unit_size*(i+1), unit_size, unit_size*(i+1), unit_size*9 );
+	}
+	//縦線
+	g.setColor(Color.black);
+	for (int i = 0; i<9 ; i++) {
+	    g.drawLine(unit_size, unit_size*(i+1), unit_size*9, unit_size*(i+1) );
+	}
+	// 目印
+	for (int i=0; i<2; i++) {
+	    for (int j=0; j<2; j++) {
+		g.fillRect(unit_size*(3+4*i)-(unit_size/16), unit_size*(3+4*j)-(unit_size/16), (unit_size/8), (unit_size/8));
+	    }
+	}
+	//各マスの中心座標を計算した後描画
+	Point p = new Point();
+	int rad = ((unit_size/2) * 4 ) / 5;
+	for(int i=0; i<8; i++) {
+	    for(int j=0; j<8; j++) {
+		p.x = (unit_size*(i+1) + unit_size*(i+2))/2;
+		p.y = (unit_size*(j+1) + unit_size*(j+2))/2;
+		ste[i][j].paint(g, p, rad);
+	    }
+	}
+    }
+    //クリックされたマス(x,y)に石sを配置する
+    void setStone(int x, int y, int s) {
+	ste[x][y].setObverse(s);
+    }
+}
+```

line.add(ste[cx][cy].obverse); に修正しました。また質問のレイアウトを変更しました。

2019/11/27 04:42

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

	@@ -1 +1 @@
1	- javaでオセロを作成したが反転が~~上手く行えません！~~
1	+ javaでオセロを作成したが反転に問題が生じて困っている

test CHANGED Viewed

@@ -1,242 +1,64 @@
-java環境で石を反転させる時に、隣合う挟まれていない石まで反転されてしまいます。
-修正カ所などがあれば御指摘ください。
+### 前提・実現したいこと
-```ここに言語を入力
+```
-import java.awt.*;
+java環境で8×8盤面のオセロを作成した。
-import javax.swing.*;
+- 石を置けないマスには置けない。
-import java.awt.event.*;
+- 石を点(x,y)に置き、ひっくり返せる周囲の石を反転させたい。
-import java.util.*;
+```
-class Stone {
+### 注釈
-    public final static int black = 1;
+```
-    public final static int white = 2;
-    public int obverse = 0;
+手番やパスの処理がおかしい点は無視して下さい。
-    Stone() {
+```
-	obverse = 0;
-    }
-    //表面の色を設定
+### 発生している問題
-    void setObverse(int color) {
+```
-	if ( color == black || color == white)
-	    obverse = color;
+石を反転させる処理に問題が生じる
-	else
-	    System.out.println("黒か白でなければいけません");
-    }
-    //白黒を入れ替える
+(例)
-    void doReverse(int color) {
+初期配置から
-	if ( color == black )
+黒石[5][3]
-	    obverse = white;
+白石[3][2]
-	else if ( color == white )
+黒石[2][4]
-	    obverse = black;
+白石[3][5]
-	else
-	    System.out.println("黒か白でなければいけません");
+の順に配置すると意図してない[4][4]の黒石が反転される。
-    }
-    //表面の色で中心p, 半径radの円を塗り潰す
-    void paint(Graphics g, Point p, int rad) {
-	if (obverse == black) {
-	    g.setColor(Color.black);
-	    g.fillOval(p.x - 32, p.y - 32, 2*rad, 2*rad);
-	}
+```
-	else if (obverse == white) {
-	    g.setColor(Color.white);
-	    g.fillOval(p.x - 32, p.y - 32, 2*rad, 2*rad);
-	}
-    }
-}
-class Board {
+### 問題があると思われるソースコード
-    Stone[][] ste = new Stone[8][8];
+```java
-    public int num_grid_black;
-    public int num_grid_white;
-    private Point[] direction = new Point[8];
-    public int[][] eval_black = new int[8][8];
-    public int[][] eval_white = new int[8][8];
-    //コンストラクタ初期化
-    Board() {
-	for(int i=0; i<8; i++) {
-	    for(int j=0; j<8; j++) {
-		ste[i][j] = new Stone();
-	    }
-	}
-	ste[3][3].setObverse(1);
-	ste[3][4].setObverse(2);
-	ste[4][3].setObverse(2);
-	ste[4][4].setObverse(1);
-	//方向ベクトル
-	direction[0] = new Point(1,0);
-	direction[1] = new Point(1,1);
-	direction[2] = new Point(0,1);
-	direction[3] = new Point(-1,1);
-	direction[4] = new Point(-1,0);
-	direction[5] = new Point(-1,-1);
-	direction[6] = new Point(0,-1);
-	direction[7] = new Point(1,-1);
-    }
-    //盤面の中か？
-    boolean isOnBoard(int x, int y) {
-	if ( x<0 || 7<x || y<0 || 7<y)
-	    return false;
-	else
-	    return true;
-    }
-    //盤面(x,y)から方向dに向かって石を順番に取得
-    ArrayList<Integer> getLine(int x, int y, Point d) {
-	ArrayList<Integer> line = new ArrayList<Integer>();
-	int cx = x + d.x;
-	int cy = y + d.y;
-	while ( isOnBoard(cx, cy) && ste[cx][cy].obverse != 0) {
-	    line.add(ste[cy][cx].obverse);
-	    cx += d.x;
-	    cy += d.y;
-	}
-	return line;
-    }
-    //盤面(x,y)に石sを置いた場合に反転できる石の数を数える
-    int countReverseStone(int x, int y, int s) {
-	//既に石が置かれていたら、置けない
-	if (ste[x][y].obverse != 0) return -1;
-	//8方向をチェック
-	int cnt = 0;
-	for (int d=0; d<8; d++) {
-	    ArrayList<Integer> line = new ArrayList<Integer>();
-	    line = getLine(x, y, direction[d]);
-	    int n = 0;
-	    while ( n < line.size() && line.get(n) != s) n++;
-	    if (1 <= n && n < line.size()) cnt += n;
-	}
-	return cnt;
-    }
-    //石を置き他を反転させる
+ //石を置き他を反転させる
     void setStoneAndReverse(int x, int y, int s) {
@@ -278,462 +100,48 @@
     }
+```
+```java
+//盤面(x,y)から方向dに向かって石を順番に取得
-    //盤面を見る
+    ArrayList<Integer> getLine(int x, int y, Point d) {
-    void evaluateBoard()
+    ArrayList<Integer> line = new ArrayList<Integer>();
-    {
+    int cx = x + d.x;
-	num_grid_black = 0;
+    int cy = y + d.y;
-	num_grid_white = 0;
+    while ( isOnBoard(cx, cy) && ste[cx][cy].obverse != 0) {
-	for ( int i=0; i<8; i++) {
+        line.add(ste[cx][cy].obverse);
-	    for (int j=0; j<8; j++) {
+        cx += d.x;
-		eval_black[i][j] = countReverseStone(i, j, 1);
-		if (eval_black[i][j] > 0) num_grid_black++;
-		eval_white[i][j] = countReverseStone(i, j, 2);
-		if (eval_white[i][j] > 0) num_grid_white++;
-	    }
-	}
-    }
-    //盤面をコンソールに表示する
-    void printBoard() {
+        cy += d.y;
-	for ( int i=0; i<8; i++) {
-	    for (int j=0; j<8; j++) {
-		System.out.printf("%2d ", ste[j][i].obverse);
-	    }
-	    System.out.println("");
-	}
     }
-    //盤面の評価結果をコンソールに表示する
-    void printEval() {
+    return line;
-	System.out.println("Black(1):");
-	for ( int i=0; i<8; i++) {
-	    for ( int j=0; j<8; j++) {
-		System.out.printf("%2d " , eval_black[j][i]);
-	    }
-	    System.out.println("");
-	}
-	System.out.println("White(2):");
-	for ( int i=0; i<8; i++) {
-	    for ( int j=0; j<8; j++) {
-		System.out.printf("%2d " , eval_white[j][i]);
-	    }
-	    System.out.println("");
-	}
     }
+```
-    //画面描画
-    void paint(Graphics g, int unit_size) {
-	//背景
-	g.setColor(Color.black);
-	g.fillRect(0, 0, unit_size*10, unit_size*10);
-	//盤面
-	g.setColor(new Color(0, 85, 0));
-	g.fillRect(unit_size, unit_size, unit_size*8, unit_size*8);
-	//横線
-	g.setColor(Color.black);
-	for (int i = 0; i<9 ; i++) {
-	    g.drawLine(unit_size*(i+1), unit_size, unit_size*(i+1), unit_size*9 );
-	}
-	//縦線
-	g.setColor(Color.black);
-	for (int i = 0; i<9 ; i++) {
-	    g.drawLine(unit_size, unit_size*(i+1), unit_size*9, unit_size*(i+1) );
-	}
-	// 目印
-	for (int i=0; i<2; i++) {
-	    for (int j=0; j<2; j++) {
-		g.fillRect(unit_size*(3+4*i)-(unit_size/16), unit_size*(3+4*j)-(unit_size/16), (unit_size/8), (unit_size/8));
-	    }
-	}
-	//各マスの中心座標を計算した後描画
-	Point p = new Point();
-	int rad = ((unit_size/2) * 4 ) / 5;
-	for(int i=0; i<8; i++) {
-	    for(int j=0; j<8; j++) {
-		p.x = (unit_size*(i+1) + unit_size*(i+2))/2;
-		p.y = (unit_size*(j+1) + unit_size*(j+2))/2;
-		ste[i][j].paint(g, p, rad);
-	    }
-	}
-    }
-    //クリックされたマス(x,y)に石sを配置する
-    void setStone(int x, int y, int s) {
-	ste[x][y].setObverse(s);
-    }
-}
-public class Reversi extends JPanel{
-    private final static int UNIT_SIZE = 80;
-    private Board board = new Board();
-    private int turn = 1;
-    //コンストラクタ初期化
-    public Reversi() {
-	setPreferredSize(new Dimension(800, 800));
-	addMouseListener(new MouseProc());
-    }
-    public void paintComponent(Graphics g) {
-	board.paint(g, UNIT_SIZE);
-    }
-    //turn 1は黒の番
-    void changeTurn() {
-	if ( turn == 1 )  turn = 2;
-	else if ( turn ==  2 ) turn = 1;
-    }
-    void MessageDialog(String str) {
-     	JOptionPane.showMessageDialog(this, str, "情報", JOptionPane.INFORMATION_MESSAGE);
-    }
-    //黒石と白石の数を比較し終了
-    void EndMessageDialog(int num_black, int num_white) {
-	if( num_black > num_white ) {
-	    String str = "[黒:" + num_black + ",白:" + num_white + "]で黒の勝ち";
-	    JOptionPane.showMessageDialog(this, str, "ゲーム終了", JOptionPane.INFORMATION_MESSAGE);}
-	else if (num_black == num_white ) {
-	    String str = "[黒:" + num_black + ",白:" + num_white + "]で引き分け";
-	    JOptionPane.showMessageDialog(this, str, "ゲーム終了", JOptionPane.INFORMATION_MESSAGE);}
-	else {
-	    String str = "[黒:" + num_black + ",白:" + num_white + "]で白の勝ち";
-	    JOptionPane.showMessageDialog(this, str, "ゲーム終了", JOptionPane.INFORMATION_MESSAGE);}
-	System.exit(0);
-    }
-    //クリックされた時の処理用クラス
-    class MouseProc extends MouseAdapter {
-	public void mouseClicked(MouseEvent me) {
-	    Point point = me.getPoint();
-	    int btn = me.getButton();
-	    System.out.println("(" + point.x + "," + point.y + ")");
-	    board.evaluateBoard();
-	    if ( btn == MouseEvent.BUTTON1){
-		int x = point.x / UNIT_SIZE - 1;
-		int y = point.y / UNIT_SIZE - 1;
-		System.out.println("[" + x + "]" + "[" + y + "]");
-		if( 0 <= x && x < 8 && 0 <= y && y < 8 ) {
-		    if(board.eval_black[x][y] < 1){
-			System.out.println("そこに石を置けません");
-		    }
-		    else {
-			board.setStone(x, y, 1);
-			board.setStoneAndReverse(x, y, 1);
-			repaint();
-			board.printBoard();
-			board.evaluateBoard();
-			board.printEval();
-			changeTurn();
-			if( board.num_grid_black >= 1){
-			    MessageDialog("白の番です");
-			}
-			else {
-			    changeTurn();
-			    MessageDialog("あなたはパスです");
-			}
-		    }
-		}
-	    }
-	    else if ( btn == MouseEvent.BUTTON3){
-		int x = point.x / UNIT_SIZE - 1;
-		int y = point.y / UNIT_SIZE - 1;
-		System.out.println("[" + x + "]" + "[" + y + "]");
-		if( 0 <= x && x < 8 && 0 <= y && y < 8 ) {
-		    if(board.eval_white[x][y] < 1){
-			System.out.println("そこに石を置けません");
-		    }
-		    else {
-			board.setStone(x, y, 2);
-			board.setStoneAndReverse(x, y, 2);
-			repaint();
-			board.printBoard();
-			board.evaluateBoard();
-			board.printEval();
-			changeTurn();
-			if( board.num_grid_black >= 1){
-			    MessageDialog("黒の番です");
-			}
-			else {
-			    changeTurn();
-			    MessageDialog("あなたはパスです");
-			}
-		    }
-		}
-	    }
-	    //白、黒石の総数と石が置けなくなる場合を考える
-	    int num_black = 0;
-	    int num_white = 0;
-	    for(int i = 0; i<8; i++) {
-		for(int j = 0; j<8; j++) {
-		    if( board.ste[i][j].obverse == 1 ) num_black++;
-		    if( board.ste[i][j].obverse == 2 ) num_white++;
-		}
-	    }
-	    if( board.num_grid_black < 1 && board.num_grid_white < 1 )
-		{
-		    EndMessageDialog(num_black, num_white);
-		}
-	}
-    }
-    //起動
-    public static void main(String[] args) {
-	JFrame f = new JFrame();
-	f.getContentPane().setLayout(new FlowLayout());
-	f.getContentPane().add(new Reversi());
-	f.pack();
+### 試したこと
-	f.setResizable(false);
-	f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
-	f.setVisible(true);
-    }
-}
 ```
+上記のsetStoneAndReverse(int x, int y, int s){}において、反転を行った方角を示すdirection[d]を出力した。
+- 意図しない石が反転された際、予想外の方向にも反転を行っていることが分かった。
+- このことから方角を取得するgetLine(int x, int y, Point d){}に問題があると考えた。
+```

誤字

2019/11/27 04:33

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -1,3 +1,9 @@
+java環境で石を反転させる時に、隣合う挟まれていない石まで反転されてしまいます。
+修正カ所などがあれば御指摘ください。
 ```ここに言語を入力
 import java.awt.*;