随笔-204  评论-149  文章-0  trackbacks-0
Effective C++
条款31: 千万不要返回局部对象的引用,也不要返回函数内部用new初始化的指针的引用


 1 #include<iostream>
 2 #include<string>
 3 
 4 using namespace std;
 5 
 6 class SimpleCat
 7 {
 8 public:
 9     SimpleCat(void);
10     SimpleCat(int age,int weight);
11     int GetAge(){return itsAge;}
12     int GetWeight(){return itsWeight;}
13 
14 private:
15     int itsAge;
16     int itsWeight;
17     //int itsAge=5;//1>.\SimpleCat.cpp(14) : error C2864: “SimpleCat::itsAge”: 只有静态常量整型数据成员才可以在类中初始化
18 
19 public:
20     virtual ~SimpleCat(void);
21 };
22 
23 SimpleCat::SimpleCat(void)
24 {
25     cout<<"SimpleCat constructor "<<this<<endl;
26 }
27 
28 SimpleCat::SimpleCat(int age,int weight)
29 {
30     cout<<"SimpleCat constructor "<<this<<endl;
31     itsAge = age;
32     itsWeight = weight;
33 }
34 
35 SimpleCat::~SimpleCat(void)
36 {
37     cout<<"SimpleCat destructor"<<this<<endl;
38 }
39 
40 SimpleCat &TheFunction();
41 
42 int main()
43 {
44     SimpleCat myCat;
45     cout<<myCat.GetAge()<<endl;//这个值区别于java不是赋初值为0的,而是一个随机的值
46 //    cout<<myCat.itsAge<<endl;
47 
48     cout<<"------------------------"<<endl;
49     SimpleCat &rCat = TheFunction();
50     int age = rCat.GetAge();
51     cout<<"rCat is "<<age<<"yeas old!"<<endl;
52     cout<<"&rCat:  "<<&rCat<<endl;
53     SimpleCat *pCat = &rCat;
54     //delete rCat;//不能对引用使用delete
55     delete pCat;
56     //delete好像没有释放内存,怎么获取的还是原来的值
57     //可能在这个内存区域存放的还是原来的?先new string后再调用也没变,与编译器有关还是什么??
58     for(int i =0;i<10;i++)
59     {
60         //想通过创建string对象来填充之前的内存区间,好像没用
61         string *= new string("abcdefghijklmn");
62     }
63 
64     //这时问题来了,rCat.getAge()为123了
65     SimpleCat *pSecond = new SimpleCat(123,444);
66     
67     cout<<"delete pCat后再使用rCat引用会发生什么问题???"<<endl;
68     cout<<"delete pCat后 &rCat"<<&rCat<<endl;
69     cout<<"delete pCat后 rCat.age"<<rCat.GetAge()<<endl;
70 
71     cout<<"--------------------------"<<endl;
72     SimpleCat myCat2 = TheFunction();//这个会发生内存泄漏,在函数中申请的内存会得不到释放
73                                      //myCat2是通过默认的拷贝函数来进行初始化的
74     cout<<"myCat2的地址是 "<<&myCat2<<endl;
75     return 0;
76 }
77 
78 
79 //这个函数在返回时,是否会创建一个临时引用变量???
80 //TheFunctionTwo(SimpleCat &simpleCat)这个函数在传递参数时是否会创建一个临时的引用变量
81 //临时的引用变量的作用域范围是什么
82 SimpleCat &TheFunction()
83 {
84     SimpleCat *pFrisky = new SimpleCat(5,9);
85     cout<<"pFrisky: "<<pFrisky<<endl;
86     return *pFrisky;
87 }
88 
89 
90 

函数TheFunction()的返回值被赋值给对SimpleCat类的一个引用,然后使用这个引用来获得对象的成员变量age的值
为了证明函数TheFunction()在自由存储区中创建的对象被赋给了函数main()声明的引用,对rCat采取取址运算,很明显,它显示了其引用的对象的地址,并且和自由存储区中的对象的地址相匹配。
这一切看上去都很正确,但是如何释放被占用的内存,对于引用是不能使用delete运算符的,一个笔记聪明的方法是声明另一个指针,并使用由rCat得到的地址对它进行初始化。这样做确实可以释放内存,避免了内存的泄漏,但是存在一个小问题,在delete pCat后,rCat引用又代表什么呢??引用必须是一个实际对象的别名;如果引用一个空对象的话(现在就是这种情况)那么程序就是无效的。。

注意:不能过分去强调一个使用对空对象的引用的程序,也行能通过编译,但是即使程序通过了编译,它仍然是病态的,而且执行结果也是不可预料的

关于这个问题,存在这3种解决方案
第1种是在第49行声明一个SimpleCat类对象,然后从函数TheFunction()中采取值传递的方式返回它的局部对象,这时不能在自由存储区来分配对象空间,不能使用new,
第2种是在函数TheFunction()中声明一个位于自由存储区中的SimpleCat类的对象,但是让函数返回一个指向内存区域的指针,然后完成对对象的操作,调用函数可以删除这个指针
第3种可行的方案,也是正确的,是在调用函数中声明对象,然后通过引用的方式把它传递给函数TheFunction()


也意味在main函数中不能这样使用了

posted on 2009-05-11 23:01 Frank_Fang 阅读(1129) 评论(2)  编辑  收藏 所属分类: C++编程

评论:
# re: C++函数返回自由内存区间的对象的引用 2009-06-06 22:06 | yyy
条款31: 千万不要返回局部对象的引用,也不要返回函数内部用new初始化的指针的引用

