精华好帖回顾
· 一个善意的提醒：猫猫不能太胖了————兼回忆我们家聋子 (2007-7-27) hysteria	· prius V 万公里简报，同时召唤墨尔本的车友 (2013-1-18) bennytang
· 孩子的双语教育 (2009-3-3) 思思妈妈	· 兔小D 09中国行 - 湘西之旅篇 (2009-10-25) 我是兔小D

1 23 / 3 页

楼主:乱码

大家都怎么做code review.... [复制链接]

angeltw

皮靴族

发表于 2011-11-14 10:06 |显示全部楼层

此文章由 angeltw 原创或转贴，不代表本站立场和观点，版权归 oursteps.com.au 和作者 angeltw 所有！转贴必须注明作者、出处和本声明，并保持内容完整

喜欢lz的风格，呵呵。赞一个。我不写代码，我是做integration的。不过我们公司service team也有这个一个人，英国人，老头，team leader，写apex. 最爱把东西写一半丢给别的developer做，而且还说他已经做得差不多了，再让别人做个半天就好。实际上写得代码离完成还差得远。而且代码里面连基本的error handling都没有，遍历树都是只遍历一半的。公司的developer都对他抱怨不堪。一直搞不懂这种人还能在公司里混得不错。幸好我不写代码。

mxgong

银靴族

发表于 2011-11-14 10:34 |显示全部楼层

此文章由 mxgong 原创或转贴，不代表本站立场和观点，版权归 oursteps.com.au 和作者 mxgong 所有！转贴必须注明作者、出处和本声明，并保持内容完整

JIRA + fisheyes?

yourock

金靴族

发表于 2011-11-18 22:38 |显示全部楼层

此文章由 yourock 原创或转贴，不代表本站立场和观点，版权归 oursteps.com.au 和作者 yourock 所有！转贴必须注明作者、出处和本声明，并保持内容完整

consultant干吗的？

梦呓人

银靴族

发表于 2012-12-11 14:39 |显示全部楼层

此文章由梦呓人原创或转贴，不代表本站立场和观点，版权归 oursteps.com.au 和作者梦呓人所有！转贴必须注明作者、出处和本声明，并保持内容完整

无视发表于 2011-11-3 16:44
The following are guidelines we've worked out most recently and try to follow.

• Proven

码兄，

•       Correct
      o       Meets COAs
      o       Visual Signoff

指的是写出来的东西和spec和客户要求的一样，是不是？

•       Consistent
      o       Vertical Slicing
      o       Responsibilities
      o       Helpers

指的是？....

•       Repeatable
      o       Atomic, parallelisable, data safe

指的是代码原子化，从而re-usable是不是？后面的“parallelisable, data safe”怎么和Repeatable又联系起来？

无视

银靴族

发表于 2012-12-11 15:23 |显示全部楼层

此文章由无视原创或转贴，不代表本站立场和观点，版权归 oursteps.com.au 和作者无视所有！转贴必须注明作者、出处和本声明，并保持内容完整

梦呓人发表于 2012-12-11 14:39
码兄，

• Correct

1. COA - condition of acceptance, means to meet the requirement/spec

2. vertical slicing -- it's the way to separate concerns
responsibility -- single responsibility
helper -- u may use helper to cross concern cutting

3. Repeatable/Atomic, parallelisable, data safe -- there are more of slicing logic to minimal units and make it in reusable/multi-threading/asyn manner.

Don't worry too much for 3, if you check off things based on the items on that sheet, ppl get crazy and you may get beat up mate

评分

参与人数 1	积分 +6	收起理由
梦呓人	+ 6	你太有才了

查看全部评分

kanjunhai

木屐族

发表于 2012-12-18 23:41 |显示全部楼层

此文章由 kanjunhai 原创或转贴，不代表本站立场和观点，版权归 oursteps.com.au 和作者 kanjunhai 所有！转贴必须注明作者、出处和本声明，并保持内容完整

无视发表于 2011-11-3 15:44
The following are guidelines we've worked out most recently and try to follow.

• Proven

cool.

orange0328

皮靴族

发表于 2012-12-19 13:35 |显示全部楼层

此文章由 orange0328 原创或转贴，不代表本站立场和观点，版权归 oursteps.com.au 和作者 orange0328 所有！转贴必须注明作者、出处和本声明，并保持内容完整

态度决定一切

cynosure

铜靴族

发表于 2012-12-20 11:00 |显示全部楼层

此文章由 cynosure 原创或转贴，不代表本站立场和观点，版权归 oursteps.com.au 和作者 cynosure 所有！转贴必须注明作者、出处和本声明，并保持内容完整

赞同你尽自己的职责，但语气要委婉点，每个人都有自己的能力所限，应该像当年别人给我们耐心一样也给别人耐心

bullying520

皮靴族

发表于 2012-12-21 00:17 |显示全部楼层

此文章由 bullying520 原创或转贴，不代表本站立场和观点，版权归 oursteps.com.au 和作者 bullying520 所有！转贴必须注明作者、出处和本声明，并保持内容完整

有什么软件能做CODE REVIEW?

xji

铜靴族

发表于 2012-12-24 03:44 |显示全部楼层

此文章由 xji 原创或转贴，不代表本站立场和观点，版权归 oursteps.com.au 和作者 xji 所有！转贴必须注明作者、出处和本声明，并保持内容完整

bullying520 发表于 2012-12-21 00:17
有什么软件能做CODE REVIEW?

我们用Review Board
http://www.reviewboard.org/

有Eclipse的plugin。

内核

铜靴族

发表于 2012-12-24 11:33 |显示全部楼层

此文章由内核原创或转贴，不代表本站立场和观点，版权归 oursteps.com.au 和作者内核所有！转贴必须注明作者、出处和本声明，并保持内容完整

xji 发表于 2012-12-24 03:44
我们用Review Board
http://www.reviewboard.org/

+1

~~~快快长大~~~

IsDonIsGood

银靴族

发表于 2012-12-24 11:36 |显示全部楼层

此文章由 IsDonIsGood 原创或转贴，不代表本站立场和观点，版权归 oursteps.com.au 和作者 IsDonIsGood 所有！转贴必须注明作者、出处和本声明，并保持内容完整

关键看公司愿意花多少钱，和unit testing一样的，总共3人的team，deadline一个接一个，谁有那闲心code review和unit testing啊，best practice大家都懂，但是做起来就是另外一回事情了。

2020目标: 活着

mxgong

银靴族

发表于 2012-12-25 01:01 |显示全部楼层

此文章由 mxgong 原创或转贴，不代表本站立场和观点，版权归 oursteps.com.au 和作者 mxgong 所有！转贴必须注明作者、出处和本声明，并保持内容完整

bullying520 发表于 2012-12-21 00:17
有什么软件能做CODE REVIEW?

fisheye + crucible

情何以堪

1 23 / 3 页

返回列表

		自动登录	找回密码
密码			注册

精华好帖回顾

大家都怎么做code review.... [复制链接]

评分

发表回复