I want the user to press a key from keyboard and then want the computer to tell if the key is a alphabet, symbol or a number.

Question

0.00/5 (No votes)

See more:

I want the user to press a key from keyboard and want the computer to tell if the key is a alphabet, symbol or a number. But some error is there in the code. Can you point out ?

C#

Console.WriteLine("type a character");
            int character = int.Parse(Console.ReadLine());
            string type = " ";

            if ((character>=48) && (character<=57))
            {
                type = "integer";
            }

            else if ((character >= 65) && (character <= 90))
            {
                type = "uppercase alphabet";

            }


            else if ((character >= 97) && (character <= 122))
            {
                type = "lowecase character";

            }

            else
            {
                type = "symbol";
            }
           
           
            
            Console.WriteLine("the type of character that you have entered is " +type);
            Console.ReadLine();

Posted 17-Mar-13 0:43am

Allwin456

Updated 17-Mar-13 0:44am

v2

Add a Solution

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)

OriginalGriff · Answer 1 · 2013-03-17T00:52:00

The problem is that when the user types a key, you try to parse it to an int:

C#

int character = int.Parse(Console.ReadLine());

Which means convert from a string representation of a number "1234" to an integer version that can be handle mathematically.

So if the user types an "8" for example, int.Parse will return the value 8 - which is less than 48, so it will fail all your tests.

Instead, use the character directly:

C#

string userInput = Console.ReadLine();
foreach (char c in userInput)
    {
    string type = "unknown";
    if (c >= '0' && c <= '9')
        {
        type = "integer";
        }
    else if ...

Or better, use the built in functions:

C#

string userInput = Console.ReadLine();
foreach (char c in userInput)
    {
    string type = "symbol";
    if (char.IsDigit(c))
        {
        type = "integer";
        }
    else if (char.IsLetter(c))
        {
        if (char.IsLower(c))
            {
            type = "lowercase character";
            }
        else
            {
            type = "uppercase character";
            }
        }
    ...
    }