代码Review那些事 - 简书
下载简书移动应用
写了118555字，被180人关注，获得了173个喜欢
代码Review那些事
本篇推文是以前同事做分享的时候的ppt，这里我整理出来分享给大家
什么是代码Review?
代码review是指在软件开发过程中，通过对源代码进行系统性检查来确认代码实现的质量保证机制
为什么不做代码Review?
业务需求大，工作时间紧张
项目小，协作的人少，没必要
为什么要做代码Review?
提高代码质量，提升自身水平
及早发现潜在缺陷与BUG，降低事故成本
促进团队内部知识共享，提高团队整体水平
保证项目组人员的良好沟通
避免开发人员犯一些很常见，很普通的错误
总而言之目的是查找系统缺陷，保证软件总体质量和提高开发者自身水平，使项目代码更加容易维护。
代码Review的好处
在代码提交之前如果有很多双眼睛盯着看可以发现bug，这是代码审查最广为人知的好处。（人们的确可以在代码审查中发现bug，单身这些bug大部分都是显而易见的小bug，开发者分分钟可以发现，而那些真正需要花时间发现的bug通常是在代码审查中发现的）
代码审查最大的好处是纯社会性的。（如果你编程的时候知道你的同事将要看你的代码，你的编程方式会不一样，你的代码会写的更整洁，注释更加清楚，组织得更好。因为你知道其他人会看你的代码，他们的意见是你需要关注的。如果没有审查，你虽然知道人们最后会去看你的代码，但是那样不会给你一种紧迫感，也不会给你同样的个人评判的感觉。）
还有一个更大的好处就是代码审查可以传播知识。（在很多开发小组里，每个人都负责某一个核心组件，专注于自己的这一块，只要其他同事的模块不会破坏自己的代码就不会去关注，这种模式导致一个模块只有一个人熟悉对应的代码，如果一个人请教或者离职，其他人对他负责的模块将一无所知。如果采用代码审查，那么至少有两个人熟悉代码-作者和审查者。审查者知道的代码不如作者多，但是他们都熟悉代码的设计和结构，这意义重大）
Code Review的前提
重视代码review(Code Review人员是否理解了Code Review的概念和Code Review将做什么如果做Code Review的人员不能理解Code Review对项目成败和代码质量的重要程度，他们的做法可能就会是应付了事。)
代码是否已经正确的build，build的目的使得代码已
经不存在基本语法错误(我们总不希望高级开发人员或是主管将时间浪费在检查连编译都通不过的代码上吧。 )
代码执行时功能是否正确(Code Review人员也不负责检查代码的功能是否正确，也就是说，需要复查的代码必须由开发人员或质量人员负责该代码的功能的正确性。 )
开发人员是否对代码做了单元测试(这一点也是为了保证Code Review前一些语法和功能问题已经得到解决，Code Review人员可以将精力集中在代码的质量上。 )
Code Review需要注意什么？
完整性检查（Completeness）
代码是否完全实现了设计文档中提出的功能需求
代码是否已按照设计文档进行了集成和Debug
代码是否已创建了需要的数据库,包括正确的初始化数据
代码中是否存在任何没有定义或没有引用到的变量、常数或数据类型
一致性检查（Consistency）
代码的逻辑是否符合设计文档
代码中使用的格式、符号、结构等风格是否保持一致
正确性检查（Correctness）
所有的变量都被正确定义和使用
所有的注释都是准确的
所有的程序调用都使用了正确的参数个数
可修改性检查（Modifiability）
代码涉及到的常量是否易于修改(如使用配置、定义为类常量、使用专门的常量类等)
代码是否只有一个出口和一个入口（严重的异常处理除外）
健壮性检查（Robustness）
可理解性检查（Understandability）
注释是否足够清晰的描述每个子程序
是否使用到不明确或不必要的复杂代码，它们是否被清楚的注释
使用一些统一的格式化技巧（如缩进、空白等）用来增强代码的清晰度
是否在定义命名规则时采用了便于记忆，反映类型等方法
每个变量都定义了合法的取值范围
代码中的算法是否符合开发文档中描述的数学模型
可验证性检查(Verifiability)
代码中的实现技术是否便于测试
Code Review经验检查项
1、 编码规范方面检查项2、面向对象设计方面检查项
类设计和抽象是否合适
是否符合面向接口编程的思想
是否采用合适的设计模式
3、性能方面检查项
对hashtable,vector等集合类数据结构的选择和设置是否合适
有无滥用String对象的现象
是否采用通用的线程池、对象池模块等cache技术以提高性能
I/O方面是否使用了合适的类或采用良好的方法以提高性能（如减少序列化，使用buffer类封装流等）
同步方法的使用是否得当,是否过度使用
4、数据库处理方面
数据库资源是否正常关闭和释放
数据库访问模块是否正确封装，便于管理和提高性能
是否采用合适的事务隔离级别
资源泄漏处理方面检查项 cursor
5、通讯方面检查项
socket通讯是否存在长期阻塞问题
6、重复代码7、其他
日志是否正常输出和控制
配置信息如何获得,是否有硬编码
怎么更有效的做Code Review
一次评审量要低于 200–400 行代码缺陷密度 就是每 1000 行代码之中所发现的错误（bug）数
每小时低于 300–500 LOC 检查率的目标
花足够的时间进行适当缓慢的评审，但是不要超过 60-90 分钟但反过来说，评审代码所花的时间不得低于五分钟，就算代码只有一行也是如此。通常来说，单行的代码也会影响到整个的系统，所以花上五分钟时间去检查更改可能造成的结果是值得的
确定在评审开始之前代码开发者已经注释源代码了
使用检查表，因为它能极大地影响代码开发者和评审者的结果另外一个有用的概念就是 个人检查表 。每个人一般都会犯 15-20 个错误（bug）。如果您注意到了一些典型的错误（bug），那么您就可以开发自己的个人检查表
确认缺陷得到了修复
最后，让Code Review成为一种习惯
The biggest thing that makes Google’s code so good is simple:code review
欢迎关注我的公众号：wwjblog
如果觉得我的文章对您有用，请随意打赏。您的支持将鼓励我继续创作！
打开微信“扫一扫”，打开网页后点击屏幕右上角分享按钮
被以下专题收入，发现更多相似内容：
· 3人关注
如果觉得我的文章对您有用，请随意打赏。您的支持将鼓励我继续创作！
选择支付方式：商品编号：
京 东 价：
[定价：￥]
支　　持：
搭配赠品：
所 在 地：北京 海淀区
服务支持：
加载中，请稍候...
加载中，请稍候...
加载中，请稍候...
EffectiveObjective-C2.0-编写高质量iOS与OSx代码的52个有效方法
加载中，请稍候...
商品介绍加载中...
扫一扫，精彩好书免费看
服务承诺：
京东平台卖家销售并发货的商品，由平台卖家提供发票和相应的售后服务。请您放心购买！
注：因厂家会在没有任何提前通知的情况下更改产品包装、产地或者一些附件，本司不能确保客户收到的货物与商城图片、产地、附件说明完全一致。只能确保为原厂正货！并且保证与当时市场上同样主流新品一致。若本商城没有及时更新，请大家谅解！
权利声明：京东上的所有商品信息、客户评价、商品咨询、网友讨论等内容，是京东重要的经营资源，未经许可，禁止非法转载使用。
注：本站商品信息均来自于合作方，其真实性、准确性和合法性由信息拥有者（合作方）负责。本站不提供任何保证，并不承担任何法律责任。
印刷版次不同，印刷时间和版次以实物为准。
价格说明：
京东价：京东价为商品的销售价，是您最终决定是否购买商品的依据。
划线价：商品展示的划横线价格为参考价，该价格可能是品牌专柜标价、商品吊牌价或由品牌供应商提供的正品零售价（如厂商指导价、建议零售价等）或该商品在京东平台上曾经展示过的销售价；由于地区、时间的差异性和市场行情波动，品牌专柜标价、商品吊牌价等可能会与您购物时展示的不一致，该价格仅供您参考。
折扣：如无特殊说明，折扣指销售商在原价、或划线价（如品牌专柜标价、商品吊牌价、厂商指导价、厂商建议零售价）等某一价格基础上计算出的优惠比例或优惠金额；如有疑问，您可在购买前联系销售商进行咨询。
异常问题：商品促销信息以商品详情页“促销”栏中的信息为准；商品的具体售价以订单结算页价格为准；如您发现活动商品售价或促销信息有异常，建议购买前先联系销售商咨询。
加载中，请稍候...
加载中，请稍候...
加载中，请稍候...
加载中，请稍候...
加载中，请稍候...
加载中，请稍候...
加载中，请稍候...
浏览了该商品的用户还浏览了
加载中，请稍候...
价　格： 到
　　　
iframe(src='///ns.html?id=GTM-T947SH', height='0', width='0', style='display: visibility:')ios基础问题，求大神解答_ios开发吧_百度贴吧
&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&签到排名：今日本吧第个签到，本吧因你更精彩，明天继续来努力！
本吧签到人数：0成为超级会员，使用一键签到本月漏签0次！成为超级会员，赠送8张补签卡连续签到：天&&累计签到：天超级会员单次开通12个月以上，赠送连续签到卡3张
ios基础问题，求大神解答收藏
能解一道是一道，谢谢了
会答的题写下题号就好，谢谢了
多视图编程一、填空题1.在sqlite中，代表数据库连接对象的是_____
2.在sqlite中，如果打开文件成功，则结果是_____
5.在sqlite中，代表SQL语句的对象是_____
8.sqlite中要执行sql语句，必须把它编译成字节码，使用的语句是_____
二、选择题1.打开一个数据库文件后，要查看数据库如何创建，需要使用以下的哪个命（）A..tablesB..selectC..findD..schema2.以下哪种数据类型不是sqlite中的基本类型（）A. real
D. nil3.以下哪个不是UIView设定的转换类型（）A. UIViewAnimationTransitionNone B. UIViewAnimationTransitionFlipFromBottom C. UIViewAnimationTransitionFlipFromRight D. UIViewAnimationTransitionCurlDown4.当看到两个或多个代码块很相似的时候，应该考虑能否将这些代码块合并成一个代码块，这个叫做代码块的（）A. 重组
D. 维护5.以下哪个是UITableViewDelegate协议中规定的方法（）A.tableView: numberOfRowsInSection:
B.tableView: heightForRowAtIndexPath:C.tableView: cellForRowAtIndexPath: D.numberOfSectionsInTableView:
三、判断题1.使用sqlite是数据存储的一种方法（）2.text是sqlite中一种基本的数据类型（）3.UIViewAnimationTransitionFlipFromRight是UIView中一种转换类型（）4.numberOfSectionsInTableView是UITableViewDelegate协议中规定的一个方法（）5.在命令行终端里，使用sqlite3 database name. db既可以创建没有的数据库，也可以打开已经存在的数据库（）6.在视图转换中，presentViewController是使用压栈的方式进行视图转换的（）7.实现tableView cellForRowAtIndexPath:方法返回整个UITableView（）8.如果查看一个sqlite中表格的名称，可以使用.tables命令（）9.@property默认的性质是readonly（）10.setAnimationTransition: forView ：cache cache为No时转换过程中不需要升级动画帧（）
四、问答题4.分别写出UITableViewDataSource和UITableViewDelegate中的两个方法（5）
五、编程题1.用Objective-C语言编写代码打开一个叫做data.db的数据库文件（5）2.在Controller A的点击方法中，用presentViewController编写代码实现界面的切换（10）3.编写代码，初始化一个字典对象，键和值可以自己规定，在cellForRowAtIndexPath方法中取得键和值（10）4.编写核心动画代码实现pageCurl的转换（15）
核心课程一、填空题1.iOS中本地化需要用到的宏是____
2.在iOS中，代表触摸对象的类是_____
3.在iOS中，应用程序可以通过_____来使用照相机和照片库4.在iOS本地化过程中要传递两个参数，第一个参数是_____，第二个是____
5.iOS触摸对象中能记录轻击次数的属性是_____
7.在iOS中，如果要让视图可以接受多点触摸，需要将_____属性设置为YES8.在iOS触摸中，要修改视图的位置信息，要声明一个_____的数据，才能把视图原始的frame读出
二、选择题1.UITouchPhase类型是一个枚举类型，其中代表接触点无移动的是（）A.UITouchPhaseEndedB.UITouchPhaseMovedC.UITouchPhaseStationaryD.UITouchPhaseCancelled2.创建本地化的字符串文件是（）格式的文本文件A. UTF-8
C. NSString
D. GBK3.以下哪个不是UITouch类的属性（）A. window
C. timestamp
D. event4.要想返回触摸点在窗口的位置，应将locationInView中的参数设置为（）A. nil
D. NSNull5.如果不希望视图对触摸事件有回应，则需将视图的userInteractionEnabled的属性设置为（）A.Nil
三、判断题1.iOS中的轻击操作一般是针对一个手指的，因此[touches anyObject]中只包含一个操作对象（）2.UIEvent包含了整个触摸过程中的所有触摸对象（）3.目前iOS提供的本地化只包括文本信息和资源文件本地化（）4.在UIKit类库中，和iPhone设备的晃动功能紧密相关的类是UIMonitor类和UIAccelerometer类（）5.用户不但可以使用iPhone内置照相机拍摄照片，还可以从iPhone的照片库中选择照片（）6.响应者链是按照视图控制器、父视图、第一响应者这个顺序递推的（）7.中iOS API中，UIResponder类定义了响应者对象的所有方法，这个类一般作为接口来使用，继承自它的子类即可响应事件（）8.通过UIAcceleration类可以可以获取iPhone硬件设备上的加速度数据（）9.响应者对象可以响应事件并对事件做处理（）10.手势是由在事件中的一系列触摸构成的（）
四、问答题1.简述在iOS模拟器上如何设置不同的语言（5）2.写出四个常用的UIResponsder中定义的事件处理函数（5）3.iOS本地化时使用getstrings完成的顺序是什么（5）4.简述如何UIImagePickerController来实现图像选取功能（5）
五、编程题1.假设你现在在一个UIView的类中，编写一个代码，实现该视图的背景色变为蓝色（5）2.iOS网络开发中，如果网络没有连接，则会弹出UIAlertView，编写一段代码实现UIAlertView中的字符串能本地化（the title string :Warning， the message string: No Internet connection， cancel string: Close）（10）3.利用全局变量，使用开始触摸和触摸移动这两个方法实现滑动（10）4.编写程序，开始单击时视图背景颜色延迟0.5秒变蓝色，双击时候背景颜色变红色（15）
一、填空题1.NSXMLParser采用的解析模式是_____
2.在iOS中，网路编程的最底层是_____
3.NSURLConnection是基于_____协议来发送请求的5.iOS传统的并发模式是多线程，但线程的伸缩性不强，所以iOS采用_____来解决并发问题6.在iOS开发中，替代多线程的两个常用技术是_____和_____
7.Operation objects会产生_____通知，可以监控任务的进程8.在sqlite和plist文件所保存的内容都属于_____
二、选择题1.以下哪个不是和iOS线程开发相关的类（）A.NSThreadB.NSURLC.NSRunLoopD.NSOperation2.RSS是以下哪个选项的一种格式（）A. HTML
D. XHTML3.NSURLConnection发送请求后，如果收到数据，则调用以下哪个方法（）A. connectionDidFinishLoading
B. connection: didReceiveResponse:C. connection: didReceiveData
D. parser: didStartElement:4.以下哪个不是线程安全的类（）A. NSData
B. NSDictionary
C. NSArray
D. NSMutableArray5.以下哪个选项不属于XML（）A.标记语言
B. 数据存储格式
C. 数据交换格式
三、判断题1.NSURLConnection发送的请求是同步请求（）2.NSXMLParser是基于C语言的解析器（）3.线程用于指定一个正在执行的可执行程序（）4.NSThread可以使用detachNewThreadSelecor:toTarget:withObject:类方法来生成一个新的线程（）5.NSMutableDictonary是线程安全的类（）6.RunLoop是一个运行循环，它的目的是让线程忙的时候处理事件，不忙的时候线程休眠（）7.RunLoop中的定时源传递异步事件（）8.LibXML不仅可以解析XML文件，还可以产生XML文件（）9.提交到Operation Queues的任务必须是NSOperation对象（）10.多数开发从来都不需要实现并发operation对象，只需要将operations添加到operation queue（）
四、问答题1.请列举使用多线程的好处（5）2.简述XML中节点和元素的区别（5）3.请简述同步请求和异步请求的区别（5）4.简述XML解析通常使用的两种模式以及它们的区别（5）
五、编程题1.编写一个使用NSURLConnection发送请求的程序，不需要写代理函数，URL为（5）2.假设传递给NSXMLParser的是responseData，编写使用NSXMLParser解析XML的代码，不需要写代理函数（10）3.对以下XML文档，编写程序使用NSStrig产生该文档（10）&address& &name&John Smith&/name& &street&1 Ocean Blvd&/street& &city&Isle Of Palms&/city& &/address&4.使用LibXML编写程序产生上题中的XML文档（15）
这还有考试么
类似面试题么
题目也是够老的，不过我只能写一半的样子，话说数据解析不是喜欢json嘛
登录百度帐号推荐应用
为兴趣而生，贴吧更懂你。或求大神帮忙讲解一下下面这段IOS代码_百度知道
求大神帮忙讲解一下下面这段IOS代码
((void)0)#endif#if ITTLOGLEVEL_INFO &lt., ;, .;
} ((void)0)#else
#define ITTDCONDITIONLOG(condition.)
ITTDPRINT(xx, ;
NSLog(@&= ITTMAXLOGLEVEL
#define ITTDINFO(xx, &#92.., ##__VA_ARGS__)..)
((void)0)#endif#if ITTLOGLEVEL_WARNING &
ITTDPRINT(xx.., , ##__VA_ARGS__)#else
#define ITTDINFO(%s&quot, ##__VA_ARGS__)#else
#define ITTDWARNING(xx., .;%s(%d), ##__VA_ARGS__)#else
#define ITTDPRINT(xx., ##__VA_ARGS__)#else
#define ITTDERROR(xx, ., __PRETTY_FUNCTION__..)
((void)0)#endif#ifdef ITTDEBUG
#define ITTDCONDITIONLOG(condition., xx, ....., __PRETTY_FUNCTION__)#if ITTLOGLEVEL_ERROR &lt, .)
((void)0)#endif#define ITTDPRINTMETHODNAME() ITTDPRINT(@&quot.)
ITTDPRINT(xx.)
ITTDPRINT(xx.: &= ITTMAXLOGLEVEL
#define ITTDWARNING(= ITTMAXLOGLEVEL
#define ITTDERROR(xx, __LINE__.) { if ((condition)) { &#92, , .#define ITTDEBUG#define ITTLOGLEVEL_INFO
10#define ITTLOGLEVEL_WARNING
3#define ITTLOGLEVEL_ERROR
1#ifndef ITTMAXLOGLEVEL#ifdef DEBUG
#define ITTMAXLOGLEVEL ITTLOGLEVEL_INFO#else
#define ITTMAXLOGLEVEL ITTLOGLEVEL_ERROR#endif#endif#ifdef ITTDEBUG
#define ITTDPRINT(xx
提问者采纳
作用是在编译ios程序前对程序文本做一些预处理。其中有些是if else 你应该猜到什么意思，不知道你明白没有，没有可以追问，语义分析之类的。所有这些都是编译之前做的， 例如上面出现的 #define ITTLOGLEVEL_INFO
10 这是替换功能的语句，语法分析，这种叫宏，可以把程序中出现ITTLOGLEVEL_INFO的地方用10替换，即开始做词法分析。然后开始做编译，仅仅是替换文本而已。讲远了，叫预处理，完了之后生成的代码就是要编译的源文件这些东西不是ios的代码，是xcode编译器识别的代码
谢谢，，定义这些宏是不是在其他的.m或.h文件中用啊？？方便加我qq么？大神，，非常感谢~
你说的对，是在点.m或者.h文件中用到，在使用c++的时候还有.mm文件， 其中可以用来方便替换代码，可以做条件编译（就是在满足某个宏的条件下编译哪些代码，不编译哪些代码）qq我已经加你了，另外我不是大神。
提问者评价
其他类似问题
为您推荐：
ios的相关知识
等待您来回答
下载知道APP
随时随地咨询
出门在外也不愁}