I want to create 5 random numbers between 2 numbers and display in my textbox

Question

Im just learning C# at uni and seem to be struggling to get this random number to out put in a textbox. I havnt learnt a language before this so apologies if its a simple question.

I want to create 5 random numbers, between two different numbers specified by the user. Then print the 5 numbers generated into a textbox to display them.

private void button1_Click(object sender, EventArgs e) {

        int firstnum = Int32.Parse(txtboxnum.Text);
        int secondnum = Int32.Parse(txtboxnum2.Text);


        Random random = new Random();
        int randomNumber = random.Next(firstnum, secondnum);

        Int32 loop = 0;
        do
        {
            loop++;
        }
        while (loop > 5);

        string myString = randomNumber.ToString();

       string  txtboxanswer = myString;

I would be much appreciative of any suggestions where im going wrong. As i just cant seem to get the answer to appear in my textbox. I know i must be close. I hope :)

Thanks for all help in advance.

you need all the 5 values in same text box altogether at a time? — Karthik
– Karthik, Commented Oct 24, 2012 at 18:58

Sam I am says Reinstate Monica · Accepted Answer · 2012-10-24 19:02:52Z

1

by the way, this is how I'd do it

        int firstnum = 0;
        int secondnum = 1;
        Random random = new Random();

        int[] randomnums = new int[5];

        for (int i = 0; i < randomnums.Length; i++)
        {
            randomnums[i] = random.Next(firstnum, secondnum);
        }

and then you can do something like

string myString = randomnums[1].ToString()

to get the string representation of a particular number.

answered Oct 24, 2012 at 19:02

Sam I am says Reinstate Monica

31.3k12 gold badges74 silver badges101 bronze badges

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

Comments

Björn Kaiser · Accepted Answer · 2013-02-21 10:56:47Z

1

You are close enough. With minimum editing of your code, it should be pretty much like this

int firstnum = Int32.Parse(txtboxnum.Text);
    int secondnum = Int32.Parse(txtboxnum2.Text);


    Random random = new Random();


    Int32 loop = 0;
    do
    {
       int randomNumber = random.Next(firstnum, secondnum);
       string myString = randomNumber.ToString();
       TextBox t= new TextBox();
       t.Text=myString; 
       t.Left=0;t.Top=loop * 20;
       this.Controls.Add(t);

       loop++;
    }
    while (loop < 5);

For more on adding controls dynamically check this

edited Feb 21, 2013 at 10:56

Björn Kaiser

9,9024 gold badges39 silver badges57 bronze badges

answered Oct 24, 2012 at 19:01

Arjun Shetty

1,5851 gold badge15 silver badges36 bronze badges

2 Comments

Nikola Davidovic Over a year ago

It should be while(loop < 5);

Nikola Davidovic Over a year ago

Also you should put the code for generating new random number inside the loop. For each iteration there should be new generated random number.

Sleiman Jneidi · Accepted Answer · 2012-10-24 19:01:51Z

0

Simply, you did not set the Text property of the TextBox to the result.

txtBoxName.Text= myString

answered Oct 24, 2012 at 19:01

Sleiman Jneidi

23.4k14 gold badges60 silver badges80 bronze badges

3 Comments

user1735367 Over a year ago

string txtboxanswer.Text = myString; This gives me a red line under the txtboxanswer.Text am i missing something else?

user1735367 Over a year ago

got it. take the string out at the start. Thanks

Sleiman Jneidi Over a year ago

@user1735367 I assume that you have a TextBox called txtBoxName

CloneXpert · Accepted Answer · 2012-10-24 19:01:03Z

0

you are on the right track, I think the only thing you are missing is to assign the string to a textbox. Just place another textbox (let's say answertxtbox) and write in the end of your loop the following:

answertxtbox.Text += txtboxanswer + ", ";

answered Oct 24, 2012 at 19:01

CloneXpert

2462 silver badges9 bronze badges

Comments

Rohit Vats · Accepted Answer · 2012-10-24 19:13:26Z

0

Try this :

Random rand = new Random();
for (int i = 0; i < 5; i++)
{
    int a = rand.Next(Min, Max);
    textBox1.Text =  textBox1.Text + string.Format(" {0} ", a.ToString());
}

edited Oct 24, 2012 at 19:13

Rohit Vats

81.5k13 gold badges170 silver badges193 bronze badges

answered Oct 24, 2012 at 19:00

Ali Vojdanian

2,1232 gold badges32 silver badges49 bronze badges

1 Comment

Rohit Vats Over a year ago

What's the use of for loop here? You are overwriting last value after every iteration.

RobH · Accepted Answer · 2012-10-24 19:24:46Z

Just because everyone seems to be answering...

 int firstnum = Int32.Parse(txtboxnum.Text);
 int secondnum = Int32.Parse(txtboxnum2.Text);
 Random random = new Random();

 List<int> results = new List<int>();
 for (int i =0; i < 5; i++)
 {
     results.Add(random.Next(firstnum, secondnum));
 }

 answertxtbox.Text = String.Join(",", Array.ConvertAll<int, String>(results.ToArray(), Convert.ToString));

Note: I wouldn't actually do it this way but it does illustrate that there are many, many ways of solving the same problem.

Collectives™ on Stack Overflow

I want to create 5 random numbers between 2 numbers and display in my textbox

6 Answers 6

Comments

2 Comments

3 Comments

Comments

1 Comment

Comments

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

Comments

2 Comments

3 Comments

Comments

1 Comment

Comments

Related