How Do I Use A Console.Readline In An If Statement

Question

0.00/5 (No votes)

See more:

Hello. I am trying to make a game like dungeons and dragons in c# using a console application. I am trying to use if statements with readlines and it won't let me. this is the code that i put in.

C#

if(Console.ReadLine == ("Attack"))
{
    //Take away health from the goblin.
}

Posted 25-Aug-15 4:50am

Coder1999

Add a Solution

Comments

PIEBALDconsult 25-Aug-15 10:56am

Ah, that causes one of those less-helpful compile errors. But a small dose of RTFM should fix you right up.
https://msdn.microsoft.com/en-us/library/system.console.readline(v=vs.110).aspx

Andy Lanng 25-Aug-15 11:26am

My boss once asked me what RTFM meant. Obviously I replies "Read the Manual". He paused looked confused, then started laughing his ass off :P

OP: Methods need braces '()'. Google Console.ReadLine and you'll find 1000's of answers

Maciej Los 25-Aug-15 13:34pm

I thought that RTMF means: Read The F...ing Manual! :laugh:

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 · Accepted Answer · 2015-08-25T05:37:00

For starters, the problem is pretty simple: ReadLine is a method, and that means you need to call it as a method by appending brackets that you could put parameters in:

C#

if(Console.ReadLine() == "Attack")
{
    //Take away health from the goblin.
}

But I woudln't do that anyway.
The reason why not is pretty simple: what if it's "throw" or "runaway" instead? Becasue you don't save the user input, you can't test it for anything else afterwards.
So instead, read teh input, and then check it:

C#

string userResponse = Console.ReadLine();
if (userResponse == "Attack")
{
    //Take away health from the goblin.
}
else if (userResponse == "Runaway")
}
    ...

And I wouldn't do it like that either! :laugh:

C#

string userResponse = Console.ReadLine();
switch (userResponse.ToLower())
    {
    case "attack":
        //Take away health from the goblin.
        ...
        break;
    case "runaway":
        // Run aaaaaaway!
        ...
        break:
    default:
        Console.WriteLine("I'm sorry, I didn't understand that!");
        break;
    }

Try it - you should find it's easier to read.