bug日记2019.02.21

昨天写代码的时候写出了一个bug，在此记录一下。

一下是对bug的简化，问题是同样的，不过略去了一些无关紧要的细节

简单说来就是从vcA跳转vcB，vcB有个属性是在跳转前设置的。

跳转代码如下

BViewController *vcB = [[BViewController alloc] init];
vcB.isPhone = YES/NO;
[vcA.navigationController pushViewController:vc animated:YES];

vcB里边有个titleLabel属性是根据vcB的isPhone设置的

-(void)setIsPhone:(BOOL)isPhone{
    _isPhone = isPhone;
    self.titleLabel.text = isPhone ? @"手机" : @"邮箱";
}

在vcB里titleLabel是这样添加的

-(UILabel *)titleLabel{
    if (!_titleLabel) {
        _titleLabel = [UILabel new];
        [self.view addSubview:_titleLabel];
    }
    return _titleLabel;
}

- (void)viewDidLoad {
    [super viewDidLoad];
    // Do any additional setup after loading the view.
    self.titleLabel.sd_layout
        .topSpaceToView(self.view, 30)
        .leftSpaceToView(self.view, 17)
        .rightSpaceToView(self.view, 17)
        .heightIs(24);
}

这时跳转到vcB就会发现titleLabel不能正确的显示。

如果在viewDidLoad里使用直接设置titleLabel的frame的话就可以正确显示。

不过这里我不想使用直接设置frame的方式来布局，所以还要继续排查

我猜测可能跟-(void)setIsPhone:(BOOL)isPhone里设置了titleLabel.text有关系，把那句代码注释掉之后titleLabel的确可以正确显示了。

也就是说在设置titleLabel的约束之前设置了titleLabel的text（确切的说是跳转到vcB之前设置了titleLabel的text属性）的话就会导致跳转之后再用SDAutoLayout设置titleLabel的约束就会显示不出来（暂未查明原因）。

所以我给vcB添加了一个初始化方法:

-(instancetype)initWithPhone:(BOOL)isPhone{
    if (self = [super init]) {
        _isPhone = isPhone;
    }
    return self;
}

再在titleLabel的get方法里边加多一句

-(UILabel *)titleLabel{
    if (!_titleLabel) {
        _titleLabel = [UILabel new];
        _titleLabel.text = self.isPhone ? @"手机" : @"邮箱";
        [self.view addSubview:_titleLabel];
    }
    return _titleLabel;
}

这样因为跳转之前没有调用setIsPhone方法，所以不会对titleLabel.text进行设置，也就不会发生titleLabel不能正确显示的问题。