Skip to content
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

Feature/command /myrole all #1290

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

schwKatz
Copy link
Contributor

ホスト用コマンド /myrole all
機能:ゲーム中に全員に自分の役職説明を送信する

初心者向けに役職説明を任意のタイミングで送信できる
/myroleコマンド入力による役職透け防止のため

@yurinakira
Copy link
Collaborator

yurinakira commented Jul 25, 2023

コード外で2点
・このコマンドの利用用途が限られていて、あまり需要を感じない。
=>前提として試合始まる前に役職を把握しておく必要があります。
=>把握できないのであればホストの説明不足や、要素過剰なレギュレーションです。

/myroleコマンド入力による役職透け防止のため

とありますが、参加者がホストに依頼する依頼する時点で役職透けが発生するので問題の解消方法としては不適切です。
=>使用タイミングとしては会議中に限られると思いますが、テキストチャットオンリーでやられるプレイヤーにとっては、テキストが流れてしまうのでストレスになるかと思います。

・All checks have failed
1 failing check
上記failedしていますね。

@Hyz-sui
Copy link
Collaborator

Hyz-sui commented Jul 25, 2023

Actionsの失敗はPRの問題ではないので大丈夫です

@schwKatz
Copy link
Contributor Author

このコマンドの利用用途ですが、
前提として試合始まる前に役職を把握しておく必要については確かにその通りです

ただ、ランダムアサインも可能になり、また習熟度の差により、説明は余分にでもできた方が
親切だと考えました
すべての役職を完全に把握して役職をみるより、アサインされた役職の説明が見える方が
より把握しやすいと思っています
また各参加者の役職の誤認防止、再確認のためにも使えると考えています

MOD初心者向けとして、役職やコマンドを知らなくても体験できると誘えることで、
MOD参加へのハードルが下がることも良いことだと思っています

テキストが流れてしまって・・・というところを考慮して、初回会議での自動送信より
ホストコントロールによる送信が良いと考えています

@Hyz-sui
Copy link
Collaborator

Hyz-sui commented Jul 25, 2023

大前提として,初心者に十分説明せずにゲームを開始している時点で,その部屋主は正常なゲームを進行しようとする意思を持っていないと思われるので,焦点を当てるべきところとして役職透け云々はあまり適切ではないように思います
それを別として,/n/n r等を部屋主が実行することで全員に送ることができる仕様になっている以上,/myroleについても同等の操作を行う機能があった方が仕様として一貫性はあるのかなと思います


本筋とは関係ありませんが,少なくともこういった比較的規模の大きな変更ではPRのタイトルを適切に付けてほしいです
「Feature/command /myrole all」では一見して何のことやらわからずタイトルとしての意味をなしていません
また,コミット e887ace についてですが,このコミットで行われた変更は少なくとも「修正」ではないと思います
次からは「追記」「記述を追加」等適切な動詞を使っていただけるとコミットの内容や意図がより摑みやすいかと思います

@Hyz-sui Hyz-sui added the New Feature/新機能 役職・ゲームモード以外の新機能 label Jul 25, 2023
@yurinakira
Copy link
Collaborator

yurinakira commented Jul 26, 2023

親切だと考えました
すべての役職を完全に把握して役職をみるより、アサインされた役職の説明が見える方が
より把握しやすいと思っています
また各参加者の役職の誤認防止、再確認のためにも使えると考えています

MOD初心者向けとして、役職やコマンドを知らなくても体験できると誘えることで、
MOD参加へのハードルが下がることも良いことだと思っています
大前提として,初心者に十分説明せずにゲームを開始している時点で,その部屋主は正常なゲームを進行しようとする意思を持っていないと思われるので,焦点を当てるべきところとして役職透け云々はあまり適切ではないように思います

"MOD初心者向け"であるならば、マッドとかシェリフくらい知名度が高く説明も容易で理解しやすい役職に限ってレギュを決めるべきだと考えています。
MOD初心者に役職盛りの部屋をいきなり体験させるのは初心者のことを考えていない部屋主の自己満足に近いです...。
それに自分の役職だけではなく他者の役職の能力を知らないと公平なプレイにならないので、
"MOD初心者向け"を目的とするのであれば、「全員が全員の役職説明を見れる」ようにする必要があります。

