編集履歴

質問編集履歴

解決策を思いついたので記入。現在、修正作業中。

2018/03/20 23:21

投稿

hilty8

スコア15

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -58,6 +58,24 @@
+追記２ 問題箇所の予想(対処中) (18/03/21 - 08:10)
+---
+MVCにおけるViewを担当している、マス目の処理を担うSquareクラス(extends StackPane)の処理に関して、
+前のターンにBoardLatticeインスタンスに追加したSquareインスタンスを更新するのではなく、
+その上に新たなSquareインスタンスを追加していることが原因だと考えています。
+そのため、BoardLatticeクラスの描画仕様を変更するつもりでいます。
+描画の方法に関して助言頂ければ幸いです。
+宜しくお願い致します。
 ```java
 public class Main extends Application {

主に、コードの補足、ModelであるLogManager、描画のメソッド、Controllerクラスの設定したBind、など追記しました。

2018/03/20 23:21

投稿

hilty8

スコア15

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -30,12 +30,34 @@
 わかりにくいところなどは指摘していただければ、なるべく早く追記します。
 宜しくお願い致します。
+追記（18/03/21 - 00:20）
+---
+・PieceTypeというのは、個人的に定義しているenumクラスです。コマなし、コマA～コマC、の計4つを持ちます。
+・盤面の描画の際、本当はint[][] ではなくPieceType[][] を用いていたので、実際のコードに沿った形に修正しました。
+・先ほどまではGameBoardクラスと表記していましたが、コードではBoardLatticeクラスという名前で使用していたので、実際のコードに沿って名前を修正しました。
+・MVCのModel部分であるLogManagerクラスの挙動は、標準出力を用いて１つずつ確認したので、このクラスにエラーは無いと判断しています。
+見直しをしたので、syntaxErrorは無いと思いますが、ミスがあれば即時対応します。
+宜しくお願い致します。
 ```java
 public class Main extends Application {
@@ -50,15 +72,11 @@
     root.getChildren().add(controller.gameBoard);
-    (以下略)
+    (以下はStageやSceneに追加しているだけなので省略)
-}
+}
-```
+```
@@ -68,15 +86,27 @@
 public class BoardLattice extends GridPane {
+    private Boolean turnPlayer1 = true;
+    public IntegerProperty propertyRow = new SimpleIntegerProperty(-1);
+    public IntegerProperty propertyCol = new SimpleIntegerProperty(-1);
     Square[][] squares;
+    public BoardLattice(PieceType[][] boardStatus, Boolean argsTurnPlayer1){
-    public GameBoard(int[][] status){
+        rendering(boardStatus, argsTurnPlayer1);
-        rendering();
-    }
+    }
-    public void rendering(int[][] status){
+    public void rendering(PieceType[][] boardStatus, Boolean argsTurnPlayer1){
+        turnPlayer1 = argsTurnPlayer1;
         //描画の処理
@@ -86,12 +116,42 @@
             for(int j=0; j<5; j++){
-                // 前回のターンの結果を消去した後、新たに描画処理を実行
+                // 前回のターンの結果を消去した
                 super.getChildren().remove(squares[i][j]);
+                // ImageViewクラスを継承したPieceクラスのインスタンスを生成
+                Piece piece = new Piece(boardStatus[i][j]);
+                squares[i][j].getChildren().add(piece);
                 super.add(squares[i][j], i, j);
+                int row = i;
+                int col = j;
+                squares[i][j].setOnMouseClicked(event -> {
+                    propertyRow.setValue(row);
+                    propertyCol.setValue(col);
+                };
+                // 手番ではないPlayerの駒が載ったsquares[i][j]をクリックしてもEventが発生しないように設定
+                switch(boardStatus[i][j]){
+                    // 条件分岐のみ省略
+                    squares[i][j].setDisable(true);
+                }
             }
         }
@@ -148,15 +208,35 @@
 public class LogManager {
+    public Boolean turnPlayer1;
+    // 選択した駒の行、列の番号
+    public NumberBinding bindRow;
+    public NumberBinding bindCol;
     // 駒の移動方向を指すStringProperty
     public StringProperty direction = new SimpleStringProperty("");
-    public int[][] tmpLog;
+    public int turn;
+    public PieceType[][][] mainLog = new PieceType[256][5][5]; // 256手番もあればゲームが終わる想定
+    public PieceType[][] tmpLog = new PieceType[5][5];
     public LogManager(){
+        turn = 1;
+        turnPlayer1 = true;
-        tmpLog = new int[5][5];//細かい数値は省略します
+        initializeMainLog();
     }
@@ -166,11 +246,63 @@
     }
+    // 以下の移動の処理に関しては期待通りの結果が出ているため、各メソッドの詳細は省略しています。
     public void moveMethod(){
-        // tmpLogを更新
+        // tmpLogを更新。
+        setTmpLog();
+        //NumberBinding, StringPropertyの値を用いて移動処理を実行する
+        whichDirection(direction.getValue());
+        // 実際の移動処理
+        movePiece(bindRow.intValue(), bindCol.intValue());
+        // 処理結果を格納したtmpLogの値をMainLogに書き込み
+        updateMainLog();
-    }
+    }
+    private void setTmpLog(){
+        // mainLog[turn]をtmpLogにコピー（省略）
+    }
+    private void whichDirection(String str){
+        // 移動処理に用いるint型の変数を設定（省略）
+    }
+    private void updateMainLog(){
+        // ターン数＋１
+        turn += 1;
+        // 処理結果を格納したtmpLogの値をMainLogに書き込み（省略）
+        // 手番の入れ替え
+        turnPlayer1 = !turnPlayer1;
+    }
+    private void initializeMainLog(){
+        // MainLog[0], MainLog[1]の２つにおいて、初期状態のコマ配置を指定
+    }
 }
@@ -192,14 +324,20 @@
     public Controller() throws IOException {
+        logManager = new LogManager();
-        boardLattice = new BoardLattice(int[][] status);
+        boardLattice = new BoardLattice(logMangager.getLog());
         directionButton = FXMLLoader.load(getClass().getResource("DirectionButton.fxml"));
+        // Binding設定１ - 選択した駒を示す情報のBinding
+        logManager.bindRow = Bindings.add(boardLattice.propertyRow, 0);
+        logManager.bindCol = Bindings.add(boardLattice.propertyCol, 0);
         // 移動ボタンのコントローラクラス、DirectionButtonControllerのStringPropertyと、logManagerのStringPropertyをBind
         logManager.direction.bind(DirectionButtonController.direction);
@@ -208,13 +346,13 @@
         logManager.direction.addListener( (ObservableValue<? extends String> observable, String oldValue, String newValue) -> {
-            // logManager内のtmpLogを更新
+            // logManager内の移動処理メソッドを実行
             logManager.moveMethod();
             // boardLatticeの再描画
-            boardLattice.rendering(logManager.getLog());
+            boardLattice.rendering(logManager.getLog(), logManager.turnPlayer1);
         }
@@ -230,12 +368,14 @@
 public class Square extends StackPane {
-    public Square(int num){
+    public Square(){
+        super.setPrefSize(70,70);
-        // 引数に従ってStackPaneに駒画像を追加
+        // MouseOver,MouseExitのEventに従って背景を変化(省略)
-    }
+    }
-}
+}
-```
+```

以下の説明を補足。１、MVCの役割を担当するクラス群　２、インスタンスに追加しているChangeListener　３、ChangeListenerから再描画までの処理

2018/03/20 15:34

投稿

hilty8

スコア15

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -46,9 +46,9 @@
     Controller controller = new Controller();// MVCモデルのコントローラ
-    VBox vbox = new VBox();
+    VBox root = new VBox();
-    vbox.getChildren().add(controller.gameBoard);
+    root.getChildren().add(controller.gameBoard);
     (以下略)
@@ -58,57 +58,165 @@
+```java
+//MVCのView部分１ - ゲーム盤
+public class BoardLattice extends GridPane {
+    Square[][] squares;
+    public GameBoard(int[][] status){
+        rendering();
+    }
+    public void rendering(int[][] status){
+        //描画の処理
+        squares = new Square[5][5];
+        for(int i=0; i<5; i++){
+            for(int j=0; j<5; j++){
+                // 前回のターンの結果を消去した後、新たに描画処理を実行
+                super.getChildren().remove(squares[i][j]);
+                super.add(squares[i][j], i, j);
+            }
+        }
+    }
+}
+```
+```java
+//MVCのView部分２ - 移動ボタンのコントローラクラス
+public class DirectionButtonController implements initializable {
+    // 移動ボタンが押されたとき、このコントローラクラスのStringPropertyの値に格納される
+    public static StringProperty direction = new SimpleStringProperty("have not choosen!");
+    // FXML内に定義したボタン（Polygonクラス）
+    @FXML Polygon upButton;
+    // 移動ボタンにEventを実装
+    EventHandler<MouseEvent> upMethod = (event) -> {
+        direction.setValue("up");
+    };
+    @Override
+    public void initialize(URL url, ResourceBandle rb){
+        upButton.setOnMouseClicked(upMethod);
+    }
+}
+```
+```java
+//MVCのModel部分
+public class LogManager {
+    // 駒の移動方向を指すStringProperty
+    public StringProperty direction = new SimpleStringProperty("");
+    public int[][] tmpLog;
+    public LogManager(){
+        tmpLog = new int[5][5];//細かい数値は省略します
+    }
+    public int[][] getLog(){
+        return tmpLog;
+    }
+    public void moveMethod(){
+        // tmpLogを更新
+    }
+}
+```
 ```java
 public class Controller {
+    //MVCモデルにおける、View: BoardLattice、Model: LogManager
-    public GameBoard gameBoard;
+    public BoardLattice boardLattice;
+    public LogManager logManager;
+    public GridPane directionButton;
-    public Controller(){
+    public Controller() throws IOException {
-        gameBoard = new GameBoard(int[][] status);
+        boardLattice = new BoardLattice(int[][] status);
+        directionButton = FXMLLoader.load(getClass().getResource("DirectionButton.fxml"));
+        // 移動ボタンのコントローラクラス、DirectionButtonControllerのStringPropertyと、logManagerのStringPropertyをBind
+        logManager.direction.bind(DirectionButtonController.direction);
+        // LogManagerのStringPropertyが更新されたとき、移動処理を実行（ChangeListenerをラムダ式で定義）
+        logManager.direction.addListener( (ObservableValue<? extends String> observable, String oldValue, String newValue) -> {
-        // int[][] status を更新するメソッドなどありますが、省略します。
+            // logManager内のtmpLogを更新
+            logManager.moveMethod();
+            // boardLatticeの再描画
+            boardLattice.rendering(logManager.getLog());
-    }
+        }
-}
-```
-```java
-public class GameBoard extends GridPane {
-    Square[][] squares;
-    public GameBoard(int[][] status){
-        rendering();
-    }
-    public void rendering(int[][] status){
-        //描画の処理
-        squares = new Square[5][5];
-        for(int i=0; i<5; i++){
-            for(int j=0; j<5; j++){
-                // 前回のターンの結果を消去した後、新たに描画処理を実行
-                super.getChildren().remove(squares[i][j]);
-                super.add(squares[i][j], i, j);
-            }
-        }
     }