大公司的核心项目代码也不是那么美好(c++)

2021-02-20 10:03:29 +08:00
 whi147

不同的页面,相似的功能,没有抽象全是复制粘贴。想改成模版元编程或者二级指针抽象,发现又不是完全复制,都是把结构体换了个名字复制,二十几个文件顿时丧失优化兴趣。反正能跑就跑算了

12245 次点击
所在节点    程序员
94 条回复
sillydaddy
2021-02-20 13:53:09 +08:00
成本大于收益,所以不抽象。

可能抽象后的代码扩展性不强。
可能别人理解起来会困难。
可能抽象所需时间比较长。
可能这些代码仅仅是局部的,无关全局。

即使是涉及到架构方面的代码,还有一个权衡利弊,判断是否要重构的过程。何况其他的代码呢!
TimRChen
2021-02-20 13:59:24 +08:00
@bthulu 什么都不做肯定是没问题的,敢于尝试的都是错的。doge
atonku
2021-02-20 13:59:32 +08:00
大公司的核心业务又不是优化代码!他们首先定一个高工资,让小公司的社畜不停的骚动,然后制定一堆自己都不执行的乱七八糟的标准,去拖小公司的后退。这就是核心业务
paradoxs
2021-02-20 14:03:35 +08:00
上次公司来了个新人,写的代码,一些基础的东西封装的挺好的。

我秒懂了,这人是培训班出来的。

真的自学科班出身,哪有这样封装的,都是能用就行。
dwSun
2021-02-20 14:05:00 +08:00
@atonku #43 这就是 google 在干的事情,所以我一直觉得要远离 google 的产品
uselessVisitor
2021-02-20 14:06:58 +08:00
不写第三方框架很少用到抽象封装或者设计模式吧。。最近学了一些,感觉都很抽象
zhigang1992
2021-02-20 14:07:32 +08:00
quceng
2021-02-20 14:26:36 +08:00
哪个大公司啊?
lakehylia
2021-02-20 14:47:57 +08:00
首先完成需求,拿到 KPI 。然后再重构,缩小代码大小,又拿到 KPI 。接着继续完成需求,循环往复。有 KPI 就有重构得动力,没有就没人愿意动。
firefox12
2021-02-20 14:59:20 +08:00
XML 倒是抽象 复杂,你们怎么都去用 json ?口嫌体正直。

每一座屎山都是一段历史,没有经历过的人 真的没资格去评论。
chendl111
2021-02-20 15:15:06 +08:00
能用就行.jpg
whi147
2021-02-20 15:23:42 +08:00
@atonku 厉害
whi147
2021-02-20 15:24:43 +08:00
@quceng 安防行业
coolesting
2021-02-20 15:42:33 +08:00
能运行就不重构,否则,你会无限地加班 !!
shm7
2021-02-20 15:44:33 +08:00
"没有抽象全是复制粘贴"
相信我,有时候完全以 DRY 为代码规范来写的,一样有阅读的问题。

“不同的页面,相似的功能” 假如页面变化较大,一个页面要改一个 小组件来实现,一个不要改怎么搞?

也许这么写是比较好读又好改的折中方案呢。
zjl03505
2021-02-20 15:46:16 +08:00
@whi147 #53 你说到安防行业,行业领头的那两三个确实远远比不上一般的互联网大公司。
whi147
2021-02-20 15:48:11 +08:00
@zjl03505 对啊,写的也不是很舒服
mapoor
2021-02-20 15:53:26 +08:00
如果核心代码抽象做的不好,那依托以此的产品可想而知,必定没有任何竞争力。
你说的核心代码真的是核心吗?
Avedge
2021-02-20 15:56:43 +08:00
毕竟很多历史因素,改改说不定就扯出一堆新问题。
whi147
2021-02-20 15:58:53 +08:00
@mapoor 这个项目是多项目结构,也就是有 100 多个 dll,我就在其中一两个里写,有的代码封装的很好,有些代码不是

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

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

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

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

© 2021 V2EX