C# 코드는 단계별로 예상 결과 만 제공합니까?

Question

좋아요. 주사위를 던지려면 ...

코드를 단계별로 실행하면 정상적으로 작동하며 '결과'에 정확한 수의 결과가 포함되어 있으며 코드를 남겨 둘 때 임의로 표시됩니다. 똑같은 번호를 만들어 낸 것과 똑같은 일을합니다.

나는 이것이 볼 수없는 논리적 오류라고 확신하지만 몇 시간 동안은 그다지 신경 쓰지 않아도 상황이 개선되지 않으므로 도움이 많이 필요하다. :)

class Dice 
{ 

    public int[] Roll(int _throws, int _sides, int _count) 
    { 
     Random rnd = new Random(); 
     int[] results = new int[_throws]; 
     // for each set of dice to throw pass data to calculate method 
     for (int i = 0; i < _throws; i++) 
     { 
      int thisThrow = Calculate(_sides, _count); 
      //add each throw to a new index of array... repeat for every throw 
      results[i] = thisThrow; 
     } 

     return results; 
    } 


    private int Calculate(int _sides, int _count) 
    { 
     Random rnd = new Random(); 
     int[] result = new int[_count]; 
     int total = 0; 
     //for each dice to throw put data into result 
     for (int i = 0; i < _count; i++) 
     { 
      result[i] = rnd.Next(1, _sides); 
     } 
     //count the values in result 
     for (int x = 0; x < _count; x++) 
     { 
      total = total + result[x]; 
     } 
     //return total of all dice to Roll method 
     return total; 
    } 
} 

Answer 1

첫 번째 실수 : 단일 인스턴스를 사용하고, 통과, 임의의 여러 인스턴스를 사용하지 마십시오 그 다른 매개 변수와 함께.

Answer 2

생성자에게 임의의 시드를 지정합니다. 그것이 문제이다.

http://msdn.microsoft.com/en-us/library/aa329890%28VS.71%29.aspx

Random r = new Random(DateTime.Now.Millisecond); 

Answer 3

"임의의 rnd = 새 임의();" 그것은 현재 시간에 의해 파종된다. 시간이 걸리는 코드를 디버깅 할 때마다 매번 다르게 시드됩니다.

무작위로 1 인스턴스를 만들고 모든 곳에서 참조하십시오.

Answer 4

숫자를 만들 때마다 임의의 클래스를 만듭니다. 이렇게하면 열매 맺는 결과를 얻을 수 있습니다.

는 여기를 참조하십시오 : FROM MSDN

이 문제는 여러 사람이 아니라 하나의 임의의 객체를 생성 방지 할 수 있습니다.

성능을 향상 시키려면 하나의 임의 번호를 생성하기 위해 새로운 임의 개체를 반복적으로 만드는 대신 하나의 임의 개체를 만들어 시간 경과에 따라 많은 난수를 생성하십시오.

예. 등 무작위 이미지를 선택 주사위, 카드 게임 같은 것들에 대한 ... 앞서 언급 된 내용 외에도

Answer 5

...

를 사용하여 랜덤를 임의의 개인 인스턴스를 만듭니다. 보안을 위해 임의의 숫자를 생성해야하는 경우 System.Security.Cryptography.RandomNumberGenerator를 사용하십시오. 이 간단한 예제에서는 임의의 정수를 만드는 방법을 보여줍니다.

 RandomNumberGenerator gen = RandomNumberGenerator.Create(); 
     byte[] myBytes = new byte[4]; 
     gen.GetBytes(myBytes); 
     int myValue = (BitConverter.ToInt32(myBytes, 0)); 

보안이 필요한 경우가 아니면 이것을 사용하지 마십시오. 성능은 Random 클래스의 성능보다 작습니다. 나는 당신이 무작위로 씨앗을 뿌리기 위해 이것을 사용할 수 있다고 생각하지만 과잉이라고 할 수 있습니다.

편집 : 내가 이것을 테스트 한 적이 없다는 것이 나에게 발생했습니다. 빠른 성능 테스트는 다음을 보여주었습니다.

1,000,000 임의의 숫자 : RandomNumberGenerator : 2.6 초 랜덤 : .015 초.

그래서 무작위는 약 150 배 빠릅니다.