请问如何说服工作 5、6 年的同事不要把两三千行的逻辑写在一个类里?

2016-04-21 01:14:56 +08:00
 qingchengdev

我毕业快一年了。和另外一个搭档同事都是最近刚入职的新人。之前偶尔代码 Review 的时候发现他很喜欢把逻辑都写在同一个类里,不过注释用中文写得很详尽,所以还是能读的,但我觉得这样不太好,而且以后我们两个可以说是互为相依为命的角色,所以一直想找个机会告诉他。 所以趁着这个一个新的模块开始的时候,对模块的整体结构提了一些建议,整个模块可能两三千行代码的规模,建议划分成 4~5 个类来实现,他也同意这样划分在逻辑上是很清晰的,但觉得这样太细了,可能有些过度设计了。他最后坚持要写在一个类里,觉得代码分散在四五个文件里面可以维护,集中在一个文件里面也是可以维护的。我听到这样的想法突然觉得无力反驳,于是作罢了。 另外,我们的项目有充裕时间。

事后我也有反省自己,我是不是控制欲太强,太喜欢指手画脚了呢?虽然我心里确实有倾向,也有答案。但还是想来听听前辈们怎么说。

4806 次点击
所在节点    职场话题
27 条回复
karlakte
2016-04-21 12:49:56 +08:00
他软的源码 一个文件超过 1 万行的都有。关键看文件内代码的模块功能 /业务相关性。允许拆开,当然是拆开好咯。
harry890829
2016-04-21 12:51:34 +08:00
我刚进新公司,正在看项目代码,纯 c 的代码,文件近 2w 行,有一个函数就是 7.5k 行
jy01264313
2016-04-21 14:20:55 +08:00
打车求教,如果说服一个工作 10+ 的 J2EE 人不用要 tab 换行?总有一些 python 代码被他无意间碰到,最后不能运行了。
loryyang
2016-04-21 14:23:30 +08:00
直接说,难!
这种事情可以尝试走这个路线,私下找技术负责人,提出这个建议,然后促使技术负责人立下规定,所有组员都要遵守,你就可以拿这个当令箭了
xdazz
2016-04-21 14:44:16 +08:00
制定編碼規範,強制執行的話可以在中央倉庫設置 hook 檢查,不符合規範的直接 reject 掉不讓提交。好好溝通。
Tneciv
2016-04-21 18:30:41 +08:00
@jy01264313 悄悄改下他的 ide 设置就好了
jy01264313
2016-04-22 09:31:57 +08:00
@Tneciv 我现在的解决方案是,看见 J2EE 的人就直接 PASS

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

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

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

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

© 2021 V2EX