实在忍不住了,接手的代码太 shi 了

2019-03-15 15:56:27 +08:00
 zycpp
语言 C++

0、变量命名随意,驼峰、下划线混用。
1、函数、变量没注释,意思全靠猜
2、单个函数极长,四五百行的函数随处可见。
3、c style 和 c++ style 混用
4、源文件编码混乱,有 u8,u8+bom,gbk 都有
5、代码冗余,一样功能的函数重复实现好几次,只是换了个名字…
6、不明意义的数字满天飞,你猜我这里的 128 跟那边的 128 是不是同一个意思?

溜了溜了
8705 次点击
所在节点    程序员
60 条回复
l00t
2019-03-15 18:53:40 +08:00
能用就行。接手这种状态的代码也是正常情况,用不着惊讶。
Justin13
2019-03-15 18:57:13 +08:00
见过 1500 行的类,也见过 1800 行函数(数据引擎)
iceheart
2019-03-15 19:08:18 +08:00
这都小问题,到处是全局变量静态变量的才是真的有问题,头文件里静态变量声明才是问题
w01230
2019-03-15 19:32:43 +08:00
换个角度想想,有不少公司代码规范也是不同情况下命名规则不同。
anyele
2019-03-15 19:37:12 +08:00
这种代码不是正常操作?
guokeke
2019-03-15 19:41:19 +08:00
如果向老大吐槽,只会让老大觉得你不够专业。要么硬着头皮搞,要么走。
zmxnv123
2019-03-15 20:06:50 +08:00
有本书叫程序员的呐喊,提到当接手一个复杂的 c++项目时的解决方法..







看到 c++想都不用想,赶紧跑路就对了。
lepig
2019-03-15 20:26:18 +08:00
稍微截点图给兄弟们开开眼呐
AngryMagikarp
2019-03-15 21:16:17 +08:00
这个应该是好多代程序员的结果。
chinese_zmm
2019-03-15 21:19:10 +08:00
是维护还是添加新功能继续二次开发?维护建议不要改,针对 issue 解决就是
noe132
2019-03-15 21:30:51 +08:00
我见过 200 多行的 if 套 if,各种重叠 if,里面各种 magic number

溜了溜了
timetolo
2019-03-15 22:55:45 +08:00
感觉像是我刚写完的代码
timetolo
2019-03-15 22:57:08 +08:00
@noe132 我见过九级嵌套 if,并且还画流程图了
jim1002
2019-03-15 22:57:13 +08:00
理解烂代码也是一门技术
lhr_lovehoney
2019-03-15 23:01:03 +08:00
祖传代码 hhhh
xiaottt
2019-03-15 23:56:45 +08:00
传家包,不要乱动

if it works, don't fix it!
fvckDaybyte2
2019-03-15 23:57:03 +08:00
5000 行的 h 头文件路过……
harde
2019-03-15 23:59:14 +08:00
说点恶心的,我们去公共厕所,厕所里很多屎,气味也很臭,但是我们并不是来吃屎的,我们只要解决自己的,为屎山再贡献一点高度就可以了。。。

靠。。。我都写不下去了。。。
Lighfer
2019-03-16 00:02:07 +08:00
和我去年入职情况一样,画了一个月理清逻辑后像头儿申请重构了,可以看看这个片段:
https://www.zhihu.com/question/20402322/answer/389075475
ymcj001
2019-03-16 00:13:57 +08:00
@timetolo 这算是很良心的了。感谢他吧。

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

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

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

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

© 2021 V2EX