使用swift懒加载需要注意的陷阱修改老代码后，发现UITableView会在创建cell时闪退，原因是在调用deque

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

样例

我把问题代码简化后如下：

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。那么这个问题是怎么产生的类？

问题是怎么产生的？

首先我们要清楚两个知识点：

lazy var懒加载不是线程安全的
在UIViewController中，成员变量view没有初始化及viewDidLoad方法被调用之前，只要调用了成员变量view，就会立即初始化view并调用viewDidLoad方法。

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

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

在调用let testVC = TestTableViewController.init()初始化控制器后，我们立即调用了testVC.tableView.isScrollEnabled = false，这个时候会进入tableView的懒加载部分：

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位置代码如下：

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?

我的博客：星的天空