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

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

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

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

18525 次点击
所在节点    程序员
141 条回复
tt67wq
2020-01-21 19:03:47 +08:00
@herich 如果代码正常运行,而且封装也没太大的问题,只是你个人感觉不好,那有问题的是你啊
katsusan
2020-01-21 19:07:05 +08:00
新建个 project 把他的私有库 fork 过去,依赖都改成指向这个公共 project 下的项目,可以防止删库跑路这样的行为 /狗头.
herich
2020-01-21 19:15:42 +08:00
@katsusan 他可能会改某些接口或者功能,fork 过去还要同步更新才行

@tt67wq 要是写在公司的仓库我没意见,拿公司钱为公司贡献代码嘛,这样大量引用自己私人仓库里的东西隐患挺大的
angith
2020-01-21 19:22:27 +08:00
那你一个人都包了呀
dapang1221
2020-01-21 19:23:14 +08:00
@zlyuancn 你同事吐槽你代码(试试有没有
herich
2020-01-21 19:25:22 +08:00
@angith 一言不合就重构吗,要是自己的编码风格同事不喜欢呢?互相重构吗🤣
katsusan
2020-01-21 19:27:26 +08:00
@herich 改的话就改这个公共仓库,在公司里的产出原则上属于公司,要不让他把 github 账号交出来也行。
wd
2020-01-21 19:28:55 +08:00
这明显你们公司的问题,在公司写的代码版权都是公司的,你们公司允许随意开源吗?
hyy1995
2020-01-21 19:32:06 +08:00
我看了下他仓库,一堆自己封装的东西,项目里面这么用,有够自恋。你们没技术总监?他这样用自己的库,那这个项目他走之后,谁敢接啊
xy90321
2020-01-21 19:32:19 +08:00
反感就去推倒重来,又懒得重来那就自己默默承受,很简单的道理
pursuer
2020-01-21 19:59:52 +08:00
我希望和我合作的人封装好(指 IDE 比较强大的项目且封装代码层次清晰命名准确的)有不懂的开着 IDE 一步步跟过去就明白了,封装得好的他就是驼峰下划线大括号换行不换行混用我都不介意(除非命名空间污染)。
herich
2020-01-21 20:39:35 +08:00
@wd 公司规模不大,这个项目一共就两三个人开发,还没有什么规范的

@hyy1995 这确实是个大问题,自己写的爽,别人接手火葬场
@xy90321 我觉得既然是一个共同开发的项目,虽然暂时没有文档规范,但是这公司的项目也写得太 personal 了吧,不能愉快合作呀
DJQTDJ
2020-01-21 20:41:50 +08:00
如果公司的库封到自己库里,那么律师函警告
UsherOu
2020-01-21 20:56:20 +08:00
要么重构,要么忍,风格这玩意很难改成统一的
Reficul
2020-01-21 20:58:10 +08:00
大不了 fork,删除又怎么样? 会封装总比不封装强,封装了你不爽就按照接口重构一个新的,replace 过去不就完了?
robertredd
2020-01-21 21:00:34 +08:00
俺要是老板 看懂了肯定要他改 这不是往公司代码植入其他依赖吗 特别不可靠
SnailLin
2020-01-21 21:02:16 +08:00
要我肯定就举报了,公司的代码使用自己的私有库,还公开在 GitHub 上,安全和维护都会有问题。
blless
2020-01-21 21:26:42 +08:00
vendor 啊 固化一下依赖,依赖直接跟着项目走就是了,github 删了你们照样可以用 go.mod replace 慢慢替换嘛
blless
2020-01-21 21:30:29 +08:00
不过说真的,你们这样基本上架构内部用法跟技术栈都可以猜个七七八八。虽然不见得是啥重要信息,不过谁知道哪天出了漏洞啥的。比如你们的 mgo,casbin 之类的
lasuar
2020-01-21 21:44:57 +08:00
做事方式有问题,这些库得需要维护吧?保证毫无 bug 吗?要维护再给他提 pr ?这不是搞些事情出来在做。
封装思想是 OK 的,但是应该直接封装在 codebase 里面。

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

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

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

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

© 2021 V2EX