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

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

新規登録して質問してみよう
ただいま回答率
85.50%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Q&A

解決済

1回答

1065閲覧

C#のRandom.Nextが思った通りに動きません

rinmeda317

総合スコア12

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

0グッド

1クリップ

投稿2017/11/29 13:44

###前提・実現したいこと
C#で配列の中身を4回ランダムに作成し、コンソールに表示させようとしたのですが、何故か

Gene0 = 1111000111
Gene1 = 1101101000
Gene2 = 1101101000
Gene3 = 1101101000

というように、何度やり直しても二つ目以降が同じになってしまいます。どこがいけないのでしょう?

###該当のソースコード

c#

1using System; 2 3class Origin //初期データの作成 4{ 5 Random rnd = new Random(); 6 7 public int[] OriginGene() 8 { 9 int[] Gene = new int[10]; 10 for (int i = 0; i < 10; i++) 11 { 12 Gene[i] = rnd.Next(0, 2); 13 } 14 return Gene; 15 } 16} 17 18class Run //実行 19{ 20 public static void Main() 21 { 22 for (int i = 0; i < 4; i++){ 23 Origin gene = new Origin(); 24 int[] Gene = gene.OriginGene(); 25 Console.Write("Gene" + i + " = "); 26 foreach (int ge in Gene) 27 { 28 Console.Write(ge); 29 } 30 Console.WriteLine(" "); 31 } 32 } 33}

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

ozwk

2017/11/29 14:01

mainのforループ内で適当にsleep()などでウェイトかけるとどうなりますか?
rinmeda317

2017/11/29 14:10

sleep()とは何ですか?
guest

回答1

0

ベストアンサー

Randomはデフォルトだとシード値をインスタンス生成時点の時刻で作るので
短いループ内でインスタンス複数作ると高確率でシードが被ります

投稿2017/11/29 14:17

編集2017/11/29 14:20
ozwk

総合スコア13512

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

rinmeda317

2017/11/29 14:22

なるほど、遅延させなければならなかったのですね
ozwk

2017/11/29 14:26

いや、遅延させるのは実際それが原因か確かめる方法の一つで 普通は1つのrandomインスタンス使いまわすとか、シード値が確実にずれるように何かしら手を打つとかします
rinmeda317

2017/11/29 14:31

randomインスタンスを使いまわすとはどうやるのでしょうか
ozwk

2017/11/29 14:43

staticで持つとか外部で作って引数で渡すとか
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問