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

相応しい/相応しくない武器 の判定反転 #82

Closed
Hourier opened this issue Feb 12, 2021 · 1 comment
Closed

相応しい/相応しくない武器 の判定反転 #82

Hourier opened this issue Feb 12, 2021 · 1 comment
Assignees
Labels
refactor 処理の整理、可読性の向上

Comments

@Hourier
Copy link
Member

Hourier commented Feb 12, 2021

OSDNより移行。41319

現在はicky (相応しくない)という変数が使われているが、相応しい時は!icky である
これは負論理でよろしくないので、suitable (相応しい)に変更し、相応しくない時は!suitable とする
なお負論理のコードは他にもあるが、まとめて実施するとブランチが膨れ上がるので都度対応とする

@Hourier Hourier added the refactor 処理の整理、可読性の向上 label Feb 12, 2021
@Hourier Hourier added this to the 3.0.0正式リリース milestone Feb 12, 2021
@Hourier Hourier self-assigned this Feb 12, 2021
@iks3
Copy link
Contributor

iks3 commented Mar 21, 2021

ペナルティの条件が、装備している武器が ickyである(不愉快である) 時なので、条件自体は正論理ですから、suitable にすることで条件が負論理( !suitable のときにペナルティ)になってしまうので、変えない方がよろしい気がします。
日本語で「不」愉快ですので反対に思えますが、不愉快の反対は愉快ではありませんので、言葉の上でも同じかと。
ただ、ickyではわかりにくいので、badとかhateとかに変えるのはアリかと思います。

@iks3 iks3 closed this as completed Mar 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 処理の整理、可読性の向上
Projects
None yet
Development

No branches or pull requests

2 participants