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

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

  •  
  •   victorwu34 · 232 天前 · 4089 次点击
    这是一个创建于 232 天前的主题,其中的信息可能已经有所发展或是发生改变。

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

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

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

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

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

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

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

    用工具来强化代码规范能省不少无意义的争论,可以使用 git 的 pre-commit hook 配合代码格式化工具(Prettier,Black)来做
        27
    v2chou   230 天前
    没有一点代码洁癖
        28
    Jackietsui   230 天前
    骑驴找马,公司没有管的你自己的力量根本不够
    关于   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   962 人在线   最高记录 5043   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.3 · 24ms · UTC 22:33 · PVG 06:33 · LAX 14:33 · JFK 17:33
    ♥ Do have faith in what you're doing.