Comparing a string and a char c#

Question

I am working on a console Hangman game in c#. I am trying to compare user inputted letter against the letters in the random word I have generated. Error i get is "Operator "==" cannot be applied to operands of type "string" and "char". What other ways could I go about doing this? Ive googled a lot but I haven't found any ideas.

public static void LetterChecker(string word)
    {
        int userGuesses = 6;
        string userInputGuess;

        while(userGuesses > 0)
        {
            Console.WriteLine("Please guess a letter");
            userInputGuess = Console.ReadLine();
            foreach(var letter in word)
            {
                if(userInputGuess == letter)
                {
                    Console.WriteLine("this letter is in word, guess again");
                }
                else
                {
                    Console.WriteLine("Incorrect guess");
                    userGuesses--;
                }
            }
        }
    }

You can use Console.ReadKey().KeyChar instead of Console.ReadLine() since you want only a single character from the user. — Ousmane D.
– Ousmane D., Commented Mar 14, 2018 at 22:28

Camilo Terevinto · Accepted Answer · 2018-03-14 22:34:18Z

6

Instead of using Console.ReadLine which reads an entire line, use Console.ReadKey:

Obtains the next character or function key pressed by the user.
The pressed key is displayed in the console window.

char userInputGuess;
...
Console.WriteLine("Please guess a letter");
userInputGuess = Console.ReadKey().KeyChar;

edited Mar 14, 2018 at 22:34

answered Mar 14, 2018 at 22:29

Camilo Terevinto

32.2k7 gold badges95 silver badges127 bronze badges

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

Comments

m3n7alsnak3 · Accepted Answer · 2018-03-14 22:43:39Z

0

The return type of Console.ReadLine() is a string. By this your userInputGuess is of type string, and this is why you receive the error. With slight modification your code can work. Instead of:

if(userInputGuess == letter)

use:

if(userInputGuess[0] == letter)

This will read the first letter of your line. But as you may guess, this is not the best solution in this case.

The better approach would be to read only one letter from the console. Like this:

var userInputGuess = Console.ReadKey().KeyChar; (and get rid of the previous declaration)

The result of this is of type char, and you won't have a problem in the comparison.

edited Mar 14, 2018 at 22:43

answered Mar 14, 2018 at 22:38

m3n7alsnak3

3,1661 gold badge18 silver badges24 bronze badges

2 Comments

Camilo Terevinto Over a year ago

Two things here: userInputGuess[0] would throw an IndexOutOfRangeException if the user just pressed enter (generating an empty string); and userInputGuess would have to be declared as char for the second approach to work

m3n7alsnak3 Over a year ago

Agreed. I missed that for the char declaration.

Collectives™ on Stack Overflow

Comparing a string and a char c#

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related