Silkroad Online Forums

Very simple to fix.

Open Notepad and copy this code exactly:



Save as lol.cmd and then run it. Restart your computer, and your problems will be fixed.

what if he uses linux?

I want to make a program that will ask me a question, and I have to answer with yes or no, and it will put some message...

Hahaha, thank you very much, but that is TOOO much for me...
I dont know 50% of those commands ... but I'll test it
Thank you very much

I get error : "'Static' can only be specified for objects and functions"

@DotCom

Not really complicated, It's a nice way of handling it, maybe not the best example for people that are new to it, but i like it.

lol fangyuan, you really complicated such a simple example program. Most of the stuff is not neccessary. Hang on a sec...





---------------------------------------------------------------------------

#include<iostream>
#include <string>
#include <conio.h>
using namespace std;

int main()
{
string str;
cout<<"Silkroad Online Forum is awesome, right?";
getline(cin, str);
if (str == "yes")
{
cout<<"Exactly";
}
else if (str == "no")
{
cout<<"You fail :p";
}
else
{
cout<<"Wrong input entered";
// Since we consider "yes" or "no" as correct input
}
getch();
return 0;
}


---------------------------------------------------------------------------


Didn't really compile this one, dont think i need to.

Explanation:
First of all you cannot input a word (yes, no) into a variable declared as a single char (char a). You have to use strings or an array of characters as NuclearSilo mentioned above. In this case i used strings and was neccesary to include the string class first. Declared a string variable called "str" where i would save the string entered by the user input. Getline is a method which can save and entire line entered by the user,including spaces, into a string variable. Its correct syntax is "getline(inputStreamVariable, stringVariable)" where the inputStreamVariable refers to cin (cause thats what we want to do) and stringVariable to the variable we want to store the string. Also using getline method doesnt require you to allocate space for the string variable when you declare it (not so important for now). The rest is self explanatory i guess. Hope it helped you

I'm working in devC++ btw

Your dos compiler sucks. I suggest you get Visual studio 6, its a really good compiler for C++ and has a good debugger too. And you wont need to include those old C language syntax which C++ managed to get rid off.

#include <iostream>
#include <string>
using namespace std;
int main()
{
string a;
cout<<"Silkroad Online Forum is awesome, right?";
do {
cin >> a;
if (a == "yes")
cout<<"Exactly\n";
else
cout<<"Try again n1gger\n";
} while ( a != "yes");
return 0;
}

This is one way you could do it using strings.

In your original code, you had 'yes' instead of "yes".

The while command is a loop that continues until the condition is false. It works just like a normal sentence in English.

In this implementation, I declare an integer validityFlag to keep track of whether or not an input statement is valid or not. When the statement is valid, I change validityFlag to 1. When the while loop reaches the end, it checks the condition again and realizes that validityFlag is no longer 0 and exits. The while loop always reads through the entire code inside it, then goes back to the beginning of the code and checks the condition again to decide whether or not to go back inside the loop or exit out.

Also, note that this the program is case-sensitive. "Yes" is not the same as "yes". If you want to take either you can use the || operator which means "or".

1)conio.h is deprecated... dont ever use that preprocessor directive anymore.
2)a char accepts only 1 char... if you want to test to see if it's "yes" or "no", u want to use a string since the two words has more than 1 char.
3)use a do-while loop... it's best for what u are trying to do
4)dont put all those uncessary curly braces... its only needed for compound statements. for 1, it's unnecssary.
5) for the love of god, dont use getch()


Here's a much better version of your code. study the code and u will see why it's much better.

int main() is the technically correct heading for the main function heading..... not void main()
why on earth do you want to use a boolean flag and make? this is C++, not C
\n is better and more efficient than endl

fanyang probably use a programming wizard that generate a template... lots of unnecessary code/do nothing code that no human that knows C++ would type

What? You can tell me what's wrong and "un-human" with the code.

The STL Containers are very powerful, and people should get used to it, even for "simple" programs. I'm sure everyone in his school will have the same code like him, hence I suggested std::map, because it will make his different, and outstanding.

Tried it, and works awesome... about getch() and <conio.h> ; I simply cannot do it without it, since we are programming in dev C++.... (in school)

AFAIK, int main() and void main() are pretty much the same depending on how you want the program to end.

I used the boolean to show how while works with the condition. *Edit* I copied that part from an old C file I had and added some C++ into it, which is why it's there. While (!=) is much better.

I learned \n in C and endl in C++ which is why I used that here. I can't comment on their efficiency.

you've got to be kidding me right? u think using a template library that adds 585747 lines of code to a program is human when a simple 10 line program can do the exact thing better and faster?

Feel free to add a check to see if it's really faster, like I said, it may be more code, but it will make it outstanding. Everyone in his school will have the same code, which is boring and I doubt the teacher wants them to use the same approach. Also it's not that much code, it's ~50 lines of code total, and it would be usable for future projects, cause the teacher may want them in the next project to use more than just few strings, I doubt it'd look good with so many if statements.