search

kzrnm / ac-library-csharp

42 stars 5 forks source link

Add NumOperators #38

Closed kzrnm closed 4 years ago

kzrnm commented 4 years ago

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

key-moon commented 4 years ago

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

kzrnm commented 4 years ago

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

key-moon commented 4 years ago

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

kzrnm commented 4 years ago

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

kzrnm commented 4 years ago

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

key-moon commented 4 years ago

ありがとうございます!