O**o
发帖数: 2071 | 1 这里大公司的牛人多，借问一下哈。
我觉得我需要一个code review process。但是不知道要怎么弄。我曾经有个
consultant, 他给过我一些coding standard and code review check points.
这个coding standard有个二十来页，无非是些naming convention, comments,
javadoc这些个东西，跟当年java.sun.com，现在oracle网站上面的没什么大区别，排
版好看一些而已。
然后这个code review check list, 共80项，号称每个module/web page都要查这80项
。靠，这怎么搞啊？这不是坑姐吗？80项每一项都够查一阵子的。这玩意工作量很大啊
，我觉着比coding工作量还大啊。谁有这么多闲功夫一条一条琢磨啊？我不可能一个
developer写出code陪俩Sr. developers做code review啊。
难道就是走个过场？那负责code review的同志不是很惨？sign off之后那就是责任啊
。再说我确实code有点乱，确实想搞这么个process，不是想走过场的。除了general
coding standard, 还有security standard，跟那个什么OWASP有关。
请问各位牛人，你们公司里都是怎么弄这个的？要预留出多少时间给code review呢？
code review如果发现有问题又怎么处理？
感谢感谢！
===================
附：
从80条里面随手paste几个：
All the variables (member variables, local variables) are initialized. (
Either to some object or null or 0)
Judge whether String.length() / String.getBytes().length() should be
used.
Unnecessary imports are not present.
Check whether it is not a double loop. |
z******i
发帖数: 19 | 2 style can be checked by scripts
it's the easy part of code reviews
getting your reviewer to undertand and agree with your code is the hard part.
you write code, your reviewer will spend more than you to review
【在 O**o 的大作中提到】
: 这里大公司的牛人多，借问一下哈。
: 我觉得我需要一个code review process。但是不知道要怎么弄。我曾经有个
: consultant, 他给过我一些coding standard and code review check points.
: 这个coding standard有个二十来页，无非是些naming convention, comments,
: javadoc这些个东西，跟当年java.sun.com，现在oracle网站上面的没什么大区别，排
: 版好看一些而已。
: 然后这个code review check list, 共80项，号称每个module/web page都要查这80项
: 。靠，这怎么搞啊？这不是坑姐吗？80项每一项都够查一阵子的。这玩意工作量很大啊
: ，我觉着比coding工作量还大啊。谁有这么多闲功夫一条一条琢磨啊？我不可能一个
: developer写出code陪俩Sr. developers做code review啊。
|
t*****g
发帖数: 1275 | 3 checkstyle plugin for eclipse?
【在 O**o 的大作中提到】
: 这里大公司的牛人多，借问一下哈。
: 我觉得我需要一个code review process。但是不知道要怎么弄。我曾经有个
: consultant, 他给过我一些coding standard and code review check points.
: 这个coding standard有个二十来页，无非是些naming convention, comments,
: javadoc这些个东西，跟当年java.sun.com，现在oracle网站上面的没什么大区别，排
: 版好看一些而已。
: 然后这个code review check list, 共80项，号称每个module/web page都要查这80项
: 。靠，这怎么搞啊？这不是坑姐吗？80项每一项都够查一阵子的。这玩意工作量很大啊
: ，我觉着比coding工作量还大啊。谁有这么多闲功夫一条一条琢磨啊？我不可能一个
: developer写出code陪俩Sr. developers做code review啊。
|
O**o
发帖数: 2071 | 4 感谢回复。
那在你们的SDLC里面，真的有code review这么一个过程，而且allocate more time
and resource on the code review than coding?
part.
【在 z******i 的大作中提到】
: style can be checked by scripts
: it's the easy part of code reviews
: getting your reviewer to undertand and agree with your code is the hard part.
: you write code, your reviewer will spend more than you to review
|
d********t
发帖数: 587 | 5 我们公司code review的时候把checklist各项事先就分派到个人，严查+sign off by
reviewer's name.
【在 O**o 的大作中提到】
: 这里大公司的牛人多，借问一下哈。
: 我觉得我需要一个code review process。但是不知道要怎么弄。我曾经有个
: consultant, 他给过我一些coding standard and code review check points.
: 这个coding standard有个二十来页，无非是些naming convention, comments,
: javadoc这些个东西，跟当年java.sun.com，现在oracle网站上面的没什么大区别，排
: 版好看一些而已。
: 然后这个code review check list, 共80项，号称每个module/web page都要查这80项
: 。靠，这怎么搞啊？这不是坑姐吗？80项每一项都够查一阵子的。这玩意工作量很大啊
: ，我觉着比coding工作量还大啊。谁有这么多闲功夫一条一条琢磨啊？我不可能一个
: developer写出code陪俩Sr. developers做code review啊。
|
O**o
发帖数: 2071 | 6 那么，有没有一个general expectation on how long the code review should take?
如果严查，这东西不比coding本身花的时间短啊。
【在 d********t 的大作中提到】
: 我们公司code review的时候把checklist各项事先就分派到个人，严查+sign off by
: reviewer's name.
|
b*******y
发帖数: 12791 | 7 我讨厌死给人review code了，尤其是check in notes不交代因果，code也没有
comments的，看死了
part.
【在 z******i 的大作中提到】
: style can be checked by scripts
: it's the easy part of code reviews
: getting your reviewer to undertand and agree with your code is the hard part.
: you write code, your reviewer will spend more than you to review
|
l*****u
发帖数: 12114 | 8 我以前打工的时候最稀饭这个了。可以堂而皇之地拖和午睡。
【在 b*******y 的大作中提到】
: 我讨厌死给人review code了，尤其是check in notes不交代因果，code也没有
: comments的，看死了
:
: part.
|
b*******y
发帖数: 12791 | 9 屁股后面一堆活，怎么拖都是自己的阿
【在 l*****u 的大作中提到】
: 我以前打工的时候最稀饭这个了。可以堂而皇之地拖和午睡。
|
d****d
发帖数: 699 | 10 Code review is as importment as design, coding, and testing. Thinking about
this way, if the author left the project, can some one else understand the
code? |
r********r
发帖数: 964 | 11 review就是看个大概吧？除非跟自己关系密切的部分，基本就是走马观花，
看看没有大问题就放行
take?
【在 O**o 的大作中提到】
: 那么，有没有一个general expectation on how long the code review should take?
: 如果严查，这东西不比coding本身花的时间短啊。
|
