修改老代码后，发现UITableView会在创建cell时闪退，原因是在调用dequeueReusableCell(withIdentifier:)创建cell时返回了nil。但是检查代码，确认在viewDidLoad注册了这个cell，按道理不应该返回nil。后面分析才发现，由于lazy var不是线程安全的，在碰到viewDidLoad的某个特殊调用时机时就会出现这个问题，而且代码可能在大部分场景正常运行，然后出现一些看起来莫名其妙的bug！
iOS学习资料可关注个人资料领取
样例
我把问题代码简化后如下：
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的提前执行。
如果你有所收获，不如动动小手指头双击一下。
如需iOS资料包，可关注个人主页领取哦。
作者：星的天空
