乱数で範囲を指定したい

Question

前提・実現したいこと

rand関数を使って範囲を指定して乱数を発生させたいです。
-0.05から0.05の範囲に指定したいのですが、実行すると、0.00しか表示されません。
ここに質問の内容を詳しく書いてください。
関数が呼び出されるたびに、乱数を返したいです。
（例）PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

エラーメッセージ

該当のソースコード

#include <stdio.h>
#include <stdlib.h>
int rnd_generate(double min, double max);

int rnd_generate(double min,double max)
{
	double dt = 0;
	
	dt = min + (double)(rand() / (max - min + 1.0)) * (RAND_MAX + 1.0);

	return dt;
}
int main(void)
{
	
		printf("%f\n", rnd_generate(-0.05, 0.05));

	return 0;
}

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

int rnd_generate(double min,double max)
{
...
return dt;
}

返り値をint型にキャストする処理が走っています。適切な型(double)を指定してください。

---

rnd_generateの実装自体も間違っていそうな雰囲気です。
関数を適切に分けた方が書きやすいと思いますが。

double d_rand(void) {
    return 1. * rand() / RAND_MAX;
}
double d_rand_range(double min, double max) {
    return d_rand() * (max - min) + min;
}

Comments

[h_proc]

回答ありがとうございます。恐れ入りますが、int型にキャストする処理というのはどういうものを指すのでしょうか。関数を分けて再度作ってみようと思います。やはり、関数一つだと難しいのでしょうか。

[LouiS0616]

> int型にキャストする処理というのはどういうものを指すのでしょうか

戻り値にint型を指定しているので、return dt; は実際には return int(dt); のように動作します。
小数点以下切り捨てです。

> 関数一つだと難しいのでしょうか。

一般に、物事は適切に分割すると分かり易くなります。
私の場合『0から1の浮動小数の乱数を返す関数』が切り離されている方がしっくりくるだけです。

[h_proc]

丁寧に教えていただき、ありがとうございました。無事、ソースコードが完成し、正しく実行されました。

Answer 2

r.c

#include <stdio.h>
#include <stdlib.h>

double rnd_generate(double range_0, double range_1) {
  double t = (double)rand() / RAND_MAX;
  return range_0 + (range_1 - range_0) * t;
}

int main(void) {

  for (int i = 0; i < 10; i++) {
    printf("%f\n", rnd_generate(-0.05, 0.05));
  }
  printf("\n");

  for (int i = 0; i < 10; i++) {
    printf("%f\n", rnd_generate(0.05, -0.05));
  }
  printf("\n");

  for (int i = 0; i < 10; i++) {
    printf("%f\n", rnd_generate(0.05, 0.05));
  }
  printf("\n");

  return 0;
}

実行例

範囲指定の数値の大小関係が逆転していたり、同じ値を指定したときにも、範囲中の乱数が得られることがわかります。

rnd_generate（）中の t は 0.0 ,, 1.0 の範囲の値になります。
t = 0 なら、range_0 を返します。
t = 1 なら range_1 を返します。
t = 0.5 なら range_0 から renge_1 へ 半分だけ進んだ値を返します。

Comments

[h_proc]

回答ありがとうございます。同様のソースコードを書いてみたのですが、やはり、0.00と表示されてしまいます。もう少し考えてみます。

[katoy]

> ... 同様のソースコード ...

質問文の中の int rnd_generate(double min,double max) を
double rnd_generate(double range_0, double range_1)に置き換えるだけで済むはずです。
int rnd_generate(...) と int を返すままになっているのではと想像されます。double rnd_generate(...) とすることが重要な変更点の１つです。