关于团队合作的问题! - V2EX
LeeChP

关于团队合作的问题!

  •  
  •   LeeChP Sep 26, 2019 via iPhone 3241 views
    This topic created in 2423 days ago, the information mentioned may be changed or developed.

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

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

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

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

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

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

    头大!

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

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

    不过据我观察,国内相当多的程序员都缺乏这种意识,即便是大厂的码农也同样如此。
    反倒是外企的程序员相对会更注重这一块。
    yk88king
        18
    yk88king  
       Oct 25, 2019
    给份简历看看 [email protected]
    About     Help     Advertise     Blog     API     FAQ     Solana     933 Online   Highest 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 80ms UTC 20:01 PVG 04:01 LAX 13:01 JFK 16:01
    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