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

Reportに持ち越しかの判定だけでなく、持ち越しが発生したかの判定が必要 #26

Closed
drakontia opened this issue Sep 29, 2023 · 2 comments

Comments

@drakontia
Copy link
Collaborator

今、持ち越しかの判定でisCarryOverがあると思いますが、その凸で持ち越しが発生したかの判定も必要だと思われます。
あとで集計する際に削りか撃破だけだと持ち越しが発生しているかの区別をするのが困難です。
なので、hasCarryOverという発生判定を付ける必要があると思います。

@yuina1056
Copy link
Owner

カラム名がよろしくないだけかも。
isCarryOverは撃破して持ち越しが発生したユーザーに対してフラグを立てる想定で準備した。
ただ、持ち越し凸かの判定もないと、残持越凸の計算を行えないので、どちらにせよフラグは増やす必要がある。

@yuina1056
Copy link
Owner

#72 で対応済み

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

2 participants