CodeReview规范 CodeReivew 标准 通用原则 提交 PR 的代码必须保证自测通过 只 review 代码规范、业务逻辑，不 review 架构设计（那是写代码前应该做的事情） 干掉重复代码，合理复用代码 合理命名变量、函数、API 的名字（具体参考各语言规范） 删除无用代码（包括以 ...
分类：
其他好文 时间：
2017-02-22 13:28:56
阅读次数：
158
什么是代码评审（CodeReview）? 代码评审也称代码复查，是指通过阅读代码来检查源代码与编码标准的符合性以及代码质量的活动。 Jupiter提供了代码行级别的评审批注功能，方便评审参与人了解具体是哪些行代码存在问题。 同时，它也比较符合常规的评审流程，被评审人提供待审代码->评审人线下提出个人 ...
分类：
编程语言 时间：
2017-01-10 11:27:57
阅读次数：
205
我们先来看看Code Reivew的用处： Code reviews 中，可以通过大家的建议增进代码的质量。 Code reviews 是一个传递知识的手段，可以让其它并不熟悉代码的人知道作者的意图和想法，从而可以在以后轻松维护代码。 Code reviews 也鼓励程序员们相互学习对方的长处和优点 ...
分类：
其他好文 时间：
2016-09-26 10:49:56
阅读次数：
120
昨天下午部门的日会的CodeReview期间出了一个很有意思的事。 大家对一个上传Excel的功能提出了异议。 这个函数是解析Excel对必填和部分业务规则做一些校验，有问题的数据标示一下保存回Excel返给用户，把通过较验的数据保存到数据库中。 大家质疑的点是以上的逻辑是混在一起的，程序员的理由是 ...
分类：
其他好文 时间：
2016-09-20 23:43:33
阅读次数：
130
一，CI系统流程开发人员将本地Git仓库中的代码更新后，push到Gerrit系统上对应的项目仓库中；该动作会触发Gerrit系统发送邮件通知给Owner，Owner此时需要登录GerritWeb页面，进行CodeReview(代码审核)。如果审核通过，则Owner进行Submit提交动作。此时开发人员push的代码才会Me..
分类：
其他好文 时间：
2016-05-13 04:57:43
阅读次数：
5969
phabricator开放源代码的软件开发平台Web应用程序，用以帮助软件公司开发更好的软件。在代码审查(CodeReview)方面，Facebook做了一个可视化的工具，现已开源，叫Phabricator;工程师可以在页面上非常方便的针对每一段(单行或者多行)代码进行交互讨论;负责审查的工程师可以接受代..
訪问这个地址： https://codereview.chromium.org/1141283003 主要分为3类：png资源文件、java代码、C++代码，曾经以为这部分不开源的代码主要是C++，事实上不是这种。
分类：
移动开发 时间：
2016-02-29 10:32:13
阅读次数：
176
http://blog.qt.io/blog/2016/01/18/qt-charts-2-1-0-release/ 下载地址： https://codereview.qt-project.org/#/admin/projects/qt/qtcharts 已经在qt5.3.1上面编译通过 可以直接集
分类：
其他好文 时间：
2016-02-02 21:35:15
阅读次数：
1994
1.风格务必保持一贯性(Consistent)一位同胞顶着我的鼻子问，为什么我们的Java代码缩进格式非得是这样，而不能是他那样，他就是喜欢他自己的这一种，因此他写的代码总是用他自己习惯的风格。结果在CodeReview里被大家毙掉，责令修改。因此他是大大地不服。就是风格一贯性问题。其实他的风格，本...
分类：
编程语言 时间：
2016-01-23 07:51:41
阅读次数：
251
找到的方法有两种：1）http://www.cnblogs.com/liuyazhou/archive/2009/11/02/1594364.html2）http://codereview.stackexchange.com/questions/20871/single-instance-wpf-a...
