Password verification

[roots]

I am not even getting the error statement back. I am guessing that the issue is in main and not in the other functions. I am unsure of where. The if statement looks valid to me.

cin.getline(passWord,20)

if (testPassWord(passWord))
	{
		cout << "Please wait - your password is being verified" << endl;
	}
	else
	{

		cout << "Invalid password. Please enter a password "
			<< "with exactly 5 letters and 3 digits" << endl;
		cout << "For example, my37RuN9 is valid" << endl;
	}

bool testPassWord(char custPass[])
{
	int numLetters, numDigits, length;

	length = strlen(custPass);
	numLetters = countLetters(custPass);
	numDigits = countDigits(custPass);
	if (numLetters == 5 && numDigits == 3 && length == 8)
	{
		return true;
	}
	else
	{
		return false;
	}
}

[AbstractionAnon]

Your testPassWord function looks fine, as does the call to it.

I would suggest putting a cout of passWord just before you call testpassWord to see what you are passing. Or step through it using a debugger.

[roots]

It displays what is typed just fine.

I am not sure what you meant by step through it using the debugger?

[AbstractionAnon]

You could have a problem in your countLetters or countDigits functions. Since you did not provide those, it's hard to say.

I am not sure what you meant by step through it using the debugger?

http://www.cplusplus.com/forum/articles/28767/

[roots]

I'm still working at this and will update if I find a solution.

int countLetters(char *strPtr)
{
	int occurs = 0;
	while (*strPtr != '\0')
	{
		if (isalpha(*strPtr))
		{
			occurs++;
			strPtr++;
		}
	}
	return occurs;
}

int countDigits(char *strPtr)
{
	int occurs = 0;
	while (*strPtr != '\0')
	{
		if (isdigit(*strPtr)) 
		{				  
			occurs++;
			strPtr++;
		}
	}
	return occurs;
}

[Thomas1965]

int countLetters(char *strPtr)
{
	int occurs = 0;
	while (*strPtr != '\0')
	{
		if (isalpha(*strPtr))
		{
			occurs++;
			strPtr++;
		}
	}
	return occurs;
}

You need to advance your pointer also when !isalpha(*strPtr)
Same with countDigits

Why don't you use a for loop and strlen() ?

[roots]

Silly me! Thanks Thomas1965.
And AbstractionAnon for the debugging tips.