質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.58%

乱数で範囲を指定したい

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 2,034

h_proc

score 59

 前提・実現したいこと

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/ツールのバージョンなど)

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+2

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;
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/11/23 15:05

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

    キャンセル

  • 2018/11/23 15:09

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

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

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

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

    キャンセル

  • 2018/11/23 15:19

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

    キャンセル

+1

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 へ 半分だけ進んだ値を返します。
 

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/11/23 15:13

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

    キャンセル

  • 2018/11/23 15:51

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

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.58%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る