对于近阶段公司代码 review 小结
来新公司,给公司的SDK review了一下。发现了不少小问题,在此总结一下。
(我下面说明问题可能是很简单,但是搞清楚某些问题还是花了些时间的,大家引以为戒吧)
先谈谈处理的问题:
1.某天QA说有游戏在iphone5 上测试,启动闪退。
我通过日志捕捉到,是因为用了openssl库,这个库包含了libcrypto.a, 这个库 和 老款的 iphone 处理器 架构冲突,导致crash。
我检查了openssl引入是为了做 https 的验证(为了验证服务器证书,其实是画蛇添足)。但是这部分 验证是完全没有必要的,细节就不说了,移除依赖解决了问题。
ps:当初面试时候,面试官说公司用的https时双向验证,我后来发现是 单向验证。:)
2.内存泄露问题
我通过leaks 检测,我们SDK 的 泄露是无处不在。因为用的是AFN框架,原开发人员也对其进行了封装。他们认为的单例封装。我给下伪代码:
ServerHttpManager.m + (instancetype)manager
{
ServerHttpManager *instance = [super manager];
return instance;
}
我只想说 WTF!
AFN建议需要自己用单例封装一下,不然会有泄露。但是,这样跟没封装一样。改为:
+ (instancetype)manager
{
static ServerHttpManager *instance = nil;
static dispatch_once_t onceToken;
dispatch_once(&onceToken, ^{
instance = [super manager];
}); return instance;
}
网络请求泄露问题就解决了。当然还有其它一些block的泄露,伪代码:
__block XXViewController * weakSelf = self;
大兄弟估计是MRC那边还没有过渡到 ARC,SDK是用的ARC啊,因为这个也导致好几处的bug。
把__block 改成 __weak 就解决了。还有等等。。。
3. iOS8.x系统bug
由于项目里某些特殊的需求会出现以下伪代码:
[[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设备里会闪退,意思说你 presentViewController未完成就开始了dismissViewControllerAnimated 。不要问为什么会有这种逻辑,有时候你会用到其中一层嵌套。
修改后伪代码:
[[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 不当导致的问题
伪代码:
bool isDevMode = [[[NSBundle mainBundle] infoDictionary] objectForKey:@"IS_DEV_MODE"] ;
然后在info.plist 里 配置 YES or NO ,但是读出得 总是true。
这个问题也是细节问题,通过代码review 才发现。
bool isDevMode = [[[[NSBundle mainBundle] infoDictionary] objectForKey:@"IS_DEV_MODE"] boolValue];
因为这个问题虽小,但导致我们某个功能无法使用。
5.检测网络连接状况
伪代码:
NSURL *baseURL = [NSURL URLWithString:[self getAPPBaseURL]];
AFHTTPSessionManager *manager = [[AFHTTPSessionManager alloc] initWithBaseURL:baseURL];
[manager.reachabilityManager setReachabilityStatusChangeBlock:^(AFNetworkReachabilityStatus status) {
switch (status) {
case AFNetworkReachabilityStatusReachableViaWWAN:
........
}
}];
其实这个URL传入一点作用没有,这样使用还会导致内存泄露,改成:
AFNetworkReachabilityManager * reachabilityManager = [AFNetworkReachabilityManager sharedManager];
[reachabilityManager setReachabilityStatusChangeBlock:^(AFNetworkReachabilityStatus status) {
switch (status) {
case AFNetworkReachabilityStatusReachableViaWWAN:
........
}
}];
6.苹果内支付的问题
通过代码review发现有3个地方有问题
问题a:SKPaymentTransactionStateRestored 没有处理(只是做了finish)。
问题a和 问题b同时存在都会出现丢单的问题,假如支付成功,在finish之前app意外情况crash或是没有网络了。重新登陆SKPaymentTransactionStateRestored 里仅仅finish了,没有其它处理,所以这个订单丢了,用户支付了,我们服务器还是未支付状态。
问题b:支付成功后,finish订单 和 请求服务器验证 同时处理。
跟a类似,订单finish 应该在 得到服务器验证处理结果 后,不然在服务器请求这段时间出问题,依然是用户付款了,但是服务器订单状态是未支付。
问题c:考虑到会有其它越狱渠道的支付,在支付服务器验证回来客户端处理优先处理sdk反馈的结果。
应该依赖服务器处理的结果,不能优先以sdk以及其它的处理。
另外,内支付的 各种错误码也是千奇百怪。很多苹果都是没有公开的,我不止一次跟苹果技术沟通一些特殊错误码的详情,但是他们一直不肯说明那些错误码代表什么。只是说是他们内部用的。哎。。。。下一篇文章我讲讨论一下千奇百怪的内支付错误码,以及可能出现的状况。
7.封装的sdk,多次调用会触发多次问题
一般我们用第三方不论是分享还是推送,你 [xxSDK initWithKey:@"xxxxxxx"]; 调用多次也是没有问题的,但是我们的SDK封装有点小问题,你调用多次,将会触发多次回调或是相关动作,修复倒是很简单就不提了。
对于近阶段公司代码 review 小结的更多相关文章
- Gerrit代码Review实战
代码审核(Code Review)是软件研发质量保障机制中非常重要的一环,但在实际项目执行过程中,却因为种种原因被Delay甚至是忽略.在实践中,给大家推荐一款免费.开放源代码的代码审查软件Gerri ...
- 敏捷开发:代码Review
热情高涨 代码走查作为一种流程形式,起初大家的参与热情非常高涨. 因为,自己可以学习到别人一些巧妙的思想,自己的代码和习惯都暴漏出来. 这个过程中不断地吸收和改正. 但是...... 我们一开始组织的 ...
- 部署代码review和CI
公司原先搭了一个代码Review的服务器,由于历史原因,装的是一个32bit的Ubuntu系统,后来由于需要,需要安装gitlab,由于gitlab需要64位系统,所以临时凑合了个vagrant,本质 ...
- 由学习《软件设计重构》所想到的代码review(一)
前言 对于一个程序猿来讲怎样来最直接的来衡量他的技术能力和产出呢?我想最直观的作法是看他的代码编写能力,就拿我常常接触的一些程序猿来看,他们买了非常多技术重构类书籍.可是看完后代码编写能力并没有显著提 ...
- 代码review的流程
以前我们一直都是如果要进行代码review的时候,要不我们就直接用idea来进行查看,根据不同的来查看 但是我们都是看代码的不同来进行来实现的,其实我们不需要这样,我们可以使用工具Phabricato ...
- SAP-MM:收货转储时提示 M7053“只能在公司代码 **** 的期间 2014/04 和 2014/03 中记账”
错误信息 消息号M7053 解决方法 Step 1.使用MMPV进入"关闭账期"界面. Step 2.输入"公司代码"."期间".& ...
- 前端代码质量保障之代码review
经验丰富的程序员和一般程序员之间的最大区别,不仅体现在解决问题的能力上, 还体现在日常代码的风格上.掌握一门技术可能需要几月,甚至几周就够了. 好的习惯风格养成却需数年. 团队成员之间需要合作,代码需 ...
- 研发团队如何借助Gitlab来做代码review
代码review是代码质量保障的手段之一,同时开发成员之间代码review也是一种技术交流的方式,虽然会占用一些时间,但对团队而言,总体是个利大于弊的事情.如何借助现有工具在团队内部形成代码revie ...
- Gitlab来做代码review
Gitlab来做代码review 代码review是代码质量保障的手段之一,同时开发成员之间代码review也是一种技术交流的方式,虽然会占用一些时间,但对团队而言,总体是个利大于弊的事情.如何借助现 ...
随机推荐
- iOS开发之--如何修改TabBarItem的title的字体和颜色/BarButtonItem的title的字体大小和颜色/添加背景图片,并添加点击方法
在进行项目的过程中,我们往往会遇到各种各样的自定义颜色和字体,下面提供一种修改系统自带的TabBarItem的字体和颜色的方法,希望能帮到大家: [[UITabBarItem appearance] ...
- 第七篇:使用 CUDA 进行计算优化的两种思路
前言 本文讨论如何使用 CUDA 对代码进行并行优化,并给出不同并行思路对均值滤波的实现. 并行优化的两种思路 思路1: global 函数 在 global 函数中创建出多个块多个线程对矩阵每个元素 ...
- poj_1125 Floyd最短路
题目大意 N个股票经纪人,每个股票经纪人都会将得到的消息传播给另外一些股票经纪人,传播的速度均不固定,且从A传到B的速度和B传到A的速度不一定相等.给定一个消息,并不一定能够传遍所有的股票经纪人,因为 ...
- 单反手动对焦M档,AV,TV,P,A,A-DEP
今天科普一下单反相机的自动对焦设置,以佳能EOS600为例,在镜头的后方会看到一个af,mf的开关.af为auto focus的缩写,自动对焦.mf为手动对焦 自动曝光模式的种类: AV:光圈优先自动 ...
- Groovy基本句法
Groovy基本句法 Gradle作为一个构建工具自然不会自己去创造一门语言来支撑自己,那么它用的是哪门子语言呢?什么语言能写成这样: task hello { doLast { println 'H ...
- HDFS 常用Shell命令
HDFS Shell命令 概述 HDFS Shell命令允许使用命令行在HDFS存储中进行文件夹和文件操作. 如文件夹的增删改查.文件的增删改查等. 开始练习hadoop时,打开Linux之后要用 s ...
- Oracle Schema Objects(Schema Object Storage And Type)
One characteristic of an RDBMS is the independence of physical data storage from logical data struct ...
- [转]通过apk签名使应用程序有系统权限
[转]通过apk签名使应用程序有系统权限 (2013-01-08 13:40:50) 转载▼ 标签: it 分类: Android 出处:http://blog.csdn.net/doom66151/ ...
- 会议室预订系统 td 宽度 php 浏览器 兼容性
w获取浏览器标识 <style> .w > td { <?php $wua=$_SERVER['HTTP_USER_AGENT']; if(strpos($wua, 'Chro ...
- Genymotion 模拟器的sd卡的位置
今天用genymotion测试一个例子,发现要用sdcard,虽然可以再DDMS的 File Explore 下看到 sdcard目录,也可以看到/mnt/sdcard 目录,但是往他那里传文件,建目 ...