zoukankan      html  css  js  c++  java
  • 【11】在operator=中处理“自我赋值”

    1、自我赋值,看起来愚蠢,但是却合法。有些自我赋值一眼就可看出来。有些自我赋值是潜在的。比如:a[i] = a[j]; *px = *py; 甚至不同类型的指针,都指向同一个地址,也是自我赋值,这一类自我赋值,很难识别,因此对自我赋值要有一定的防范。

    2、对于资源管理类auto_ptr和shared_ptr,自我赋值是安全的。如果自行管理资源,比如Widget中有个Bitmap* pb;copy赋值如下:

    1 Widget& Widget::operator=(const Widget& rhs)
    2 {
    3  delete pb;
    4  pb = new Bitmap(*(rhs.pb));
    5  return *this;
    6 }

    假如是自我赋值,delete销毁了this.pb,也是销毁了rhs.pb,rhs.pb指向垃圾,new出来的Bitmap也有问题。

    3、怎么解决上面的问题,增加一个等同测试,如下:

     1 Widget& Widget::operator=(const Widget& rhs)
     2 {
     3  if(this == &rhs)
     4  {
     5   return *this;
     6  }
     7 
     8  delete pb;
     9  pb = new Bitmap(*(rhs.pb));
    10  return *this;
    11 }

    4、上面的方法还是有问题,考虑new Bitmap的时候出现异常,这种情况下,老的对象已经销毁,新的又没有分配成功,导致pb指向垃圾。怎么解决这个问题呢?
      问题的关键是,先删除老的,再去分配新的。因此,可以这样解决:先把老的记录下来,再去分配新的,然后再去删除老的,如果分配新的出现异常,退出方法,老的没有删除,还是原来的值。如下:

     1 Widget& Widget::operator=(const Widget& rhs)
     2 {
     3  if(this == &rhs)
     4  {
     5   return *this;
     6  }
     7 
     8  Bitmap* old = pb;
     9  pb = new Bitmap(*(rhs.pb));
    10  delete old;
    11  return *this;
    12 }

    5、再仔细分析一下,上面不需要等同测试,即使相同也是没有问题的,只不过导致Bitmap的copy构造。这种情况下,考虑等同测试的成本和Bitmap copy构造的成本,如果等同的频率很高,添加等同测试,否则去掉等同测试。
    6、有没有更好的办法呢?

      既然可能出现自我赋值,我可以先对rhs做个copy temp,然后this和temp交换,这样也就不需要考虑自我赋值了。特别注意:下面的交换只是交换指针值,并不交换指向的内容,可以认为是浅交换。

    1 Widget& Widget::operator=(const Widget& rhs)
    2 {
    3     Widget temp(rhs);
    4     swap(temp);
    5     return *this;
    6 }

    7、上面的做法还可以优化,形参表中使用传值,方法内直接使用swap(rhs); 这种方法很好地利用了传值的特点,具有技巧性,但是失去了清晰性,不推荐这种做法。

  • 相关阅读:
    彩色文件夹、彩色文件、图标标记(MultiColorWin)快速操作
    git 国内源
    Mysql 日期格式化 复杂日期区间查询
    项目版本管理Git使用详细教程
    SpringBoot 优雅配置跨域多种方式及Spring Security跨域访问配置的坑
    SpringBoot Spring Security 核心组件 认证流程 用户权限信息获取详细讲解
    SpringBoot包扫描之多模块多包名扫描和同类名扫描冲突解决
    初阶绘图
    变量与档案存取
    结构化程式和自定义函数
  • 原文地址:https://www.cnblogs.com/nzbbody/p/3527481.html
Copyright © 2011-2022 走看看