Hi!
Okay, your premise is sound, but the execution could use some work. If you don't mind me turning into an evil code reviewer, I'll point out some ways to make your code better.
using std::cout;
using std::cin;
using std::endl;
This is my personal preference. I think it's a good one because I'm good at convincing my coworkers to use it.
Any using statements should be at the smallest possible scope. That usually means putting a
using namespace std; at the top of a bunch of functions or even in a loop or if body. You shouldn't use it globally, and the highest scope should be at the top of a namespace. Anything global should be prefixed with std:: and that's not a big problem because global things that need a standard name should be kept to a minimum. You can ignore all of this for now though, it's not really important when learning C++. :eek:
char x [1];
char c [1];
char C [1];
char p [1];
char P [1];
char e [1];
char E [1];
char d [1];
char D [1];
char q [1];
char Q [1];
You can make arrays of 1, but it doesn't really make much sense. An array of 1 char is basically a single char that's just harder to access directly. Instead of just saying x, you have to say x[0], or *x. That's a notational inconvenience.
This is the big problem. cin.getline() will read n - 1 characters and append the array with '\0'. No matter what you type, x will always be an empty string.
Stream input is really tricky. I hate it.
But in this case you can just use cin.get() to read one character and things will work. If you add other input later on, you need to re-evaluate that assumption because it might not be true anymore.
There are three problems here. First, x is an array and the first character must be accessed with x[0] or *x. Otherwise you're just taking the address of the first character, and that's probably not what you were expecting.
Second, the = operator is assignment. You want the == operator for comparison. Finally, c and C are treated as variables, not character literals. You need to wrap them in single quotes.
Here's my take on the code.
#include <iostream>
int main()
{
using namespace std;
char x;
cout << "You can perform the following tasks: \n";
cout << "(p) Print a Sentence \n";
cout << "(c) Capitalize the Sentence \n";
cout << "(e) Encode the Sentence \n";
cout << "(d) Decode the Sentence \n";
cout << "(q) Quit \n";
cout << "Please Select one... \n";
cin.get( x );
if (x == 'c' || x == 'C')
cout << "Test Successful\n";
else
cout << "Test Failed\n";
return 0;
}
Ask questions! I love helping.
Offline 
Marked 
