ここでの問題は、どのクラスがどのタスクを処理するのかを明確に説明していないことだと思います。各クラスが何をすべきかについての良い説明だと思うことを説明してから、アイデアを説明する汎用コードの例を示します。コードの結合性が低く、循環参照が実際にはないことがわかります。
各クラスの機能を説明するところから始めましょう。
GameStateクラスは唯一のゲームの現在の状態に関する情報が含まれている必要があります。ゲームの過去の状態や将来の動きについての情報を含めるべきではありません。チェスのどのマスにどのピースが入っているか、またはバックギャモンのどのポイントに何人のチェッカーがいるのかに関する情報のみが含まれている必要があります。GameStateチェスやバックギャモンで倍増キューブについてのキャスリングに関する情報と同様に、いくつかの追加情報が含まれている必要があります。
Moveクラスは少しトリッキーです。私は、ムーブをプレイしGameStateた結果を指定することで、プレイするムーブを指定できると言います。ですから、移動は単にとして実装できると想像できますGameState。ただし、（たとえば）goでは、ボード上の1つのポイントを指定することで、移動を指定する方がはるかに簡単だと想像できます。Moveこれらのケースのいずれかを処理できるように、クラスに十分な柔軟性が必要です。したがって、Move実際には、クラスはpre-move GameStateを取得して新しいpost-moveを返すメソッドとのインターフェースになりますGameState。
現在、RuleBookクラスはルールに関するすべてを知る責任があります。これは3つのものに分類できます。イニシャルGameStateが何であるか、どの動きが合法であるかを知る必要があり、プレイヤーの一人が勝ったかどうかを知ることができる必要があります。
また、GameHistoryクラスを作成して、行われたすべての動きと発生したすべての動きを追跡することもできGameStatesます。新しいクラスが必要なのは、その前に来たGameStateすべてのGameStates を知ることは単一の責任ではないと判断したためです。
これで、これから説明するクラス/インターフェースを終了します。Boardクラスもあります。しかし、さまざまなゲームのボードは十分に異なっているため、ボードで一般的に何ができるかはわかりにくいと思います。次に、ジェネリックインターフェイスを提供し、ジェネリッククラスを実装します。
最初はGameStateです。このクラスは特定のゲームに完全に依存しているため、一般的なGamestateインターフェイスやクラスはありません。
次ですMove。前述したように、これは、移動前の状態を取得して移動後の状態を生成する単一のメソッドを持つインターフェイスで表すことができます。このインターフェイスのコードは次のとおりです。
package boardgame;
/**
*
* @param <T> The type of GameState
*/
public interface Move<T> {
T makeResultingState(T preMoveState) throws IllegalArgumentException;
}
型パラメーターがあることに注意してください。これは、たとえば、 ChessMovepre-moveの詳細を知る必要があるためChessGameStateです。したがって、たとえば、クラス宣言はChessMoveなり
class ChessMove extends Move<ChessGameState>、
すでにChessGameStateクラスを定義しているはずです。
次に、ジェネリックRuleBookクラスについて説明します。コードは次のとおりです。
package boardgame;
import java.util.List;
/**
*
* @param <T> The type of GameState
*/
public interface RuleBook<T> {
T makeInitialState();
List<Move<T>> makeMoveList(T gameState);
StateEvaluation evaluateState(T gameState);
boolean isMoveLegal(Move<T> move, T currentState);
}
再び、GameStateクラスの型パラメーターがあります。RuleBookは初期状態が何であるかを知っているはずなので、初期状態を与えるメソッドを追加しました。RuleBookはどの動きが合法であるかを知っているはずなので、特定の状態で合法であるかどうかをテストし、特定の状態で合法的な動きのリストを提供するメソッドがあります。最後に、を評価するメソッドがありGameStateます。RuleBookいずれかのプレイヤーが既に勝ったかどうかを説明する責任があるだけで、ゲームの途中で誰がより良い位置にいるのではないことに注意してください。誰がより良い立場にあるかを決定することは複雑なことであり、それを独自のクラスに移すべきです。したがって、StateEvaluationクラスは実際には次のように指定された単なる列挙型です。
package boardgame;
/**
*
*/
public enum StateEvaluation {
UNFINISHED,
PLAYER_ONE_WINS,
PLAYER_TWO_WINS,
DRAW,
ILLEGAL_STATE
}
最後に、GameHistoryクラスについて説明しましょう。このクラスは、ゲームで到達したすべての位置とプレイされた動きを記憶する役割を果たします。できるはずの主なことはMove、再生されたままのレコードです。を元に戻す機能を追加することもできますMove。以下に実装があります。
package boardgame;
import java.util.ArrayList;
import java.util.List;
/**
*
* @param <T> The type of GameState
*/
public class GameHistory<T> {
private List<T> states;
private List<Move<T>> moves;
public GameHistory(T initialState) {
states = new ArrayList<>();
states.add(initialState);
moves = new ArrayList<>();
}
void recordMove(Move<T> move) throws IllegalArgumentException {
moves.add(move);
states.add(move.makeResultingState(getMostRecentState()));
}
void resetToNthState(int n) {
states = states.subList(0, n + 1);
moves = moves.subList(0, n);
}
void undoLastMove() {
resetToNthState(getNumberOfMoves() - 1);
}
T getMostRecentState() {
return states.get(getNumberOfMoves());
}
T getStateAfterNthMove(int n) {
return states.get(n + 1);
}
Move<T> getNthMove(int n) {
return moves.get(n);
}
int getNumberOfMoves() {
return moves.size();
}
}
最後に、Gameすべてを結びつけるクラスを作成することを想像できます。このGameクラスは、現在の内容をGameState確認し、誰が誰を持っているのか、誰が持っているのか、どの動きをプレイできるのか、そして動きをプレイできるようにするメソッドを公開することになっています。私は以下の実装を持っています
package boardgame;
import java.util.List;
/**
*
* @author brian
* @param <T> The type of GameState
*/
public class Game<T> {
GameHistory<T> gameHistory;
RuleBook<T> ruleBook;
public Game(RuleBook<T> ruleBook) {
this.ruleBook = ruleBook;
final T initialState = ruleBook.makeInitialState();
gameHistory = new GameHistory<>(initialState);
}
T getCurrentState() {
return gameHistory.getMostRecentState();
}
List<Move<T>> getLegalMoves() {
return ruleBook.makeMoveList(getCurrentState());
}
void doMove(Move<T> move) throws IllegalArgumentException {
if (!ruleBook.isMoveLegal(move, getCurrentState())) {
throw new IllegalArgumentException("Move is not legal in this position");
}
gameHistory.recordMove(move);
}
void undoMove() {
gameHistory.undoLastMove();
}
StateEvaluation evaluateState() {
return ruleBook.evaluateState(getCurrentState());
}
}
このクラスではRuleBook、電流GameStateが何であるかを知る責任はありません。それがGameHistoryの仕事です。だから、現在の状態が何であるかをGame尋ね、法的動きが何であるかを言う必要がGameHistoryあるRuleBookとき、Gameまたは誰かが勝ったかどうかにこの情報を提供します。
とにかく、この答えのポイントは、各クラスが何に責任があるのかを合理的に決定し、各クラスを少数の責任に焦点を合わせ、各責任を一意のクラスに割り当ててから、クラス分離される傾向があり、すべてがコーディングしやすくなります。うまくいけば、それは私が与えたコード例から明らかです。
RuleBook例えば取っState引数として、および有効な返さMoveListすなわち、「次何ができるのか、私たちが今いる場所ここにあるの？」