对于近阶段公司代码 review 小结】的更多相关文章

来新公司,给公司的SDK review了一下.发现了不少小问题,在此总结一下. (我下面说明问题可能是很简单,但是搞清楚某些问题还是花了些时间的,大家引以为戒吧) 先谈谈处理的问题: 1.某天QA说有游戏在iphone5 上测试,启动闪退. 我通过日志捕捉到,是因为用了openssl库,这个库包含了libcrypto.a, 这个库 和 老款的 iphone 处理器 架构冲突,导致crash. 我检查了openssl引入是为了做 https 的验证(为了验证服务器证书,其实是画蛇添足).但是这部分…
代码审核(Code Review)是软件研发质量保障机制中非常重要的一环,但在实际项目执行过程中,却因为种种原因被Delay甚至是忽略.在实践中,给大家推荐一款免费.开放源代码的代码审查软件Gerrit. 1. Why Code Review Code Review是什么? Code Review最直观的解释即看代码.常规的做法为自己看,有时代码逻辑问题可能自己看不出来,需要找同事一起看,在大家知识体系相对平均的情况下可能需要花钱专门的公司帮助查看. Code Review需要看哪些?对于刚入职…
热情高涨 代码走查作为一种流程形式,起初大家的参与热情非常高涨. 因为,自己可以学习到别人一些巧妙的思想,自己的代码和习惯都暴漏出来. 这个过程中不断地吸收和改正. 但是...... 我们一开始组织的代码走查是一个很重的会议形式. 参加的人有写这段代码的人(小菜).比较有经验的开发(大佬) 如果为了再隆重一些,请一些领导也参与其中. 但是...... 我上面提过了,会议很重,协调时间这个事情就是一个很费时间的事情. 还有就是,大家恨不得对每一句代码都发表自己的意见,往往非常低细枝末节. 导致会议…
公司原先搭了一个代码Review的服务器,由于历史原因,装的是一个32bit的Ubuntu系统,后来由于需要,需要安装gitlab,由于gitlab需要64位系统,所以临时凑合了个vagrant,本质就是一个纯粹的虚拟机,感觉不爽,这两天终于抽出时间来重新整理了一下.基于Ubuntu 18.04 x64版本和Docker来部署,减少后面换机器换系统可能导致的重复安装工作. Docker安装 Docker安装还是比较简单的. $ sudo apt-get install apt-transport…
前言 对于一个程序猿来讲怎样来最直接的来衡量他的技术能力和产出呢?我想最直观的作法是看他的代码编写能力,就拿我常常接触的一些程序猿来看,他们买了非常多技术重构类书籍.可是看完后代码编写能力并没有显著提高.有人说可以用代码review工具啊,可是像市面上的这些代码review工具.仅仅能帮助我们解决表面的bug和规范点.还无法帮助我们发现更深层次的设计问题. 以下我将结合<软件设计重构>这本书谈谈在进行代码review的时候.须要关注的哪些点. 一.技术债务 何为技术债务? 技术债务是有意或无意…
以前我们一直都是如果要进行代码review的时候,要不我们就直接用idea来进行查看,根据不同的来查看 但是我们都是看代码的不同来进行来实现的,其实我们不需要这样,我们可以使用工具Phabricator的 工具进行提交,无奈,自己在windows的安装目前不行,所以就用公司的代码review的工具 1.我们先使用git到你自己的分支wangjing18-dev,然后到master里面,pull下来,接下来再到自己的分支wangjing-dev,接下来 git merge master,合并mas…
错误信息   消息号M7053   解决方法 Step 1.使用MMPV进入"关闭账期"界面. Step 2.输入"公司代码"."期间"."会计年度"后,执行(F8). Step 3.使用MMRV进入"查看打开的账期"界面,当前期间仍为"2014/04". Step 4.同 Step 1.Step 2 操作,将期间改为 "05". Step 5.同 Step 3 操作…
经验丰富的程序员和一般程序员之间的最大区别,不仅体现在解决问题的能力上, 还体现在日常代码的风格上.掌握一门技术可能需要几月,甚至几周就够了. 好的习惯风格养成却需数年. 团队成员之间需要合作,代码需要日后可维护,个体的能力和习惯存在差异. 故保证代码质量及风格,就需要制定一定的规则,按项目周期(最好是在上预发之前)组织进行集体代码review. 一 目的 1 保证代码质量 自己的代码要给别人看,在开发过程中就会刻意的注意一些规范,写法及逻辑严谨性. 2 扼杀潜在的风险 程序员会去自测,即使有某…
代码review是代码质量保障的手段之一,同时开发成员之间代码review也是一种技术交流的方式,虽然会占用一些时间,但对团队而言,总体是个利大于弊的事情.如何借助现有工具在团队内部形成代码review的流程与规范,是team leader或技术管理者需要考虑的问题.本文分享一种基于Gitlab代码merge流程的code review方法,以供参考与探讨.如有更好的方法,欢迎交流. 1. 设置成员角色 首先需要对你团队的成员分配角色,在Gitlab groups里选择一个group,然后左边菜…
Gitlab来做代码review 代码review是代码质量保障的手段之一,同时开发成员之间代码review也是一种技术交流的方式,虽然会占用一些时间,但对团队而言,总体是个利大于弊的事情.如何借助现有工具在团队内部形成代码review的流程与规范,是team leader或技术管理者需要考虑的问题.本文分享一种基于Gitlab代码merge流程的code review方法,以供参考与探讨.如有更好的方法,欢迎交流. 1. 设置成员角色 首先需要对你团队的成员分配角色,在Gitlab group…
对于代码review个人也有些小小的看法: 1.首先我觉得我们所有开发人员要弄明白 现在Code Review 的目的 ,凡事不弄明白目的,无法做好完成一件事情,个人觉得有以下一些目的: a)可以在项目早期就能够发现代码中的BUG ,提测后可以尽快的释放开发资源:b)同时可以达到知识共享 ,避免我们所有开发人员犯一些很常见,很普通低级的错误 :c)保证项目组人员的良好沟通 ,项目的代码更容易维护 大家还有希望补充上 2.Code Review 很容易变得没有意义或是流于形式,进入 Code Re…
SVN和Git代码管理小结  之前,先用的是SVN,后来用了Git,最近又在用SVN.  关于代码管理,写几句.    由于自己参与的项目,人通常不超过10个人,版本不是很多,协作比较正常,感觉SVN和Git足够用,差别也不算明显.之前有认真阅读过SVN的一些资料,现在是非常模糊了.  目前,没有兴趣研究SVN的原理.细致用法,Git也就算了.    只是写点简短的实用做法:    Trunk:最新代码,有新代码就提交上去  Tags:稳定的不变代码,线上发布1.0,1.1,1.2.项目上线时,…
代码审核(Code Review)是软件研发质量保障机制中非常重要的一环,但在实际项目执行过程中,却因为种种原因被Delay甚至是忽略.在实践中,给大家推荐一款免费.开放源代码的代码审查软件Gerrit. 1.Why Code Review Code Review是什么 Code Review最直观的解释即看代码.常规的做法为自己看,有时代码逻辑问题可能自己看不出来,需要找同事一起看,在大家知识体系相对平均的情况下可能需要花钱专门的公司帮助查看. Code Review需要看哪些?对于刚入职场或…
上周没事扫扫系统原来的代码,突然发现这样一段代码: if(log.isInfoEnable()) { log.info("ID"+userID+"pwd"+userPwd); } 因为之前写的日志类都是公司内部自己的日志类,对log4j用的并不多.当时看了这段代码觉得很奇怪,代码意思我是明白的,log本身就可以根据打印权限,判断当前是否打印呀,为什么专门要加上这样一条判断呢?后来查了一下,发现是由于性能方面考虑的. 如果直接写入这样一段话 log.info(&quo…
最近阶段正在学习Js框架的使用,目前正在详细了解JQuery EasyUI 的使用. jQuery EasyUI 框架帮助我们轻松建立站点: easyui是一个基于jquery的集成了各种用户界面的插件. 使用easyui你不需要写太多javascript代码,一般情况下你只需要使用一些html标记来定义用户接口. easyui非常简单,但是功能非常强大. 因为EasyUI有以上的优点,所以我们可以借助它来做我们的后台管理页面,在掌握了一些JQuery基础的前提下,可以进一步的学习JQuery又…
转自:http://blog.lichengwu.cn/java/2013/11/24/use-findbugs-code-review/ 介绍 Findbugs是一个代码静态分析工具,用来找出Java代码中的bug.通过分析字节码并与一组缺陷模式匹配,找出代码中的问题. 目前Findbugs支持除了java 8以外的所有版本编译的字节码文件分析. Findbugs不仅提供了可视化界面,还有Idea,Eclipse插件. Findbugs有谷歌和SUM/(Oracle)支持,在JDK源码编译中,…
一.一个叫声接口和几只鸭子 从一个叫声接口开始. {<HeadFirst设计模式>Delphi代码之模式小结 } { 一个叫声接口                            } { 编译工具:Delphi2010 for win32          } { E-Mail :guzh-0417@163.com             } unit uQuackable; interface type IQuackable = interface(IInterface) proced…
前言 有时候我们需要在应用启动时执行一些代码片段,这些片段可能是仅仅是为了记录 log,也可能是在启动时检查与安装证书 ,诸如上述业务要求我们可能会经常碰到 Spring Boot 提供了至少 5 种方式用于在应用启动时执行代码.我们应该如何选择?本文将会逐步解释与分析这几种不同方式 CommandLineRunner CommandLineRunner 是一个接口,通过实现它,我们可以在 Spring 应用成功启动之后 执行一些代码片段 @Slf4j @Component @Order(2)…
版本发布之后,外网新增了一些crash, 从mini dump看,的确是有异常的,619行crash了. 代码如下: 奔溃的地方如下 可以看到是Zip add的时候指针空了, 为什么呢? 结合代码,可以发现这个地方算便宜是有问题的,万一length 小于size呢?.. 修复方案就不另说了,其实这个问题我一开始从dump 没看出来,还是实际该代码重现才发现的. 最终找到问题了再来看,经验就是: windbg不会骗人,如果有源代码,就多review吧.不是每次都有条件重现的.…
作业链接 结对项目-第一阶段 优秀作业推荐 本次博客作业虽然是简单总结,但是以下作业中都不乏有思考.有亮点的精彩内容,推荐给同学们阅读学习. 山鸣谷应,相得益彰--杰对项目-第一阶段总结 该组对于可能出现的极端数据,使用 JProfiler进行了压力测试.并分析了测试结果,改进了相应代码以提高程序性能. 初见,结对编程!(上) 该组对于程序设计部分的介绍相当出色.绘制了指令的处理流程图,将程序所需实现的模块列出并绘制草图,清晰地展现了程序设计中思考的过程. 结对编程-第一阶段 该组设计部分的介绍…
在此结构中,一个 PPC 处理器作为监管处理器,与大量的 SPE流处理器相连通,组成了一个工作流水线. 对于一个图形处理过程来说,某个 SPE 可负责提取数据,另一个 SPE 负责变换,再另一个负责存回.这样可构成一道完完整整的流水线,大大提高了处理速度. 在我们搭建框架之初一般会设计一个ViewController基类,并在基类ViewDidLoad中设置一个随机的背景颜色.并通过touch手势来进行界面的跳转,以此来设计最开始的一个界面跳转框架,并通过界面颜色的变幻来验证我们界面跳转是否有做…
在创建阶段系统通过下面的几个步骤来完成对象的创建过程 为对象分配存储空间 开始构造对象 从超类到子类对static成员进行初始化 超类成员变量按顺序初始化,递归调用超类的构造方法 子类成员变量按顺序初始化,子类构造方法调用 本文重点演示第三步到第五步: Grandpa类 package com.xinye.test; public class Grandpa { { System.out.println("执行Grandpa的普通块"); } static { System.out.p…
1. dp px 相互转换---------------public class DensityUtil { /** * 根据手机的分辨率从 dip 的单位 转成为 px(像素) */ public static int dip2px(Context context, float dpValue) { final float scale = context.getResources().getDisplayMetrics().density; return (int) (dpValue * sc…
1.           Framework.QueryInfo info1 = new Framework.QueryInfo();            //string Sql = Holworth.Utility.Utility.EntityRowMapper.getSql("//data//ValueAtRiskCalculationService//NoneQuery2");            //info.CustomSQL = Sql;            inf…
FrmMain.cs中存在问题 1. int i=0 设定为了全局常量且未在类顶部,出现问题时不好查找 i 属于常用临时变量,设定全局变量容易引起混乱 2.定义的全局变量但仅在一处方法中使用,定义全局变量过多 3.变量名及控件名等意义不明确又缺少注释,如顶部定义的全局变量 ; ; private string oldPath = null; ; ; private string quotaNum = null; 其他类似 timer1,timer2,l1,l2等等... 4. 存在多处重复或相似…
RELULayer层 bottom[0]->count=n*c*w*h=50*96*56*56 count=50*96*56*56,根据bottom_data[i]访问所有的数据(多维数组都是一维数组那个样子存储的,所以这样就可以访问所有的数据),同时说明,一批50是一起计算的 conv_layer卷积层 blobs=2,一个是w,一个是b,blobs[0]表示的是w blobs_[0]->count=96*3*7*7=14112 说明了每个核只有一个bias,不像weight,在不同通道上面…
两个url 共用一个视图 url url(r'^books/$', views.BookViewSet.as_view({"get":"list","post":"create"}),name="book_list"), #{"get":"list","post":"create" } 当参数传递给.as_view 里面会经过一…
1.replace替换 <%@Page Language="C#"%> <% string strID=Request["id"]; strID = strID.Replace("'","");//单引号过滤 //strID==Globals.HtmlEncode(strID) Response.Write(strID); %> 2.转换 <%@ Page Language="C#&qu…
目录: 1.with读.写文件 (1)with读文件 (2)with写文件 2.requests爬虫 (1)get请求 (2)post请求 1.with读.写文件 (1)with读文件 (2)with写文件 https://www.cnblogs.com/andy9468/p/11493062.html 2.requests爬虫 (1)get请求 https://www.cnblogs.com/andy9468/p/11492910.html (2)post请求 https://www.cnbl…
github - git  init/add/commit/reset/log/status/stash pop/checkout/branch    新入职到公司,地址:   git clone http:/xxxxx      git init    git remote add origin http://xxxxx   git pull origin master         git init    git remote add origin http://xxxxx   git f…