[snafu117]

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Windows.Forms;


namespace randomNumber
{
    class Program
    {
        static void Main(string[] args)
        {
            Random numb = new Random();      
            int n = numb.Next(1000);          
            string r;

            string results = "";              
            int counter = 0;

           


        do
        {
                n = numb.Next(1000);
                r = Convert.ToString(n);
            counter++;
            results += counter + ")\t" + r + "\t";
        }

            
                for (int i=2; i <= 1000; i=i+2) //THIS IS THE PART OF THE CODE THATS SUPPOSED TO DISPLAY THE EVEN
                Console.WriteLine(i);           // NUMBERS OF THE 100 RANDOMS, BUT I KEEP GETTING ERROrs
                                                // what am i doing wrong?
        
            
        while (counter < 100&&n<1000);

        

            MessageBox.Show(results, "Random Numbers" , MessageBoxButtons.OK);



        }
    }
}

[modi123_1]

Quote

//THIS IS THE PART OF THE CODE THATS SUPPOSED TO DISPLAY THE EVEN
// NUMBERS OF THE 100 RANDOMS, BUT I KEEP GETTING ERROrs
// what am i doing wrong?

It's always good to share your errors your are getting..

[Curtis Rutland]

"between 0 & 1"

...there are no numbers between 0 and 1.

[Ryano121]

Unless they mean 100 random decimal numbers between 0 and 1?

In which case you want to be using the .NextDouble method instead of .Next.

But then again they seem to be wanting to find all the even numbers, in which case using decimals wouldn't work.

This post has been edited by Ryano121: 27 March 2012 - 12:40 PM

[modi123_1]

Bah.. there are plenty of numbers between 0 and 1.. not many of them are WHOLE numbers but they are numbers regardless of their handicaps..

[snafu117]

they are not decimals, they are integers. the code finds 100 random integers between 0 and 1000, this part of the code is done, now im trying to sort the even numbers into a separate column in the message box that appears after i run the code, i keep getting syntax errors, not exactly sure what the syntax errors are but there are an error for the for and while

[Ryano121]

What do you mean you don't know what the error messages are?

Copy and paste the errors you get when you try to compile in Visual Studio and post them here.

[snafu117]

"The name 'MessageBox' does not exist in the current context"

"The name 'MessageBoxButtons' does not exist in the currentcontext"


looks like the errors for "for" and "while" have went away. anyway those are the messages that appear now.

[tlhIn`toq]

Start by cleaning up your code.

What's with lines 33 and 34 sitting between your DO and WHILE statements? If you frak up all the syntax and completely confuse the compiler you will get errors where you think you shouldn't.

Since you are using a random number to start with, just adding 2 to it doesn't guarantee its even. Use the modulus function to determin if it is even

x = myRandom.NextRandom();
if (x%2 == 0) // then its even

[negligible]

snafu117, on 27 March 2012 - 09:37 PM, said:

"The name 'MessageBox' does not exist in the current context"

Most of the time this error means you are missing a Using directive.

Right click on the red underlined "MessageBox" in your code, then you'll seeing an option called "Resolve" above refactor which should fill in the missing directive for you.

The other common one is trying to use something out of it's scope, for example trying to use a variable you declared in method in another separate method. (Instead of declaring it at class level so you can use it in both)

This post has been edited by negligible: 28 March 2012 - 08:04 AM

[tlhIn`toq]

negligible, on 28 March 2012 - 09:01 AM, said:

snafu117, on 27 March 2012 - 09:37 PM, said:

"The name 'MessageBox' does not exist in the current context"

Most of the time this error means you are missing a Using directive.

There is no need for guessing about that. The OP does have the proper using directive.
05 using System.Windows.Forms;



Mind you he's trying to call it from a console app. I'm not sure that's even possible. Hmmm... Something to play with

Like I said... Fix your code. You can't have statements between the Do{} and While blocks.
Visual Studio is even telling you what it expected. Just follow the tooltips.



Yep, if you fix the stuff its telling you to fix, you are indeed able to use a MessageBox from a console app.

[snafu117]

thank you all :] i think i have it figured out! ill be sure to + you guys! thanks a lot!