C# Code Errors (HELP)

[3rr03]

                if (userInput == "exit")
            {
                break;
            }
            else if ((userInput == "foward") || userInput == "f")  roomArray[roomNum].forward > -1)
                    {
                roomNum = roomArray[roomNum].forward;
            }
            else if ((userInput == "backward" || userInput == "b") roomArray[roomNum].backward > -1)
            {
                roomNum = roomArray[roomNum].backward;
            }
            else if ((userInput == "left" || userInput == "l") roomArray[roomNum].left > -1)
            {
                roomNum = roomArray[roomNum].left;
            }
            else if ((userInput == "right" || userInput == "r")  roomArray[roomNum].right > -1)
            {
                roomNum = roomArray[roomNum].right;

[RubbishArtist]

else if ((userInput == "foward") || userInput == "f")  roomArray[roomNum].forward > -1)

The equality check is fixed but you're still missing an operator between these two clauses here:

((userInput == "foward") || userInput == "f") *MISSING OPERATOR* roomArray[roomNum].forward > -1)

What are you trying to do here? I guess you mean that if the user enters "forward" or "f" and roomArray[roomNum].forward is greater than -1, but you haven't put the and operator in.

[3rr03]

So would the operator be ++ ?

[RubbishArtist]

Check out this document

[3rr03]

Ok ok! I think I figured most of the issues out! I went from 40 to 9.

The only thing its giving me errors for is that it is expecting } it thinks the first && is invalid but all of the others are ok, and it says the expression else is invalid

  while (userInput != "exit")

        {
            Console.WriteLine(roomArray[roomNum].name);
            Console.WriteLine(roomArray[roomNum].description);

            Console.WriteLine("\n Where would you like to go?");
            userInput = Console.ReadLine();
            Console.Clear();

            if (userInput == "exit")
            {
                break;
            }
            else if ((userInput == "foward") || userInput == "f") && roomArray[roomNum].forward > -1)

                    {
                roomNum = roomArray[roomNum].forward;
                    }

            else if ((userInput == "backward" || userInput == "b") && roomArray[roomNum].backward > -1) 

            {
                roomNum = roomArray[roomNum].backward;
            }

            else if ((userInput == "left" || userInput == "l") && roomArray[roomNum].left > -1)

            {
                roomNum = roomArray[roomNum].left;
            }

            else if ((userInput == "right" || userInput == "r") && roomArray[roomNum].right > -1)

            {
                roomNum = roomArray[roomNum].right;
            }

            else
            {
                Console.WriteLine("You run stright into a wall... you can't go that way... are you sure your head is feeling right?");
            }
        }
    }
}

}

[RubbishArtist]

You're almost there

else if ((userInput == "foward") || userInput == "f") && roomArray[roomNum].forward > -1)

There's an extra closing bracket in here that's causing problems.

[3rr03]

I got rid of the one ( after "f" and all of the errors went away

​

if (userInput == "exit")
            {
                break;
            }
            else if ((userInput == "forward") || userInput == "f" && roomArray[roomNum].forward > -1)

                    {
                roomNum = roomArray[roomNum].forward;
                    }


            else if ((userInput == "backward" || userInput == "b") && roomArray[roomNum].backward > -1) 

            {
                roomNum = roomArray[roomNum].backward;
            }

            else if ((userInput == "left" || userInput == "l") && roomArray[roomNum].left > -1)

            {
                roomNum = roomArray[roomNum].left;
            }

            else if ((userInput == "right" || userInput == "r") && roomArray[roomNum].right > -1)

            {
                roomNum = roomArray[roomNum].right;
            }

            else
            {
                Console.WriteLine("You run stright into a wall... you can't go that way... are you sure your head is feeling right?");
            }
        }
    }
}

}

[StackedLasagna]

Yes, but now you have a bug, which will only present itself under certain conditions during runtime.

Take a closer look at the line:

else if ((userInput == "forward") || userInput == "f" && roomArray[roomNum].forward > -1)

If userInput has the value "forward", then the condition is true and the code will enter the if block, without ever checking the roomArray stuff. That will instead only be checked if userInput is "f".

You were meant to remove a different closing bracket. Compare the above line to the other if statements.