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

Add NumOperators #38

Merged
merged 4 commits into from
Sep 12, 2020
Merged

Add NumOperators #38

merged 4 commits into from
Sep 12, 2020

Conversation

kzrnm
Copy link
Owner

@kzrnm kzrnm commented Sep 12, 2020

#13, #24 での数値計算用インターフェイスを追加

@key-moon
Copy link
Contributor

実装ありがとうございます。遅くなってごめんなさい🙇
ゼロについては struct 制約をつけているので default が 0 になるであろうという判断で良いでしょうか。(BigInteger も同様の制約に従っているので、私もこれで問題ないと考えます。)

@kzrnm
Copy link
Owner Author

kzrnm commented Sep 12, 2020

はい、default == 0 前提の想定です。

@key-moon
Copy link
Contributor

ありがとうございます。比較については IComparable<T>.CompareTo を使用したほうが実装としては簡潔で綺麗にはなりますが、別途 Greater 等のオーバーライドを用意した方が高速にはなります。内部的に使用する型ですし、高速化を前提として用意してしまうのが良いのではないかと思うのですが、いかがでしょうか?

@kzrnm
Copy link
Owner Author

kzrnm commented Sep 12, 2020

そうですね。高速さもですが、コードのわかりやすさにも関わるので追加したいと思います。

@kzrnm
Copy link
Owner Author

kzrnm commented Sep 12, 2020

比較演算子を追加しました。
ついでにドキュメントも追加しました。

@key-moon
Copy link
Contributor

ありがとうございます!

@key-moon key-moon merged commit 3f5ac8d into kzrnm:master Sep 12, 2020
@kzrnm kzrnm deleted the Algebra branch September 12, 2020 15:44
@takytank takytank mentioned this pull request Sep 12, 2020
kzrnm referenced this pull request in FKbelm/ac-library-csharp-old Nov 12, 2020
kzrnm added a commit that referenced this pull request Feb 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants