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

关于团队合作的问题!

  •  
  •   LeeChP · 2019-09-26 17:10:47 +08:00 via iPhone · 2551 次点击
    这是一个创建于 1888 天前的主题,其中的信息可能已经有所发展或是发生改变。

    你们修改方法的时候,会把其他人的调用到该方法的代码也一并修改掉吗?

    很蛋疼的一件事,id 使用 long,但是有个人,非要用字符串,理由就是他从 redis 获取值的时候,需要使用字符串来作为 key,我一脸懵逼,不能转吗?

    昨天确定用 long 以后,他今天在我开发的那个类里,添加了个方法,id 依旧用字符串,并且有个需要扩展的参数,他没加,直接写死。我就改了 id 类型,添加了需要的参数,提醒他我改了他的方法,我提醒他的目的还有一个,就是让他把他写的模块有调用到该方法的,都去改一下。他当时没回我!

    高潮来了,下午他 pull 了后,没改,一片红,然后怪我没把他的其他模块也改掉,我根本不想理他。他就在群里发消息,意思就是提醒我们改了某个地方,就把用到那个方法的其他地方,也全部改了,不管是不是改的人写的。

    说真的,我是第一次遇见这种要求。之前都是有改动,提交了代码,都在组里通知一声,大家都是各改各的。而且蛋疼的是,这哥们还在我开发的类里添东西,我是看不懂这个操作的。

    前几天也是,我在改 A 类,他要修改里面的几个东西,我说你不用管,我来添加就好,免得同时操作一个文件,会出现冲突,结果他不管,还是改了,又浪费了时间去 merge。

    头大!

    18 条回复    2019-10-25 13:19:00 +08:00
    LeeChP
        1
    LeeChP  
    OP
       2019-09-26 17:13:26 +08:00 via iPhone
    而且有一点很操蛋,往 git 上提交了代码,从来不说,我的习惯是,早上 pull,下班 push,他经常提交代码,但是从来不说,没吃都得花时间 merge 他的提交,头疼
    40huo
        2
    40huo  
       2019-09-26 17:14:56 +08:00   ❤️ 1
    难道不是一个人一个分支?
    LeeChP
        3
    LeeChP  
    OP
       2019-09-26 17:19:25 +08:00 via iPhone
    @40huo 我也很纳闷,都是就是让我们提交到 dev。在上家公司就是一个人一个分支,由老大来把代码合并到主分支
    jackleelss123
        4
    jackleelss123  
       2019-09-26 17:54:58 +08:00
    @LeeChP 让你们共同的老大来决定规则!
    LeeChP
        5
    LeeChP  
    OP
       2019-09-26 17:58:22 +08:00 via iPhone
    @jackleelss123 算俅,再苟一阵子就滚蛋
    jackleelss123
        6
    jackleelss123  
       2019-09-26 18:04:53 +08:00
    @LeeChP 为啥你滚蛋?找你们技术负责人沟通啊! id 命名没有一个共同的规则吗?我个人怎么方便怎么来嘛!
    每个公司或多或少都有坑逼,碰到坑逼你就滚蛋?没有完美的企业,你往哪里滚呢!找你们技术负责人沟通一样,让他出面制定变量命名的规范啊!
    jybox
        7
    jybox  
       2019-09-26 18:13:10 +08:00
    没仔细看,如果他的改动会导致其他人的代码编译不过,那么连别人的代码一起改掉不是非常合理且必要的么?

    我觉得你们缺少的可能是 PR Review 的机制,他可以改完发 PR,但要等其他涉及到的人 Review 再合并进主分支。
    halk
        8
    halk  
       2019-09-26 18:57:44 +08:00
    @jybox +1
    ffeii
        9
    ffeii  
       2019-09-26 19:00:55 +08:00 via iPhone
    @jybox +1
    LeeChP
        10
    LeeChP  
    OP
       2019-09-26 19:17:12 +08:00 via iPhone
    @jackleelss123 每天的例会提了很多次了呀。遇到坑比我不一定滚蛋啊,遇到坑比还装大佬的,我真的是懒得去争。
    oneisall8955
        11
    oneisall8955  
       2019-09-26 19:21:25 +08:00 via Android
    改动影响到别人,提前说一句你改还是我该就 OK 了,都没啥问题呀。
    LeeChP
        12
    LeeChP  
    OP
       2019-09-26 19:21:25 +08:00 via iPhone
    @jybox 蛋疼的要死这种开发模式,我不可能一个个路径去翻,而且也是对方不遵守约定造成的,所以我提交后,立马群里通知了我改了哪个方法。有调用到的,自己门儿清。遵守约定,我也就没必要去改了。其实更坑爹的还有。两个不同实体硬写在一起。指出来后还在那争!
    LeeChP
        13
    LeeChP  
    OP
       2019-09-26 19:26:19 +08:00 via iPhone
    @oneisall8955 主要是方法很简单。但是他很不爽,要我把他调用的也给改过来,我哪知道他在哪里调用了?我自己一堆事儿呢。我懒得理他,最后他见我不鸟他,一副大佬的样子在群里发话。
    这货把两个实体类合并的时候,我写的大部分作废,让我去适配他的做法,我没鸟他,不然后面改回来我得暴走!
    LeeChP
        14
    LeeChP  
    OP
       2019-09-26 19:28:51 +08:00 via iPhone
    @jackleelss123 老大昨天就说了,拆他合并的实体,唯一标志用 long,今天看他还是用字符串,我是懵逼家不爽的
    jackleelss123
        15
    jackleelss123  
       2019-09-26 19:41:22 +08:00
    @LeeChP 你让老大来出面处理,你不用理他!
    LeeChP
        16
    LeeChP  
    OP
       2019-09-26 19:58:10 +08:00 via iPhone
    @jackleelss123 也是,反正我做好我的模块就行了。
    scipio
        17
    scipio  
       2019-09-27 10:02:24 +08:00
    作为提供服务方不能尽力保证接口的向下兼容性,在某种程度上是缺乏职业道德的体现。

    不过据我观察,国内相当多的程序员都缺乏这种意识,即便是大厂的码农也同样如此。
    反倒是外企的程序员相对会更注重这一块。
    yk88king
        18
    yk88king  
       2019-10-25 13:19:00 +08:00
    给份简历看看 [email protected]
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   3524 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 22ms · UTC 04:56 · PVG 12:56 · LAX 20:56 · JFK 23:56
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.