Random возвращает одни и те же значения

[pa6kevi4]

Скажите, почему Random возвращает одни и те же значения???
Тот который за цвет отвечает. Цвет у всех фигур получается одинаковым....

 
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace Number3
{
    class Program
    {
        static void Main(string[] args)
        {
            /*Rectangle figure;

            figure = new Rectangle(20, 10);
            System.Console.Write(figure.Color + " " + figure.getTheType() + " with the following area: " + figure.Area + "\n");

            figure = new Square(20, 10);
            System.Console.Write(figure.Color + " " + figure.getTheType() + " with the following area: " + figure.Area + "\n");

            figure = new Circle(20, 10);
            System.Console.Write(figure.Color + " " + figure.getTheType() + " with the following area: " + figure.Area + "\n");

            figure = new Triangle(20, 10);
            System.Console.Write(figure.Color + " " + figure.getTheType() + " with the following area: " + figure.Area + "\n");
            */

            Rectangle figure;

            int[] figureArray;
            figureArray = new int[15];
            Random rand = new Random();

            for (int count = 0; count < 15; count++)
            {
               figureArray[count] = rand.Next(4) + 1;
               switch (figureArray[count])
               {
                   case 1: figure = new Rectangle(rand.Next(50) + 1, rand.Next(50) + 1); break;
                   case 2: figure = new Square(rand.Next(50) + 1, 0); break;
                   case 3: figure = new Circle(rand.Next(50) + 1, 0); break;
                   case 4: figure = new Triangle(rand.Next(50) + 1, rand.Next(50) + 1); break;
                   default: figure = new Rectangle(0, 0); break;
               }
               System.Console.Write(figure.Color + " " + figure.getTheType() + " , his area: " + figure.Area + "\n");
            }

            System.Console.ReadKey();
        }
    }

    class Rectangle
    {
        protected double a;
        public double A
        {
            set
            {
                a = value;
            }
            get
            {
                return a;
            }
        }
        protected double b;
        public double B
        {
            set
            {
                b = value;
            }
            get
            {
                return b;
            }
        }
        public string Color
        {
            get
            {
                return defColor(); 
            }
        }
        public double Area
        {
            get
            {
                return getArea();
            }
        }

        //constructor
        public Rectangle(double a_side, double b_side)
        {         
            a = a_side;
            b = b_side;
        }

        public virtual string getTheType()
        {
            return "rectangle";
        }

        public virtual double getArea()
        {
            return a * b;
        }

        public string defColor()
        {
            Random rnd = new Random();
            int num = rnd.Next(7) + 1;
            switch (num)
            {
                case 1: return "Red";
                case 2: return "Orange";
                case 3: return "Yellow"; 
                case 4: return "Green"; 
                case 5: return "Lightblue"; 
                case 6: return "Blue"; 
                case 7: return "Purple"; 
                default: return "Unknown color"; 
            }
        }
    }

    class Square : Rectangle
    {
        public Square(double a_side, double b_side)
            : base(a_side, b_side)
        {
        }

        public override string getTheType()
        {
            return "square";
        }

        public override double getArea()
        {
            return a * a;
        }
    }

    class Circle : Rectangle
    {
        public Circle(double radius, double b_side)
            : base(radius, b_side)
        {
        }

        public override string getTheType()
        {
            return "circle";
        }

        public override double getArea()
        {
            return 3.14 * (a * a);
        }

    }

    class Triangle : Rectangle
    {
        public Triangle(double catet_a, double catet_b)
            : base(catet_a, catet_b)
        {
        }

        public override string getTheType()
        {
            return "triangle";
        }

        public override double getArea()
        {
            return (a * b) /2;
        }
    }
}

[Serge_Bliznykov]

попробуйте заменить строчку

           Random rand = new Random();

на строчку

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

[pa6kevi4]

Сообщение от Serge_Bliznykov

попробуйте заменить строчку
на строчку

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

Всё к сожалению по прежнему.

Заметьте, у меня 2 рандома.
Random rnd = new Random(DateTime.Now.Millisecond);
Random rand = new Random(DateTime.Now.Millisecond);

Я их оба заменил, но не сработало.

[pa6kevi4]

ну кто знает в чем дело?

[BOBAH13]

Вроде так

var value = (new Random()).Next(int.MaxValue);

[Serge_Bliznykov]

pa6kevi4
рандом, судя по всему построен на получении числа иницилизируя последовательность от системного таймера. скорость выполнения такова, что вызовы defColor() происходят БЫСТРЕЕ, чем успевает смениться таймер.

пути решения проблемы.
Либо, перекроить программу, вынеся определения генератора случайных чисел
Random rnd = new Random();
из процедуры public string defColor()
куда нибудь в высший уровень (на уровень Class Program, например).


либо изменить принцип генерации случайной величины.
например, так:

        public string defColor()
        {
            int numPrev = (new Random()).Next(7) + 1;
            int num = (new Random()).Next(7) + 1;
            while (num == numPrev) {
              num = (new Random()).Next(7) + 1;            	
            }
            switch (num)
      ......

[pa6kevi4]

Спасибо, сам бы точно не догадался