piny940 / python-reverse Goto Github PK

モードってのはCPUモードかPvPモードかってこと

色を引数にとって、その色の石を置ける場所(座標)のリストを返す関数と、
引数なしで現在のプレイヤーが置ける場所のリストを返す関数があるとよさそう？
プレイヤーが石を置ける場所を、ハイライトするのに必要

現状、controller.pyから石の数にアクセスするには
self.__reversi.get_board().get_white_stone_counts()とする必要があって大変。
あと、引数の数を減らすために、石の数は
{ Stone.White: m, Stone.Black: n }
の形で渡したいと思っている(各石の数はView.create_windowの引数にも渡す必要があるが、白い石の数と黒い石の数を分けて渡すとなると引数が多くなるし引数の名前も長くなるので1つの変数にまとめてしまいたいから)ので、この形で石の数を返す関数を作ってほしい(controller.pyから石の数にアクセスする場面は現状2箇所あり、その2箇所でわざわざdictionaryにまとめる処理を書くのは大変)

大部分は書けるようになったはず
まだ書けない部分(=controller.pyに足りてない関数)がある場合はissueに投げといてください

表題の通りです。
関数名はどんなのが良いんだろう。request_notify_player_wins(self, color) みたいな感じかな？

• インデントはスペース4つ
• テストは tests/ ディレクトリに置いていく

とりあえずこれで書き始めてしまったんですけどこれでよろしいですか、というお伺いと、あと他にも規約みたいなので追加すべきものがあればコメントしてくださいというお話です。