Слабая личность в блоках

В этой проверке нет необходимости, и она дает вам ложное чувство безопасности.

Вот проблема:

__weak typeof(self) weakSelf = self;
dispatch_async(dispatch_get_main_queue(), ^{
    if (!weakSelf) { return; }
    // THE LINE OF INTEREST
    [weakSelf doSomething];
});

В THE LINE OF INTEREST какой-то другой поток может очистить последнюю сильную ссылку на self, после чего weakSelf устанавливается в nil. Таким образом, сообщение doSomething отправляется на nil, что является «безопасным» (оно ничего не делает), но может оказаться не таким, как вы ожидали!

Хуже, если вы хотите предпринять другое действие, когда weakSelf равно нулю, например.

__weak typeof(self) weakSelf = self;
dispatch_async(dispatch_get_main_queue(), ^{
    if (weakSelf) {
        [weakSelf doSomething];
    } else {
        [someOtherObject doSomethingElse];
    }
});

В этом случае между моментом, когда блок проверяет, что weakSelf не является nil, и временем, когда он отправляет сообщение doSomething, weakSelf может стать nil, и ни doSomething, ни doSomethingElse фактически не будут выполняться.

Правильное решение такое:

__weak typeof(self) weakSelf = self;
dispatch_async(dispatch_get_main_queue(), ^{
    typeof(self) strongSelf = weakSelf;
    if (strongSelf) {
        [strongSelf doSomething];
    } else {
        [someOtherObject doSomethingElse];
    }
});

В этом случае копирование weakSelf в strongSelf (по умолчанию сильное) является атомарным. Если weakSelf было нулем, strongSelf будет нулем. Если weakSelf не было nil, strongSelf не будет nil и будет строгой ссылкой на объект, предотвращая его освобождение до сообщения doSomething.

Это кажется совершенно ненужным, поскольку вызов сообщения на nil не является операцией. (Ничего не произошло)

^{
    [weakSelf doSomething]; //Does nothing if weakSelf is nil
}

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

^{
    // Here I don't want to add weakSelf as an observer if it's nil
    if (!weakSelf) return;

    [OtherClass addObserverForSomething:weakSelf];
}

Слабые ссылки не сохраняют указанный объект. Если никто другой не сохраняет его, объект освобождается, а слабые ссылки ссылаются на nil.

Поэтому возможно, что ваш код выполняется с weakSelf, который ссылается на nil. Но это вовсе не повод его проверять. Особенно в Objective-C вы используете определенное поведение, если отправляете сообщение nil. т. е. это идеальный код, если вы устанавливаете свойство, используя ссылку nil. Просто уходит в никуда.

Конечно, иногда вы не хотите взаимодействовать с nil. В таком случае вы должны проверить его.

Кстати: вам нужно weakSelf только в некоторых, очень редких случаях. Это городская легенда, что в замыканиях общие ссылки на self должны быть слабыми, чтобы предотвратить циклы удержания. Это было неправдой, это неправда и никогда не будет правдой.