search

sash2104 / carcassonne

An AI for carcassonne
0 stars 0 forks source link

タイルが置かれた位置を管理するクラスの実装 #2

Closed mikarru closed 6 years ago

mikarru commented 6 years ago

タイルが置かれた位置を管理するクラスを実装しました。(どこにどういうタイルが置かれてて、どこが空いていて、次どこに置けるのか、といった情報を管理します。) 将来的にはboardクラスがこのクラスを使ってタイルの位置情報を管理するような感じになります。 (Java実装のTilePositionManager.javaに対応します。)

あたりでコメントもらえるとありがたいです ><

(テストはどうしよう？)

mikarru commented 6 years ago

ちゃんとしたテストの仕組みが整うまで簡易的なテストで対応するようにしました。

sash2104 commented 6 years ago

テストも書いてくれてありがとう！ C++だと、テストのフレームワークはCppUnitとかCppUtestとかgoogletestあたりが有名らしいんだけど、 googletestが一番使いやすそう & 利用者多くてドキュメントも豊富そうだからgoogletest使うのが良い気がしてる

sash2104 commented 6 years ago

クラスの設計はこれで良さそう。 記法で気になった点だけ幾つかコメントしました。

mikarru commented 6 years ago

レビューありがとうございます。修正しました。 なるほど、特にこだわりないからgoogletestでいいよ。導入はもう少し先になるかもしれんけど。

sash2104 commented 6 years ago

修正ありがとう。マージします。 そうね、テストの本格導入はもう少し先で良いと思う。