你们修改方法的时候,会把其他人的调用到该方法的代码也一并修改掉吗?
很蛋疼的一件事,id 使用 long,但是有个人,非要用字符串,理由就是他从 redis 获取值的时候,需要使用字符串来作为 key,我一脸懵逼,不能转吗?
昨天确定用 long 以后,他今天在我开发的那个类里,添加了个方法,id 依旧用字符串,并且有个需要扩展的参数,他没加,直接写死。我就改了 id 类型,添加了需要的参数,提醒他我改了他的方法,我提醒他的目的还有一个,就是让他把他写的模块有调用到该方法的,都去改一下。他当时没回我!
高潮来了,下午他 pull 了后,没改,一片红,然后怪我没把他的其他模块也改掉,我根本不想理他。他就在群里发消息,意思就是提醒我们改了某个地方,就把用到那个方法的其他地方,也全部改了,不管是不是改的人写的。
说真的,我是第一次遇见这种要求。之前都是有改动,提交了代码,都在组里通知一声,大家都是各改各的。而且蛋疼的是,这哥们还在我开发的类里添东西,我是看不懂这个操作的。
前几天也是,我在改 A 类,他要修改里面的几个东西,我说你不用管,我来添加就好,免得同时操作一个文件,会出现冲突,结果他不管,还是改了,又浪费了时间去 merge。
头大!
1
LeeChP OP 而且有一点很操蛋,往 git 上提交了代码,从来不说,我的习惯是,早上 pull,下班 push,他经常提交代码,但是从来不说,没吃都得花时间 merge 他的提交,头疼
|
2
40huo 2019-09-26 17:14:56 +08:00 1
难道不是一个人一个分支?
|
3
LeeChP OP @40huo 我也很纳闷,都是就是让我们提交到 dev。在上家公司就是一个人一个分支,由老大来把代码合并到主分支
|
4
jackleelss123 2019-09-26 17:54:58 +08:00
@LeeChP 让你们共同的老大来决定规则!
|
5
LeeChP OP @jackleelss123 算俅,再苟一阵子就滚蛋
|
6
jackleelss123 2019-09-26 18:04:53 +08:00
@LeeChP 为啥你滚蛋?找你们技术负责人沟通啊! id 命名没有一个共同的规则吗?我个人怎么方便怎么来嘛!
每个公司或多或少都有坑逼,碰到坑逼你就滚蛋?没有完美的企业,你往哪里滚呢!找你们技术负责人沟通一样,让他出面制定变量命名的规范啊! |
7
jybox 2019-09-26 18:13:10 +08:00
没仔细看,如果他的改动会导致其他人的代码编译不过,那么连别人的代码一起改掉不是非常合理且必要的么?
我觉得你们缺少的可能是 PR Review 的机制,他可以改完发 PR,但要等其他涉及到的人 Review 再合并进主分支。 |
10
LeeChP OP @jackleelss123 每天的例会提了很多次了呀。遇到坑比我不一定滚蛋啊,遇到坑比还装大佬的,我真的是懒得去争。
|
11
oneisall8955 2019-09-26 19:21:25 +08:00 via Android
改动影响到别人,提前说一句你改还是我该就 OK 了,都没啥问题呀。
|
12
LeeChP OP @jybox 蛋疼的要死这种开发模式,我不可能一个个路径去翻,而且也是对方不遵守约定造成的,所以我提交后,立马群里通知了我改了哪个方法。有调用到的,自己门儿清。遵守约定,我也就没必要去改了。其实更坑爹的还有。两个不同实体硬写在一起。指出来后还在那争!
|
13
LeeChP OP @oneisall8955 主要是方法很简单。但是他很不爽,要我把他调用的也给改过来,我哪知道他在哪里调用了?我自己一堆事儿呢。我懒得理他,最后他见我不鸟他,一副大佬的样子在群里发话。
这货把两个实体类合并的时候,我写的大部分作废,让我去适配他的做法,我没鸟他,不然后面改回来我得暴走! |
14
LeeChP OP @jackleelss123 老大昨天就说了,拆他合并的实体,唯一标志用 long,今天看他还是用字符串,我是懵逼家不爽的
|
15
jackleelss123 2019-09-26 19:41:22 +08:00
@LeeChP 你让老大来出面处理,你不用理他!
|
16
LeeChP OP @jackleelss123 也是,反正我做好我的模块就行了。
|
17
scipio 2019-09-27 10:02:24 +08:00
作为提供服务方不能尽力保证接口的向下兼容性,在某种程度上是缺乏职业道德的体现。
不过据我观察,国内相当多的程序员都缺乏这种意识,即便是大厂的码农也同样如此。 反倒是外企的程序员相对会更注重这一块。 |
18
yk88king 2019-10-25 13:19:00 +08:00
给份简历看看 [email protected]
|