本条款听起来很复杂,其实不然。它只是一个很简单的道理,真的,相信我。

先看第一种情况:返回一个局部对象的引用。它的问题在于,局部对象 ----- 顾名思义 ---- 仅仅是局部的。也就是说,局部对象是在被定义时创建,在离开生命空间时被销毁的。所谓生命空间,是指它们所在的函数体。当函数返回时,程序的控制离开了这个空间,所以函数内部所有的局部对象被自动销毁。因此,如果返回局部对象的引用,那个局部对象其实已经在函数调用者使用它之前被销毁了。

当想提高程序的效率而使函数的结果通过引用而不是值返回时,这个问题就会出现。下面的例子和条款23中的一样,其目的在于详细说明什么时候该返回引用,什么时候不该:

class rational { // 一个有理数类
public:
rational(int numerator = 0, int denominator = 1);
~rational();

...

private:
int n, d; // 分子和分母

// 注意operator* (不正确地)返回了一个引用
friend const rational& operator*(const rational& lhs,
const rational& rhs);
};

// operator*不正确的实现
inline const rational& operator*(const rational& lhs,
const rational& rhs)
{
rational result(lhs.n * rhs.n, lhs.d * rhs.d);
return result;
}

这里,局部对象result在刚进入operator*函数体时就被创建。但是,所有的局部对象在离开它们所在的空间时都要被自动销毁。具体到这个例子来说,result是在执行return语句后离开它所在的空间的。所以,如果这样写:

rational two = 2;

rational four = two * two; // 同operator*(two, two)


函数调用时将发生如下事件:

1. 局部对象result被创建。
2. 初始化一个引用,使之成为result的另一个名字;这个引用先放在另一边,留做operator*的返回值。
3. 局部对象result被销毁,它在堆栈所占的空间可被本程序其它部分或其他程序使用。
4. 用步骤2中的引用初始化对象four。

一切都很正常,直到第4步才产生了错误,借用高科技界的话来说,产生了"一个巨大的错误"。因为,第2步被初始化的引用在第3步结束时指向的不再是一个有效的对象,所以对象four的初始化结果完全是不可确定的。

教训很明显:别返回一个局部对象的引用。

"那好,"你可能会说,"问题不就在于要使用的对象离开它所在的空间太早吗?我能解决。不要使用局部对象,可以用new来解决这个问题。"象下面这样:

// operator*的另一个不正确的实现
inline const rational& operator*(const rational& lhs,
const rational& rhs)
{
// create a new object on the heap
rational *result =
new rational(lhs.n * rhs.n, lhs.d * rhs.d);

// return it
return *result;
}

这个方法的确避免了上面例子中的问题,但却引发了新的难题。大家都知道,为了在程序中避免内存泄漏,就必须确保对每个用new产生的指针调用delete,但是,这里的问题是,对于这个函数中使用的new,谁来进行对应的delete调用呢?

显然,operator*的调用者应该负责调用delete。真的显然吗?遗憾的是,即使你白纸黑字将它写成规定,也无法解决问题。之所以做出这么悲观的判断,是基于两条理由:

第一,大家都知道,程序员这类人是很马虎的。这不是指你马虎或我马虎,而是指,没有哪个程序员不和某个有这类习性的人打交道。想让这样的程序员记住无论何时调用operator*后必须得到结果的指针然后调用delete,这样的几率有多大呢?也是说,他们必须这样使用operator*:

const rational& four = two * two; // 得到废弃的指针;
// 将它存在一个引用中
...

delete &four; // 得到指针并删除

这样的几率将会小得不能再小。记住,只要有哪怕一个operator*的调用者忘了这条规则,就会造成内存泄漏。

返回废弃的指针还有另外一个更严重的问题,即使是最尽责的程序员也难以避免。因为常常有这种情况,operator*的结果只是临时用于中间值,它的存在只是为了计算一个更大的表达式。例如:

rational one(1), two(2), three(3), four(4);
rational product;

product = one * two * three * four;

product的计算表达式需要三个单独的operator*调用,以相应的函数形式重写这个表达式会看得更清楚:

product = operator*(operator*(operator*(one, two), three), four);

是的,每个operator*调用所返回的对象都要被删除,但在这里无法调用delete,因为没有哪个返回对象被保存下来。

解决这一难题的唯一方案是叫用户这样写代码:

const rational& temp1 = one * two;
const rational& temp2 = temp1 * three;
const rational& temp3 = temp2 * four;

delete &temp1;
delete &temp2;
delete &temp3;

果真如此的话,你所能期待的最好结果是人们将不再理睬你。更现实一点,你将会在指责声中度日,或者可能会被判处10年苦力去写威化饼干机或烤面包机的微代码。

所以要记住你的教训:写一个返回废弃指针的函数无异于坐等内存泄漏的来临。

另外,假如你认为自己想出了什么办法可以避免"返回局部对象的引用"所带来的不确定行为,以及"返回堆(heap)上分配的对象的引用"所带来的内存泄漏,那么,请转到条款23,看看为什么返回局部静态(static)对象的引用也会工作不正常。看了之后,也许会帮助你避免头痛医脚所带来的麻烦。
  回复  更多评论
  
# re: C++函数返回自由内存区间的对象的引用 2009-06-06 22:24 | yyy
const rational& four = two * two; // 得到废弃的指针;
// 将它存在一个引用中

还必须得使用引用,否则会发生内存泄漏
  回复  更多评论
  

只有注册用户登录后才能发表评论。


网站导航: