need some correction in implementing TrimLeft(wchar_t*)

[bajajv]

This is my code -

i/p is L"!?%@ VIPUL"
o/p is @ VIPUL.
desired o/p is VIPUL

Not getting the correct o/p.

class CString 
{ 
public:
	wstring wStr; 
                int getLength();
                CString& TrimLeft(wchar_t* wcChar);
};

CString& CString::TrimLeft(wchar_t* wcChar)
{
	wstring::iterator iterCString = wStr.begin();
	int iLength = this->getLength();
	wchar_t* wcTemp = wcChar;

	while (*wcTemp != L'\0')
	{
		if (*iterCString == *wcTemp)
		{
			wStr.erase(0, 1);
			iterCString++;
			wcTemp = wcChar;
			iLength = this->getLength();
		}
		else
		{
			wcTemp++;			
		}
	}
	return *this;
}

int main()
{
     	CString name = L"!?%@ VIPUL";
	wcout<<name.wStr<<endl;
	wchar_t wcArray[5];
	wcArray[0] = L'%';
	wcArray[1] = L'!';
	wcArray[2] = L'?';
	wcArray[3] = L'@';
	wcArray[4] = L' ';
	name.TrimLeft(wcArray);
	wcout<<name.wStr<<endl;
	return 0;
}

[bajajv]

Got the answer.

No need for incrementing iterator in If-else condition. Since it is set to the beginning of the string, it will automatically increment whenever first letter of wstring is deleted.

Well, plz correct me if I am wrong, but it is working that way..... :))
Going to consult some text.

[weaknessforcats]

wStr.erase(0, 1); 
            iterCString++;

The erase invalidates the iterator.

[RRick]

The correct use of erase is

 iterCString = wStr.erase(0, 1);

But there are other issues here. When you find a bad wchar, you always erase the first wchar in the string. You probably want to erase the wchar at the iterator position.

 iterCString = wStr.erase( iterCString);