Why doesn't the if statement execute?

Question

1.00/5 (1 vote)

See more:

C#

int n = 777;

            string x = n.ToString();

            Console.WriteLine(x[2]);

            if (x[2] == 7)

            {

                Console.WriteLine("no.7");

            }

//output-

//7

"no. 7" is not getting printed. Is there something which I haven't quite done right?

What I have tried:

Well, what could I have tried anyways?

Posted 27-Mar-16 2:50am

Member 12414152

Updated 27-Mar-16 3:21am

Add a Solution

Comments

[no name] 27-Mar-16 9:08am

Yes you missed something. The character representation of '7' (in x[2]) does not match to integer 7....

2 solutions

Solution 2

You should learn to use the debugger as soon as possible. Rather than guessing what your code is doing, It is time to see your code executing and ensuring that it does what you expect.

The debugger allow you to follow the execution line by line, inspect variables and you will see that there is a point where it stop doing what you expect.
Mastering Debugging in Visual Studio 2010 - A Beginner's Guide[^]
http://docs.oracle.com/javase/7/docs/technotes/tools/windows/jdb.html[^]
https://www.jetbrains.com/idea/help/debugging-your-first-java-application.html[^]

With the debugger, stop execution at the if and inspect variables, you will see what in x[2] and you will understand that 7 is not the same as "7".

You should read language documentation as soon as possible.

Posted 27-Mar-16 3:21am

Patrice T

Updated 27-Mar-16 3:45am

v2

Comments

[no name] 27-Mar-16 9:39am

In this case I think Debugger is not relevant. OP should first understand character representation. Ok Debugger can help to visualize it... No vote from my side.

Patrice T 27-Mar-16 9:51am

I agree that OP must read document as soon as possible, but knowing the debugger can't harm. and will show the problem in this case.

No vote is ok for me.

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)

OriginalGriff · Accepted Answer · 2016-03-27T03:09:00

Becasue there is a big difference between 7 and '7'
The first is a number, the second is a character.
A string is made up of a sequence of characters, so 777 as a string is "777": three characters '7' following each other.
A character can contain '7', but it can also contain 'H', 'E', 'L', 'L', or 'O' - so trying to compare it to a numeric value only works if you use the value for the character in a right character set.
And so x[2] is a single character '7' which is not the same value as 7 the number.
Try this:

C#

if (x[2] == '7')

and it'll work.