没有躲过的坑--重载赋值运算符的自我赋值

标签: c++operator
1443人阅读 评论(1) 收藏 举报
分类:

C++中有个很重要的事情,就是对于类重载赋值运算符,而达到我们想要的结果。
先看看这几行代码:

//Window 是一个类
Window w;
w = w;          // 再傻的人也不会这么干
w[i] = w[j];    // 这个情况偶尔会发生

作为一个优秀的工程师,就要考虑到任何可能的情况。

看一段更加完整的代码:

class ScrollBar {};

class Window
{
    ScrollBar *sb;
public:
    Window(ScrollBar *s) : sb(s) {}
    Window() = default;
    Window& operator=(const Window&);
};

Window& Window::operator=(const Window& rhs)
{
    delete sb;
    sb = new ScrollBar(*rhs.sb);
    return *this;
}

int main()
{
    Window w(new ScrollBar);
    Window w2(w);
}

这段代码到底有什么坑儿呢?
设想一下,如果 *this 和rhs 是同一个实例对象呢?
那么

delete sb;
sb = new ScrollBar(*rhs.sb);

就会造成严重的问题。

再delete sb后, 我们试图去访问一个已经被删除的rhs。这当然是致命的坑儿了。

跨越这个坑儿:

if (this == &rhs)
    return *this;
delete sb;
sb = new ScrollBar(*rhs.sb);
return *this;

上面这段代码几乎所有的教科书都会这么讲,但是曾经一个arcserver的工程师跟我讲,这样同样存在危险,不是完美的:

试想一下,如果我们delete sb后发生了异常怎么办?这个时候,就会有存在一个没有指向任何东西的指针。所以下面这样写会更好:

Window& Window::operator=(const Window& rhs)
{
    if (this == &rhs)
    return *this;

    ScrollBar *sbOld = sb;
    sb = new ScrollBar(*rhs.sb);
    delete sbOld;
    return *this;
}
2
0

查看评论
* 以上用户言论只代表其个人观点,不代表CSDN网站的观点或立场
    个人资料
    • 访问:3969745次
    • 积分:36859
    • 等级:
    • 排名:第121名
    • 原创:542篇
    • 转载:28篇
    • 译文:13篇
    • 评论:724条
    微信公众号
      我的微信公众号
      为你推荐最新的博文~更有惊喜等着你
    时光荏苒
      白驹过隙
    博客专栏
    文章分类
    百度统计
    Google Analytics