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

レスポンスでキーが含まれない場合の戻り値をNoneからMISSINGかそれに類ずるものに変える #150

Open
yupix opened this issue Oct 11, 2024 — with Huly for GitHub · 0 comments
Assignees
Labels

Comments

Copy link
Owner

yupix commented Oct 11, 2024

Summary

現在 hoge: NotRequired[str | None] の様な型の属性があり、キーが存在しない場合 .get("hoge") と書くことでキーが存在しない場合でも None を返している。しかしながらこれではキーが存在しないのか、キーは存在するものの値が存在しないのか判断ができない。

それを今度は str | None | Missing といった形にすることで明確に判断できるようにする。

問題点としては以下が挙げられる

  • bool(属性)した場合Falseが入ってしまう
    • わざわざboolで囲うという条件はあるものの、意図せず条件式が動いてしまいデバッグ時に原因の特定が困難になることがあるかもしれない

解決策としてはMissingの __bool__ を持たないようなクラスを用意する

@yupix yupix added the kind/Feature✨ This adds/improves/enhances a feature label Oct 11, 2024
@yupix yupix self-assigned this Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant