高手帮忙看看,这个字符串的函数有什么问题,马上给分?

[verapan]

一个VC下的函数

void strF(CString str, float f)
{
str.Format("%s%f", (LPCTSTR)str, f);
//AfxMessageBox(str);
}

这样调用这个函数会不会有问题呢?

strF(CString(_T("Hello")), 10.0);

[fingerfox]

str.Format
first release the buffer
seconde realloc the buffer
so the error occur

[Paris_Luo]

这个函数
void strF(CString str, float f)
{
str.Format("%s%f", (LPCTSTR)str, f);
//AfxMessageBox(str);
}
有问题,str的字符缓冲区将会被破坏.

应该是
void strF(CString str, float f)
{
CString another;
another.Format("%s%f",(LPCTSTR)str,f);
//AfxMessageBox(another);
}
对

[HongHuer]

void strF(CString str, float f)
{
str.Format("%s%f", (LPCTSTR)str, f);
//AfxMessageBox(str);
}

这样调用这个函数会不会有问题呢?----------没有问题

strF(CString(_T("Hello")), 10.0); -------有个临时无名CString对象，
------但是你怎么使用这个无名的对象呢？

[HongHuer]

void strF(CString str, float f)
{
CString another;//----------------------这是个栈对象，你怎么传出去？？？
another.Format("%s%f",(LPCTSTR)str,f);
//AfxMessageBox(another);
}

[wuxfBrave]

void strF(CString str, float f)
{
str.Format("%s%f", (LPCTSTR)str, f);
//AfxMessageBox(str);
}
strF(CString(_T("Hello")), 10.0);

有问题，问题是你用str去format自己

[HongHuer]

楼主，你想要干什么？？？


你的函数要完成什么样的功能？？？？？

[wuxfBrave]

（LPCTSTR)str还是对str的操作

[verapan]

我先定义了一个字符串向量vector<CString> m_strVector;
然后往向量里面加入字符串，加入前想先调整一下格式，所以用这样一个函数先将字符串格式化，然后加入到向量中去

也就是：
vector<CString> m_strVector;

void strF(CString str, float f)
{
CString another;
another.Format("%s%f",str,f);
m_strVector.push_back(another);
}

上面那么做好像没错了

但我原先的做法到底错在哪里呢？
void strF(CString str, float f)
{
str.Format("%s%f",（LPCTSTR)str,f);
m_strVector.push_back(str);
}

是因为用str格式化自己吗

[lmopq]

这是MSDN的原话

The call will fail if the string object itself is offered as a parameter to Format. For example, the following code:

CString str = "Some Data";
str.Format("%s%d", str, 123); // Attention: str is also used in the parameter list.

will cause unpredictable results.

虽然你做了（LPCTSTR)str转换，但使用的依然是str。想知道清楚还是自己查查
operator LPCTSTR ( ) const和Format的源代码吧

[HongHuer]

1
void strF(CString str, float f)
{
CString another;
another.Format("%s%f",str,f);
m_strVector.push_back(another);
}

上面那么做好像没错了
>>
错。
CString another; 是 栈对象。函数栈析构后，变量值随机。


2
但我原先的做法到底错在哪里呢？
void strF(CString str, float f)
{
str.Format("%s%f",（LPCTSTR)str,f);
m_strVector.push_back(str);
}

是因为用str格式化自己吗

>>
是啊，是自己格式化自己。


3
我有个解法，你看看。

void strF(CString str, float f)
{
str.Format("%f",f);
m_strVector.push_back(another);
}

以后的不变。
你自己调看看。

[Zark]

这个函数
void strF(CString str, float f)
{
str.Format("%s%f", (LPCTSTR)str, f);
//AfxMessageBox(str);
}
有问题,str的字符缓冲区将会被破坏.

应该是
void strF(CString str, float f)
{
CString another;
another.Format("%s%f",(LPCTSTR)str,f);
//AfxMessageBox(another);
}

[verapan]

你的解法看不懂

你说：

void strF(CString str, float f)
{
CString another;
another.Format("%s%f",str,f);
m_strVector.push_back(another);
}

上面那么做好像没错了
>>
错。
CString another; 是 栈对象。函数栈析构后，变量值随机。

push_back()应该是存储变量的一个副本吧， 也就是调用拷贝构造函数， 应该没有问题的

[booklove]

Format函数的参数里不能有变量自己。

[wy2001wy]

直接这样就可以了吧:
strF(_T("Hello"), 10.0);

[verapan]

Zark(金陵五月)
bluebohe(薄荷) ( )

能不能再说清楚点啊, 还是有点不懂

void strF(CString str, float f)
{
str.Format("%s%f", (LPCTSTR)str, f);
//AfxMessageBox(str);
}

中应该会建立一个临时对象吧, 那临时对象越界是怎么回事啊, 字符串临时对象不能改变大小吗?

谢谢

[lostgdi731]

strF("Hello",10.0);
就可以了，没问题.

[verapan]

可我在调试的时候有时会报错, 大家再帮忙看看!!

[ringphone]

函数参数最好不要用类，而用类指针或引用。
strF(CString(_T("Hello")), 10.0);
这样会先构造CString(_T("Hello"))，因为是以值传递，会以生成的这个临时对象来调用CString的复制构造函数（CString::CString(CString&))进行初始化，最后还要析构，这么一个函数调用就执行了CString::CString(LPCTSTR)和CString::CString(CString&)，以及两次~CString。

[BigF]

楼上说的是用类做参数没有效率, 但效率低和出错是两回事啊

[bluebohe]

临时对象越界吧
楼上说得没错