• Quincy
    2018-11-15
    老师,我有个问题想问,您觉得程序员该不该追求安稳。。我目前校招拿到的offer中私企和国企有点纠结。。。国企安稳,私企发展比较好,但又担心以后裁员的问题。

    作者回复: 你准备一份工作干一辈子吗?

     1
     5
  • 黄蓓
    2018-11-21
    小米的代码评审做的还不错,高级权限能+2,普通权限能+1,每次提交只有+2了才能入库

    作者回复: 这就是工具和流程给予的限制,才方便推动👍

     1
     4
  • godtrue
    2018-11-16
    我们组还好,之前也讲过必须进行代码评审的,合并代码也必须两个人,且通过架构师评审。
    胡哥,咱们同属一家公司的,看样子是不同部门或小组有不同的要求的!

    作者回复: 嗯,公司没有流程没有要求,就是自己团队约定了,要看团队所在业务和接受度。你是哪个部门?

    
     2
  • 心在飞
    2018-11-14
    我在医疗行业,code review是必要环节,每个user story接受需要填写线上code review记录(50-60号人,跨国家)。如果团队人数较少,5-6个开发,个人比较喜欢线下的code review,大家坐会议室,分享自己想法,吃吃零食,聊聊天,这样气氛会更好点。code review气氛很重要,只评论代码,不接受人身攻击!(你review别人的代码VS你的代码被人review,感觉完全不同) 然后还需要有个资深的架构师,在大家拿不定主意的时候能够拍个板。总之,code review程序员还是受益良多的!

    作者回复: 嗯,行业性质不同,程序的重要性也不同。你们这个过程还是挺不错的,我们基本很难搞成这样

    
     2
  • javaadu
    2019-12-16
    我们最近半年开始强制做代码评审,每周还有统计同晒,目前我感觉收获蛮大,对于评审人来说是一次表达自己观点、熟悉别的业务的机会;对于被review的人来说,是一次从外部审查自己的产出的机会,是一次成长的机会,可以说,好的代码评审可以促进双方进步、同时提高团队的代码质量
    
     1
  • 寇云
    2018-11-22
    「说到痛点了,时间紧,任务重,没有时间做 code review.」
    -----
    是不是悖论呢?就像写UT,写UT浪费时间。但是认识就是错误的,写UT是为了节省时间。code review 的目的也是为了让代码符合规范,可复用。也是为了节省时间。

    作者回复: 要么工具强制执行CR,要不就考验团队智慧了

    
     1
  • Franklin.du
    2018-11-14
    以前公司没有code review的工具,仅仅是当bug修改以后,等待提交代码时需要指定程序员当场review一下,看下有没有啥没注意到的缺陷。虽然不是很严格的审查,感觉还是有效果的。

    作者回复: 这是在成本和概率间权衡取舍的方式,大部分互联网软件可以接受这个一定比率出错的概率

    
     1
  • 杨少侠
    2018-11-14
    说到痛点了,时间紧,任务重,没有时间做 code review.

    作者回复: 共同的痛

    
     1
  • Allen_Go
    2019-04-10
    那我公司来说,提交业务代码后发起代码合并请求,小组长因为合并和上线的角色,鉴于锅从天上来的敬畏,都很自觉reviwer一下代码。但是鉴于时间的关系,都只能到达代码逻辑有没有问题的层面。

    作者回复: 能到这个层面已经不容易了吧

    
    
  • LieBrother
    2018-12-25
    所在的团队没有代码review,代码比较乱,每个人的风格不一样

    作者回复: 风格这个问题不是靠Review来解决的

    
    
  • 行下一首歌
    2018-12-18
    没有任何代码评审工作的公司或团队,都不值得加入。

    作者回复: 也许没这么绝对吧

    
    
  • third
    2018-12-15
    把自己想象成外人,用第三者视角看自己

    代码评审:提前对代码进行检测,有较高概率降低出错率

    多种困境
    时间成本
    效果差
    利益不好分配
    总有刁民想害朕

    自省
    自我的成长就是自省开始的。
    展开
    
    
我们在线,来聊聊吧