В выражениях, разделенных запятыми в Objective-C

Question

Недавно я столкнулся с this blog post, где обсуждаются два варианта освобождения переменных экземпляра. Чтобы дать Вам резюме:

Первый подход

- (void)dealloc { 
    [instanceVar release]; 
    [super dealloc]; 
} 

считается не оставить указатель живым до возвращения метода. Это может привести к неопределенному поведению. Второй подход

- (void)dealloc { 
    [instanceVar release], instanceVar = nil; 
    [super dealloc]; 
} 

считается более стабильным для производства кода, поскольку instaceVar устанавливается в ноль.

OK. Вот вопрос, который вы ждете: Оба решения одинаковы? Или разделение запятой выражений делает их атомарными?

Answer 1

Запятая не делает его атомарным, но в любом случае должен быть только один поток, вызывающий dealloc, иначе у вас на руках много, гораздо больших проблем. Установка instanceVar на nil не нужна, это не безопасно, потому что, если в вашей программе есть ошибка, которая каким-то образом использует оборванное значение в instanceVar, оно будет проявляться иначе, в зависимости от того, является ли оно nil или чем-то еще, в любом случае, есть ошибка, потому что ничего не должен полагаться на это значение.

С ARC это еще менее проблематично. Метод dealloc в ARC используется только для освобождения ресурсов, не поддерживающих ARC, все остальное автоматически обрабатывается для вас.

Answer 2

Сообщение в блоге связывании от 2010

Более интересным моментом является то, что в настоящее время вы должны сделать ни. Если вы используете ARC, вы не можете отправить сообщение об освобождении объектам.

Установка nil после освобождения была предназначена для предотвращения отправки неконсервированных объектов сообщения после того, как они были освобождены и вызвали сбой. Я говорю не удерживаемые объекты, потому что если бы они были сохранены, они не были бы (не должны) освобождаться неосознанно. Опять же, с ARC (до тех пор, пока вы не используете ARC-lite), вы можете пометить не удерживаемые объекты как weak, а слабые указатели - авто-ноль; то есть когда объект, на который они указывают, освобождается, указатель устанавливается на nil.

Таким образом, второй случай с указателями, установленными на nil после dealloc, считается более безопасным во время выполнения, но вам не нужно беспокоиться об этом сейчас, поскольку ARC обрабатывает это для вас.