如何应对一个项目中其他组员写的烂代码?

2020-02-27 08:48:13 +08:00
 enlight

实在看不下去了,思维毫无逻辑想到哪写到哪,代码不格式化,变量命名大小写毫无规则,model controller 混着写,各种不规范的方式导致整个代码质量降低很多经常出一些奇怪的 bug,但是碍于面子没有出 bug 的地方又不好意思改动别人写的代码。 大家有什么方法应对这种问题吗?

4277 次点击
所在节点    问与答
32 条回复
jinsongzhao
2020-02-27 11:41:42 +08:00
@danyi 你可以的,原来你就是那位组员! 23333
mcfog
2020-02-27 11:43:13 +08:00
@enlight 既然是给你加速度加来的人,那很简单啊,告诉你领导和甲方爸爸,加这个人反而速度变慢
jinsongzhao
2020-02-27 11:51:17 +08:00
因为代码即文档是最省力的开发模式,所以给他提供代码风格手册,好的风格大家都乐意接受,毕竟本人也要阅读,已经存在的代码重构它,也可以继续提升质量;本人不在了,那终归无法完美,只能改善,想想 github 上好项目的代码风格各式各样,而且不同语言风格混合开发也常见,读别人的代码提升自己也是必经之路。
charlie21
2020-02-27 12:03:33 +08:00
重写给他看,供他学习。求他
blessyou
2020-02-27 12:21:57 +08:00
当着他的面重写😝
wm5d8b
2020-02-27 12:49:17 +08:00
设计上解耦合,确保每一部分都能重构
wm5d8b
2020-02-27 12:50:55 +08:00
@charlie21 #24
@blessyou #25
这样不行,他会认为反正你会去重写,所以乱写好了
sampeng
2020-02-27 12:54:31 +08:00
外包项目跑起来就是赢啊
ianva
2020-02-27 14:41:31 +08:00
没有 code review 的机制,都是空谈
charlie21
2020-02-27 14:44:29 +08:00
@wm5d8b 职业道德会要求他去学习
mikulch
2020-02-27 16:21:34 +08:00
可能你没看过我这个代码。if 可以嵌套 5,6 层。一个方法几百行。
最牛逼的是写这个代码的人从我们公司走后竟然去了京东了。
yilingersier
2020-02-28 03:23:27 +08:00
(又不是不能用。。手动狗头

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

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

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

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

© 2021 V2EX