Thread: Problem validating

hello, I'm working on a roman numeral converter, but I'm having trouble validating that the user input is valid. Even when I input a valid roman number the following function returns false. Can someone be kind enough to point out what I'm doing wrong, and also I'm confused as how to validate that the roman number is valid, for example XIDMXM is not valid.

Code:

bool isValidSigns(char roman[20])
{
	bool valid = true;
	int i = 0;

	while(roman[i] != '\0')// && valid != false)
	{
		cout<<"= "<<roman[i]; //for testing
		if(roman[i] != 'M' ||'D' ||'C' ||'L'||'X' ||'V' ||'I')
		{
			valid = false;
		}
		i++;
	}

	return valid;
}

Thank you for the help, i tried using && but not && and != and now that I look at the code you provided it makes sense.

you could use an std::string containing "MDCLXVI", and search for the character in that string using std::string::find().

Originally Posted by Kinto

Well so far i'm getting the output I was expecting. I thought about using a string but felt more comfortable using char[]. I need more practice with string as I moved from C to C++.

Yes, because std::string is much harder to work with, right?

Code:

int main( void )
{
    std::string
        txt;
    txt = "foo";
    txt += "bar";    
    if( txt == "foobar" )
        /* do something*/;
    return 0;    
}

Versus:

Code:

int main( void )
{
    char
        txt[ 1024 ];
    strncpy( txt, "foo", sizeof str );
    strncat( txt, "bar", sizeof str );
    if( strncmp( txt, "foobar", sizeof str ) == 0 )
        /* do something*/;
    return 0;    
}

When people ask me why I drive standard over an automatic, I simply tell that I get bored and like switching gears. well enough of bad analogies thanks for the suggestions and the sample code.