Random value(in C#)

Banishah · June 24th, 2009, 06:55 AM

Hi......this is my question...
1. Write a function that generates a random integer between two integers passed as parameters. Build an ASP.NET page which allows you to enter the lower and upper limits, and generates a set of random numbers in that range
Guide: You may use the following code for the function that generate the random integer
Random r = new Random();
int rnd = r.Next(10);
return ((b-a)*(rnd)/10 + a);

and my code is.....

publicpartialclassQues1 : System.Web.UI.Page{
protectedvoid Page_Load(object sender, EventArgs e){
double min1, max1,num;
int md,randNum;
Random r=newRandom();
min1 = Convert.ToInt32(TextBox1.Text);
max1 = Convert.ToInt32(TextBox2.Text);
if (IsPostBack){
randNum = randomNum(min1, max1);//ERROR APPEAR HERE
Label1.Text = randomNum + " ";
}
}

int randomNum(int min, int max)
{
Random r = newRandom();
int md = r.Next(10);
return ((max - min) * (md) / 10 + min);
}
}
and anyone solve my error......

Lee Dumond · June 24th, 2009, 12:25 PM

You have several problems.

The main problem is that this code doesn't even compile correctly, as you are calling randomNum incorrectly when setting the label text.

The second problem is that this logic does not belong in the Page_Load method, It should be placed in an event handler for the control doing the postback (presumably a button click).

Third, your logic is way too complex. You are declaring variables that you don't need, and declaring them as the wrong type. Also, there is no need to call the Randon constructor twice.

Code:

using System;
public partial class Ques1 : System.Web.UI.Page
{
   protected void Page_Load(object sender, EventArgs e)
   {
   }
   protected void Button1_Click(object sender, EventArgs e)
   {
      int min = Convert.ToInt32(TextBox1.Text);
      int max = Convert.ToInt32(TextBox2.Text);
      Label1.Text = randomNum(min, max).ToString();
   }
   private int randomNum(int min, int max)
   {
      Random r = new Random();
      int rnd = r.Next(10);
      return ((max - min) * rnd / 10 + min);
   }
}

gbianchi · June 24th, 2009, 01:01 PM

Lee.. this thread is duplicate...
Random value(in C#)

and it looks too much like homework..

Lee Dumond · June 24th, 2009, 01:13 PM

Yes, I figured it was homework. What's wrong with asking for help with your homework?

gbianchi · June 24th, 2009, 01:17 PM

Nothing.. but Isn't a thread for homework?? He's learning, asking is a good way to learn :)...

But isn't better to point him in the right direction, instead of solving the problem???

Lee Dumond · June 24th, 2009, 01:26 PM

Quite frankly, based on what he posted, it appeared he didn't have a good grasp on the tools required to even begin solving the problem, no matter how many "pointers" he was given.

I teach some computer classes at a local college. Sometimes, if I assign a series of questions, I find it helps to show the students a complete solution to one of the easy ones. Then, by studying my solution, they can work out the rest on their own.