How to solve error C2039: 'size' : is not a member of 'CString'

Question

0.00/5 (No votes)

See more:

Hi ,

In my program I got this error C2039: 'size' : is not a member of 'CString'.
How to solve it? help me.

C++

BOOL CEMailDlg::Validate() 
{  
	CString m_sFrom;
	
	for(int a = 0; a < m_sFrom.size(); a++) 
	{  
		if(m_sFrom.at(a) = '@') 
			return true;
   }   return false;
}

thank you

Posted 9-May-11 23:36pm

degorio

Updated 9-May-11 23:43pm

Sandeep Mewara

v2

Add a Solution

5 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

OriginalGriff · Answer 1 · 2011-05-09T23:57:00

Solution 1

Did you mean to use GetLength? MSDN[^]

for(int a = 0; a < m_sFrom.GetLength(); a++)

Posted 9-May-11 23:57pm

OriginalGriff

Comments

[no name] 10-May-11 6:24am

I just wonder what part of the error message he did not understand. Then again, not everybody can be a master of the obvious.

OriginalGriff 10-May-11 6:30am

That's me! Master of the Obvious! :)
It's nice to be recognised as Master of Something... :laugh:

[no name] 10-May-11 6:34am

'Master of the Obvious' usually is not meant to be a title of honor and prestige and usually bestowed upon people who can't think their way out of a paper bag. I would not want to use it for you, but if you insist...

tolw · Answer 2 · 2011-05-09T23:58:00

Solution 3

If memory serves: try m_sFrom.GetLength() instead

EDIT: Just read the code - for what you are trying to do wouldn't it be better to use:

return (m_sFrom.Find( '@' ) != -1)

instead of searching manually?

Posted 9-May-11 23:58pm

tolw

Updated 10-May-11 0:01am

v2

ShilpiP · Answer 3 · 2011-05-10T00:51:00

Solution 4

"size" and "at" is a member function of basic_string.
1) include basic_string header file
#include <string>
using namespace std;

Replace CString m_sFrom; with string m_sFrom;

2) If you still want to use CString than

BOOL CEMailDlg::Validate() 
{  
	CString m_sFrom;
	
	for(int a = 0; a < m_sFrom.GetLength(); a++) 
	{  
		if(m_sFrom.GetAt(a) == '@') 
			return true;
   }   return false;
}

Posted 10-May-11 0:51am

ShilpiP

Updated 10-May-11 3:17am

v4

Comments

degorio 10-May-11 9:04am

im getting this error ,,error C2628: 'CString' followed by 'int' is illegal (did you forget a ';'?)... how to solve it

ShilpiP 10-May-11 9:18am

No one mistake I found in my code is using assignment instead of ==
if(m_sFrom.GetAt(a) == '@')
return true;

Olivier Levrey · Answer 4 · 2011-05-09T23:57:00

Solution 2

Replace size() by GetLength().
Everything is written in MSDN. Check out the documentation for the CString class.

Posted 9-May-11 23:57pm

Olivier Levrey

Comments

degorio 10-May-11 9:29am

im getting this error ,,error C2628: 'CString' followed by 'int' is illegal (did you forget a ';'?)... how to solve it

Olivier Levrey 10-May-11 9:36am

Post the code please. The code you posted in your question doesn't match the error you describe.

degorio 10-May-11 9:41am

BOOL CEMailDlg::Validate()
{
CString m_sFrom;
int a;
int GetLength();
for( a=0;a < m_sFrom.GetLength(); a++)
{
if(m_sFrom.GetAt(a) == '@')return true;
}
return false;
}

Olivier Levrey 10-May-11 9:43am

Remove the int GetLength(); line.

degorio 10-May-11 9:45am

the same error occurs

Olivier Levrey 10-May-11 9:47am

Mmmm if you removed the line, it should compile. Are you sure the error is not on another line? Can you copy the code giving the error again?

degorio 10-May-11 9:53am

BOOL CEMailDlg::Validate()
{
CString m_sFrom;
int a;
for( a=0;a < m_sFrom.GetLength(); a++)
{
if(m_sFrom.GetAt(a) == '@')return true;
}
return false;
}

BOOL CEMailDlg::Validate()
{
CString m_sFrom;
int a;
for( a=0;a < m_sFrom.GetLength(); a++)
{
if(m_sFrom.GetAt(a) == '@')return true;
}
return false;
}

error C2628: 'CString' followed by 'int' is illegal (did you forget a ';'?)

Olivier Levrey 10-May-11 9:55am

There is no "CString followed by int" there. Everything is normal. The error is somewhere else. Double-click the error message to know which line is causing this error.

degorio 10-May-11 9:57am

it shows the library functions, // Attributes & Operations

// get data length
int GetLength() const;
// TRUE if zero length
BOOL IsEmpty() const;
// clear contents to empty
void Empty();
it points int Get Length() const

Olivier Levrey 10-May-11 10:03am

"Get Length" is not valid it should GetLength (without space). But I think you just wrote it wrong...
What do you mean by "the library functions". Do you mean CString class definition? What you show is not the CString definition. At least not the one included in Visual Studio. If you are using a custom CString class, it is not a good idea...

degorio 11-May-11 1:29am

thank you,,, i solved that error,, now im having another problem im validating email id for that im using this code

void CEMailDlg::OnValidate()
{
// TODO: Add your control notification handler code here
CString m_sFrom;
for(int a=0;a < m_sFrom.GetLength(); a++)
{
if(m_sFrom.GetAt(a) == '@')
{
AfxMessageBox("valid");
}
else
{
AfxMessageBox("invalid");
}
}
}
now the email id is not validating the problem is this statement " for(int a=0;a < m_sFrom.GetLength(); a++) "
i think there is problem in "m_sFrom.GetLength();"... i dono what problem it is,, pls help me

Joan M · Answer 5 · 2011-05-10T05:13:00

This is a simple error...

Then I understand that you are novice... so I'll give you some tricks:

- When working with objects, pressing "." will show you up the intellisense which should help you to find the right method.
- Each object type is based on a class... i.e. you have a CString (class) named m_sFrom (object). So you can search help (MSDN, google, Here...) to get details on how to work with i.e. strings looking for CString.

Apart from that OrigianlGriff has given you a proper answer... (joke to OG => Obvious ;P but proper)...