实在忍不住了,接手的代码太 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 是不是同一个意思?

溜了溜了
8688 次点击
所在节点    程序员
60 条回复
hellowmykami
2019-03-15 16:46:27 +08:00
打扰了告辞
mokain
2019-03-15 17:18:51 +08:00
花一个月跟头儿说重构代码吧!
zycpp
2019-03-15 17:22:08 +08:00
@mokain 有一部分就是头儿写的😩
ifxo
2019-03-15 17:24:45 +08:00
只要结果,不管过程
yoke123
2019-03-15 17:25:02 +08:00
爽啊
javaWeber
2019-03-15 17:25:49 +08:00
我刚进某公司,唉。。
变量名有拼音,拼音首字母缩写。
方法名有的大写,甚至还有下划线。
有的函数居然长达 400 行。
虽然代码很不规范,但是我没有其他更好的公司选择。
我之前接触的都是小项目。现在做的项目大了点,有十多个服务模块。
有点担心需求没法及时完成 。
learnshare
2019-03-15 17:27:14 +08:00
正常的,而且领导通常不让你优化
archxm
2019-03-15 17:27:30 +08:00
你行的
finab
2019-03-15 17:28:09 +08:00
不知道为什么,感觉你说的这些问题还不算太 shi ,我完全扛得住
cookbook
2019-03-15 17:29:54 +08:00
处理这种代码是我的强项,梳理的过程很爽
TingHaiJamiE
2019-03-15 17:32:46 +08:00
“能跑的线上代码不要动”
Kirscheis
2019-03-15 17:41:38 +08:00
小问题。。去年刚参与了某行业一个著名软件的开发,里面上千行的函数随处可见,还有各种早期 C,Fortran 代码的残留。。照样可以硬着头皮写下去。。。
ily433664
2019-03-15 17:42:10 +08:00
应该是很多人接手过
983081882
2019-03-15 17:42:15 +08:00
我也差不多 把老代码 Java 和 C++都有 翻译成 Java 还不能随心所欲 老代码日志随便打 111 222 333 随处可见 魔法值没注释 简直头大 最可气的看着老代码有的的地有逻辑问题 这是翻译还是不翻译 好挣扎
ODD10
2019-03-15 17:42:27 +08:00
@zycpp #3 妙哉妙哉
tcdh
2019-03-15 17:44:45 +08:00
曾经接受过一个内核模块,两千多个全局变量,一个函数最长 3000 行。花了 6 个月才改完。
across
2019-03-15 17:56:20 +08:00
不是封装太差我就扛得住,上面几个大部分都是风格问题。
算法或者 MagicNumber,如果没有点注释说明倒是读起来比较累····
jmc891205
2019-03-15 18:18:40 +08:00
是 c++的项目的话可以理解
应该有一段不短的历史了
说不定还是由一段古老的纯 c 代码发展而来
Leigg
2019-03-15 18:32:21 +08:00
别乱动,出问题了跑不脱
dinjufen
2019-03-15 18:50:36 +08:00
可能是很多人接手过的,都不想改罢了

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

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

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

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

© 2021 V2EX