class Widget { ... }; Widget w; ... w = w; // assignment to self |
class Base { ... }; class Derived: public Base { ... }; void doSomething(const Base& rb, // rb and *pd might actually be Derived* pd); // the same object |
class Bitmap { ... }; class Widget { ... private: Bitmap *pb; // ptr to a heap-allocated object }; |
Widget& Widget::operator=(const Widget& rhs) // unsafe impl. of operator= { delete pb; // stop using current bitmap pb = new Bitmap(*rhs.pb); // start using a copy of rhs’s bitmap return *this; // see Item 10 } |
Widget& Widget::operator=(const Widget& rhs) { if (this == &rhs) return *this; // identity test: if a self-assignment, // do nothing delete pb; pb = new Bitmap(*rhs.pb); return *this; } |
这个也能工作,但是我在前面提及那个先前版本的 operator= 不仅仅是自赋值不安全(self-assignment-unsafe),它也是异常不安全(exception-unsafe)的,而且这个版本还是有异常上的麻烦。详细地说,如果 "new Bitmap" 表达式引发一个异常(可能因为供分配的内存不足或者因为 Bitmap 的拷贝构造函数抛出一个异常),Widget 将以持有一个被删除的 Bitmap 的指针而告终。这样的指针是有毒的,你不能安全地删除它们。你甚至不能安全地读取它们。你对它们唯一能做的安全的事情就是花费大量的调试精力来断定它们起因于哪里。
碰巧的是,使 operator= 异常安全(exception-safe)也同时弥补了它的自赋值安全(self-assignment-safe)。这就导致了更加通用的处理自赋值问题的方法就是忽略它,而将焦点集中于取得异常安全。Item 29 更加深入地探讨了异常安全,但是在本 Item 中,已经足够我们在很多情况下来观察它,仔细地调整一下语句的顺序就可以得到异常安全(exception-safe)(同时也是自赋值安全)的代码。例如,在这里,我们只要注意直到我们拷贝了 pb 指向的目标之后,才能删除它: