C++ question

[Anon]

Since i couldnt find any good c++-forum and many here probably now it, i have a question.
What is wrong with the code below?
I know it\'s something with the for-loop. It\'s suppose to check if there\'s only 1 \"@\", and if there\'s more than 1 it should type out \"Address may only contain 1\".

CString iText;

UpdateData(TRUE);

iText = m_MailAddress;

if (iText.Left(1) == \"@\")
m_MessageToUser = \"Address started with @\";

else if (iText.Right(1) == \"@\")
m_MessageToUser = \"Address ended with @\";

else if
int number=0;
for(int i=0;i<iText.length();i++){

if(iText.at(i)==\'@\')number++;
}
m_MessageToUser = \"Address may only contain 1 @\";

else
m_MessageToUser = \"Address is OK\";

UpdateData(FALSE);

}

[Anon]

I'd be surprised if there are no C++ coding forums.

Anyway, the problem is that your loop is inside the if() statement instead of outside.

[Anon]

So what exactly should the right code look like?

[Anon]

IFiu coded that you oughta now

[Anon]

c++ forum, code.box.sk/

[Anon]

Try this (I haven't checked it so there may still be a few bugs):

---

CString iText;

UpdateData(TRUE);

iText = m_MailAddress;

int number=0;
for(int i=0;i<iText.length();i++)
{
if(iText.at(i) == '@')
number++;
}

if (iText.Left(1) == "@")
{
m_MessageToUser = "Address started with @";
}
else if (iText.Right(1) == "@")
{
m_MessageToUser = "Address ended with @";
}
else if (number>1)
{
m_MessageToUser = "Address may only contain 1 @";
}
else
{
m_MessageToUser = "Address is OK";
UpdateData(FALSE);

}