C# Not understanding this code...............

Question

1.00/5 (1 vote)

See more:

Can some please help me this is my code

C#

// the death movment
private void timer2_Tick(object sender, EventArgs e)
{
    int movement = (label2.Text == "This Is Level 1") ? 5 : 15;
    // level 1
    if (label2.Text == "This Is Level 1")
    {
        if (Death.Width == 64)
        {
            Death.Location = new Point(Death.Location.X + movement, Death.Location.Y);
        }

        if (Death.Location == new Point(390, 202))
        {
            Death.Width = 63;
        }
        if (Death.Width == 63)
        {
            Death.Location = new Point(Death.Location.X - movement, Death.Location.Y);
        }

        if (Death.Location == new Point(30, 202))
        {
            Death.Width = 64;
        }
    }
    // level 2
    if (label2.Text == "This Is Level 2")
    {
        if (Death.Width == 64)
        {
            Death.Location = new Point(Death.Location.X + movement, Death.Location.Y);
        }

        if (Death.Location == new Point(390, 202))
        {
            Death.Width = 63;
        }
        if (Death.Width == 63)
        {
            Death.Location = new Point(Death.Location.X - movement, Death.Location.Y);
        }

        if (Death.Location == new Point(30, 202))
        {
            Death.Width = 64;
        }
    }

}

I tried asking this in another post but i got no answer, so anyway, i dunno what this part does.

int movement = (label2.Text == "This Is Level 1") ? 5 : 15;

at this end part

? 5 : 15;

Posted 20-Feb-11 23:00pm

C.CoderCreator

Updated 21-Feb-11 2:39am

fjdiewornncalwe

v2

Add a Solution

4 solutions

Solution 3

int movement = 15;
if (label2.Text == "This Is Level 1")
    movement = 5;

Posted 20-Feb-11 23:16pm

Manas Bhardwaj

Comments

Sunasara Imdadhusen 21-Feb-11 5:21am

Hmm, good call!!

Manas Bhardwaj 21-Feb-11 5:38am

thnx :)

Solution 4

This:

C#

int movement = (label2.text == "This is Level 1") ? 5 : 15;

Is the same as saying this:

C#

int movement;
if (label2.text == "This is Level 1")
{
    movement = 5;
}
else
{
    movement = 15;
}

It simply takes less room, and therefore keeps the code nice and tidy.

And by the way, if you do that, you do NOT need both blocks of code (as I indicated in the answer you're talking about). Go back and look at it again.

Posted 21-Feb-11 0:06am

#realJSOP

Updated 21-Feb-11 0:08am

v4

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)

Nuri Ismail · Accepted Answer · 2011-02-20T23:05:00

Solution 1

See what the documentation says[^]. :)

After reading the documentation about the conditional operator

(?:)

you will figure out that if the text of label2 is exactly equal to "This Is Level 1" then movement will be 5 otherwise movement will be 15. So you can rewrite the same line as:

C#

int movement = 15; // Initially 15

// Check if we need to assign 5 as value
if(label2.Text == "This Is Level 1")
  movement = 5;

See that there is no need for else block, because I choose to initialize movement with 15. I hope it is all clear now. :)

Posted 20-Feb-11 23:05pm

Nuri Ismail

Updated 20-Feb-11 23:45pm

v4

Comments

Manas Bhardwaj 21-Feb-11 5:53am

+5

Nuri Ismail 21-Feb-11 5:57am

Thank you, Manas!

Nuri Ismail 21-Feb-11 5:57am

@EdMan196: Thanks a lot for the typo fix.

Ed Nutting 21-Feb-11 6:22am

No worries ;)

Dalek Dave 21-Feb-11 6:06am

Good Call.

Nuri Ismail 21-Feb-11 6:31am

Thank you, DD!

Sunasara Imdadhusen 21-Feb-11 6:51am

Too good ):

Nuri Ismail 21-Feb-11 9:00am

Thank you, Sunasara!

fjdiewornncalwe 21-Feb-11 8:42am

+5. I was going to make a note an say you should call it the ternary operator, and not the conditional operator, but in the MSDN documentation you link to, they do actually call it the conditional. Why do those guys always rename stuff on us.... :)

Nuri Ismail 21-Feb-11 8:59am

Thank you very much, Marcus! :)

And about the operator check out this link: http://en.wikipedia.org/wiki/Ternary_operation

A citation from the above link:
'Many programming languages that use C-like syntax feature a ternary operator, ?:, which defines a conditional expression. Since this operator is often the only existing ternary operator in the language, it is sometimes simply referred to as "the ternary operator", though it is more accurately referred to as the conditional operator.'

I didn't know about this Wikipedia content and I found it now, but I remember some time ago we've had a discussion with my classmates at university exactly about these two terms and the conclusion was that using 'the conditional operator' is more accurate. All of this is a matter of taste of course, no matter what term is used more important is to know what is hiding behind it.

Thanks again and best regards,
Nuri

Sunasara Imdadhusen · Accepted Answer · 2011-02-20T23:06:00

Solution 2

C#

int movement = (label2.Text == "This Is Level 1") ? 5 : 15;

OR

C#

int movement = 0;
if(label2.Text == "This Is Level 1")  
   movement = 5;
else 
   movement = 15;

Thanks,
Imdadhusen

Posted 20-Feb-11 23:06pm

Sunasara Imdadhusen

Updated 20-Feb-11 23:07pm

v2

Comments

Manas Bhardwaj 21-Feb-11 5:53am

+5

Dalek Dave 21-Feb-11 6:07am

Good Answer.