是不是大部分公司的开发都没有单元测试和代码评审 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
请不要在回答技术问题时复制粘贴 AI 生成的内容
itechnology
V2EX    程序员

是不是大部分公司的开发都没有单元测试和代码评审

  •  a href="Javascript:" Onclick="downVoteTopic(922133);" class="vote">
  •   itechnology 2023-03-08 10:20:00 +08:00 4972 次点击
    这是一个创建于 996 天前的主题,其中的信息可能已经有所发展或是发生改变。

    理想情况下肯定得有单元测试和代码评审。但就我待过的几家公司来看,只遇到一家要求要有单元测试,而且还规定了覆盖率,但其他公司基本都是因为工期卡的很紧,你有这个心,但上面时间不允许,经常是做的需求刚刚上线,下一个需求立马就来了,根本没有时间给你做代码评审,单元测试也同理。

    34 条回复    2023-03-09 01:32:29 +08:00
    matrix1010
        1
    matrix1010  
       2023-03-08 11:04:19 +08:00
    代码评审可能有,但水平不够的代码评审可能没什么正面作用。单元测试就难说了,我猜大部分国内公司都没有。甚至某些公司的开源项目都没有单元测试
    Tatsuyo
        2
    Tatsuyo  
       2023-03-08 11:06:38 +08:00
    噗,按照下一期签永久合同前就裁员的人力策略
    应该木有人想要迭代超长期的项目吧
    不准备做超长期项目的话,单元测试的成本甚至会高于纯人力吧
    tyrone2333
        3
    tyrone2333  
       2023-03-08 11:10:12 +08:00   2
    需求都做不完你还想写单元测试?
    d873139022
        4
    d873139022  
       2023-03-08 11:13:44 +08:00
    无,最多 sonar 扫扫
    duan602728596
        5
    duan602728596  
       2023-03-08 11:14:15 +08:00
    反正我们是有 code review 和各种单元测试、e2e 测试的。每次 push 代码都要跑一个小时的各种 pipeline 。
    j1132888093
        6
    j1132888093  
       2023-03-08 11:16:36 +08:00   1
    单元测试没有,但是有 code review
    tool2d
        7
    tool2d  
       2023-03-08 11:17:01 +08:00
    提交代码时,自动跑一次 lint 工具吧。

    强制所有人单元测试没太大必要,除非模块划分是极细的。
    Q81vlT1d47v4fq42
        8
    Q81vlT1d47v4fq42  
       2023-03-08 11:18:42 +08:00
    各种持续集成流水线还是有, review 基本就是走过场, 单元测试? 你需求搞完了么
    kennir
        9
    kennir  
       2023-03-08 11:25:17 +08:00
    单元测倾向有,但是需求来了根本没时间加,只能期望做完需求有时间补,但是这种也就是不了了之了
    msg7086
        10
    msg7086  
       2023-03-08 11:28:10 +08:00
    对我司来说测试是开发的一部分。
    如果一个需求你只写完了代码,没写测试,那这个需求的完成进度是 33%。
    DinnyXu
        11
    DinnyXu  
       2023-03-08 11:30:30 +08:00
    有冒烟用例,等同于单元测试吧,但是等级比单元测试低一等。CR
    DinnyXu
        12
    DinnyXu  
       2023-03-08 11:30:46 +08:00
    @DinnyXu CR 是有的
    nyakoy
        13
    nyakoy  
       2023-03-08 11:34:46 +08:00
    都没有,
    dayeye2006199
        14
    dayeye2006199  
       2023-03-08 11:42:18 +08:00 via Android
    我司都有。感觉也不是特别费事。Tech lead 需要担起责任来,把流水线搭起来。这个属于一次投入,长期受益的事儿
    NoKey
        15
    NoKey  
       2023-03-08 11:44:12 +08:00   1
    没有代码 review 的,多轮人员迭代后,代码还能看不
    Desdemor
        16
    Desdemor  
       2023-03-08 11:54:25 +08:00
    不知道别人,日常开发的时候都会写,感觉这样心里有底,后续变更,再跑测试感觉也方便
    dcty
        17
    dcty  
       2023-03-08 12:11:11 +08:00
    code review 大致分为两种:
    1 、程序员间的“挑刺”,你这里这样写不好, 应该这样写。
    2 、业务上的 backup 。

    以上就是开个玩笑。
    bk201
        18
    bk201  
       2023-03-08 12:27:54 +08:00
    你都是民工了,操心的事情还挺多。这种事情是质量需要去做的 kpi ,而不是你。
    fengjianxinghun
        19
    fengjianxinghun  
       2023-03-08 12:31:24 +08:00 via iPhone   2
    没有,直接线上 debug ,客户当测试
    Building
        20
    Building  
       2023-03-08 12:41:15 +08:00 via iPhone
    这难道不是取决于你写的东西吗?
    人家敲个三天就下线的业务代码,你也要这个单元测试吗?
    你要写框架代码,不写单元测试和例子,这……谁敢用
    timle1029
        21
    timle1029  
       2023-03-08 12:57:39 +08:00
    全有:
    Code Review
    Unit Test
    Integration Test
    Canary Test
    后两个在 pipeline 里跑。不写 Unit test 的 dry run build 就跑不过,还轮不到让别人看你的代码
    wangtian2020
        22
    wangtian2020  
       2023-03-08 13:11:27 +08:00
    刚入职的时候公司搞过代码评审,我写的正常代码被一顿批,后来就没时间开会领导就不管了。但是开发前一直是有多次的设计评审的
    knightdf
        23
    knightdf  
       2023-03-08 13:18:43 +08:00
    @fengjianxinghun 哈哈,线上环境就是测试环境
    shyangs
        24
    shyangs  
       2023-03-08 14:24:32 +08:00
    首先要有 需求文件 ,文件。不然的立基是什呢?
    zxCoder
        25
    zxCoder  
       2023-03-08 15:50:19 +08:00
    一些基于开源的小公司可能反而有
    jones2000
        26
    jones2000  
       2023-03-08 15:51:17 +08:00
    开发不用写测试, 都是测试小组写的。
    unco020511
        27
    unco020511  
       2023-03-08 15:58:08 +08:00
    我们有,代码如果要进入主干,得两个核心开发审代码,发现的问题改完才可以合代码
    KedaArray
        28
    KedaArray  
       2023-03-08 16:21:05 +08:00
    想问下,做 Java 开发的 其他时间自己学下测试,对工作有没有帮助
    wanguorui123
        29
    wanguorui123  
       2023-03-08 16:25:33 +08:00 via iPhone
    业务复杂的公司都有上线代码评审,接口测试、还有黑盒测试、预发布环境、灰度发布。
    nicocho
        30
    nicocho  
       2023-03-08 16:37:44 +08:00 via Android
    前几天安卓加了点代码,发现谷歌自有的模块都有单元测试代码。瞄了几眼 mock 啥的,有时间想学着写写。
    F281M6Dh8DXpD1g2
        31
    F281M6Dh8DXpD1g2  
       2023-03-08 16:38:57 +08:00
    code review 除了吹毛求疵之外真看不出来啥
    还是集成测试来的实在
    wu67
        32
    wu67  
       2023-03-08 16:49:57 +08:00
    最多就提交的时候 review 一下, 核心纯代码逻辑简单过一下测试. 其他的, 想想就好.
    kingcanfish
        33
    kingcanfish  
       2023-03-08 22:08:20 +08:00
    为了单测率而单测
    EscYezi
        34
    EscYezi  
       2023-03-09 01:32:29 +08:00 via Android
    哈哈,何止是“这个需求刚上线下个需求就来”,经常一个需求没做完下一个需求已经出来了,隔三差五插一些紧急需求,能按时上线已经很不容易了。
    至于上线之后出问题?先上线再说
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     2651 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 28ms UTC 12:53 PVG 20:53 LAX 04:53 JFK 07:53
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86