Problem with GetDlgItemText()

The follwing codes doesn't work:

Code:

		  
TCHAR must_be_str;
TCHAR entered[MAX}PATH];
strcpy(must_be_str,"b");
GetDlgItemText(hDlg, IDC_ENTER, entered, 5); 
itoa(must_be, must_be_two, 10);

if(entered == must_be_str)
{
    MessageBox(NULL, "It worked", "yay", MB_OK);
}
else
{
    MessageBox(NULL, "Failed", "Damn", MB_OK);
    MessageBox(NULL, entered, "Value of 'entered'", MB_OK);
    MessageBox(NULL, must_be_str, "Value of 'must_be_str", MB_OK);
}

When this code runs, 'entered' shows up as blank and 'must_be_str' shows up as 0. Whats goin on?

I put this on the Windows Programming board becuase there is obviously a problem with what I did with GetDlgItemText()...

Thanks

Try replacing

entered[MAX}PATH];

with

entered[MAX_PATH];


I'm not sure why your compiler doesn't give you an error about an undefined identifier "MAX}PATH". MAX_PATH is predefined to be a sufficiently large memory space to hold whatever you want. MAX}PATH isn't anything. That could be the problem.

that is just a typo that i must have done when I typed it out here - i didn't copy/paste. In the source, there's no brace.

>>> strcpy(must_be_str,"b");

You are using strcpy() to copy to a non-string destination...

>>> if(entered == must_be_str)

You cannot compare strings that way, either use strcmp() if you want to comapre things as strings, or subscript the array to compare character values...

>>> GetDlgItemText(hDlg, IDC_ENTER, entered, 5);

Check the return value, if it is zero, then the call failed, so call GetLastError() and see what went wrong...

(From http://www.cprogramming.com/cboard/s...threadid=17871)

>>> I already know C++ and Windows programming in C++.

Are you sure?!?!

Originally posted by adrianxw


>>> if(entered == must_be_str)

You cannot compare strings that way, either use strcmp() if you want to comapre things as strings, or subscript the array to compare character values...

In C++ you can compare strings that way

>>> In C++ you can compare strings that way

Not those kind of strings, you need to use one of the string classes with the == operator overloaded to acheive that.

>>Are you sure?!?!

I've used GetDlgItemText() before successfully, there's just a mix up with all the string stuff...