自作Sinメソッドさん、まだ壊れている箇所があった

double Sin(double x)が二つありますが微妙にバグっていて、ループの添え字を2から始めるのが正しいです。
出力例に変化はありません。（以前のと同じで何故かバグっていない場合の正しい値になっている）

また、

// かなり適当に作ってるので、この方法ではそんなに精度はよくない。

/// 実装は割りと適当。

というコメントがありますが、これはバグのせいであって直すと (-2π, 2π) の範囲で14桁以上の精度が出るので消してしまっていいと思います。
(Math.Sinより少し劣るぐらい。(-2π, 2π)の範囲ではiが最大でも60程度でx / factがアンダーフローするから実装上の限界まで精度が出ている)

ちなみにコメントでいうと[構造化] 名前空間 > using エイリアスのサンプルの、

        /// この実装は甘い。
        /// 入力できる値は-0.1～0.1程度で、精度も4桁程度。

もバグっていた時の名残なので消してしまって大丈夫です。

関連：#234 #241

このリポジトリのこれもですかね

UfcppSample/Chapters/StructuredProgramming/Function/MyMath.cs

Lines 28 to 39 in 69137b2

    
           /// <summary> 
        
           /// sin x を求める関数。 
        
           /// テイラー展開を利用。 
        
           /// かなり適当に作ってるので、この方法ではそんなに精度はよくない。 
        
           /// とはいえ、10桁程度なら C# 標準の Sin 関数と値が一致するはず。 
        
           /// </summary> 
        
           static double Sin(double x) 
        
           { 
        
               double xx = -x * x; 
        
               double fact = 1; 
        
               double sin = x; 
        
               for (int i = 1; i < 100;)

Navigation Menu

Repository navigation

自作Sinメソッドさん、まだ壊れている箇所があった #372

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

	/// <summary>
	/// sin x を求める関数。
	/// テイラー展開を利用。
	/// かなり適当に作ってるので、この方法ではそんなに精度はよくない。
	/// とはいえ、10桁程度なら C# 標準の Sin 関数と値が一致するはず。
	/// </summary>
	static double Sin(double x)
	{
	double xx = -x * x;
	double fact = 1;
	double sin = x;
	for (int i = 1; i < 100;)

自作Sinメソッドさん、まだ壊れている箇所があった #372

Description

Activity

iwkjosec commented on Dec 6, 2021

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions