Swift Closures - Capturing self as weak

Question

I am trying to resolve a closure based strong reference cycle in Swift.
In the code below, object is retained by the owning view controller. ProgressHUD is a UIView that's also retained by the owning view controller. ProgressHUD is leaked every time the completion handler is called. When using the new closure capture feature, declaring self as weak or unowned does not resolve the memory leak.

object.setCompletionHandler { [weak self] (error) -> Void in
    if(!error){
        self?.tableView.reloadData()
    }
    self?.progressHUD?.hide(false)
}

However, if I declare a weak var for self outside of the closure, it fixes the memory leak, like this:

weak var weakSelf = self
object.setCompletionHandler { (error) -> Void in
    if(!error){
        weakSelf?.tableView.reloadData()
    }
    weakSelf?.progressHUD?.hide(false)
}

Any ideas as to why this is not working with Swift capturing?

there shouldn't be a retain cycle – newacct Oct 15 '14 at 00:41 — newacct, Oct 15 '14 at 00:41

Shial · Answer 1 · 2014-12-18T08:47:02.897

If you assign a closure to a property of a class instance, and the closure captures that instance by referring to the instance or its members, you will create a strong reference cycle between the closure and the instance. Swift uses capture lists to break these strong reference cycles. source Apple

source sketchyTech First, it is important to make clear that this whole issue only concerns closures where we are assigning "a closure to a property of a class instance". Keep this in mind with each rule. The rules:

use weak capture if the class instance or property is an optional
use unowned if the class instance or property is non-optional and can never be set to nil
"you must ... use the in keyword, even if you omit the parameter names, parameter types, and return type"

In answear to your question there should be no retain cycle.

dferrero · Answer 2 · 2016-08-31T19:57:02.323

5

You stated that progressHUD is retained by the owning view controller (self) and you reference it in your closure...so add it to the capture list and then use the captured variable in the closure as follows:

object.setCompletionHandler { [weak self] (error) -> Void in
    if(!error){
        self?.tableView.reloadData()
    }
    self?.progressHUD.hide(false)
}

edited Aug 31 '16 at 19:57

answered Dec 10 '15 at 04:40

dferrero

467
6
6

Unowned references are implicitly unwrapped so `self?.tableView` should be changed to `self.tableView`. – Benjamin Cheah Aug 30 '16 at 12:33
you are correct @zxzxlch, I've edited my solution by changing unowned self to weak self, and dropping the progressHUD from the capture list. There should be no retain cycle. – dferrero Aug 31 '16 at 19:58
1

How is this different to the original question? – Ashley Mills Aug 15 '18 at 07:48

score -4 · Answer 3 · edited Oct 14 '14 at 02:43

-4

Try the following:

object.setCompletionHandler { [unowned self] (error) -> () in
    if(!error){
        weakSelf?.tableView.reloadData()
    }
    weakSelf?.progressHUD?.hide(false)
}

edited Oct 14 '14 at 02:43

Fizzix

23,679
38
110
176

answered Oct 14 '14 at 02:15

kingnight

720
6
10

As noted in the original post, neither weak or unowned resolves memory leak. Thanks though. – Fergal Rooney Oct 14 '14 at 10:57

Swift Closures - Capturing self as weak

3 Answers3

Linked