expected ';' before "else"

[kcbob]

i cant seem to find were the ';' goes

void checkT(char temp[])
{
    if (temp[0] != '[' && temp[0] != '{' && temp[0] != '(');
    {
       do
       {
          cout << "Invalid first value! Must start with (,[,{" << endl;
          cout << "Write your values: " << flush;
          cin >> temp;
       } while (temp[0] != '[' && temp[0] != '{' && temp[0] != '(');

    }
    else
    {
        cout << "Pushing...." << endl;
    }
}

any other info on helping me improve this function is welcomed.

[1vRz3TCk]

if (temp[0] != '[' && temp[0] != '{' && temp[0] != '('); // remove the semicolon

[kcbob]

when i remove the else if compiles fine

[Syuf]

It's a typo: ';' after 'if' statement is excess. To improve this code:

void checkT(char str[])
{
	while(str[0] != '(' && str[0] != '[' && str[0] != '{')
	{
		cout << "Invalid first value! Must start with (,[,{" << endl
			<< "Write your values: " << flush;
		cin >> str;
	}
	cout << "Pushing...." << endl;
}

(I think "Plushing" should be written in any case after the string is checked)

[kcbob]

success!!!
thank you!!

[kcbob]

one more question

how can i have it check for and ending value of },] or )

without knowing how many characters will be typed

[Syuf]

how can i have it check for and ending value of },] or )

int len = strlen(str);
if(len > 0 && str[len - 1] != ')' && str[len - 1] != ']' && str[len - 1] != '}')
    cout << "Bad ending." << endl;