ゲーム開発 #defineを使った掛け算と関数による掛け算どっちがいいのか？

Question

提示コードですが関数を使ったラジアンを度数で取得するものと#defineを使った掛け算によるラジアンを度数で取得するやり方とありますが
表示すると180と来るはずのものが大きな誤差がありますが

この場合は#defineを使った方がいいと思われますがどういった場合は#defineの方が有利でどういった場合だと関数の方がいいのでしょうか？

#include <iostream>

#define PI ( (double)3.14159265359 )		//PI
#define RADIAN ( 360.0 / (PI * 2.0 ) )	//ラジアンを度数で取得

double getRadian(double a)
{
	double t = 360.0 / (PI * 2.0);

	return t * (double)a;
}


int main()
{
	std::cout << getRadian(PI) << std::endl;
	std::cout << PI * RADIAN << std::endl;

	return 0;
}

Comments

[actorbug]

getRadianの引数の型がintなのは、何か意図があってのことでしょうか。

[退会済みユーザー]

コーディングミスです。修正しました。

[actorbug]

型を変えても、まだ大きな誤差がでるのでしょうか。

[KoichiSugiyama]

質問からちょっと外れるかもしれませんが、手持ちの環境(Visual Studio 2015でWin32コンソールアプリケーション(C++)プロジェクトを作ってご提示されたソースをビルドして実行してみましたところ、問題とされている結果が再現しません(どちらも180になっています)。どういった環境を使用されているか記載していただいた方が良いかもしれません。

Answer 1

defineでなければいけない場合以外はdefineの方が良い場面が思いつきません。
constexprでgetRadianを定義するのが良いと思います。

Answer 2

#define じゃないと解決策がない場合にしか#define使いません僕は。

Answer 3

関数の方は計算がまずいですな

double t = (PI * 2.0)*a/360;

としよう

Comments

[退会済みユーザー]

質問ですがそれは誤差を減らすためですか？変数に一旦入れるとその時点で値が切落されるためでしょうか？

[y_waiwai]

実際に実行させてみたらどーなります？

Answer 4

C++ではinline関数が使えます。#defineより保守性が向上するそうです。