検索結果に表示された防具から、他の部位やα/β違いの防具を除外したい

[iMasanari]

ストーリー攻略中で特定のモンスターが倒せない場合など、ある防具が作れないときにはその防具の他の部位（頭、胴、腕、腰、足）やα/β違いが作れないことが多い。
なので、それらの防具を一括で除外する方法が必要である。

#69 (comment)

[iMasanari]

対応イメージ

スクショした画像を重ねているだけで、まだ未作成ですが、こんな感じです。

Originally posted by @iMasanari in #69 (comment)

今回の修正とは別に考えていたことなのですが、ArmorTable.tsxの操作に関する修正を考えていました。
iMasanariさんが添付してくれた画像の除外部分が私の添付した画像の「ArmorTable.tsx」部分で流用できそうだなと思い添付しました。

上記画像の操作について

1. すべての左押下：表示中の装備を除外 or 非除外
2. グループ名(モンスター単位)の左押下：表示中の装備をグループ(モンスター)単位で除外 or 非除外
3. 部位名の下押下：表示中の装備を部位単位で除外 or 非除外
4. Cボタン押下：テキストボックスの内容をクリア

「1, 2, 3」については、たとえば装備が１つだけ除外されている時にどのような表示にするのかと処理として「表示中装備を全て除外」するのか「表示中装備を全て非除外」にするのかという問題があります。

Originally posted by @yousuke-f in #69 (comment)

[iMasanari]

2. グループ名(モンスター単位)の左押下：表示中の装備をグループ(モンスター)単位で除外 or 非除外

チェックボックスの追加について、今回対応するかは未定。
なお、縦列（頭/胴/腕/腰/足）の一括チェックが必要になるケースがあるかどうかは要確認。

※今作には、過去作にあった頭防具の仕様（頭のみ剣士/ガンナー共用だが、ガンナー防具の方は防御力が低い）は無い

[yousuke-f]

issue作成ありがとうございます。

チェックボックスの追加について、今回対応するかは未定。
縦列（頭/胴/腕/腰/足）の一括チェックが必要になるケースがあるかどうかは要確認。

1. グループ(モンスター)単位の一括除外 or 非除外は、操作性向上を考えるなら必須だと考えます。
   表示をすべてチェックと表示をすべて除外でも良いと思いますし、チェックボックスで実装する必要は無いかも知れません。
2. 縦列（頭/胴/腕/腰/足）の一括チェックは、操作性向上を考えるなら便利だと感じます。
   モンスターは倒せるけど天鱗など、レア素材が出ないけど特定部位のみ除外したい時などに使用できると感じています。
3. 私が添付した画像のArmorTable.tsxの実装が有効だと感じています。
   現状では表示をすべてチェックと表示をすべて除外の処理がArmors.tsxにあり、メッセージやテキストボックスもArmors.tsxにあります。
   検索結果の装備詳細モーダルに除外部分の修正では、ArmorTable.tsxにチェック処理を実装できる方が有効だと感じているためです。
   ただ、チェックボックスで実装する必要は無いので他に良い案があれば、そちらの案でも良いと思います。

以上、よろしくおねがいします。

[iMasanari]

グループ(モンスター)単位の一括除外 or 非除外は、操作性向上を考えるなら必須だと考えます。

私は、5回クリックするのを1回でできるショートカットできる裏技程度と考えています。この機能をあったほうが良いと考えるものの、必須機能ではなくあくまで便利機能です。
その上で、下記について返信します。

1.について

上記の通り、シリーズでの一括チェックは私も実装したほうがよいと思っています。
ですが、checkboxに mixed な状態を実装する必要があります。なのでいったん別issueにして、先に表題の件を片付けるのも手、というのが「今回対応するかは未定」の意図です。

2.について

モンスターは倒せるけど天鱗など、レア素材が出ないけど特定部位のみ除外したい時などに使用できると感じています。

素材が同じなのは、αとβのせいぜい2つです。わざわざ専用のチェックボックスの場所を用意しなくても、2回クリックすればよいだけです。

3.について

私が添付した画像のArmorTable.tsxの実装が有効だと感じています。

こちらは「実装がラク」という認識でよろしいでしょうか。
私では、文章から設計としてのメリット・デメリットを読み取れませんでした。
（久々にコードを見たら、現状のArmorTable.tsxの設計が良いわけではなかったですが…）

ただ、チェックボックスで実装する必要は無いので他に良い案があれば、そちらの案でも良いと思います。

防具除外設定のポップアップには、「表示をすべてチェック」と「表示をすべて除外」のボタンのほうが良いと考えています。
こちらは必須機能ですので、押したときの動作がわかりやすいボタンのままにしたいです。（数が多いと取り返しがつかない操作なので、確認ダイアログも必要）

検索結果の装備詳細については、「グループ(モンスター)単位の一括除外 or 非除外」が実装済みであれば、「すべて除外」はボタンもチェックボックスも必要ないと思います。せいぜい並ぶのはα/β/γの3つですので、「グループ(モンスター)単位の一括除外」で3つを外すだけです。
（「グループ(モンスター)単位の一括除外 or 非除外」が未実装であれば、「表示をすべてチェック」と「表示をすべて除外」をつけます）

[iMasanari]

モンスターハンターRise版のシミュで除外装備設定の実装後、mhw版でも対応を行う。

関連:
iMasanari/mhrise-simulator#33

[yousuke-f]

メッセージありがとうございます。

checkboxに mixed な状態

iMasanariさんの判断にお任せます。

モンスターは倒せるけど天鱗など、レア素材が出ないけど特定部位のみ除外したい時などに使用できると感じています。

素材が同じなのは、αとβのせいぜい2つです。わざわざ専用のチェックボックスの場所を用意しなくても、2回クリックすればよいだけです。

iMasanariさんの判断にお任せます。

私が添付した画像のArmorTable.tsxの実装が有効だと感じています。

ArmorTable.tsx内に除外操作が全て含まれるので、防具設定モーダル、検索結果からの装備詳細にそのまま使用できます。
複数の場所に除外処理を書く必要がないし、if文などで表示/非表示を切り替える必要もないです。

防具除外設定のポップアップには、「表示をすべてチェック」と「表示をすべて除外」のボタンのほうが良いと考えています。
こちらは必須機能ですので、押したときの動作がわかりやすいボタンのままにしたいです。（数が多いと取り返しがつかない操作なので、確認ダイアログも必要）

押したときの動作がわかりやすいボタンについてはiMasanariさんの実装のほうがわかりやすいと思います。
私の示した実装だと説明が無いので操作しないと結果が想像しにくいとは思います。

返信、遅くなってすみません。
以上、よろしくおねがいします。