还是字符串处理效率问题

[ldq]

CString DelSeparator(CString inStr, CString spstr, int icut)
{
	CString tmpstr="";
	CString retStr, backStr;
	int i;
	spstr.MakeUpper();
	if (spstr !="")
	{
		if (spstr=="SP")
		{
			spstr = " ";
		}	

		do 
		{
		  i = inStr.Find(spstr);
		  if (i<0)
		  {		  
			  break;
		  }
		  tmpstr = tmpstr + inStr.Left(i);
		  inStr  = inStr.Mid(i+1,inStr.GetLength()-i); 
		} while(1);
    
		  retStr  = tmpstr + inStr;
		  backStr = retStr.Left(retStr.GetLength()-icut);
	}
	else
	{
	      backStr = inStr.Left(inStr.GetLength()-icut);
	}		 	  
		return backStr;
	
}

CString 效率好像真不怎样，新学C++ 不知该怎么个改法 这个函数就是传入一段字符串 然后去掉传入字符串中指定的一个字符

[ryfdizuo]

求效率就直接用char*写。

[用户 昵称]

就TCHAR * + memcmp之类的就能实现。

[s874664]

物是人非 这种感觉你体会过吗

[smwhotjay]

[ldq]

新手 还真不会转换 我全是用CString 写的 现在遇到了字符串长度为9000+的 转换的还真慢

[ldq]

哎 自已改了下

CString DelSeparator(CString inStr, CString spstr, int icut)
{
	CString tmpstr="";
	CString retStr, backStr;
	int i;
	spstr.MakeUpper();
	if (spstr !="")
	{
		if (spstr=="SP")
		{
			spstr = " ";
		}	

		do 
		{
		  i = inStr.Find(spstr);
		  if (i<0)
		  {		  
			  break;
		  }
		  //tmpstr = tmpstr + inStr.Left(i);
		  //inStr  = inStr.Mid(i+1,inStr.GetLength()-i); 
		  inStr.Replace(spstr,"");
		} while(1);
          retStr = inStr;
		  //retStr  = tmpstr + inStr;
		  backStr = retStr.Left(retStr.GetLength()-icut);
	}
	else
	{
	      backStr = inStr.Left(inStr.GetLength()-icut);
	}		 	  
		return backStr;
	
}

现在好多了 怎么就没人提醒下我呢 求改进

[副组长]

试试这段代码，只能单字节的串。破坏原串。
你那个CString反复从头找估计是慢了。

void DelSeparator(char* inStr, char* spstr)
{
	char* pStr;
	char* tmpstr = inStr;
	int dl = strlen(spstr);
	int i,sl;
	do
	{
		sl = strlen(tmpstr);
        pStr = strstr(tmpstr,spstr);
        if( NULL == pStr)
        {
            break;
        }
		sl -= (pStr-tmpstr);
		tmpstr = pStr;
		for(i=0; i<sl-dl; i++)
		{
			*(tmpstr+i) = *(pStr+dl+i);
		}
		*(tmpstr+i) = 0;
    }while(1);
}
int _tmain(int argc, _TCHAR* argv[])
{
	char str[] = "123456788901234567887340";
	DelSeparator(str,"788");

	return 0;
}

[woncomp]

养成良好的习惯
对于字符串尽量使用 char*

你会发现在传参数的时候还有各项操作的时候都简单很多，也高效很多