第二次代码评审

1.判断list是否为空,
最好用list.isempty(),不用list.size()
2.枚举是单例的,单例可以直接用==判断相等
3.for循环里面的代码,可以封装成方法
4.update table(a = a+1),不要在代码里相加。对于Oracle
5.for update
6.圈复杂度不能大于15
7.for(i;i<条件;i++),条件不能是表达式,否则会每次循环都计算一次
8.应用日期和数据库日期,选用数据库日期
9.空行,一个空行就可以,没必要两个

接口规范会议:
1.上下文,header头里放入版本号
2.复杂对象返回类型,耦合依赖。
3.面向对象与面向过程的互相转换
4.调用方式,
httpclient,spring rest template,rpc
5.RPC客户端负责抛出异常
6.throwable
7.DTO成对儿出现

时间: 2024-09-30 23:19:55

第二次代码评审的相关文章

同行代码评审过程中的实践经验

声明:该文经我翻译后首次发表在伯乐在线上,不论什么形式的转载都请标明原处. 数百万年前,猿从树上下来,进化出了对生拇指,终于.变成了人类. 我们以相似的眼光来看下强制性代码评审(Code Review):好像是一种能在软件开发这块广阔的领域里将人类从兽里分离出来的东西. 只是,我有时候会从我们的团队成员里听到以下这种评论: "这个项目的代码评审根本就是浪费时间." "我没有时间做代码评审. " "我的项目公布延期了.都是由于我那懦弱的同事还没有做不论什么评

6.我们真的做了代码评审

静态代码检查工具StyleCode. 上一章我说了设计评审和测试用例评审都是为了提升产品质量问题,后来我们又做了代码评审,但是代码评审比设计评审难搞多了,对于开发来说最在意的就是自己的代码,让别人对自己的代码指指点点,虽然是好的建议但也会让人不爽. 准备搞代码评审之前以前就尝试过,搞着搞着最后的结果只是变成一种形式而已,大家热情都不高,反而给团队增加了负面的情绪.在现在团队中由于代码质量问题,减少bug的产出,在回顾会议中团队提出可以试一下代码评审,是对这个解决办法存在担忧的.我们采用style

麻省理工18年春软件构造课程阅读04“代码评审”

本文内容来自MIT_6.031_sp18: Software Construction课程的Readings部分,采用CC BY-SA 4.0协议. 由于我们学校(哈工大)大二软件构造课程的大部分素材取自此,也是推荐的阅读材料之一,于是打算做一些翻译工作,自己学习的同时也能帮到一些懒得看英文的朋友.另外,该课程的阅读资料中有许多练习题,但是没有标准答案,所给出的答案均为译者所写,有错误的地方还请指出. 译者:李秋豪 审校: V1.0 Thu Mar 8 22:58:41 CST 2018 本次课

对代码评审的感想(回忆篇)

对代码评审的感想(回忆篇) 回想之前钱老板开代码评审流程的讨论会,有事没去,有些遗憾,所以谈下这个当前最fashion的话题,分享下之前对代码等评审的体验. 之前在CFT的研发氛围还是比较重视代码评审的,完成度也比较好,原因有以下几个: 业务特性.我们是的业务跟钱打交道的,所以一发生问题,损失的都是钱,所以追责很严,惩罚很重,而且罚钱一般都罚老大,所以质量上的规范容易从上往下推动,事半功倍.当然线上出现问题,往往会让QA(质量管理)追着问原因以及要求拿出改进措施,作为小兵也很烦很害怕有木有~ 严

自动提交Git branch代码评审到Review Board系统

背景 敏捷软件开发中,越小的反馈环,意味着软件质量越容易得到保证. 作为组件团队,我们的开发任务中,往往存在一些特性涉及到几十个功能点,开发周期持续数周或数月的情况.如何在开发过程中保证软件质量,是个很重要的话题.进行有效的细粒度的代码评审,是常见的手段之一.但是这一希望在落地时,多多少少会遇到些来自方方面面的阻力: Review Board不支持Git branch的代码评审提交: Git不熟,不知道怎么生产正确的patch文件来提交到Review Board上: Review Board不会

使用ReviewBoard进行代码评审经验总结

代码评审 代码评审(CodeReview),顾名思义是对代码进行评审,是软件工程的活动之一. 通过代码评审可以保证代码质量,促进团队知识共享--好处多多. 版本控制与代码评审 软件工程的各个活动总是离不开工具的支持. 代码评审工具首先必须和版本控制工具相结合的. 现在主流的两种版本控制工具:SVN和GIT. GIT有个Google开发的代码评审工具Gerrit,可以在提交前进行代码评审,评审通过之后才允许提交到版本库. 其次,代码托管平台GitLab(号称是GitHub的开源实现)也可以用来进行

PHP 代码评审的 10 个提示

http://www.oschina.net/translate/top-10-php-code-review-tips PHP 代码评审的 10 个提示 以下是这篇文章介绍的重点:1.业务功能2.框架相关的编码指南3.面向对象的原则4.PHP的特异性标准5.编程相关的最佳实践6.设计模式7.代码覆盖率8.安全9.异常处理10.集成模式在进入细节之前,请让我提提我的想法,我认为,涵盖各个方面的代码质量,以下8个参数(ISO 25000平方标准)会有不同的代码审查标准. 功能的适用性可维护性可用性

最新apache+svn+reviewboard实现在线代码评审

本文重点说reviewboard的安装 作用,在线代码评审工具. --------------------------------------------------------------------------- mysql安装 yum -y install gcc gcc-c++ make cmake autoconf automake ncurses* bison* zlib* expat* openssl* apr* neon* yum -y install mysql-server

《第二行代码》来了

转载请注明出处:http://blog.csdn.net/sinyu890807/article/details/52032038 本文同步发表于我的微信公众号,扫一扫文章底部的二维码或在微信搜索 郭霖 即可关注,每天都有文章更新. 回顾 <第一行代码>从出版到现在已经有两年多的时间了.当然,如果是从我开始写书的那天算起的话,时隔已经三年半之久了.回顾这三年半的时光,不得不感慨,Android 技术的发展真是日新月异. 记得在2013年我刚开始动笔写<第一行代码>的时候,Googl