codereview的正确姿势 - 未名空间MITBBS历史存档

国际科技财经博客移民网络热点娱乐民生时事公众号

Redian新闻

>未名空间

>JobHunting - 待字闺中

codereview的正确姿势

codereview的正确姿势# JobHunting - 待字闺中

l*g2018-11-20 08:11

1 楼

我看有人问，有人回。比起什么dsb的贴有营养。
我有个文档，我觉得挺好。但是弄出来前，先听听大家的意见。每个公司文化不一样。
我先炮个转：code review是show你的visibility，也是别人对的起你才给你review。
所以两方面：给code review一定要有constructive，别bb
别人给的任何意见要虚心，即使不同意，要找到理由，得到别
人同意。这叫collaboration

C*12018-11-20 08:11

2 楼

有人喜欢在cr里面乱写怎么办

m*s2018-11-20 08:11

3 楼

用心写的review，往往会得罪人，真正能心平气和学习讨论的凤毛麟角。所以，何必呢
，何苦呢？

z*g2018-11-20 08:11

4 楼

看不看都一样，关键要能递交

d*n2018-11-20 08:11

5 楼

楼上几位所在环境这么差的吗？
给review花时间花精力当然是真心实意好好写
要不然图个啥？

l*g2018-11-20 08:11

6 楼

分公司，分人。
有些人看见别人给review把你当仇人，有人把你当爸爸。。。怎么破。

【在 d*******n 的大作中提到】

: 楼上几位所在环境这么差的吗？
: 给review花时间花精力当然是真心实意好好写
: 要不然图个啥？

l*g2018-11-20 08:11

7 楼

请问您在flag？

【在 m**********s 的大作中提到】

: 用心写的review，往往会得罪人，真正能心平气和学习讨论的凤毛麟角。所以，何必呢
: ，何苦呢？

m*s2018-11-20 08:11

8 楼

也许这些大公司能不一样吧

【在 l*******g 的大作中提到】

: 请问您在flag？

d*32018-11-20 08:11

9 楼

分情况：
如果心平气和的看，确实看不懂，可以和ta先1:1/direct messaging 就说我不理解你
写的是什么意思，could you elaborate? 也许对方不是乱说，而是在code review里说
不清自己的想法。如果ta乱说，你可以说thanks for the feedback, I don’t
necessarily agree with you tho. 如果来回几次还这样，可以告诉manager，让
manager去和ta谈。
有些时候code review feedback确实让人生气，一定不要在生气的时候respond。一来
不解决问题，二来容易被抓小辫子。

: 有人喜欢在cr里面乱写怎么办

【在 C*******1 的大作中提到】

: 有人喜欢在cr里面乱写怎么办