Добавить числа путем генерации случайных задач, содержащих числа

Question

public TestClass{ 

    public Task<int> GetRandomNumber() { 
       return Task.FromResult(new Random().Next(0, 1500)); 
      } 
    } 



public class Default 
    { 
     static void Main(string[] args) 
     { 
      var test = new TestClass(); 
      List<int> adddata = new List<int>(); 
      for (int i = 0; i < 3; i++) 
      { 
       var result = Task.Run(() => test.GetRandomNumber()); 
       Console.WriteLine("The values that will be added are :{0}", result.Result); 
       adddata.Add(result.Result);  
      } 
      Console.WriteLine("The value is :{0}", adddata.Sum(v => v)); 
    } 
} 

Проблема случайного числа возвращает одинаковые числа, 68, 68, 122, когда он должен возвращать отличное число, что я делаю неправильно. Я пытаюсь узнать, как использовать задачу в C# .THANKS!

Answer 1

Потому что каждый раз вы генерируете новый экземпляр Random. Создайте объект Random один раз. Как Random не поточно, мы должны использовать генератор случайных чисел, который:

public static class ThreadSafeRandom 
{ 
    private static Random global = new Random(); 

    [ThreadStatic] 
    private static Random local; 

    public static int Next() 
    { 
     Random inst = local; 
     if (inst == null) 
     { 
      int seed; 
      lock (global) 
      { 
       seed = global.Next(); 
      } 

      local = inst = new Random(seed); 
     } 

     return inst.Next(); 
    } 
} 

А теперь потреблять:

public TestClass 
{ 
    public Task<int> GetRandomNumber() 
    { 
     return Task.FromResult(ThreadSafeRandom.Next()); 
    } 
} 

Edit:

В стороне Обратите внимание, что вы выполняете свой делегат в потоке threadpool и сразу же синхронно блокируете его с помощью Result. Я предполагаю, что вы хотите сделать, это выполнить их в параллель:

static void Main(string[] args) 
{ 
    var test = new TestClass(); 

    Task<int>[] addData = Enumerable.Range(0, 4) 
            .Select(_ => Task.Run(() => test.GetRandomNumber())) 
            .ToArray(); 

    Task.WaitAll(addData); 
    foreach (var result in addData) 
    { 
     Console.WriteLine("The values that will be added are :{0}", result.Result); 
    } 

    Console.WriteLine("The value is :{0}", adddata.Select(x => x.Result).Sum()); 
} 

Edit 2:

Согласно @ChrisL правильно указывая Random не поточно, я модифицировал код для использования беспорядочного случайного генератора, предоставленного the PFX team.

Answer 2

Посмотрите на этот пост здесь: Thread safe random number generation

Короче: Random() не поточно и не было разработано, чтобы быть.