來新公司壁熄，給公司的SDK review了一下岗憋。發(fā)現(xiàn)了不少小問題，在此總結(jié)一下。
（我下面說明問題可能是很簡單捅儒，但是搞清楚某些問題還是花了些時(shí)間的，大家引以為戒吧）
先談?wù)勌幚淼膯栴}：
1.某天QA說有游戲在iphone5 上測試券躁，啟動(dòng)閃退艾扮。
我通過日志捕捉到，是因?yàn)橛昧薿penssl庫筐高，這個(gè)庫包含了libcrypto.a, 這個(gè)庫 和 老款的 iphone 處理器 架構(gòu)沖突搜囱，導(dǎo)致crash。
我檢查了openssl引入是為了做 https 的驗(yàn)證（為了驗(yàn)證服務(wù)器證書柑土，其實(shí)是畫蛇添足）犬辰。但是這部分 驗(yàn)證是完全沒有必要的，細(xì)節(jié)就不說了冰单，移除依賴解決了問題幌缝。
ps：當(dāng)初面試時(shí)候，面試官說公司用的https時(shí)雙向驗(yàn)證诫欠，我后來發(fā)現(xiàn)是 單向驗(yàn)證涵卵。：）
2.內(nèi)存泄露問題
我通過leaks 檢測，我們SDK 的 泄露是無處不在荒叼。因?yàn)橛玫氖茿FN框架轿偎，原開發(fā)人員也對其進(jìn)行了封裝。他們認(rèn)為的單例封裝被廓。我給下偽代碼：
ServerHttpManager.m
+ (instancetype)manager
{
ServerHttpManager *instance = [super manager];
return instance;
}
我只想說 WTF坏晦！
AFN建議需要自己用單例封裝一下，不然會(huì)有泄露嫁乘。但是昆婿，這樣跟沒封裝一樣。改為：
+ (instancetype)manager
{
static ServerHttpManager *instance = nil;
static dispatch_once_t onceToken;
dispatch_once(&onceToken, ^{
instance = [super manager];
});
return instance;
}
網(wǎng)絡(luò)請求泄露問題就解決了蜓斧。當(dāng)然還有其它一些block的泄露仓蛆，偽代碼：
__block XXViewController * weakSelf = self;
大兄弟估計(jì)是MRC那邊還沒有過渡到 ARC，SDK是用的ARC啊挎春，因?yàn)檫@個(gè)也導(dǎo)致好幾處的bug看疙。
把__block改成__weak 就解決了。還有等等直奋。能庆。。
3 . iOS8.x系統(tǒng)bug
由于項(xiàng)目里某些特殊的需求會(huì)出現(xiàn)以下偽代碼：
[[Utils getCurrentVC] presentViewController:XXViewController animated:NO completion:^ {
[XXViewController dismissViewControllerAnimated:NO completion:^ {
DDViewController *vc = [DDViewController sharedInstance];
[[Utils getCurrentVC] presentViewController:vc animated:YES completion:nil];
}];
}];
在iOS8.x設(shè)備里會(huì)閃退脚线，意思說你 presentViewController未完成就開始了dismissViewControllerAnimated 搁胆。不要問為什么會(huì)有這種邏輯，有時(shí)候你?會(huì)用到其中一層嵌套。
修改后偽代碼：
[[Utils getCurrentVC] presentViewController:XXViewController animated:NO completion:^ {
dispatch_after(0, dispatch_get_main_queue(), ^{
[XXViewController dismissViewControllerAnimated:NO completion:^ {
dispatch_after(0, dispatch_get_main_queue(), ^{
DDViewController *vc = [DDViewController sharedInstance];
[[Utils getCurrentVC] presentViewController:vc animated:YES completion:nil];
});
}];
});
}];
4.讀info.plist 不當(dāng)導(dǎo)致的問題
偽代碼：
bool ?isDevMode = [[[NSBundle mainBundle] infoDictionary] objectForKey:@"IS_DEV_MODE"] ;
然后在info.plist 里 配置 YES or NO 丰涉，但是讀出得 總是true拓巧。
這個(gè)問題也是細(xì)節(jié)問題，通過代碼review 才發(fā)現(xiàn)一死。
bool ?isDevMode = [[[[NSBundle mainBundle] infoDictionary] objectForKey:@"IS_DEV_MODE"] boolValue];
因?yàn)檫@個(gè)問題雖小肛度，但導(dǎo)致我們某個(gè)功能無法使用。
5.檢測網(wǎng)絡(luò)連接狀況
偽代碼：
NSURL *baseURL = [NSURL URLWithString:[self getAPPBaseURL]];
AFHTTPSessionManager *manager = [[AFHTTPSessionManager alloc] initWithBaseURL:baseURL];
[manager.reachabilityManager setReachabilityStatusChangeBlock:^(AFNetworkReachabilityStatus status) {
switch (status) {
case AFNetworkReachabilityStatusReachableViaWWAN:
........
}
}];
其實(shí)這個(gè)URL傳入一點(diǎn)作用沒有投慈，這樣使用還會(huì)導(dǎo)致內(nèi)存泄露承耿，改成：
AFNetworkReachabilityManager * reachabilityManager = [AFNetworkReachabilityManager sharedManager];
[reachabilityManager setReachabilityStatusChangeBlock:^(AFNetworkReachabilityStatus status) {
switch (status) {
case AFNetworkReachabilityStatusReachableViaWWAN:
........
}
}];
6.蘋果內(nèi)支付的問題
通過代碼review發(fā)現(xiàn)有3個(gè)地方有問題
問題a：SKPaymentTransactionStateRestored 沒有處理（只是做了finish）。
問題a和 問題b同時(shí)存在都會(huì)出現(xiàn)丟單的問題伪煤，假如支付成功加袋，在finish之前app意外情況crash或是沒有網(wǎng)絡(luò)了。重新登陸SKPaymentTransactionStateRestored 里?僅僅finish了抱既，沒有其它處理职烧，所以這個(gè)訂單?丟了，用戶支付了防泵，我們服務(wù)器還是未支付狀態(tài)蚀之。
問題b：支付成功后，finish訂單 和 請求服務(wù)器驗(yàn)證 同時(shí)處理捷泞。
跟a類似足删，訂單finish 應(yīng)該在 得到服務(wù)器驗(yàn)證處理結(jié)果 后，不然在服務(wù)器請求這段時(shí)間出問題锁右，依然是?用戶付款了失受，但是服務(wù)器訂單狀態(tài)是未支付。
問題c：考慮到會(huì)有其它越獄渠道的支付咏瑟，在支付服務(wù)器驗(yàn)證回來客戶端處理優(yōu)先處理sdk反饋的結(jié)果拂到。
應(yīng)該依賴服務(wù)器處理的結(jié)果，不能優(yōu)先以sdk以及其它的處理响蕴。
另外谆焊，內(nèi)支付的 各種錯(cuò)誤碼也是千奇百怪。很多蘋果都是沒有公開的浦夷，我不止一次跟蘋果技術(shù)溝通一些特殊錯(cuò)誤碼的詳情，但是他們一直不肯說明那些錯(cuò)誤碼代表什么辜王。只是說是他們內(nèi)部用的劈狐。哎。呐馆。肥缔。。下一篇文章我講討論一下千奇百怪的內(nèi)支付錯(cuò)誤碼汹来，以及可能出現(xiàn)的狀況续膳。
7.封裝的sdk改艇，多次調(diào)用會(huì)觸發(fā)多次問題
一般我們用第三方不論是分享還是推送，你[xxSDK initWithKey:@"xxxxxxx"]; 調(diào)用多次也是沒有問題的坟岔，但是我們的SDK封裝有點(diǎn)小問題谒兄，你調(diào)用多次，將會(huì)觸發(fā)多次回調(diào)或是相關(guān)動(dòng)作社付，修復(fù)倒是很簡單就不提了承疲。
