条款21:必须返回对象时,别勉强返回其reference

pass by reference的滥用情况

reference的方式返回必然需要创建一个对象,耗费资源,这已经是一个糟糕的行为。

1. 返回reference指向一个local对象(也就是在stack空间创建的对象)
const Ra& operator*(const Ra& lhs, const Ra& rhs)
{
	Ra ret(lhs.n*rhs.n, lhs.d*rhs.d);
	return ret;
}

local对象在函数退出前就被销毁了,这是一个“无定义行为”,十分危险,返回指针指向一个local对象的行为同理。

2. 在heap内构建一个对象,并返回reference指向它。
const Ra& operator*(const Ra& lhs, const Ra& rhs)
{
	Ra* rp = new Ra(lhs.n*rhs.n, lhs.d*rhs.d);
	return *rp;
}

需要调用者实施delete,即使调用者意识良好,依然可能造成内存泄漏:

Ra w, x, y, z;
w = x * y * z; //operator*(operator*(x, y), z)

调用了两次new,调用者无法取得operator*返回的reference背后隐藏的那个指针,无法对该指针实施delete。

3. 返回的reference指向class内的static对象。

这确实避免了(函数调用时)构造函数被调用,但引发了其他问题:

Ra a, b, c, d;
if((a*b) == (c*d)){
	//当乘积相等时,做适当的动作
}

表达式始终为true,两次operator*调用确实各自改变了static对象的值,但由于它们返回的都是reference,指向同一对象,调用端始终看到的是static对象的“现值”。

当你必须使用by-value时,就交给编译器“优化”吧。

请记住:
绝不要返回pointer或reference指向一个local对象,或返回reference指向一个heap-allocated对象,或返回pointer或reference指向一个local static对象。

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值