Если класс A использует класс B, а класс A является делегатом класса B, нормально ли, если в делелоке класса B для делегата установлено значение nil? Я видел код, обычно сбрасывающий делегата на nil внутри Dealloc класса A, но не был уверен, что реальная разница делает это так или иначе.
например Это обычный способ:
// somewhere in class A
- (void) someFunc {
self.b = [[B alloc] init];
self.b.delegate = self;
}
- (void) dealloc {
self.b.delegate = nil;
[self.b release];
}
Да, вы должны установить для свойства делегата класса B значение nil в Dealloc класса A.
Это не проблема управления памятью, потому что свойства делегата должны быть помечены как назначенные, а не сохраненные, чтобы избежать циклов сохранения (иначе Dealloc никогда не будет вызываться). Проблема в том, что в противном случае класс B мог бы отправить сообщение классу A после того, как он был выпущен.
Например, если у classB есть вызов delagate, чтобы сказать, что он скрыт, и classB освобождается сразу после classA, он будет сообщать уже освобожденному classA, вызывая сбой.
И помните, вы не всегда можете гарантировать выполнение заказа, особенно если они автоматически освобождаются.
Так что да, обнулите свойство делегата в делелоке classA.
Насколько я знаю, лучше всего (назначать) делегата, чтобы вы избегали циклических ссылок на счетчики сохранения для подобных ситуаций. Если вы правильно настроили свойство, т.е.:
@property (assign) id<BDelegate> delegate;
Вам не нужно выполнять какое-либо управление памятью в Dealloc, так как счетчик сохранения не увеличивается, когда вы вызываете self.b.delegate = self; -- в отличие от использования (сохранить) или (копировать)
Есть смысл? Было бы хорошо установить для делегата значение nil, но какой в этом смысл?
Сначала несколько наблюдений...
[super dealloc] в конце вашего собственного метода Dealloc.-dealloc, так как 'b' все равно будет уничтожен. Если возможно, что другие объекты имеют ссылку на 'b' (это означает, что он может пережить 'a'), установите для делегата значение nil.self.b.delegate = self эквивалентно [[self getB] setDelegate:self] — это просто синтаксический сахар, из-за которого создается впечатление, что вы обращаетесь к ivar напрямую, но на самом деле это не так.self.b сохраняет значение (свойство установлено на «назначить»), у вас есть утечка памяти.Вот как я, вероятно, написал бы это:
- (void) someFunc {
b = [[B alloc] init];
b.delegate = self; // or [b setDelegate:self];
}
- (void) dealloc {
b.delegate = nil;
[b release];
[super dealloc];
}
self.b = [[B alloc] init];вы никогда не должны использоватьselfна левой стороне, если вы используетеallocна правой стороне, если@propertyдляbсохраняется. Просто хотел добавить. - person thesummersign schedule 22.03.2012