V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
burnbrid
V2EX  ›  程序员

组员写代码喜欢复制代码,复制完不格式化代码, 代码的写的歪七八扭的。今天说他,他改了重新提交,但是他的态度就是完全不在乎,一点都不是上心,经常犯。这是我的问题还是组员的问题,我要不要严格要求组员格式化代码,还是不管他们,只要代码能运行就行。

  •  
  •   burnbrid · 2019-09-24 11:01:56 +08:00 · 14165 次点击
    这是一个创建于 1669 天前的主题,其中的信息可能已经有所发展或是发生改变。
    129 条回复    2019-09-25 19:33:05 +08:00
    1  2  
    yws112358
        101
    yws112358  
       2019-09-24 17:13:35 +08:00   ❤️ 2
    BTW:建议格式化下你的标题和问题描述 。
    我没看出哪是标题,哪是描述。
    funnypc
        102
    funnypc  
       2019-09-24 17:19:27 +08:00
    22 楼就是我想说的
    qiaobeier
        103
    qiaobeier  
       2019-09-24 17:21:36 +08:00
    有很多代码检查工具,提交前检查一次,不符合规则禁止提交。需要搭配 code review 一起服用。
    duzhihao
        104
    duzhihao  
       2019-09-24 17:45:49 +08:00
    让他 go out
    bigpigeon
        105
    bigpigeon  
       2019-09-24 17:48:42 +08:00
    用.git hook 和 gitlab ci,没必要自己操心
    vimiix
        106
    vimiix  
       2019-09-24 17:50:38 +08:00
    qwertyyb
        107
    qwertyyb  
       2019-09-24 19:34:21 +08:00 via iPhone
    遇到过大佬写的代码,用编辑器打开后一片红,lint 错误触目惊心,但是人家是大佬,我一个新人人微言轻,能怎么办?
    laravel
        108
    laravel  
       2019-09-24 19:53:50 +08:00
    用 lint 检查,不符合的不让提交
    Karpov
        109
    Karpov  
       2019-09-24 20:29:17 +08:00
    团队格式化工具很多啊···可以去发掘一下·
    test0x01
        110
    test0x01  
       2019-09-24 23:10:51 +08:00 via Android
    强制格式检查,不过不 review 不准 merge
    20015jjw
        111
    20015jjw  
       2019-09-24 23:49:45 +08:00 via Android
    lint 啊
    yegle
        112
    yegle  
       2019-09-25 00:54:54 +08:00
    哈哈哈哈蛤你发个贴把内容写到标题里还好意思说别人哈哈哈哈哈哈哈哈
    boywang004
        113
    boywang004  
       2019-09-25 01:17:33 +08:00
    上 lint 绑定 push hook,硬刚谁怕谁啊!
    ericgui
        114
    ericgui  
       2019-09-25 04:10:06 +08:00
    你这样组长没权威呗,不能阻止不好的代码进来
    reus
        115
    reus  
       2019-09-25 06:18:56 +08:00
    @2255cn 随意扣钱是违法行为,法盲。
    reus
        116
    reus  
       2019-09-25 06:19:48 +08:00
    @opengps 不以为耻?
    reus
        117
    reus  
       2019-09-25 06:22:11 +08:00
    @RunPiggy 不遵守团队规范就踢出团队,这还要利诱?你看到垃圾怎么处理?直接扫地啊。
    JCZ2MkKb5S8ZX9pq
        118
    JCZ2MkKb5S8ZX9pq  
       2019-09-25 06:49:02 +08:00
    为啥我想到了富坚义博……
    fxxwor99LVHTing
        119
    fxxwor99LVHTing  
       2019-09-25 07:23:14 +08:00
    标题中也说了,这是态度问题。
    那么,
    1. 让 ta 滚蛋
    2. 忍住 /忽视 ta
    3. 如果标题说的就是楼主本人的话,劝你端正态度,要不干脆转行算了,每天被逼迫干着自己不愿干的事情,也是一种折磨 /煎熬。
    fhsan
        120
    fhsan  
       2019-09-25 07:38:33 +08:00
    我以前就是复制粘贴代码,现在最起码改改优化下。
    meppy
        121
    meppy  
       2019-09-25 09:29:26 +08:00
    ctrl+shift+F
    m9rco
        122
    m9rco  
       2019-09-25 09:43:52 +08:00
    22 楼就是我想说的
    skylancer
        123
    skylancer  
       2019-09-25 09:45:03 +08:00
    自己都把所有的内容写在标题里让我看了都头大,自己都做不到让人赏心悦目还有资格 bb 其他人
    Torpedo
        124
    Torpedo  
       2019-09-25 10:00:50 +08:00
    加个 git hook,绑定一个 lint
    nianyu
        125
    nianyu  
       2019-09-25 10:33:07 +08:00
    公司的问题, 如果一开始有严格的制度他们自然会遵守,如果一开始没有制度突然要求 他们肯定不在乎啊
    zishaofei221
        126
    zishaofei221  
       2019-09-25 10:38:22 +08:00
    代码重复率检测。
    工具能搞的 为啥要人肉
    iehp
        127
    iehp  
       2019-09-25 10:42:49 +08:00
    gitlab+sonarqube 解君愁。。。
    nicebird
        128
    nicebird  
       2019-09-25 11:10:32 +08:00
    静态检查,实在不行就开了
    realpg
        129
    realpg  
       2019-09-25 19:33:05 +08:00
    不用要求,扣钱就好
    1  2  
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   3062 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 34ms · UTC 11:01 · PVG 19:01 · LAX 04:01 · JFK 07:01
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.