215

如何在 xcode 中避免此警告。这是代码片段：

[player(AVPlayer object) addPeriodicTimeObserverForInterval:CMTimeMakeWithSeconds(0.1, 100)
queue:nil usingBlock:^(CMTime time) {
    current+=1;

    if(current==60)
    {
        min+=(current/60);
        current = 0;
    }

    [timerDisp(UILabel) setText:[NSString stringWithFormat:@"%02d:%02d",min,current]];///warning occurs in this line
}];
4

7 回答 7

522

这里的捕获self来自于您的隐式属性访问-self.timerDisp您不能在将被.selfselfself

self您可以通过在访问timerDisp块之前创建一个弱引用来解决此问题：

__weak typeof(self) weakSelf = self;
[player addPeriodicTimeObserverForInterval:CMTimeMakeWithSeconds(0.1, 100)
                                     queue:nil
                                usingBlock:^(CMTime time) {
                                                current+=1;

                                                if(current==60)
                                                {
                                                    min+=(current/60);
                                                    current = 0;
                                                }

                                                 [weakSelf.timerDisp setText:[NSString stringWithFormat:@"%02d:%02d",min,current]];
                                            }];
于 2013-01-28T06:39:16.963 回答
54 
__weak MyClass *self_ = self; // that's enough
self.loadingDidFinishHandler = ^(NSArray *receivedItems, NSError *error){
    if (!error) {
       [self_ showAlertWithError:error];
    } else {
       self_.items = [NSArray arrayWithArray:receivedItems];
       [self_.tableView reloadData];
    }
};

还有一件非常重要的事情要记住：不要直接在块中使用实例变量，将其用作弱对象的属性，示例：

self.loadingDidFinishHandler = ^(NSArray *receivedItems, NSError *error){
        if (!error) {
           [self_ showAlertWithError:error];
        } else {
           self_.items = [NSArray arrayWithArray:receivedItems];
           [_tableView reloadData]; // BAD! IT ALSO WILL BRING YOU TO RETAIN LOOP
        }
 };

并且不要忘记这样做：

- (void)dealloc {
    self.loadingCompletionHandler = NULL;
}

如果您将传递不被任何对象保留的弱副本，则可能会出现另一个问题：

MyViewController *vcToGo = [[MyViewCOntroller alloc] init];
__weak MyViewController *vcToGo_ = vcToGo;
self.loadingCompletion = ^{
    [vcToGo_ doSomePrecessing];
};

如果vcToGo将被释放然后这个块被触发我相信你会因为无法识别的选择器崩溃到vcToGo_现在包含变量的垃圾箱。试着控制它。

于 2013-02-18T16:22:01.430 回答
47

更好的版本

__strong typeof(self) strongSelf = weakSelf;

创建对该弱版本的强引用作为块中的第一行。如果当块开始执行时 self 仍然存在并且没有回退到零，则此行确保它在整个块的执行生命周期中持续存在。

所以整个事情会是这样的：

// Establish the weak self reference
__weak typeof(self) weakSelf = self;

[player addPeriodicTimeObserverForInterval:CMTimeMakeWithSeconds(0.1, 100)
                                 queue:nil
                            usingBlock:^(CMTime time) {

    // Establish the strong self reference
    __strong typeof(self) strongSelf = weakSelf;

    if (strongSelf) {
        [strongSelf.timerDisp setText:[NSString stringWithFormat:@"%02d:%02d",min,current]];
    } else {
        // self doesn't exist
    }
}];

我已经多次阅读这篇文章。这是Erica Sadun撰写的一篇关于 如何在使用块和 NSNotificationCenter 时避免问题的优秀文章

快速更新：

例如，在 swift 中，一个带有成功块的简单方法是：

func doSomeThingWithSuccessBlock(success: () -> ()) {
    success()
}

当我们调用这个方法并且需要self在成功块中使用时。我们将使用[weak self]和guard let功能。

    doSomeThingWithSuccessBlock { [weak self] () -> () in
        guard let strongSelf = self else { return }
        strongSelf.gridCollectionView.reloadData()
    }

这种所谓的强弱舞蹈被流行的开源项目使用Alamofire。

