Расстановка чисел

[Defunate]

Задача такая:расставить числа из заданного диапазона,без повторения.

  public static void Main()
        {
            int m = 10;
            int n = 10;
            List<int>[] mylist = new List<int>[4];
            for (int i = 0; i < mylist.Length; i++)
            {
                mylist[i] = new List<int>();
                mylist[i].AddRange(Generate(m, n));
            }
           }

       static IEnumerable<int> Generate(int max, int count)
       {
           return Generate(new Random(), max, count);
        }

        static IEnumerable<int> Generate(Random random, int max, int count)
        {
            max++;
            if (max < count)
                throw new ArgumentException();

            var indices = new int[count];
            for (var i = 0; i < count; i++)
            {
                indices[i] = random.Next(1,max);
                max--;
            }

            for (var i = 0; i < count; i++)
            {
                var value = indices[i];
                for (var j = i - 1; j >= 0; j--)
                {
                    if (indices[j] <= value)
                        value++;
                }

                yield return value;
            }
        }

Вопрос собственно такой,почему при заполнении каждой коллекции,в каждой коллекции оно ставит числа одинаково,а нужно-каждый раз поновому

[dampirik]

А может имеет смысл создать массив из элеметов по порядку 1.2.3.4.5.6... и потом их просто перемешать?

[Defunate]

Так получиться что у меня всегда опять тоже будет перемешивание по одному принципу,первое-туда,второе-сюда и тд.мне же надо каждый раз новую последовательность.

[dampirik]

Я не совсем понимаю задание походу... нужно взять последовательность (упорядочную) определенной длинны и перемешать ее в хаотично порядке? так? и у Вас получается, что каждый раз после перемешивания последовательности равны?

[Defunate]

Обьясню на числах: допустим у меня есть 5 чисел 1,2,3,4,5
мне нужно 1-ый раз допустим получить 2,1,3,5,4 дальше получить 1,3,5,2,4 и так далее,тоесть каждый раз получать новую перестановку из заданного множества

[Скарам]

Попробуй так, думаю, дело в этом:

return Generate(new Random(DateTime.Now.Millisecond), max, count);

Или сохранять ссылку на Random.

[Defunate]

return Generate(new Random(DateTime.Now.Millisecond), max, count);

Вторая коллекция получила новые значения,а последующие-как и вторая коллекция)

[Скарам]

using System;
using System.Collections.Generic;

namespace ConsoleApplication1
{
    internal class Program
    {
        public static void Main()
        {
            int m = 10;
            int n = 10;
            var mylist = new List<int>[4];
            for (int i = 0; i < mylist.Length; i++)
            {
                mylist[i] = new List<int>();
                mylist[i].AddRange(Generate(m, n));
            }
        }

        private static Random random = new Random(DateTime.Now.Millisecond);
        private static readonly object syncLock = new object();

        private static IEnumerable<int> Generate( int max, int count)
        {
            max++;
            if (max < count)
                throw new ArgumentException();

            var indices = new int[count];
            for (int i = 0; i < count; i++)
            {

                lock (syncLock)
                    indices[i] = random.Next(1, max);
                    max--;
                
            }

            for (int i = 0; i < count; i++)
            {
                int value = indices[i];
                for (int j = i - 1; j >= 0; j--)
                {
                    if (indices[j] <= value)
                        value++;
                }

                yield return value;
            }
        }
    }
}

Попробуй так.

[Defunate]

Спасибо!всё работает,а не мог бы ты объяснить в чём была проблема-то

[Скарам]

Случайные числа на самом деле не случайны, а псевдослучайны. По сути последовательность чисел зависит от числа основания. Т.е. если задавать одно и тоже число в конструторе random получишь одну и ту жке последовательность. В конструкторе по умолчанию random() задается какая-то костанта, поэтому сколько его не вызывай после получишь одинаковые последовательности (твой первый код) . Я предложил генерить от времени, но, видимо, милисекунды не успевали меняться (мое первое предложение). Сохранив ссылку на экземпляр класса мы избавились от необходимости менять основание и последовательности получились разные. Подробнее можно почитать в википедии.)