关于团队合作的问题！

[LeeChP]

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

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

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

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

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

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

头大！

[LeeChP]

而且有一点很操蛋，往 git 上提交了代码，从来不说，我的习惯是，早上 pull，下班 push，他经常提交代码，但是从来不说，没吃都得花时间 merge 他的提交，头疼

[40huo]

难道不是一个人一个分支？

[LeeChP]

@40huo 我也很纳闷，都是就是让我们提交到 dev。在上家公司就是一个人一个分支，由老大来把代码合并到主分支

[jackleelss123]

@LeeChP 让你们共同的老大来决定规则！

[LeeChP]

@jackleelss123 算俅，再苟一阵子就滚蛋

[jackleelss123]

@LeeChP 为啥你滚蛋？找你们技术负责人沟通啊！ id 命名没有一个共同的规则吗？我个人怎么方便怎么来嘛！
每个公司或多或少都有坑逼，碰到坑逼你就滚蛋？没有完美的企业，你往哪里滚呢！找你们技术负责人沟通一样，让他出面制定变量命名的规范啊！

[jybox]

没仔细看，如果他的改动会导致其他人的代码编译不过，那么连别人的代码一起改掉不是非常合理且必要的么？

我觉得你们缺少的可能是 PR Review 的机制，他可以改完发 PR，但要等其他涉及到的人 Review 再合并进主分支。

[halk]

@jybox +1

[ffeii]

@jybox +1

[LeeChP]

@jackleelss123 每天的例会提了很多次了呀。遇到坑比我不一定滚蛋啊，遇到坑比还装大佬的，我真的是懒得去争。

[oneisall8955]

改动影响到别人，提前说一句你改还是我该就 OK 了，都没啥问题呀。

[LeeChP]

@jybox 蛋疼的要死这种开发模式，我不可能一个个路径去翻，而且也是对方不遵守约定造成的，所以我提交后，立马群里通知了我改了哪个方法。有调用到的，自己门儿清。遵守约定，我也就没必要去改了。其实更坑爹的还有。两个不同实体硬写在一起。指出来后还在那争！

[LeeChP]

@oneisall8955 主要是方法很简单。但是他很不爽，要我把他调用的也给改过来，我哪知道他在哪里调用了？我自己一堆事儿呢。我懒得理他，最后他见我不鸟他，一副大佬的样子在群里发话。
这货把两个实体类合并的时候，我写的大部分作废，让我去适配他的做法，我没鸟他，不然后面改回来我得暴走！

[LeeChP]

@jackleelss123 老大昨天就说了，拆他合并的实体，唯一标志用 long，今天看他还是用字符串，我是懵逼家不爽的

[jackleelss123]

@LeeChP 你让老大来出面处理，你不用理他！

[LeeChP]

@jackleelss123 也是，反正我做好我的模块就行了。

[scipio]

作为提供服务方不能尽力保证接口的向下兼容性，在某种程度上是缺乏职业道德的体现。

不过据我观察，国内相当多的程序员都缺乏这种意识，即便是大厂的码农也同样如此。
反倒是外企的程序员相对会更注重这一块。

[yk88king]

给份简历看看 [e-mail]