有关更多信息，请查看swift-style-guide

于 2014-05-13T10:40:33.560 回答
15

在另一个答案中，蒂姆说：

您不能在将由 self 强烈保留的块中引用 self 或 self 上的属性。

这并不完全正确。只要您在某个时候打破循环，您就可以这样做。例如，假设您有一个触发的计时器，它有一个保留 self 的块，并且您还在 self 中保留了对计时器的强引用。如果你总是知道你会在某个时候破坏计时器并打破循环，这很好。

就我刚才的情况而言，我收到了以下代码的警告：

[x setY:^{ [x doSomething]; }];

现在我碰巧知道，只有当它检测到方法以“set”开头时，clang 才会产生这个警告（以及另一种我不会在这里提到的特殊情况）。对我来说，我知道没有保留循环的危险，所以我将方法名称更改为“useY：”当然，这可能并不适用于所有情况，通常你会想要使用弱引用，但是我认为值得注意我的解决方案，以防它帮助其他人。

于 2013-03-25T00:51:23.050 回答
5

很多时候，这实际上并不是一个保留周期。

如果你知道它不是，你就不需要把毫无结果的弱者带到这个世界上。

Apple 甚至通过UIPageViewController其.你无疑会提到你自己）。

这是一些编译器指令，用于从那一行代码中删除警告：

#pragma GCC diagnostic push
#pragma clang diagnostic ignored "-Warc-retain-cycles"
    [self.pageViewController setViewControllers:@[newViewController] direction:navigationDirection animated:YES completion:^(BOOL finished) {
        // this warning is caused because "setViewControllers" starts with "set…", it's not a problem
        [self doTheThingsIGottaDo:finished touchThePuppetHead:YES];
    }];
#pragma GCC diagnostic pop
于 2019-01-24T21:55:19.287 回答
1

在提高精度和风格上增加两分钱。在大多数情况下，您只会self在此块中使用一个或几个成员，很可能只是为了更新滑块。选角self是多余的。相反，最好是明确的并且只在块内转换您真正需要的对象。例如，如果它是 的实例UISlider*，比如说_timeSlider，，只需在块声明之前执行以下操作：

UISlider* __weak slider = _timeSlider;

然后只需slider在块内使用。从技术上讲，这更精确，因为它将潜在的保留周期缩小到只有您需要的对象，而不是内部的所有对象self。

完整示例：

UISlider* __weak slider = _timeSlider;
[_embeddedPlayer addPeriodicTimeObserverForInterval:CMTimeMake(1, 1)
     queue:nil
     usingBlock:^(CMTime time){
        slider.value = time.value/time.timescale;
     }
];

此外，被强制转换为弱指针的对象很可能已经是内部的弱指针self，并且可以最大限度地减少或完全消除保留周期的可能性。在上面的例子中，_timeSlider实际上是一个存储为弱引用的属性，例如：

@property (nonatomic, weak) IBOutlet UISlider* timeSlider;

就编码风格而言，与 C 和 C++ 一样，变量声明最好从右到左阅读。按此顺序声明SomeType* __weak variable从右到左读起来更自然：variable is a weak pointer to SomeType.

于 2015-12-22T18:54:04.073 回答
1

我最近遇到了这个警告，想更好地理解它。经过一番反复试验，我发现它源于一个方法以“添加”或“保存”开头。Objective C 将以“new”、“alloc”等开头的方法名称视为返回一个保留对象，但没有提及（我可以找到）任何关于“add”或“save”的内容。但是，如果我以这种方式使用方法名称：

[self addItemWithCompletionBlock:^(NSError *error) {
            [self done]; }];

我将在 [self done] 行看到警告。但是，这不会：

[self itemWithCompletionBlock:^(NSError *error) {
    [self done]; }];

我将继续使用“__weak __typeof(self)weakSelf = self”方式来引用我的对象，但我真的不喜欢这样做，因为它会混淆未来的我和/或其他开发人员。当然，我也不能使用“添加”（或“保存”），但这更糟，因为它带走了方法的含义。

于 2018-12-17T22:57:47.073 回答