反感同事写的代码怎么办?

2020-01-21 17:44:51 +08:00
 herich

不谈技术好坏,就编码风格来说。和同事协同开发,同事特别喜欢搞一些封装,整个项目前期主要是他一个人撸的,项目中有不少他自己写(封装)的库,

本人有一些代码洁癖, 能用内置库和官方库解决问题的绝不使用第三方库尤其是个人的库,项目中处处都是 zxxx 这些代码,感觉被 qj,比较反感这种风格,怎么破?

19840 次点击
所在节点    程序员
141 条回复
ace12
2020-01-22 11:48:06 +08:00
粘包警犬立即出动
Chase2E
2020-01-22 11:53:41 +08:00
你这同事有点意思,刚毕业的吧?估计还没有接受过社会的毒打
herich
2020-01-22 12:05:13 +08:00
@DJQTDJ 那也可以说公司项目用自己的库呢?

@SnailLin 安全和后期维护的确是个大问题


@lbfeng 测试应该都差试过,目前能跑通,正常运行,以后随着他的这些库越来越多,估计只有他一个人看得懂项目代码了


@drackzy go 的生态不像 java,python 那么成熟,所以免不了要封装适合业务的模块,但这种一个功能就搞成一个开源仓库,还弄到自己的私人帐号下,项目到处是这种代码,合作起来比较头疼


@zzcworld go 的很多优秀的第三方包很多都是在个人的 github 仓库呀...
herich
2020-01-22 12:28:12 +08:00
@kwrush code view 是没有的,所以代码写得很随意,不过他有这习惯,之前写 python 也是这样的,pypi 上一大堆他的模块,docker hub 也有他的一堆封装的镜像(我是是在搞不懂为啥 docker 也要封装一下)


@littlebaozi 对齐这个格式化一下就好了,问题不大,而他的这种代码外人很难弄的


@mouyong go 语法比较简洁,加上 gofmt,代码格式没啥问题,就是不喜欢项目中这种夹杂很多私人仓库

@sdushn 组长领导不会关注这些,他们只关心业务进度
herich
2020-01-22 12:40:25 +08:00
@Chase2E 我才是刚毕业一两年的🤣,他说他写过五年 c#,后来行业不景气,就写 python 了,也就比我大两岁而已。之前写 python 就有这习惯,lxml 有 xpath,他说他自己写个解析引擎,写了个 zxpath,后来 zxpath 又不要了,写了个 zxpath2,现在 pypi 上还有这两个模块,不过很久没更新了,之前的 python 项目也有大量的这种他私人的模块,docker hub 上也有一堆他的 ubuntu,golang,python 镜像,我是真的无语╯﹏╰,所以来吐个槽。不过业务代码他能力是比我厉害不少的
ql562482472
2020-01-22 13:25:31 +08:00
的确很不妥 遇到这种应该全部重新封装 base 包在公司仓库里
dawn009
2020-01-22 13:42:05 +08:00
@zzcworld 这取决于许可证。使用得当没有任何问题。
8kFT2l6aoU9566Bg
2020-01-22 14:07:47 +08:00
@tyrealgray 其实上 go module 的库也是删不了的,goproxy 有缓存
iweus
2020-01-22 14:13:53 +08:00
看了一下他的库,有些几百行不到的代码也要整成一个库
SpencerCJH
2020-01-22 14:33:55 +08:00
肯定这个同事的想法和能力,但他做的这些事情增加了很大的 risk,道理其实大家都懂的.

看不惯就跑呗,年后面试去了[doge]
yulon
2020-01-22 14:35:29 +08:00
重构造轮子就算了,这种套皮有什么意思= =
tsui
2020-01-22 15:00:21 +08:00
这种反复重复造轮子是自己觉得写简历好看么?
挺搞笑的。。。
linZ
2020-01-22 15:14:30 +08:00
尽量少几个库。。。不然只要有人接手,就得重复写东西了,到时候一百个地方写同一个方法,保证教你做改动的时候改的欲仙欲死
vicvinc
2020-01-22 15:30:10 +08:00
zstrimg 是啥..string ?
herich
2020-01-22 15:57:56 +08:00
@tsui 这个写在简历里怕是减分项吧🤣,猛地一看以为是个大牛,仔细一看基本都是封装别人的库造些轮子
1nakaELYBbsXbZxY
2020-01-22 17:04:11 +08:00
先沟通一下吧
herich
2020-01-22 19:59:48 +08:00
@vicvinc 我以前点进去一看是个把图片转换成字符图的库,是把其他三方库的代码拿过来的改的,再封装成自己的库🌞
muller
2020-01-22 20:59:35 +08:00
1.首先整个技术组有没有 code review 机制,这个是解决代码冲突障碍的起步
2.CTO 或者高点小组长是否阅读过他的代码,是否支持,默认即支持
3.一般公司发展到一定阶段都有自己的封装集成,golang 系统包让你调用 当然也让你来集成,在 IBM ctrip 我们大部分包都是自己内部二次定制开发集成,来适应公司的业务,封装是减少重复代码量,把要写一千行代码的封装暴露在一个 api 方法,完全没毛病,而且鼓励
4.看别人集成代码自己不得不调用 感觉自己被 QJ,你认为你有代码洁癖,放在 leader 和 team 眼中可能会认为你缺少基本的协作精神
5.他到底代码比你厉害不厉害? beyond you 就服从认怂,talk is cheapest,show me the code !!
mengzhuo
2020-01-22 23:09:25 +08:00
公司开发的代码不能放外网啊,这妥妥开除+吃官司的。
learningman
2020-01-23 02:09:05 +08:00
@tt67wq php 怎么了,2020 年了还歧视 php,php 什么时候才能真正的站起来,冷抖哭

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/639736

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX