Как избежать захвата себя в блоках при реализации API?

короткий ответ:

вместо self непосредственно, вы должны получить доступ к нему косвенно, из ссылки, которая не будет сохранена. если вы не используете автоматический подсчет ссылок (ARC), вы можете сделать это:

__block MyDataProcessor *dp = self;
self.progressBlock = ^(CGFloat percentComplete) {
    [dp.delegate myAPI:dp isProcessingWithProgress:percentComplete];
}

The __block ключевые слова помечают переменные, которые могут быть изменены внутри блока (мы этого не делаем), но также они не сохраняются автоматически при сохранении блока (если вы не используете ARC). Если вы сделаете это, вы необходимо убедиться, что после освобождения экземпляра MyDataProcessor больше ничего не будет пытаться выполнить блок. (Учитывая структуру вашего кода, это не должно быть проблемой.)подробнее о __block.

если вы используете ARC семантика __block изменения и ссылка будут сохранены, и в этом случае вы должны объявить его __weak вместо.

ответ

допустим, у вас был такой код это:

self.progressBlock = ^(CGFloat percentComplete) {
    [self.delegate processingWithProgress:percentComplete];
}

проблема здесь заключается в том, что self сохраняет ссылку на блок; между тем блок должен сохранить ссылку на self для того, чтобы получить его свойство делегата и отправить делегату метод. Если все остальное в вашем приложении освобождает ссылку на этот объект, его счетчик сохранения не будет равен нулю (потому что блок указывает на него), и блок не делает ничего плохого (потому что объект указывает на него), и поэтому пара объектов будет просачиваться в кучу, занимая память, но навсегда недостижимый без отладчика. Трагично, правда.

этот случай можно легко исправить, сделав это вместо этого:

id progressDelegate = self.delegate;
self.progressBlock = ^(CGFloat percentComplete) {
    [progressDelegate processingWithProgress:percentComplete];
}

в этом коде self сохраняет блок, блок сохраняет делегат, и нет циклов (видимых отсюда; делегат может сохранить наш объект, но это из наших рук прямо сейчас). Этот код не будет рисковать утечкой таким же образом, потому что значение свойства делегата захватывается, когда блок создал, а не искал, когда он выполняется. Побочным эффектом является то, что если вы измените делегат после создания этого блока, блок все равно будет отправлять сообщения об обновлении старому делегату. Произойдет это или нет, зависит от вашего приложения.

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

self.dataProcessor.progress = ^(CGFloat percentComplete) {
    [self.delegate myAPI:self isProcessingWithProgress:percentComplete];
};

вот вы проходите self непосредственно делегировать в вызове метода, так что вы должны получить его в где-то там. Если у вас есть контроль над определением типа блока, лучше всего было бы передать делегат в блок в качестве параметра:

self.dataProcessor.progress = ^(MyDataProcessor *dp, CGFloat percentComplete) {
    [dp.delegate myAPI:dp isProcessingWithProgress:percentComplete];
};

это решение позволяет избежать цикла сохранения и всегда вызывает текущий делегат.

если вы не можете изменить блок, вы могли бы разберись с этим. Причина, по которой цикл сохранения является предупреждением, а не ошибкой, заключается в том, что они не обязательно пишут doom для вашего приложения. Если MyDataProcessor может освободить блоки, когда операция будет завершена, прежде чем его родитель попытается освободить его, цикл будет нарушен, и все будет очищено должным образом. Если бы вы могли быть уверены в этом, то правильным было бы использовать #pragma чтобы подавить предупреждения для этого блока кода. (Или используйте флаг компилятора для каждого файла. Но не отключайте предупреждение для всего проекта.)

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

__weak MyDataProcessor *dp = self; // OK for iOS 5 only
__unsafe_unretained MyDataProcessor *dp = self; // OK for iOS 4.x and up
__block MyDataProcessor *dp = self; // OK if you aren't using ARC
self.progressBlock = ^(CGFloat percentComplete) {
    [dp.delegate myAPI:dp isProcessingWithProgress:percentComplete];
}

все три из вышеперечисленных дадут вам ссылку без сохранения результата, хотя все они ведут себя немного по-разному:__weak попытается обнулить ссылку, когда объект будет выпущен;__unsafe_unretained оставит вас с недопустимым указателем;__block фактически добавит еще один уровень косвенности и позволит вам изменить значение ссылки изнутри блока (не имеет значения в этом случае, так как dp нигде не используется).

что это лучшие будет зависеть от того, какой код вы можете изменить, а что нельзя. Но, надеюсь, это дало вам некоторые идеи о том, как поступить.