-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
状態遷移図 #27
状態遷移図 #27
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
全体的にとてもよく書けていると思います!!!MileStoneの設定を忘れずにお願いします!!
指示忘れなのですが、こんな感じでissue numを書くとPRと同期してくれます → Close #10
Pull RequestをIssueにリンクする - GitHub Docs
(mermaid、いいですね……僕も書き換えようかな)
review内容について
先手後手区分する側に石を打つが抜けています
気づいたので書いちゃいました
先手後手について
状態遷移図は基本的に、仕様の自由度を除けば一意に決まるものだと思っています。
曖昧かもしれないな、と思った点は「何の状態遷移図なのか」(画面、片方のユーザー、2人のユーザー)というところです。
先手後手を区別しない側は片方のユーザー、する側は画面、の遷移図になっている気がしています。
一応目的が「フロントエンドの画面設計」なので、画面主体でよいかなと少しだけ思いましたが、正解が僕もわからないです……
状態遷移図を使うとわかりやすいんじゃない?は僕の発想で、慣例的なものをあまり知らないのですよね。
なので、えいやと決めてしまってよいと思います。
ただ、主体を明示的に示す必要はあると思います。
ドキュメント名はとりあえずの命名でUXにしてしまっているので、もし画面主体の場合は変更してもらえると助かります。(screen state transitionとかになるのかなあ……)
修正と引き分け部分の追加をしました。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ややこしいタスクをこなしてもらい、ありがとうございました!
とても良くできていると思います!!
2024/08/22 2:07 コメント部分の返信を忘れていました……明日考えます……!!
2024/08/22 9:01 返信しました!!
review内容について
先手後手番をそれぞれ書くことに悩んでいたようなので、サブルーチンとして切り出してみました。
reviewがものすごく見づらくなってしまい申し訳ないですが、ローカルにコピペするなりして確認してみてください!
切り出した部分をこちらで若干編集入れてしまったので、記載しておきます。
以下の項目がreview項目と思っていただければ……
1. 勝敗確認
勝敗判定のロジックについて、特に言及がなかったですが、ここで仮決めしてもよいと思います。
量子五目では両者同時に5目揃う可能性があるため、リスクある観測者側が有利になるようにしてみました。
全然変えてもらって構いません。
2. 命名規則を若干変更
状態が大文字スネークケーズ、分岐を小文字スネークケースに統一しました。
特に意図は無く、いつの間にか手が動いてしまいました……
その他の修正依頼
フォルダ名の変更もお願いします!!
コメント返信
確かに……🤔🤔 |
サブルーチンにしました! 勝敗確認に関しては、元の動画でも観測者勝利となっていたので、そのまま使わせていただきました! 盤面が埋まった時は、どちらが観測をどのくらい残しているのかによって、判定方法がかなりややこしくなりそうなのでひとまず引き分けのままにしています。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ありがとうございます!!とてもきれいにまとまっていると思います。
折角ここまで色々と考えてきたので、その内容を簡単でよいので書き残すとよいと思います。
ドキュメントの役割として、実装の指針となる「設計書」以外にも、なぜそのような仕様になっているかの「理由書」も実は含まれており、将来的にこちらが結構役に立ちます。
現状でもdiscordやPR上にちゃんと書き残してくれているのでとてもありがたいです!
項目ごとに一言程度でよいので、そう決めた背景・理由(「理由は特にない」も重要な情報です)を書き残しておきましょう。
僕がとりあえず思いついた項目しか書いていないので、勿論項目を増やしてもらって構いません!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
こういうレビューって編集テキスト(今回ならREADME.md)をローカルに落として見るのが正解なのかな マーメイドなるダイアグラム作成ツール?の文法諸々知らないのでオンラインエディアで見ときました。
通話でも伝えましたが一応テキストでも残しておきます。
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!完璧です!!!
QKや連珠をちゃんと参考資料としてどうすべきか候補を出し、その中でよさそうなものをちゃんと決め切る、というのをしっかりとやり遂げてもらいましたが、結構難しいことです。
とても見やすくまとまっていますし、mermaidにしたことで保守性もよく、後でこのドキュメントを見た人は泣いて喜ぶと思います。
### 観測時の勝敗条件について | ||
このゲームでは1回の観測で、両者同時に勝利条件を満たすことがある。 | ||
その場合、この仕様では観測者が有利になるように、観測者勝利としている。 | ||
本家量子五目並べ(QuizKnock)でも、両者が同時に勝利条件を満たした時は観測者勝利となっている。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
これ、めちゃめちゃいい補足だと思います!!!!!
裏どりはめんどくさいけど、とっても大事!!!!
やったこと
状態遷移図の作成
問題点
・先手と後手をまとめて書くかどうか
まだやっていないこと
・相手手番の待機
・両方が観察を使いきった時と盤面が全部埋まった時の対応