回答編集履歴
3
追記
test
CHANGED
|
@@ -6,29 +6,35 @@
|
|
|
6
6
|
|
|
7
7
|
|
|
8
8
|
|
|
9
|
-
永久ループは**ループ終了条件を満たさない**時に発生す
|
|
9
|
+
永久ループは**ループ終了条件を満たさない**時に発生します。ループが正しく終わるのかをSystem.out.println()で変数の内容を出力してみたり、デバック実行でステップ実行をしてみてくださいな。
|
|
10
10
|
|
|
11
11
|
|
|
12
12
|
|
|
13
|
-
1,
|
|
13
|
+
1, `paintComponent`に描画以外の処理を書かないようにします。
|
|
14
14
|
|
|
15
|
-
`panitComponet`がすることは盤面
|
|
15
|
+
`panitComponet`が担当することは、現在の盤面表示/手番表示などの画面表示に関わる事のみです。
|
|
16
|
-
|
|
17
|
-
一定時間後に処理を行いたい場合(例えばCPUの思考ルーチンなど)は、[javax.swing.timer](https://docs.oracle.com/javase/jp/10/docs/api/javax/swing/Timer.html)を使います。
|
|
18
16
|
|
|
19
17
|
|
|
20
18
|
|
|
21
|
-
|
|
19
|
+
一定時間後に処理を行いたい場合(例えばCPUの思考ルーチン)は、[javax.swing.timer](https://docs.oracle.com/javase/jp/10/docs/api/javax/swing/Timer.html)や[CompletableFuture](https://docs.oracle.com/javase/jp/10/docs/api/java/util/concurrent/CompletableFuture.html)を使います。
|
|
22
20
|
|
|
21
|
+
|
|
22
|
+
|
|
23
|
+
`panitComponet`で呼び出している、`putStoneJudge();`と`turnPass();`は
|
|
24
|
+
|
|
23
|
-
|
|
25
|
+
マウスクリック時のイベントに移動してクリック時に判定をする方にします。
|
|
24
26
|
|
|
25
27
|
クリック時にクリック座標を元にして`ban`の置き換え処理を行います。
|
|
26
28
|
|
|
27
|
-
■コマが置けるかの条件は
|
|
28
29
|
|
|
29
|
-
1, 白、黒コマが設置されていない。
|
|
30
30
|
|
|
31
|
+
■コマが置けるかの判断条件は
|
|
32
|
+
|
|
33
|
+
1, 白、黒コマが置かれていない。
|
|
34
|
+
|
|
31
|
-
2, 置くことで
|
|
35
|
+
2, その位置に置くことで敵ゴマを反転できる。
|
|
36
|
+
|
|
37
|
+
参考:[オセロ（リバーシー）で駒をひっくり返す処理がわからない。](https://teratail.com/questions/81087)
|
|
32
38
|
|
|
33
39
|
|
|
34
40
|
|
|
@@ -38,27 +44,25 @@
|
|
|
38
44
|
|
|
39
45
|
2, 盤面情報は`Board`クラスとして管理し`clone`を実装するとアシスト機能でも流用できるかと。
|
|
40
46
|
|
|
41
|
-
|
|
42
|
-
|
|
43
47
|
3, `static`にするとクラス変数となるので、インスタンス間で共有されてしまいます。
|
|
44
48
|
|
|
45
49
|
これも不具合を発生させやすいので、避けたほうがよいかと。
|
|
46
50
|
|
|
51
|
+
4, 変数:`ban`と変数:`putStone`と同じ盤面情報を2つの変数で管理していますが、
|
|
52
|
+
|
|
53
|
+
どちらの変数の値も正しく更新する必要が発生するため。難易度が高くなります。
|
|
54
|
+
|
|
55
|
+
banの全要素に対して駒が置けるかどうかをループでチェックし、
|
|
56
|
+
|
|
57
|
+
置ける座標(x,y)をArrayListに格納すると二重管理が防げます。
|
|
47
58
|
|
|
48
59
|
|
|
49
|
-
4, 変数:`ban`と変数:`putStone`と同じ盤面情報を2つの変数で管理していますが、
|
|
50
|
-
|
|
51
|
-
どちらの変数の値も正しく更新する必要が発生するため。
|
|
52
|
-
|
|
53
|
-
難易度が高くなります。
|
|
54
|
-
|
|
55
|
-
`ban`のマスの値が0:未設置かつ置いたことでコマを反転できるか判定した方が二重管理が防げます。
|
|
56
60
|
|
|
57
61
|
コマ反転に関しては白、黒、未配置以外に壁マスを作ると、縦、横、斜めの判定処理の使い回しが可能です。
|
|
58
62
|
|
|
59
63
|
|
|
60
64
|
|
|
61
|
-
5,
|
|
65
|
+
5, コマは以下のような`enum`型を作ると便利です。
|
|
62
66
|
|
|
63
67
|
|
|
64
68
|
|
2
追記
test
CHANGED
|
@@ -14,17 +14,29 @@
|
|
|
14
14
|
|
|
15
15
|
`panitComponet`がすることは盤面の表示とどちらの手番かの表示などです。
|
|
16
16
|
|
|
17
|
+
一定時間後に処理を行いたい場合(例えばCPUの思考ルーチンなど)は、[javax.swing.timer](https://docs.oracle.com/javase/jp/10/docs/api/javax/swing/Timer.html)を使います。
|
|
18
|
+
|
|
17
19
|
|
|
18
20
|
|
|
19
21
|
`panitComponet`で呼び出している、`putStoneJudge();`と`turnPass();`を
|
|
20
22
|
|
|
21
23
|
どこに移動するですがマウスクリック時のイベントです。
|
|
22
24
|
|
|
23
|
-
クリック時にクリック
|
|
25
|
+
クリック時にクリック座標を元にして`ban`の置き換え処理を行います。
|
|
26
|
+
|
|
27
|
+
■コマが置けるかの条件は
|
|
28
|
+
|
|
29
|
+
1, 白、黒コマが設置されていない。
|
|
30
|
+
|
|
31
|
+
2, 置くことで相手駒を反転できる。
|
|
24
32
|
|
|
25
33
|
|
|
26
34
|
|
|
35
|
+
この2つの条件です。
|
|
36
|
+
|
|
37
|
+
|
|
38
|
+
|
|
27
|
-
2, 盤面情報は`Board`クラスとして
|
|
39
|
+
2, 盤面情報は`Board`クラスとして管理し`clone`を実装するとアシスト機能でも流用できるかと。
|
|
28
40
|
|
|
29
41
|
|
|
30
42
|
|
1
追記
test
CHANGED
|
@@ -4,9 +4,9 @@
|
|
|
4
4
|
|
|
5
5
|
> 永遠にループ
|
|
6
6
|
|
|
7
|
-
永久ループは**ループ終了条件を満たさない**時に発生するので、無限ループを作っていないか、ループが正しく終わるのかをSystem.out.println()で変数を出力してみたり、デバック実行をしてみてくださいな。
|
|
8
7
|
|
|
9
8
|
|
|
9
|
+
永久ループは**ループ終了条件を満たさない**時に発生するので、無限ループを作っていないか、ループが正しく終わるのかをSystem.out.println()で変数を出力してみたり、デバック実行をしてみてくださいな。
|
|
10
10
|
|
|
11
11
|
|
|
12
12
|
|
|
@@ -16,31 +16,33 @@
|
|
|
16
16
|
|
|
17
17
|
|
|
18
18
|
|
|
19
|
-
|
|
19
|
+
`panitComponet`で呼び出している、`putStoneJudge();`と`turnPass();`を
|
|
20
20
|
|
|
21
|
+
どこに移動するですがマウスクリック時のイベントです。
|
|
22
|
+
|
|
21
|
-
クリック時にクリック情報を元にして
|
|
23
|
+
クリック時にクリック情報を元にして`ban`の置き換え処理を行います。
|
|
22
24
|
|
|
23
25
|
|
|
24
26
|
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
2, 盤面情報はBoardクラスとして、管理すると他のクラスから使いやすいかと。
|
|
27
|
+
2, 盤面情報は`Board`クラスとして、管理すると他のクラスから使いやすいかと。
|
|
28
28
|
|
|
29
29
|
|
|
30
30
|
|
|
31
|
-
3, staticにするとクラス変数となるので、インスタンス間で共有されてしまいます。
|
|
31
|
+
3, `static`にするとクラス変数となるので、インスタンス間で共有されてしまいます。
|
|
32
32
|
|
|
33
33
|
これも不具合を発生させやすいので、避けたほうがよいかと。
|
|
34
34
|
|
|
35
35
|
|
|
36
36
|
|
|
37
|
-
|
|
38
|
-
|
|
39
37
|
4, 変数:`ban`と変数:`putStone`と同じ盤面情報を2つの変数で管理していますが、
|
|
40
38
|
|
|
41
|
-
|
|
39
|
+
どちらの変数の値も正しく更新する必要が発生するため。
|
|
42
40
|
|
|
41
|
+
難易度が高くなります。
|
|
42
|
+
|
|
43
|
-
|
|
43
|
+
`ban`のマスの値が0:未設置かつ置いたことでコマを反転できるか判定した方が二重管理が防げます。
|
|
44
|
+
|
|
45
|
+
コマ反転に関しては白、黒、未配置以外に壁マスを作ると、縦、横、斜めの判定処理の使い回しが可能です。
|
|
44
46
|
|
|
45
47
|
|
|
46
48
|
|