Simple If Else C# Statement

Question

    private void ProcessFrame(object sender, EventArgs arg)
    {
        Wrapper cam = new Wrapper();

        //show the image in the EmguCV ImageBox
        WebcamPictureBox.Image = cam.start_cam(capture).Resize(390, 243, Emgu.CV.CvEnum.INTER.CV_INTER_CUBIC).ToBitmap();
        FaceDetectedLabel.Text = "Faces Detected : " + cam.facesdetected.ToString();
    }

I am working on a C# windows application. I am stuck with a simple question:

How can i do a if else with the condition: If "cam.facesdetected.ToString()" if equal or more than 2 do sth, else do nothing.

I tried this, but it does not seems to work. Can anyone help me?

        cam.facesdetected = abc;
        MessageBox.Show("The detected faces is:" + abc);

        if (abc >= 2)
        {
            //Do action 
        }

        else
        {
            //Do nothing
        }

What does "but it does not seems to work" mean? What is the error? — zimdanen
– zimdanen, Commented Dec 4, 2012 at 18:02
It looks like you want to parse the string "2" into an integer, is that correct? — BrMcMullin
– BrMcMullin, Commented Dec 4, 2012 at 18:03
It means that the return value is "0" for abc when messagebox is pop up but however, cam.facedeteced actual value is "1" cause its a webcame face detection return value — Derek
– Derek, Commented Dec 4, 2012 at 18:03

Michael · Accepted Answer · 2012-12-04 18:02:09Z

2

You could:

if (Convert.ToInt32(abc) > 2)
   DoWork()

Although it would probably be wise to declare ABC as an integer to begin with, if it is always an integer.

answered Dec 4, 2012 at 18:02

Michael

1,8332 gold badges17 silver badges28 bronze badges

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

3 Comments

Servy Over a year ago

Considering that the string appears to come from user provided input int.TryParse should probably be used instead.

BrMcMullin Over a year ago

I was just thinking the same thing.

Michael Over a year ago

Agreed. The details of the question were a bit.. vague.. so i generalized the answer.

David Yaw · Accepted Answer · 2012-12-04 18:02:27Z

2

I believe you've got your if statement backwards.

abc = cam.facesdetected;

Now you can operate on abc, as you had listed.

answered Dec 4, 2012 at 18:02

David Yaw

28.1k5 gold badges65 silver badges95 bronze badges

Comments

Nikola Davidovic · Accepted Answer · 2012-12-04 18:08:50Z

0

I think that you should use it without assigning any new variable. There is no need to use the abc variable. You can use cam.facesdetected directly (I suppose that it is a number) like this:

    MessageBox.Show("The detected faces is:" + cam.facesdetected.ToString());

    if (cam.facesdetected >= 2)
    {
        //Do action 
    }

    else
    {
        //Do nothing
    }

answered Dec 4, 2012 at 18:08

Nikola Davidovic

8,6761 gold badge31 silver badges34 bronze badges

Comments

lante · Accepted Answer · 2012-12-04 18:10:56Z

0

Dont use .ToString() when you use numbers.

You are assigning the variable in a wrong way, this should be

var abc = cam.facesdetected;

if cam.facesdetected is not a number then use

var abc = Convert.ToInt32(cam.facesdetected);

and then

if (Convert.ToInt32(abc) >= 2)
{
   //Do action 
}

answered Dec 4, 2012 at 18:10

lante

7,3924 gold badges39 silver badges59 bronze badges

Comments

Ravindra Bagale · Accepted Answer · 2012-12-04 18:14:18Z

0

you can only assign leftside variable by rightside variable

LHS=RHS

you have wrongly assigned it must be abc = cam.facesdetected;

and you can check whether it is greater than & equal to 2 by using TryParse Method

   bool result = Int32.TryParse(abc, out number);
      if (result)
      {
         if(number>=2)
        {
          //dowork;   
         }   
      }

edited Dec 4, 2012 at 18:14

answered Dec 4, 2012 at 18:02

Ravindra Bagale

17.7k9 gold badges47 silver badges70 bronze badges

1 Comment

BrMcMullin Over a year ago

I believe your syntax to call TryParse is incorrect. See here: msdn.microsoft.com/en-us/library/f02979c7.aspx

Collectives™ on Stack Overflow

Simple If Else C# Statement

5 Answers 5

3 Comments

Comments

Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

3 Comments

Comments

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related