friend: const Rational // see Item 3 for why the Operator*(const Rational& lhs, // return type is const const Rational& rhs); }; operator* 的这个版本以传值方式返回它的结果,而且假如你没有担心那个对象的构造和析构的代价,你就是在推卸你的专业职责。假如你不是迫不得已,你不应该为这样的一个对象付出成本。所以问题就在这里:你是迫不得已吗?
const Rational& operator*(const Rational& lhs, // warning! bad code! const Rational& rhs) { Rational result(lhs.n * rhs.n, lhs.d * rhs.d); return result; } 你可以立即否决这种方法,因为你的目标是避免调用构造函数,而 result 正像任何其它对象一样必须被构造。一个更严重的问题是这个函数返回一个引向 result 的引用,但是 result 是一个局部对象,而局部对象在函数退出时被销毁。那么,这个 operator* 的版本不会返回引向一个 Rational 的引用——它返回引向一个前 Rational;一个曾经的 Rational;一个空洞的、恶臭的、腐败的,从前是一个 Rational 但永不再是的尸体的引用,因为它已经被销毁了。任何调用者甚至于没有来得及匆匆看一眼这个函数的返回值就马上进入了未定义行为的领地。这是事实,任何返回一个引向局部变量的引用的函数都是错误的。(对于任何返回一个指向局部变量的指针的函数同样成立。)
那么,让我们考虑一下在堆上构造一个对象并返回引向它的引用的可能性。基于堆的对象通过使用 new 而开始存在,所以你可以像这样写一个基于堆的 operator*:
const Rational& operator*(const Rational& lhs, // warning! more bad const Rational& rhs) // code! { Rational *result = new Rational(lhs.n * rhs.n, lhs.d * rhs.d); return *result; } 哦,你还是必须要付出一个构造函数调用的成本,因为通过 new 分配的内存要通过调用一个适当的构造函数进行初始化,但是现在你有另一个问题:谁是删除你用 new 做出来的对象的合适人选?
即使调用者尽职尽责且一心向善,它们也不太可能是用这样的方案来合理地预防泄漏:
Rational w, x, y, z;
w = x * y * z; // same as operator*(operator*(x, y), z) 这里,在同一个语句中有两个 operator* 的调用,因此 new 被使用了两次,这两次都需要使用 delete 来销毁。但是 operator* 的客户没有合理的办法进行那些调用,因为他们没有合理的办法取得隐藏在通过调用 operator* 返回的引用后面的指针。这是一个早已注定的资源泄漏。
const Rational& operator*(const Rational& lhs, // warning! yet more const Rational& rhs) // bad code! { static Rational result; // static object to which a // reference will be returned
result = ... ; // multiply lhs by rhs and put the // prodUCt inside result return result; } 就像所有使用了 static 对象的设计一样,这个也会立即引起我们的线程安全(thread-safety)的混乱,但那是它的比较明显的缺点。为了看到它的更深层的缺陷,考虑这个完全合理的客户代码:
bool operator==(const Rational& lhs, // an operator== const Rational& rhs); // for Rationals
Rational a, b, c, d;
... if ((a * b) == (c * d)) { do whatever’s appropriate when the products are equal; } else { do whatever’s appropriate when they’re not; } 猜猜会怎么样?不管 a,b,c,d 的值是什么,表达式 ((a*b) == (c*d)) 总是等于 true!
我无法拿出示例代码来肯定这个设计,但我可以概要说明为什么这个想法应该让你羞愧得无地自容。首先,你必须选择一个 n 作为数组的大小。假如 n 太小,你可能会用完存储函数返回值的空间,与刚刚名誉扫地的 single-static 设计相比,在任何一个方面你都不会得到更多的东西。但是假如 n 太大,就会降低你的程序的性能,因为在函数第一次被调用的时候数组中的每一个对象都会被构造。即使这个我们正在讨论的函数仅被调用了一次,也将让你付出 n 个构造函数和 n 个析构函数的成本。假如“优化”是提高软件效率的过程,对于这种东西也只能是“悲观主义”的。最后,考虑你怎样将你所需要的值放入数组的对象中,以及你做这些需要付出什么。在两个对象间移动值的最直接方法就是通过赋值,但是一次赋值将要付出什么?对于很多类型,这就大约相当于调用一次析构函数(销毁原来的值)加上调用一次构造函数(把新值拷贝过去)。但是你的目标是避免付出构造和析构成本!面对的结果就是:这个方法绝对不会成功。(不,用一个 vector 代替数组也不会让事情有多少改进。)