What is wrong with this code (C++)

[LeftEyeNine]

Wow, looking at these, I feel like a CEO of a very serious corporation which is dedicated to conquer the world.

Keep on guys, I'll pump the wages

[Husar]

That code looks pretty easy, I should try C++, it's to Java like German is to Dutch or so.
Also I noticed there is a fault here:

std::cout<< "1=Easy\n";
std::cout<< "2=Medium\n";
std::cout<< "3=Hard\n";
std::cin>> Option;

if (Option == EASY_LEVEL)

in the if-clause you check whether option is EASY_LEVEL, but since option is an integer, it cannot be a string and besides that, option was only assigned a number so that can never ever return true.
You either need to use a different check or turn option into a string and add three more lines and another variable and....


Despite Blodrast saying this was bad style, I would use

if (Option == 1)

Or you could change the start to say:

#include <string>

int Option;
string difficulty;
bool Word_Complete = false;

std::cout<< "Game intructions\n";
std::cout<< "Please select a difficulty level\n";
std::cout<< "1=Easy\n";
std::cout<< "2=Medium\n";
std::cout<< "3=Hard\n";
std::cin>> Option;

if (option == 1) {
difficulty = "EASY_LEVEL"}

if (option == 2) {
difficulty = "MEDIUM_LEVEL"}

if (option == 3) {
difficulty = "HARD_LEVEL"}

the three ifs could also be a case-option but then you could later check with

if (difficulty == "EASY_LEVEL")
{
std::cout<< "Display scambled word Easy\n";
}

but in Java checking strings does not work like this IIRC, so the condition may need a big change.
Ok, guess this really needs something else to check what is inside the string, but I would actually prefer numbers for such a small program anyway...

[Alexander the Pretty Good]

The_Doctor - the program ends after displaying the difficulty level because you haven't put anything after the switch statement that displays the difficulty level. If you want the program to continue after that, either add what else the code needs to do or put the entire thing in a loop as Sigurd Fafnesbane has done.

I would recommend using <iostream>instead of <iostream.h> and then put the phrase "using namespace std;" under it. Then you won't have to put std:: in front of every cout and cin. Just a little trick.

Husar - he's defined EASY_LEVEL to be 1, so the compiler treats every "EASY_LEVEL" as a 1. I guess Java doesn't have that feature...

[Husar]

Husar - he's defined EASY_LEVEL to be 1, so the compiler treats every "EASY_LEVEL" as a 1. I guess Java doesn't have that feature...

You're right, I didn't notice that.
At least my lazy brain got to think about some code again.

[The_Doctor]

It works now.

I tried the code on one of the computers in university and it worked.

I wonder why it does not work on my computer?

[LeftEyeNine]

It works now.

I tried the code on one of the computers in university and it worked.

I wonder why it does not work on my computer?

So when are we moving on to conquest, boys? Don't spoil my fun, hey!

[caravel]

Nothing appears to be wrong with that code in your first post apart from the booleans need to be initialised.

[Blodrast]

Nothing appears to be wrong with that code in your first post apart from the booleans need to be initialised.

There is. He wants loops, but his while loops don't do anything, because they end in a semicolon. His code IS, both syntactically, and semantically, correct, yes, but I very much doubt that was what he wanted (while loops that do nothing), at least from what he explained.

[KukriKhan]

So when are we moving on to conquest, boys? Don't spoil my fun, hey!

LoL. I think you install his mini-game, and pick "Medium". No one ever picks medium. Then 'the plan' is revealed.