使用swift懒加载需要注意的陷阱_星的天空

title: 使用swift懒加载需要注意的陷阱 date: 2021-12-27 17:00:10 tags:

修改老代码后，发现UITableView会在创建cell时闪退，原因是在调用dequeueReusableCell(withIdentifier:)创建cell时返回了nil。但是检查代码，确认在viewDidLoad注册了这个cell，按道理不应该返回nil。后面分析才发现，由于lazy var不是线程安全的，在碰到viewDidLoad的某个特殊调用时机时就会出现这个问题，而且代码可能在大部分场景正常运行，然后出现一些看起来莫名其妙的bug！

样例

我把问题代码简化后如下： ```obj-c class TestTableViewController: UIViewController { /// 使用懒加载创建tableView lazy var tableView: UITableView = { print("start init testLabel, isViewLoaded (self.isViewLoaded)") let tableView = UITableView.init(frame: self.view.bounds) print("created tableView (tableView)") tableView.delegate = self tableView.dataSource = self return tableView }()

override func viewDidLoad() {
    super.viewDidLoad()
    print(#function)
    view.addSubview(tableView)

    print(#function, "tableView \(tableView) register cell")
    // 注册cell
    tableView.register(UITableViewCell.self, forCellReuseIdentifier: "cell")
}

}

extension TestTableViewController: UITableViewDataSource, UITableViewDelegate { func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int { return 10 }

func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    let cell = self.tableView.dequeueReusableCell(withIdentifier: "cell")!
    cell.textLabel?.text = "\(indexPath.row)"
    return cell
}

}

// 调用方式如下 @IBAction func showTestTableViewVC(_ sender: Any) { let testVC = TestTableViewController.init() // 引起问题的关键代码 testVC.tableView.isScrollEnabled = false self.navigationController?.pushViewController(testVC, animated: true) } `` 如果你已经一眼就看出了问题所在，那么就没有必要看下去了。如果你没有看出来，也不要着急，这个问题确实挺隐蔽的。上述代码运行后，会出现报错：TestTableViewController.swift:29: Fatal error: Unexpectedly found nil while unwrapping an Optional value`。那么这个问题是怎么产生的类？

问题是怎么产生的？

首先我们要清楚两个知识点： 1. lazy var懒加载不是线程安全的 2. 在UIViewController中，成员变量view没有初始化及viewDidLoad方法被调用之前，只要调用了成员变量view，就会立即初始化view并调用viewDidLoad方法。

第二点有点隐蔽，例如在viewDidLoad方法调用之前调用self.view.bounds就会触发。

上述代码运行后的Log输出如下：

在调用let testVC = TestTableViewController.init()初始化控制器后，我们立即调用了testVC.tableView.isScrollEnabled = false，这个时候会进入tableView的懒加载部分： ``obj-c lazy var tableView: UITableView = { print("start init testLabel, isViewLoaded \(self.isViewLoaded)") // 注意，这里调用了self.view，会导致viewDidLoad`被提前调用！ let tableView = UITableView.init(frame: self.view.bounds) print("created tableView (tableView)") tableView.delegate = self tableView.dataSource = self return tableView }()

override func viewDidLoad() { super.viewDidLoad() print(#function) view.addSubview(tableView)

print(#function, "tableView \(tableView) register cell")
// 注册cell
tableView.register(UITableViewCell.self, forCellReuseIdentifier: "cell")

} `` 我们先定义这次要创建的tableView为A。这部分懒加载代码由于错误的调用了self.view，导致self.view初始化和viewDidLoad方法被提前调用，此时成员变量tableView还没有被初始化完成，而viewDidLoad方法中又调用了tableView，由于lazy不是线程安全的，所以又递归进入了上述初始化tableView的逻辑，这个时候self.view已经被创建了，所以会初始化完成，我们定义这次创建的tableView为B，这个时候控制器持有的tableView对象是B，它会在viewDidLoad方法的这次调用中注册cell。上述逻辑跑完后，A才紧随其后完成创建，并替换B成为控制器的新成员变量，而且由于viewDidLoad已经被调用过了，在self.navigationController?.pushViewController(testVC, animated: true)方法调用后，viewDidLoad不会再被调用，所以A`是没有注册cell的。

运行到这时，控制器持有了A，而控制器的view通过addSubview持有了它的子视图B，图示如下：其中B对象在viewDidLoad方法中注册了cell，而A对象并没有注册，所以在代理方法中创建cell时返回了nil，导致了crash。如果对这部分不理解，可以多看几遍代码和日志，理顺下调用流程。

crash位置代码如下： obj-c func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell { // self.tableView是对象A，它并没有注册cell。 // 代理方法传递过来的tableView是对象B，它注册了cell，直接使用它则不会crash let cell = self.tableView.dequeueReusableCell(withIdentifier: "cell")! cell.textLabel?.text = "\(indexPath.row)" return cell } 而这个问题的隐蔽性在于存在两个UITableView对象，如果在代理方法中不使用self.tableView而是使用代理方法传递过来的tableView，那么程序不会crash，而且显示正常。而后续会不会出现奇奇怪怪的问题，就完全看你的运气了。

当然这个问题埋的隐蔽性并不止于此，当外部不调用tableView属性时，例如不像样例代码那样调用testVC.tableView.isScrollEnabled = false，那么在viewDidLoad方法中会正常执行tableView的初始化，一切都是正常的。但是一旦哪位同事在外部调用了一次，那么潘多拉魔盒就打开了～

解决方案

要解决这种问题，需要我们有良好的编码规范。首先，要强化lazy不是线程安全的概念，在懒加载中只做这个变量初始化的事情，尽量避免其它变量及逻辑的混入。在UIViewController及其子类的懒加载逻辑中，避免对view的调用。我看很多人喜欢在懒加载逻辑中调用view.addSubView()或view.bounds，这是不太对的，因为在isViewLoaded为false的情况下，对view的调用就代表着viewDidLoad方法的提前调用，这让程序的逻辑变得有些混乱，除非你能保证在viewDidLoad之后调用这个属性。

其次，在编码过程中，要注意权限的控制，设计合适的接口，这样对使用者更友好，也能规避很多异常场景，当然这对开发者的要求较高，需要平常多加修炼和积累了。

关于OC

另外需要注意的是，OC的懒加载也有同样的问题。但是OC可以优化写法避免出现这个问题，而Swift不行。

关键代码如下： ``` - (UITableView *)tableView { if (!_tableView) { // 第一种用法：这样调用会出现异常 // _tableView = [[UITableView alloc] initWithFrame:self.view.bounds]; // 第二种用法：这样是正常的 _tableView = [[UITableView alloc] init]; _tableView.frame = self.view.bounds;

    _tableView.delegate = self;
    _tableView.dataSource = self;
}
return _tableView;

}

`` 上述代码中的第二种用法不会出现问题，是由于在_tableView.frame = self.view.bounds;这行代码才引入的self.view，此时_tableView` 已经有值，后续代码不会执行。

虽然没有问题，但是不推荐这样使用，因为它还是引起了viewDidLoad的提前执行。

参考资料

Swift - is lazy var thread-safe?

我的博客：星的天空