状態遷移図

[azubiwa]

やったこと

状態遷移図の作成

問題点

・先手と後手をまとめて書くかどうか

まだやっていないこと

・相手手番の待機
・両方が観察を使いきった時と盤面が全部埋まった時の対応

[oginoshikibu]

全体的にとてもよく書けていると思います！！！MileStoneの設定を忘れずにお願いします！！
指示忘れなのですが、こんな感じでissue numを書くとPRと同期してくれます → Close #10
Pull RequestをIssueにリンクする - GitHub Docs
（mermaid、いいですね……僕も書き換えようかな）

review内容について

先手後手区分する側に石を打つが抜けています
気づいたので書いちゃいました

先手後手について

状態遷移図は基本的に、仕様の自由度を除けば一意に決まるものだと思っています。
曖昧かもしれないな、と思った点は「何の状態遷移図なのか」（画面、片方のユーザー、2人のユーザー）というところです。
先手後手を区別しない側は片方のユーザー、する側は画面、の遷移図になっている気がしています。
一応目的が「フロントエンドの画面設計」なので、画面主体でよいかなと少しだけ思いましたが、正解が僕もわからないです……
状態遷移図を使うとわかりやすいんじゃない？は僕の発想で、慣例的なものをあまり知らないのですよね。
なので、えいやと決めてしまってよいと思います。

ただ、主体を明示的に示す必要はあると思います。
ドキュメント名はとりあえずの命名でUXにしてしまっているので、もし画面主体の場合は変更してもらえると助かります。（screen state transitionとかになるのかなあ……）

[azubiwa]

修正と引き分け部分の追加をしました。
盤面が埋まった時に観測をせず引き分けにしているのですが、1度観測してからの方が良さそうな気がします(盤面が埋まった状態で観測すれば勝敗が決まりそうなので)。

[oginoshikibu]

ややこしいタスクをこなしてもらい、ありがとうございました！
とても良くできていると思います！！
2024/08/22 2:07 コメント部分の返信を忘れていました……明日考えます……！！
2024/08/22 9:01 返信しました！！

review内容について

先手後手番をそれぞれ書くことに悩んでいたようなので、サブルーチンとして切り出してみました。
reviewがものすごく見づらくなってしまい申し訳ないですが、ローカルにコピペするなりして確認してみてください！

切り出した部分をこちらで若干編集入れてしまったので、記載しておきます。
以下の項目がreview項目と思っていただければ……

1. 勝敗確認

勝敗判定のロジックについて、特に言及がなかったですが、ここで仮決めしてもよいと思います。
量子五目では両者同時に5目揃う可能性があるため、リスクある観測者側が有利になるようにしてみました。
全然変えてもらって構いません。

2. 命名規則を若干変更

状態が大文字スネークケーズ、分岐を小文字スネークケースに統一しました。
特に意図は無く、いつの間にか手が動いてしまいました……

その他の修正依頼

フォルダ名の変更もお願いします！！

[oginoshikibu]

コメント返信

修正と引き分け部分の追加をしました。 盤面が埋まった時に観測をせず引き分けにしているのですが、1度観測してからの方が良さそうな気がします(盤面が埋まった状態で観測すれば勝敗が決まりそうなので)。

確かに……🤔🤔
勝敗が決まるまで観測を繰り返す、とかでよいかもしれませんね！！

[azubiwa]

サブルーチンにしました！

勝敗確認に関しては、元の動画でも観測者勝利となっていたので、そのまま使わせていただきました！

盤面が埋まった時は、どちらが観測をどのくらい残しているのかによって、判定方法がかなりややこしくなりそうなのでひとまず引き分けのままにしています。

[oginoshikibu]

ありがとうございます！！とてもきれいにまとまっていると思います。

折角ここまで色々と考えてきたので、その内容を簡単でよいので書き残すとよいと思います。
ドキュメントの役割として、実装の指針となる「設計書」以外にも、なぜそのような仕様になっているかの「理由書」も実は含まれており、将来的にこちらが結構役に立ちます。

現状でもdiscordやPR上にちゃんと書き残してくれているのでとてもありがたいです！
項目ごとに一言程度でよいので、そう決めた背景・理由（「理由は特にない」も重要な情報です）を書き残しておきましょう。
僕がとりあえず思いついた項目しか書いていないので、勿論項目を増やしてもらって構いません！！

[1m-N00b]

lgtm

こういうレビューって編集テキスト(今回ならREADME.md)をローカルに落として見るのが正解なのかな マーメイドなるダイアグラム作成ツール?の文法諸々知らないのでオンラインエディアで見ときました。

[oginoshikibu]

通話でも伝えましたが一応テキストでも残しておきます。
ここのrich diffのiconを押すと、git hub上でそのmdがどう表示されるか見ることが出来ます。

こういうレビューって編集テキスト(今回ならREADME.md)をローカルに落として見るのが正解なのかな マーメイドなるダイアグラム作成ツール?の文法諸々知らないのでオンラインエディアで見ときました。

[oginoshikibu]

LGTM！！完璧です！！！

QKや連珠をちゃんと参考資料としてどうすべきか候補を出し、その中でよさそうなものをちゃんと決め切る、というのをしっかりとやり遂げてもらいましたが、結構難しいことです。
とても見やすくまとまっていますし、mermaidにしたことで保守性もよく、後でこのドキュメントを見た人は泣いて喜ぶと思います。

[oginoshikibu]

これ、めちゃめちゃいい補足だと思います！！！！！
裏どりはめんどくさいけど、とっても大事！！！！