Thread: "if" confusion

I'm working on a program. This function you are about to see is a button. When the button is pressed, the program checks to see how two edit boxes are doing and then answers them. The problem is, the first if statement always executes, even if it shouldn't. What's wrong?


void PasswordChange::OnOK()
{
if(m_OldPassword.IsEmpty() == TRUE && m_NewPassword.IsEmpty() == TRUE)
{
MessageBox("Both are empty");
CDialog::OnOK();
}
else
{
}
if(m_OldPassword.IsEmpty() == FALSE && m_NewPassword.IsEmpty() == TRUE)
{
MessageBox("New Is Empty");
CDialog::OnOK();
}
else
{
}
CDialog::OnOK();
}

try to compare the contains of the edit box with ""
like
if(edit1.text == "" && edit2.text == "")
{
// for the things i used use your own
}

That won't work, because the if is checking to see "IF" a function is true. The " 's don't belong.

Break it down.

Catch the returns from both the functions into temp ints. You will then see what is going on.

We know that the IF function works so there must be a problem with the returns from one of the functions or in the capture of the passwords.

Your function is probably running perfectly. Both strings are empty. You need to update the data and let the framework call the functions that link the string variables to the textboxes.

Code:

void PasswordChange::OnOK() 
{ 
      UpdateData(); // Important !!!

     // yourstuff here

Thanks, nvoigt.