[poncho4all]

So i have this code, where im trying to generate random numbers into an array, but i dont want the numbers of the array to be equal, or repeat in other words ive tryed alot of things but i guess im not looking at this the correct way, there must be an easy way to do this. Im new to C# so i dont know all the methods yet.

private void btnStart_Click(object sender, EventArgs e)
		{
			Random ran = new Random();
			int[] randomNum = new int[6];
			for (int j = 0; j < 6; j++)
			{
				randomNum[j] = ran.Next(imageNameArray.Length);
			}
			picOne.Image = Image.FromFile(imageNameArray[randomNum[0]]);
			picTwo.Image = Image.FromFile(imageNameArray[randomNum[1]]);
			picThree.Image = Image.FromFile(imageNameArray[randomNum[2]]);
			picFour.Image = Image.FromFile(imageNameArray[randomNum[3]]);
			picFive.Image = Image.FromFile(imageNameArray[randomNum[4]]);
			picSix.Image = Image.FromFile(imageNameArray[randomNum[5]]);
		}

basicly when I hit the start button this code will run, what it has to do is create a 6 new slots in an array and then fill them with random numbers wich will be used to select six images from a file. Sometimes it gets reapeted images and i dont want that, the way to do according to me is by going throw the random numbers array and see that they have no reapeted ones.

But i fail at that, if anyone could help me i would greatly apreciate it.

This post has been edited by poncho4all: 20 December 2009 - 03:58 PM

[Renagado]

You could swap the items in imageNameArray first(randomly) and then load/use them in order of the array. That way you'll never get the same image more than once. It's the same kind of algorithm as what is used for shuffling a virtual deck of cards, search for that is my advice.

[remorseless]

private void btnStart_Click(object sender, EventArgs e)
		{
			Random ran = new Random();
			int[] randomNum = new int[6];
			for (int j = 0; j < 6; j++)
			{
								int current_int = ran.Next(imageNameArray.Length);
								while(randomNum.contains(current_int))current_int= ran.Next(imageNameArray.Length);
				randomNum[j] = current_int;
			}
			picOne.Image = Image.FromFile(imageNameArray[randomNum[0]]);
			picTwo.Image = Image.FromFile(imageNameArray[randomNum[1]]);
			picThree.Image = Image.FromFile(imageNameArray[randomNum[2]]);
			picFour.Image = Image.FromFile(imageNameArray[randomNum[3]]);
			picFive.Image = Image.FromFile(imageNameArray[randomNum[4]]);
			picSix.Image = Image.FromFile(imageNameArray[randomNum[5]]);
		}

I haven't tested it, it's just on the spot coding, tell me if it works or not.

[poncho4all]

so contain is a member of the array class, and it will go throw the array looking for similarities on the new number and each of the number in the array, right?

[remorseless]

poncho4all, on 21 Dec, 2009 - 09:13 AM, said:

so contain is a member of the array class, and it will go throw the array looking for similarities on the new number and each of the number in the array, right?

Yeah, It'll go through each of the elements in the array and look for similarities in what you entered. Again, make sure to check that it works because I'm not too positive that it works, I wrote it on the spot without checking. Hope I helped

[poncho4all]

You sure did helped it works just fine, thanks much.

[FlashM]

I think that a solution that Renagado proposed is really best one, because it's simple and gives you just the result you need.

Of course, generating random numbers and then getting image files is also a possibility, but is not real most optimal.