C# method and table random numbers [closed]

Question

I've method which put the numbers 1 to 9 in a random order. Each number appears only once. The method should return an integer table.

This is my code which facing the wall, I ran out of ideas with this code. I know that this code is 100% wrong.

    class Program
    {
        static void Main(string[] args)
        {
            int luvut = { 1, 2, 3, 4, 5, 6, 7, 8, 9 };
            Console.WriteLine(luvut);
            Console.ReadKey();
        }

        private static int Kuuluuko(int luvut)
        {
            for (int i = 0; i < luvut.Length; i++)
            {

                return;

            }
        }
    }
}

So Kuuluuko is supposed to randomize them? What have you tried? Also if you compile this what errors do you get? — Prescott
– Prescott, Commented Dec 16, 2011 at 9:18
please explain in more details what you want to do, not clear question — Arsen Mkrtchyan
– Arsen Mkrtchyan, Commented Dec 16, 2011 at 9:18

Cheng Chen · Accepted Answer · 2011-12-16 09:21:28Z

1

var randomNumbers = Enumerable.Range(1,9).OrderBy(n => Guid.NewGuid()).ToList();

answered Dec 16, 2011 at 9:21

Cheng Chen

43.7k18 gold badges118 silver badges184 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Rawling Over a year ago

Looking at the example code, I'm not sure just giving a working answer is going to be much help :)

coodienoobie Over a year ago

can you please put the hole method code

Fischermaen Over a year ago

@DannyChen: OP is a really newbie in C#, your code is great, but will this help him?

Øyvind Bråthen · Accepted Answer · 2011-12-16 09:23:42Z

0

What about something like this? Make a list, and remove the numbers from that list as you use them up. Like drawing numbers from a bowl or something like that :)

var numbers = new List<int>(new[]{ 1, 2, 3, 4, 5, 6, 7, 8, 9});
var randomnumbers = new List<int>();
var rnd = new Random();
while( numbers.Count > 0 ){
  var index = rnd.Next(0, numbers.Count);
  randomnumbers.Add(numbers[index]);
  numbers.RemoveAt(index);
}
//randomnumbers now contain the random sequence

answered Dec 16, 2011 at 9:23

Øyvind Bråthen

60.9k28 gold badges128 silver badges154 bronze badges

Comments

Fischermaen · Accepted Answer · 2011-12-16 09:24:10Z

0

Your code has a couple of errors and didn't even compile. Here is the code running and giving you what you expect:

class Program
{
    static void Main(string[] args)
    {
        int[] luvut = { 1, 2, 3, 4, 5, 6, 7, 8, 9 };
        Console.WriteLine(Kuuluuko(luvut));
        Console.ReadKey();
    }
    private static int Kuuluuko(int[] luvut)
    {
        var random = new Random();
        return luvut[random.Next(0, luvut.Length)];
    }
}

answered Dec 16, 2011 at 9:24

Fischermaen

12.5k2 gold badges43 silver badges56 bronze badges

3 Comments

Hybrid Over a year ago

is it? it still doesn't call the Kuuluuko method :)

Hybrid Over a year ago

ah, now it does! however, it still only returns one int from the list, rather than a reordered list.

Fischermaen Over a year ago

@Hybrid: Yes, I missed the call to Kuuluuko (what that means in Finnish).

Collectives™ on Stack Overflow

C# method and table random numbers [closed]

3 Answers 3

3 Comments

Comments

3 Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

Comments

3 Comments

Linked

Related