Слабо Аз в блокове

Тази проверка е ненужна и ви дава фалшиво чувство за сигурност.

Ето го проблема:

__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 е зададено на нула. Така че съобщението doSomething се изпраща до нула, което е „безопасно“ (не прави нищо), но може да не е това, което сте очаквали!

По-лошо е, ако искате да предприемете различно действие, когато weakSelf е нула, напр.

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

В този случай, между времето, когато блокът проверява, че weakSelf не е нула, и времето, когато изпрати съобщението doSomething, weakSelf може да стане нула и нито 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 не е нула, strongSelf няма да бъде нула и ще бъде силна препратка към обекта, предотвратявайки освобождаването му преди съобщението 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 трябва да са слаби, за да предотвратят цикли на запазване. Не е било вярно, не е вярно и никога няма да е вярно.