Q&A
コメントありがとうございます。
別のランダム発生の関数を試してみようと思います。
また、少し言葉足らずだったのですが、x＝0とした時に"常に"負の値が帰ってきてしまうことで現在頭を悩まされておりました。
現在C++の関数を自作しており、取得した値に対しランダム関数を用い、値の更新を行う関数を作成しています
作成した関数は以下の通りです。
int getrandom(uchar x){
srand(time(NULL));
int pattern = rand() % 21 - 10;
int x1 = x;
x1 = x1 + pattern;
return x1;
}
対象を画像ピクセルの値としているため読み込みをucharとしており、その値へint型で発生するランダム値を加算してint型でmainへと返すものにしています。そのために一度”int x1 = x;”という式を組み込んでいます。
ここで，毎回入力する値のxがprintfで出力すると，x=0の時にランダム関数で発生する値がマイナスとなります。
これは偶然なのでしょうか？毎回マイナス値となるのはおかしいと思い質問しました。何か解決策をご存知の方、お答えいただければと思います。
よろしくお願いします。
回答3件
0
こんにちは。
特に関係する部分が見当たらないので偶然のような気がします。
数回プログラムを別途起動してみて、毎回x=0の時、乱数が負になるならば、何かあるとは思いますが、謎ですね。
ところで、毎回、srand(time(NULL));するのは止めた方がよいです。
恐らく短時間の間に繰り返し呼ぶと思いますが、time(NULL)が返却する時刻の単位は意外に大きいです。1秒とか1ミリ秒とか。その間、毎回同じ種で乱数が処理化されるため、毎回同じ乱数が生成されます。
投稿2017/10/18 07:09
編集2017/10/18 07:13
総合スコア23272
0
ベストアンサー
int pattern = rand() % 21 - 10;
変数patternの値域は-10 <= pattern <= 11ですから、xが0のとき戻り値の値域はやはりおなじになりますから負になることはなんら不思議ではありません。
またすでに指摘のある通りseedを毎回初期化するのは意味不明かつ、提示のコードの場合極めて高い確率で毎回同じ値が生成されるのでやめましょう。
ところでC++をお使いとのことですが、C++においてrand関数の使用は非推奨になっています。C++11で追加されたstd::random_deviceおよびstd::mt19937の利用を強く推奨します
投稿2017/10/18 07:41
編集2017/10/18 07:43
総合スコア5850
https://wandbox.org/permlink/nIbFnkqMaM9H4bNa
当方の検証では運が良ければ正の値も出ています。
>別のランダム発生の関数を試してみようと思います。
rand/srandはC++においては使うべきではないので、この辺見てがんばってください
http://siv3d.hateblo.jp/entry/2013/02/17/231829
https://cpplover.blogspot.jp/2009/11/c0xrandom.html
確認しました。再度トライしています。
ありがとうございます。
0
x=0の時にランダム関数で発生する値がマイナスとなります。
rand()%21 は 0～20 だから
pattern = rand()%21 - 10 がマイナスになるのは当然あり得る。
x1 = x; x1 = x1 + pattern; は x = 0 なら x1 = pattern; なので
pattern < 0 であったなら、そりゃマイナス値が返ってくるでしょ。
それはそうと、呼び出しのたんびに srand(time(NULL)) してるのはなぜ?
これだと短時間に何度も getrandom(x) すると毎回同じ値が返ってくるよ?
投稿2017/10/18 07:10
総合スコア16614
コメントありがとうございます。
書き方が悪かったですが、x=0とした時に発生するpatternの値が"常に"pattern<0となってしまっていまい、悩んでおります。
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
2017/10/18 07:48
2017/10/18 07:53
2017/10/18 12:37
2017/10/18 13:21