ランダムアサインとの兼ね合いも考えて、
A.実際にアサインされた役職とアサインされた人数をリストアップし、それを元に全員に役職説明をする(割り当てられた役職と人数が透けるが、初心者としてはどの役職がいるのか分かりやすいのでこちらの方が良いかなと感じます)
B.ランダムアサインの設定をそのまま全員に送る(全役職入れてる人にとっては凄い量のチャットが飛ぶ)(そもそも初心者いるのに全役職入れるのは初心者を迎え入れるなって怒りが沸きますが...)

@yurinakira
Copy link
Collaborator

yurinakira commented Jul 26, 2023

申し訳ありませんが、
目的が"(MOD)初心者向け"であるならば、この機能の現状*では許可できません。
目的が"/myroleコマンド入力による役職透け防止"であるならば、不適切な解消方法だと思っています。

*「全員が自分の役職説明を見れる」 =>「全員が全員の役職説明を見れる」に変更の上、以下の要素も考慮して仕様を再検討する必要があると考えています。

ランダムアサインとの兼ね合いも考えて、
A.実際にアサインされた役職とアサインされた人数をリストアップし、それを元に全員に役職説明をする(割り当てられた役職と人数が透けるが、初心者としてはどの役職がいるのか分かりやすいのでこちらの方が良いかなと感じます)
B.ランダムアサインの設定をそのまま全員に送る(全役職入れてる人にとっては凄い量のチャットが飛ぶ)(そもそも初心者いるのに全役職入れるのは初心者を迎え入れるなって怒りが沸きますが...)

コマンドとは別件で
実際にアサインされている役職を自動アナウンスする機能(ON/OFF)の方が適切な可能性もあるので、仕様検討する必要があるかと。

=イメージ=
会議毎の自動アナウンスで、「役職(割り当て数)リスト」が表示され、
ホストが/XXXコマンドを打ち「全員が全員の役職説明」を見ることができれば初心者に寄り添えるかと思っています。

@yurinakira
Copy link
Collaborator

(初心者村2年近くやり続けているので)申し訳ありませんが初心者向け機能は厳しく見ます。

部屋主が「初心者いるけどこのコマンド打っときゃ良いだろ!」って思って初心者への説明等を省く可能性があり、
不幸な初心者が生まれる可能性が高い機能だと思っているので、かなり慎重に見ています。

@yurinakira
Copy link
Collaborator

それだともう[/myrole all]じゃないじゃん!ってなりますので根本を崩すことにはなりますが、、、色々口出しして申し訳ない........

@Hyz-sui
Copy link
Collaborator

Hyz-sui commented Jul 26, 2023

目的に対する解決になっていないので、初心者サポート向けとしてこの機能を取り入れることはないと思います
その意図を抜きにして考えれば、上で書いたように取り入れる意義はあると思います

@yurinakira
Copy link
Collaborator

はいずさんと同意見です。
『ホストの/myroleの挙動を「全員が自分の役職説明を見れる」ようにする。』
「挙動の統一化」という目的であれば問題なしかなと

@schwKatz
Copy link
Contributor Author

挙動の統一化は確かにその通りだと思います

挙動の統一化という点で考えると、ホストの /myrole で全員に送信、/myrole xx で自分にのみ送信の方が
統一されている気がしますがそちらの方が良いでしょうか
※xxは仮 適切な文字に変更が必要

ホストコマンドで、/n /n r /m は全員に送信、/n xx /n r xx /m xx は自身にのみ送信になります

@Hyz-sui
Copy link
Collaborator

Hyz-sui commented Jul 26, 2023

他のコマンドと違って,myroleは実行者によって表示される内容が異なるので,ホストが単体で実行した場合の挙動は非ホストと同様のほうが良いと思います
つまり設計的には今のままで問題ないかなと思ってますがいかがでしょうか

@schwKatz
Copy link
Contributor Author

schwKatz commented Jul 26, 2023

上記の /myrole xx については認識しやすさの問題ですが、
「実行者によって表示される内容が異なる」は基準として良いかもしれません

余談)/n xx /n r xx も使用したいタイミングがありますので実装検討しても良いかと思いました。
 ホストで設定ミスが無いか確認したいが、議論チャットを占有したくない場合など

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Feature/新機能 役職・ゲームモード以外の新機能
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants