Need the code for Randomization of numbers using c#

[OriginalGriff]

I am not even going to look at them until you have preserved the formatting - why should I even slightly struggle? Remember the "code block" button above?

No trees were harmed in the sending of this message; however, a significant number of electrons were slightly inconvenienced. This message is made of fully recyclable Zeros and Ones

[Lost User]

I will refrain form making any comment except to point out that you have the following for loop:

for (int i = 0; i < Quescnt; i++)

and within this loop you have the following statement

Quescnt++;

Using the debugger could have helped you locate this fairly quickly.

[J4amieC]

vasavi.p wrote:

and try with this too[this will go to infinite loop]

Now for extra credit (that is, beyond being able to blindly paste your code into this window) can you tell us why this goes into an infinite loop? Hint:- Documentation :Random.Next(Int32,Int32)[^] Relevant part of documentation:

minValue
    Type: System..::.Int32
    The inclusive lower bound of the random number returned.

maxValue
    Type: System..::.Int32
    The exclusive upper bound of the random number returned. maxValue must be greater than or equal to minValue.

[J4amieC]

Irrespective of this error, s/he's not going to get 6 unique random numbers from a range of only 5 possibilities.

[DeepakMega]

Read this first How to post questions[^]

[Lost User]

J4amieC wrote:

Irrespective of this error, s/he's not going to get 6 unique random numbers from a range of only 5 possibilities.

I thought s/he's complaining that it goes into an infinite loop. There seems little point in worrying about anything else until that logic is fixed.

[J4amieC]

I thought it was going into an infinite loop because its looking for the next unique random number, and all possibilities have already been found. :confused:

[Skymir]

Christian Graus wrote:

I think I should become a plumber.

I've been thinking electrician. Plumbing was an idea, but you end up having to deal with septic tanks and pipes clogged with things no one ever wants to know about.

The true man wants two things: danger and play. For that reason he wants woman, as the most dangerous plaything.

[Henry Minute]

I wonder if you can help me? I've been trying to fit a new washer to my kitchen tap for some time. Should I use C# or VB.Net for this?

Henry Minute Do not read medical books! You could die of a misprint. - Mark Twain Girl: (staring) "Why do you need an icy cucumber?" “I want to report a fraud. The government is lying to us all.”

[OriginalGriff]

Neither - it's a hardware problem...

No trees were harmed in the sending of this message; however, a significant number of electrons were slightly inconvenienced. This message is made of fully recyclable Zeros and Ones

[Lost User]

J4amieC wrote:

I thought it was going into an infinite loop because its looking for the next unique random number, and all possibilities have already been found.

In the following code:

for (int i = 0; i < Quescnt; i++)
{
int RandomNumber = RandomClass.Next(28, 33);
if (!Randomnos.ToString().Contains(RandomNumber.ToString() + ","))
{
Randomnos = Randomnos + RandomNumber.ToString() + ", ";
}
else
{
Quescnt++;
}
}

every time it hits the else clause it is incrementing Quescnt so the for loop will keep going because Quescnt will always stay ahead of the loop variable.