CString的错误，快来看啊

[So1o]

在给我一个类的成员变量(CString)赋值时总是出错，
CMyClass
{
public:
SetString(LPCTSTR str) {m_str = str;}//出错

protected:
CString m_str;
}

出错位置如下：
void CString::AllocBeforeWrite(int nLen)
{
if (GetData()->nRefs > 1 || nLen > GetData()->nAllocLength) //////////此句出错！
{
Release();
AllocBuffer(nLen);
}
ASSERT(GetData()->nRefs <= 1);
}

何解？ 是不是其他地方把内存搞乱了？

[striking]

SetString(LPCTSTR str) 判断str是否是合法的字符指针.

[So1o]

str没有问题，
MyClass aa;
aa.SetString("abc");

这样都出错

[cozil]

参数改成CString,或者m_str.Format(_T("%s"),str);

[maohbao]

参数改成CString

[tudou614]

#include <afx.h>
#include <iostream>
using std::cout;
using std::cin;
using std::endl;

class CMyClass
{
public:
//我要保存一份COPY,该怎么把这个函数设计到非常强悍的地步??
CString SetString(LPCTSTR str)
{
m_str = str;
return str; //这里显然不行,因为超出作用域,销毁掉
}
protected:
CString m_str;
};


int main(void)
{
CMyClass my;
cout<<"the display is:"<<my.SetString("hello")<<endl;
cin.get();
return 0;
}

艾,记性不好,忘性到不小,楼下的提示偶一下

[So1o]

是CString分配空间时出错的，楼上各位说的都不对啊。

[nike007]

CString LPCTSTR 我记的好象是不能直接传值的

[tudou614]

艾,今天又问了一个傻问题
DEBUG:CString SetString(LPCTSTR str)改为为

LPCTSTR SetString(CString str)
{
m_str = str;
return str;
}

这样设计,这不是用牛刀杀鸡吗??呵呵

[tudou614]

void CString::AllocBeforeWrite(int nLen)
{
//假设你这里有new操作
ASSERT(.....); //在这里就要看下new出来没有
//一般都会设计成下面这种好理解的方式,而不是你那种正好倒过来的
if(GetData()->nRefs < 1 || nLen< GetData()->nAllocLength)
{
Release();
}
}