Question

OC内存管理问题。来大神。 当一个类的成员变量是另一个类的对象指针的时候，如何来写这个成员变量的s

OC内存管理问题。来大神。
当一个类的成员变量是另一个类的对象指针的时候，如何来写这个成员变量的setter方法？
比如Person类中包含Dog类的对象指针，目前想到最好的是这么写setter方法：{
if(dog != newDog)
{ [dog release]; dog = [newDog retain] } }
然后这么写 dealloc方法：{
[dog release];
NSLog(@"%@ 对象已释放",[self class]);
[super dealloc]; }
但是在main.m中假如先 [person setDog:dog1];
再[person setDog:dog2];
再[person setDog:dog1];
程序会崩溃，原因是dog1在setDog:dog2时已经被销毁或者说释放了，再去setDog:dog1明显不合法。

请问有什么好的代码方法解决这种一个类中的成员变量是对象指针时的内存管理问题么？

Answer 1

怎么会崩呢

Dog * dog1 = [[Dog alloc]init]; // dog1 引用计数为1 
[person setDog:dog1];     // dog1 在set方法中retain一次，引用计数为2
[person setDog:dog2];    // dog1被release一次，引用计数为1
[person setDog:dog1];      // dog1引用计数不为0，没有调用Dog的dealloc方法，没有被销毁

不会崩啊

就算当person的实例的引用计数为0了，调用了person的dealloc，在person的dealloc里面，dog被release一次，这时候dog的引用计数才变为1，也不会被dealloc

追问

[person setDog:dog1];
后面应该有[dog1 release]
同样[person setDog:dog2]; 后面也应该有一条释放dog2的语句对吧，你把这俩语句加上再试试

不然一个dealloc方法发送给对象指针一条release方法怎么释放掉计数器为2的对象的空间

追答

set之后release?

Dog * dog1 = [[Dog alloc]init];
[person setDog:dog1];     
[dog1 release];
[person setDog:dog2];

这就是你自己的内存管理逻辑错了啊，不是set方法的问题了。

要知道dog1在set之后release的话，不符合引用计数平衡原则啊，这样的话，在setDog方法再次被调用后dog1就会dealloc了，也就变成野指针了。而其他地方很明显是很可能还会用到dog1的。

你在person的dealloc方法里面release的是person的dog属性啊，你陷入了第二个误区：person实例dealloc后就要dealloc它所有的属性。这明显是不对的啊，因为这些属性所指向的地址（指针）很有可能是其他地方还要用的，不一定一个对象dealloc后就一定要dealloc掉它所有的属性啊。在dealloc方法里面把它们release一次是因为在set的时候retain了一次啊，一个retain对应一个release。

- (void)setDog:(Dog *)newDog
{
   if(dog != newDog) { 
       [dog release]; 
       dog = [newDog retain] ;
   } 
}

这个setter实际上是系统的实现方式，是正确的

所以综上，不是setter的问题，是你自己内存管理的问题，不应该在调用set方法后再把那个参数release一次，除非实参已经没有其他地方会用了。

当然在这里如果你用的是ARC的话就不用担心了。ARC会自动帮你在合适的位置release，比如

Dog * dog1 = [[Dog alloc]init];
[person setDog:dog1];     
[person setDog:dog2];

然后就返回方法了，这样的话，ARC会在[person setDog:dog2]前面添加[dog1 release];

如果

Dog * dog1 = [[Dog alloc]init];
[person setDog:dog1];     
[person setDog:dog2]; 
[person setDog:dog1];

ARC会在最后一个setDog:dog1后面加dog1 release。

追问

谢谢！！懂了！