sanaehirotaka / logbook-kai

「艦隊これくしょん ~艦これ~」をより遊びやすくするための外部ツール
MIT License
82 stars 49 forks source link

遠征条件の修正および遠征条件ビューの改善 #199

Closed Sdk0815 closed 4 years ago

Sdk0815 commented 4 years ago

問題解析

A4の遠征条件が現状判明している条件と異なっている。

変更内容

A4の遠征条件の定義ファイルを更新。

関連して、遠征条件の評価の logical operator (and/or)は Predicate で実装されているが、and 条件なら片方の条件が false の時点で、or 条件なら true の時点で探索を終了してしまうため、結果としてもう片方の条件が評価されない。そうなると遠征条件ビューにおいては最初に満たしていない条件が出てきた時点でそれ以降の条件を満たしているかどうかがわからず、「?」で表示されてしまう。それを解決するため、評価を行う際には operator の双方の値の評価を行ってから戻り値を計算するように変更した。

image

(従来バージョンだと火力条件を満たしていない時点で、対空以降の条件はすべて「?」で表示される。)

関連するIssue

197 (一部)