effective c++第11

shaAbca 2011-01-02 10:11:57
class Bitmap
{
};

class Widget
{
Bitmap* pb;
}

Widget& Widget::operator=(const Widget& rhs)
{
delete pb; //

Widget& Widget::operator=(const Widget& rhs)
{
Bitmap* pOrig = pb; //
}

int main()
{
Widget w;
w = w; //执行到这里时,上面两个//的意思是?
}
...全文
74 5 打赏 收藏 转发到动态 举报
写回复
用AI写文章
5 条回复
切换为时间正序
请发表友善的回复…
发表回复
  • 打赏
  • 举报
回复
要摒弃 自己给自己赋值这一操作
所以更多安全的代码里面都有
if(this == &rhs)这个操作...
yutaooo 2011-01-02
  • 打赏
  • 举报
回复

delete pOrig;的时候,在这个时间点上,pb指向一个新new出来的对象,不在是pOrig指向的那个了。


很简单的,你再想想,估计一下卡住了,呵呵。
shaAbca 2011-01-02
  • 打赏
  • 举报
回复
Widget& Widget::operator=(const Widget& rhs)
{
delete pb;
pb = new Bitmap(*rhs.pb);
return *this;
}

当w = w,(*this).pb = rhs.pb,
delete pb,rhs.pb = ?
pb = new Bitmap(*rhs.pb)时
对它进行解引用所以失败了.pb = ?

Widget& Widget::operator=(const Widget& rhs)
{
Bitmap *pOrig = pb;
pb = new Bitmap(*rhs.pb);
delete pOrig;
return *this;
}

当w = w时.
这代表pOrig = pb = rhs.pb;
pb = new Bitmap(*rhs.pb);
delete pOrig;不是pb = ?吗 rhs.pb也等于?吗
怎么是合法的呢?

yutaooo 2011-01-02
  • 打赏
  • 举报
回复

这大概是在说,对于operator=()的实现,要考虑自己赋值给自己,这种情况。

自己赋值给自己,应该什么都不发生。而,如果不加检测,直接delete pb,则有可能错误的清空自己。
無_1024 2011-01-02
  • 打赏
  • 举报
回复
Widget& Widget::operator=(const Widget& rhs)
{
delete pb; // 删除当前对象空间

Widget& Widget::operator=(const Widget& rhs)
{
Bitmap* pOrig = pb; // 构造一个基类指针指向当前的对象
}

64,654

社区成员

发帖
与我相关
我的任务
社区描述
C++ 语言相关问题讨论,技术干货分享,前沿动态等
c++ 技术论坛(原bbs)
社区管理员
  • C++ 语言社区
  • encoderlee
  • paschen
加入社区
  • 近7日
  • 近30日
  • 至今
社区公告
  1. 请不要发布与C++技术无关的贴子
  2. 请不要发布与技术无关的招聘、广告的帖子
  3. 请尽可能的描述清楚你的问题,如果涉及到代码请尽可能的格式化一下

试试用AI创作助手写篇文章吧