svn提交代码时:
1.提交代码时要先更新,如果没有冲突再提交。
2.如果更新完有冲突,就解决冲突,在其过程中不要乱删除别人的代码,删之前,应该问问 其他人。
3.提交代码前看自己的代码格式对否,空行的行数是固定的,函数的结尾需要空行;所引得包是否多,用ctrl+shift+o 整理一下。
4.那些用anntation注释的地方的上方要空一行,在每一个方法的开头也空一行。
5.在本地用ant test测试一下,成功再提交,不要影响别人的代码运行。
开发时:
1.在写测试类时,首先要考虑所传的参数是否符合要求,有时只传id而不是name,弄错了会影响进度。
2.测试用例尽量想全,空,不合法值,越界,互斥...
3.编码时,一个方法一般在20行左右,有一些对象通用的方法尽量都写在model里面去,如果还用写其他方法,那就在本类中用private写一个方法,写好注释说明,让别人看懂。
4.如果编码很麻烦,那就是可能逻辑弄错了,就得重新考虑方法了,要不就是参数传的不对等造成的。不要做下去了,得和大家考虑方法,看是否对,再做。
5.注意只在私有方法,类和接口上面写注释
6.禁止提交哪些sysout打印输出语句和一些本地的测试代码,迷惑大家
7.在方法上面写注释时要写清楚,所传的参数的类型,还有数组内的各个标注下代表的值是什么,用1,2,3...介绍清楚
8.类的上方写清楚开发者,日期,作用
9.测试类不一定要写的很全,可以都让去数据库查的,就让其去数据库去查,最终返回空就行了,互斥的条件也如此。
10.在编码时,考虑异常情况时要首先判断不合理的情况让其返回,如下面代码:
if(CollectionUtils.isEmpty(idList)) return groupDataModelList;
但是不要写成if(!CollectionUtils.isEmpty(idList)){...正常实现},不写else的实现了,这样会造成idList为空时出异常情况。
11.空行:
在if,for循环等开始的地方要注意最好在其上面空一行;
在一些声明链表,字符串等变量时,在其下面如果可以空行的话,尽量也空一行,方便清晰查看代码;
在方法体结束的return之前空一行。
12.[color=red]如果方法里某些代码都应该是属于某个对象的行为,就应该封装到该对象的里面,把方法名字定义好,明白易懂[/color]
13.等待重构
1.提交代码时要先更新,如果没有冲突再提交。
2.如果更新完有冲突,就解决冲突,在其过程中不要乱删除别人的代码,删之前,应该问问 其他人。
3.提交代码前看自己的代码格式对否,空行的行数是固定的,函数的结尾需要空行;所引得包是否多,用ctrl+shift+o 整理一下。
4.那些用anntation注释的地方的上方要空一行,在每一个方法的开头也空一行。
5.在本地用ant test测试一下,成功再提交,不要影响别人的代码运行。
开发时:
1.在写测试类时,首先要考虑所传的参数是否符合要求,有时只传id而不是name,弄错了会影响进度。
2.测试用例尽量想全,空,不合法值,越界,互斥...
3.编码时,一个方法一般在20行左右,有一些对象通用的方法尽量都写在model里面去,如果还用写其他方法,那就在本类中用private写一个方法,写好注释说明,让别人看懂。
4.如果编码很麻烦,那就是可能逻辑弄错了,就得重新考虑方法了,要不就是参数传的不对等造成的。不要做下去了,得和大家考虑方法,看是否对,再做。
5.注意只在私有方法,类和接口上面写注释
6.禁止提交哪些sysout打印输出语句和一些本地的测试代码,迷惑大家
7.在方法上面写注释时要写清楚,所传的参数的类型,还有数组内的各个标注下代表的值是什么,用1,2,3...介绍清楚
8.类的上方写清楚开发者,日期,作用
9.测试类不一定要写的很全,可以都让去数据库查的,就让其去数据库去查,最终返回空就行了,互斥的条件也如此。
10.在编码时,考虑异常情况时要首先判断不合理的情况让其返回,如下面代码:
if(CollectionUtils.isEmpty(idList)) return groupDataModelList;
但是不要写成if(!CollectionUtils.isEmpty(idList)){...正常实现},不写else的实现了,这样会造成idList为空时出异常情况。
11.空行:
在if,for循环等开始的地方要注意最好在其上面空一行;
在一些声明链表,字符串等变量时,在其下面如果可以空行的话,尽量也空一行,方便清晰查看代码;
在方法体结束的return之前空一行。
12.[color=red]如果方法里某些代码都应该是属于某个对象的行为,就应该封装到该对象的里面,把方法名字定义好,明白易懂[/color]
13.等待重构