Can someone help me figure out what wrong with my code?

Question

1.00/5 (1 vote)

See more:

==== i am trying to figure out why, when i click the spin button my code doesn't run. it freeze.

int num1;
           int num2;
           int num3;
           Random RndNumber = new Random();
           double cash;
           string addCash;
           for (; ; )
           {
               cash = Convert.ToDouble(txtAdd.Text);
               addCash = cash.ToString();
               label2.Text = addCash;
               if (label2.Text == "")
               {
                   MessageBox.Show("YOU NEED TO ADD MORE BEFORE PLAING");
                   break;
               }
               else
               {
                   num1 = RndNumber.Next(0, 10);
                   num2 = RndNumber.Next(0, 10);
                   num3 = RndNumber.Next(0, 10);

                   btnOne.Text = num1.ToString();
                   btnTwo.Text += num2.ToString();
                   btnThree.Text += num3.ToString();

                   if((btnOne.Text==btnTwo.Text)&&(btnTwo.Text==btnThree.Text))
                   {
                       cash++;
                       MessageBox.Show("Congrats! you win");
                   }
               }

What I have tried:

i am new to coding so i tried debugging.

Posted 13-May-23 3:36am

Mohamed Jawo

Updated 13-May-23 4:22am

Add a Solution

Comments

Member 15627495 13-May-23 9:43am

Hello !

what are you expecting with the "for loop" ?

Kenneth Haugland 13-May-23 9:45am

I think you have some logical flaws in your code. Why not check if the num1 == num2 and num2 == num3 instead?

0x01AA 13-May-23 9:51am

Looks like you only have to wait long enough until the random number num1, num2 and num3 will be the same...

Member 15627495 13-May-23 9:59am

your code is not freezing, but "busy state".

add the 'exit loop test' in your for( ; ; ) or use while(1){.....}

to see your program ending, you have to reach '' var1 == var 2 == var 3 'it will not happen because of strings and + operator'

but for label 2 and 3, you're adding String to your button values.

review the all label.text, to fix your code.

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Member 15627495 · Accepted Answer · 2023-05-13T04:22:00

C#

          long long cash = 0 ;
          Random RndNumber = new Random();

if (txtAdd.Text == "")
{
   MessageBox.Show("YOU NEED TO ADD MORE cucumber BEFORE PLAYING");
   break;
}
else
{
  while(1){ // infinite loop will proceed until the 3 output values are equal.
           btnOne.Text = RndNumber.Next(0, 10).toString();
           btnTwo.Text = RndNumber.Next(0, 10).toString();
           btnThree.Text = RndNumber.Next(0, 10).toString();

           if( btnOne.Text == btnTwo.Text && btnTwo.Text == btnThree.Text )
           {
              cash  += 120000 ; // random gain
              MessageBox.Show("Congrats! You Win");
              return 0 ;
           }

        }

 }