__weak, похоже, сохраняет объект

Question

Слабые свойства Objective-C должны указывать на нуль, если объект освобождается, но в этом случае слабые свойства, похоже, сохраняют объект. Рассмотрим случай:

@interface SillyObject : NSObject 

@property (nonatomic, assign) NSInteger val; 

-(void)printVal; 

@end 

@implementation SillyObject 

-(void)printVal 
{ 
    NSLog(@"%d", self.val); 
} 

@end 

-(void)saveReference 
{ 
    SillyObject* s = [SillyObject new]; 
    s.val = 100; 

    [[ObjectCache sharedInstance] addWeakRef:s callback:^(NSString* junk) { 
    [s printVal]; 
    }]; 
} 

callSillyObjectBlocks перебирает все объекты добавляются в кэш и вызывает correspondending блоков (см ниже)

-(void)callDeadObject 
{ 
    [self saveReference]; 
    [[ObjectCache sharedInstance] callSillyObjectBlocks]; 
} 

Теперь saveReference выходы и SillyObject должны быть высвобождены, но это Безразлично 't и слабая ссылка не равна нулю.

Соответствующие детали реализации кэша:

typedef void (^Callback)(NSString* junk); 

@interface CacheSlot : NSObject 

@property (nonatomic, copy) Callback callback; 
@property (nonatomic, weak) id source; 
// More irrelevant properties. 

-(instancetype)initWithRef:(__weak id)obj callback:(Callback)cb; 

@end 

@implementation CacheSlot 

@synthesize callback, source; 

-(instancetype)initWithRef:(__weak id)obj callback:(Callback)cb 
{ 
    self = [super init]; 
    if(self) 
    { 
    self.callback = cb; 
    self.source = obj; 
    } 
    return self; 
} 

@end 

@interface ObjectCache() 

// This array contains CacheSlot objects 
@property (nonatomic, strong) NSMutableArray* savedObjects; 
@end 

// Implementation. 
-(void)addWeakRef:(id)obj callback:(Callback)block 
{ 
    __weak id src = obj; 
    [self.savedObjects addObject:[[CacheSlot alloc] initWithRef:src callback:block]]; 
} 

-(void)callSillyObjectBlocks 
{ 
    for(CacheSlot* slot in self.savedObjects) 
    { 
    if(slot.source) 
    { 
     slot.callback(@"Whatever!"); 
    } 
    else 
    { 
     // Remove the slot from cache 
    } 
    } 
} 

Вызывающие saveReference первоначально должен создать временный объект, который будет освобождён как только функция выходов (что он делает, если я позвоню addWeakRef: ноль вместо).

После вызова saveReference я запускаю callSillyObjectBlocks, и соответствующий блок добавленного объекта не следует вызывать, но он вызывается с его значением. Выход:

100 

Answer 1

Всякий раз, когда у вас есть блок, который ссылается на переменную вне блока, вы должны объявить его как слабой а. В противном случае вы получите сильную ссылку. Так оно и должно быть:

-(void)saveReference 
{ 
    SillyObject* s = [SillyObject new]; 
    s.val = 100; 
    SillyObject * __weak weakSilly = s; 

    [[ObjectCache sharedInstance] addWeakRef:s callback:^(NSString* junk) { 
    [weakSilly printVal]; 
    }]; 
} 

В то же время, вы можете упростить ваш код, удаляя __weak от нескольких других мест, так как они влияют на аргументы метода только то, что будет не сохраняющим экземпляр для соответствующего времени:

-(instancetype)initWithRef:(id)obj callback:(Callback)cb { 
    ... 
} 

-(void)addWeakRef:(id)obj callback:(Callback)block 
{ 
    [self.savedObjects addObject:[[CacheSlot alloc] initWithRef:obj callback:block]]; 
}