search

jonpili / KomatsuBlokus

小松研で開発する名作ボードゲーム「Blokus」
3 stars 0 forks source link

Satoshiiiiiiiii/issue14 endgame #66

Closed satoshiiiiiiiii closed 5 years ago

satoshiiiiiiiii commented 5 years ago

issue14_endgame 一応、ゲームの勝敗と終了ができるようになったので、確認をお願いします。 ただ、終わり方がskipturn()でのエラーによってなので、そこを改善したいです。

あと、間違ってmasterにissue14_endgameをマージしてしまいました。 どのように対処すればよいでしょうか。

jonpili commented 5 years ago

先にマージ問題について話すと、 まずそれ自体は僕がリモートブランチのmaster(origin)に対してマージ許可を出さない限り問題ないから大丈夫 だからこのプルリクを完璧なものにした上で僕がマージをするよ

jonpili commented 5 years ago 
gameFinish = scoreCheck()
if gameFinish:

に関してはgameFinishを置かなくても直接if scoreCheck()でできるよ

jonpili commented 5 years ago

エラーに関してはゲーム終了時、96行目のreturn whoTurn, selectedBlock, selectedDirectionの中のselectedBlock, selectedDirectionが存在しないからだね なのでここはifとelse両方にreturn文を用意した上で、ifの方はreturn内容をNoneとかでごまかすといいかな

satoshiiiiiiiii commented 5 years ago

了解です！ 修正します

satoshiiiiiiiii commented 5 years ago

別の方法で、エラー無くゲームを終了できるようにしました。 確認をお願いします。

jonpili commented 5 years ago

選択キャンセル・重複選択・片方のみしばらくスキップを経た場合のテスト

jonpili commented 5 years ago