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

Ought to check where arguments is not null in all public methods. #11

Open
ChanyaVRC opened this issue Dec 21, 2021 · 1 comment
Open
Labels
bug Something isn't working Priority: Low Should be fixed in the future.

Comments

@ChanyaVRC
Copy link
Owner

Description

現在はアクセスレベルが public であるメソッドにおいて、Reference Types である――もしくはそうである可能性がある――引数の値が null でない事をチェックしていない。実際の利用では null が入るケースは十分に想定できるため、例え引数が not NRT である場合についても引数が null である場合は System.ArgumentNullException を throw するべきである。また、テストにて null である場合についての System.ArgumentNullException もしくは他の適切な処理を行うことを確認すべきである。

@ChanyaVRC ChanyaVRC added bug Something isn't working Priority: Low Should be fixed in the future. labels Dec 21, 2021
@ChanyaVRC
Copy link
Owner Author

修正は難しくないが、テストコードを作成することが非常に煩雑である。そのため、 #10 の解決後にコラボレーターに修正を促したい。これは現時点の計画であり、今後変更する可能性が高い。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Priority: Low Should be fixed in the future.
Projects
None yet
Development

No branches or pull requests

1 participant