V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
victorwu34
V2EX  ›  职场话题

同事不注意代码质量,应该如何沟通

  •  
  •   victorwu34 · 2019-04-22 00:33:39 +08:00 · 5759 次点击
    这是一个创建于 2054 天前的主题,其中的信息可能已经有所发展或是发生改变。

    一次是根据 id 的集合查询,查询的 api 如果 id 集合比如 3000 个以上就不支持,我以前的代码有错,3000 之外的都被忽略掉,结果同事把代码改成了循环几千次去查表。我跟他沟通这个问题,他说,就是这样,多了就不支持,那你改啊。

    还比如,一次他把几十个返给前端实体里面的字段修改了,前端很多接口拿不到这个字段,很多页面报错。这次和他沟通,我说这是你造成的 bug,你自己去改,他说,我这里的比你的 bug 还多,咱们是一个团队,这正常。 还有很多次,都是他瞎写瞎改造成的,每一次都无法沟通,别人也在帮他改错。

    已经很尊重他了,也帮助了他,我们没有代码审查,技术经理也从来不问不管,没有任何改善,不知道这种情况应该如何沟通。

    28 条回复    2019-04-23 11:01:04 +08:00
    CEBBCAT
        1
    CEBBCAT  
       2019-04-22 00:45:41 +08:00
    看起来公司里没这个文化,凭你自己也似乎很难改变一个公司。没工作的我这么分析一下:费力不讨好,骑驴找马吧
    victorwu34
        2
    victorwu34  
    OP
       2019-04-22 00:48:11 +08:00 via Android
    @CEBBCAT 你就是说无法沟通😭😭😂😂
    R4mble
        3
    R4mble  
       2019-04-22 00:54:23 +08:00 via Android
    你这两个例子我都没太看懂诶*罒▽罒*
    你是前端,你同事是后端吗
    LifStge
        4
    LifStge  
       2019-04-22 01:15:12 +08:00 via iPhone
    例子不是代码质量的问题啊 沟通问题 开发人员相处的不融洽...
    boris1993
        5
    boris1993  
       2019-04-22 01:27:35 +08:00 via Android   ❤️ 1
    这已经不是代码质量问题了,这是团队里进了个二逼

    要我说啊,首先,(联合起几个人)找领导报告这事,私下里他胡逼写的时候就损丫的骂丫的

    傻逼不值得被尊重
    zgl263885
        6
    zgl263885  
       2019-04-22 07:19:31 +08:00 via iPhone
    有些人就不适合写代码,特别是工作一年之后还是这样的话。但是这种事情又没法跟人家直说,可以跟上级领导反应,劝他做其他岗位工作,说不定做的更好。每个人都有优点和特长,与此人擅长做设计,美工,市场等。
    gramyang
        7
    gramyang  
       2019-04-22 07:32:04 +08:00
    有两种方法可选:
    一是严正秉持谁污染谁治理的策略,发现是他的问题导致的 bug 就让他自己弄。当然前提是你能够收集到足够严谨的证据来证明是他个人的问题导致的(代码截图,聊天信息,邮件记录)。
    二是把这个人的各种坑操作都记录下来(同样是代码截图,聊天记录,邮件记录等),然后等待一个时机(比如某一次出现重大事故,不管是不是他造成的都无所谓)。把搜集到的证据全部整理清除,让人一目了然,用一个邮件发给直属领导(记得属上其他看不惯这厮的同事的名字)。

    如果上面两种方法都不顶用,那么很可能这哥们是关系户,你斗不过也没必要浪费精力和他斗,跑路吧
    chinesestudio
        8
    chinesestudio  
       2019-04-22 08:11:28 +08:00 via Android
    快狠准 。写代码是习惯 做好备注 别人看得懂就行 。划水的时候可以优化代码。
    tt67wq
        9
    tt67wq  
       2019-04-22 08:33:17 +08:00
    对他说: 你代码写的像蔡徐坤!
    elgae
        10
    elgae  
       2019-04-22 08:47:29 +08:00 via iPhone
    lz 是不是在大兴上班?
    alw
        11
    alw  
       2019-04-22 09:26:55 +08:00
    谁导致的 BUG 谁修,帮忙并不是理所应当的,打死都不帮他修,难道还要我背责任?
    mawenjie
        12
    mawenjie  
       2019-04-22 09:35:37 +08:00
    我想起前公司一个同事兼负责人,他出的方案基本上就是 “就是这样,多了就不支持,那你改啊”。
    meteor957
        13
    meteor957  
       2019-04-22 09:49:08 +08:00
    还是单独沟通一下吧,脸皮薄的说一次就记住了
    LwZiye
        14
    LwZiye  
       2019-04-22 10:57:53 +08:00
    人善被人欺,直接开怼
    tiedan
        15
    tiedan  
       2019-04-22 11:09:08 +08:00
    只要没影响到自己工作 其实不用管。
    catcn
        16
    catcn  
       2019-04-22 11:45:03 +08:00
    直接改丫的代码
    itfinally23
        17
    itfinally23  
       2019-04-22 11:51:15 +08:00
    这个,在行政上会做代码检查,构建时加个插件,不合格的全部停止构建。
    tairan2006
        18
    tairan2006  
       2019-04-22 14:08:53 +08:00
    这种公司呆着没啥意思啊,建议溜
    zhpech
        19
    zhpech  
       2019-04-22 14:51:54 +08:00
    溜了吧 没啥相关文化,可能还会给你扣影响团队和谐的帽子
    katsusan
        20
    katsusan  
       2019-04-22 15:20:49 +08:00 via iPhone
    > 技术经理也从来不问不管
    都到这份上了还想啥呢
    q8164305
        21
    q8164305  
       2019-04-22 15:58:33 +08:00 via Android
    赶紧跑路吧,这种人我也遇到过,根本没法合作,后面会憋死你,人家就是把这当工作,而不是爱好,不好指责人家,只能说道不同不相为谋罢了
    lmeteor1101
        22
    lmeteor1101  
       2019-04-22 17:26:00 +08:00
    没有 leader 吗
    miao1007
        23
    miao1007  
       2019-04-22 18:58:59 +08:00 via Android
    走吧
    specita
        24
    specita  
       2019-04-22 19:19:47 +08:00
    唉,我现在也很难受,搞过两次代码评审,基本就是要求按照阿里的代码规范来,也让去下了个提示插件,结果呢,还是那个样子,idea 已经提示黄色警告了也从来不会去看...沟通了几次,每次都是点头说好...结果一个屌样.....最后自己去改了,还被别人说代码被别人动了....难受得一逼
    Priest
        25
    Priest  
       2019-04-23 08:36:53 +08:00 via Android
    @specita 用工具来强化代码规范会
    Priest
        26
    Priest  
       2019-04-23 08:43:26 +08:00 via Android
    @specita 上条信息不小心就发了

    用工具来强化代码规范能省不少无意义的争论,可以使用 git 的 pre-commit hook 配合代码格式化工具(Prettier,Black)来做
    v2chou
        27
    v2chou  
       2019-04-23 10:05:32 +08:00
    没有一点代码洁癖
    Jackietsui
        28
    Jackietsui  
       2019-04-23 11:01:04 +08:00
    骑驴找马,公司没有管的你自己的力量根本不够
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   5775 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 27ms · UTC 03:27 · PVG 11:27 · LAX 19:27 · JFK 22:27
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.