最近bugly上每天都有几个在非主线程更新UI的crash，从跟踪数据看VC crash前基本集中出现在如下几个VC中，但具体哪个方法bugly不知道没法给出，首先通过对这几个VC和关联代码仔细排查了逻辑发现没啥明显问题，后面在打包机上看到该版本Xcode有crash上报，而且信息更加具体明显，问题隐藏的比较深，代码路径比较长，复现的情况比较难，解决费了点时间，在此记录一下，这之后也提醒我需要对bugly 的crash上报做下更具体的优化才行，上报的信息都不及xcode上报的详细具体。
问题原因
1、MainViewController是一个rootViewController，viewDidLoad中注册监听token是否失效，如果token失效登出清空用户数据弹出present登录界面，同时发出一个用户需要退出的通知。
@implementation MainViewController
- (void)viewDidLoad {
[super viewDidLoad];
[[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(userToeknInvalidateHandle) name:user_token_invalidate_notification object:nil];
}
- (void)userToeknInvalidateHandle{
//退出登录并弹出登录界面
[UserDataHandler setUserLoginModel:[[LoginModel alloc] init]];
[[NSNotificationCenter defaultCenter] postNotificationName:user_loginOrLogout_notification object:@{@"code":@"0"}];
[AppDelegate loadLoginVc];
}
2、在消息列表MessageViewController的viewDidLoad中注册监听用户登录登出的通知，接收到通知会更新相应的UI界面。
@implementation MessageViewController
- (void)viewDidLoad {
[super viewDidLoad];
[[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(userLoginStatusChangeHandler:) name:user_loginOrlogout_notification_key object:nil];
}
- (void)userLoginStatusChangeHandler:(NSNotification *)notif {
if (notif.object && notif.object[@"code"] && [notif.object[@"code"] integerValue] > 0) {
//登录 更新UI
[self loadRightItemWithImage:@"msg_clear" action:@selector(rightClick)];
} else {
//退出登录 更新UI
[self loadRightItemWithImage:nil action:nil];
[self hideBadgeOnItemIndex:2];
}
}
3、获取消息列表用户基础信息的EMConversationsViewController 在下拉刷新的时候因为需要处理数据量有点大， dispatch_async了一个子线程 异步处理，再dispatch_async(dispatch_get_main_queue(), ^{回到主线程刷新UI}，逻辑没问题，问题在于 [weakself requestAvatarByCurPage];没放在主线程执行，这时就造成一个子线程去请求网络了，一般来说也没啥问题，因为网络请求的回调失败OR成功结果肯定都做了放在主线程统一处理的。
@implementation EMConversationsViewController
- (void)viewDidLoad {
[super viewDidLoad];
self.tableViewDidTriggerHeaderRefresh = ^{
[weakSelf _loadAllConversationsFromDBWithIsShowHud:NO];
}
}
- (void)_loadAllConversationsFromDBWithIsShowHud:(BOOL)aIsShowHUD {
__weak typeof(self) weakself = self;
dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^{
// 数据处理段
[weakself requestAvatarByCurPage]; // 错误源地：在此处写了刷新聊天列表数据
//回归主线程刷新UI
dispatch_async(dispatch_get_main_queue(), ^{
[weakself.tableView reloadData];
});
});
}
4、但中途如果有通知就另外说了。因为通知是不管你在哪个线程注册的，只关心你在哪个线程post发出的，主线程发出的就在主线程处理，子线程发出的就在子线程处理。
//此时是一个子线程发起网络请求
- (void)requestAvatarByCurPage {
[IMUtility.shareInstance getMessageListInfoWithUids:uids completeBlock:^(BOOL isSucc) {
if (isSucc) {
[self.tableView reloadData];
}
}];
}
}
5、网络请求的路径基本是：业务类请求->网络基础类逻辑判断判断（是否有token、该请求是否需要token）->AFnetWorking建立请求request和response；当用户token失效，该请求必须带token则回调回去的error还是在当前调用线程（没到AFnetworking那一步），发出的通知也是子线程，这时候的上层失败回调和通知接收者对UI的更新都是在子线程的，所以会造成crash。
- (void)getMessageListInfoWithUids:(NSString *)uids completeBlock:(UserMessageInfoBlock)completeBlock {
[WSAPPAPIHelper.shared.wsAPI getMessageListInfoWithUids:uids complete:^(id data) {
completeBlock(YES);
} error:^(NSError *error) {
completeBlock(NO);
}];
}
- (void)getMessageListInfoWithUids:(NSString *)uids complete:(CompleteBlock)complete error:(ErrorBlock)errorBlock {
NSMutableDictionary *parameters = [[NSMutableDictionary alloc] init];
[parameters setObject:uids?:@"" forKey:@"user_list"];
__weak WSBaseHttpAPI *SELF = self;
if ([self addCurrentUserToken:parameters isMustToken:YES error:errorBlock]) {
[SELF getRequest:UserList_URL parameters:parameters complete:complete error:errorBlock];
}
}
- (BOOL)addCurrentUserToken:(NSDictionary *)parameter isMustToken:(BOOL)isMustToken error:(ErrorBlock)error {
if (UserDataHandler.getUserInfo.token > 0) {
return YES;
} else {
if (isMustToken) {
[self didError:kErrorTokenCode strError:@"请先登录" error:error];
[[NSNotificationCenter defaultCenter] postNotificationName:user_invalidate_token_notification_key object:nil];
}
}
return isMustToken == NO;
}
解决办法
刷新请求放到主线程中，token失效放在主线程回调error，通知放在主线程post
