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

2点ほど問題があると思います #18

Open
takuyan020 opened this issue Sep 6, 2016 · 1 comment
Open

2点ほど問題があると思います #18

takuyan020 opened this issue Sep 6, 2016 · 1 comment

Comments

@takuyan020
Copy link

1つ目
コメントでも近い指摘がありましたが
JavaScriptの数値型は倍精度浮動小数点数であり
大きな整数を取り扱った場合正確な値は求められません
N=100とした場合の解、特に後半が明らかに丸め込まれていますので
もはやフィボナッチ数列ではないです

2つ目
N番目まで求めるプログラムのはずが
N-1番目までしか求められていません
for文の致命的な凡ミスです

@sifue
Copy link
Contributor

sifue commented Sep 7, 2016

指摘ありがとうございます。その通りですね。

N=100とした場合の解、特に後半が明らかに丸め込まれていますので

JavaScriptでは多倍長整数を自分で実装しない限り、正確な値を求めることができません。
この不具合を取り上げてScala基礎コースではBigIntを用いて正確な値を計算していますので、
Scala基礎コースが公開されましたらそちらをまた参照ください。

N-1番目までしか求められていません

こちらはその通りですね。じき修正されるのではと思います。

二点目が修正され次第、このissueは閉じさせてもらいます。
指摘ありがとうございました。

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

No branches or pull requests

2 participants