z****e 发帖数: 54598 | 1 写出低质量的代码的特征包括但不限于:
心安理得地写出低质量的代码
最终写出了低质量代码却不在之后改善它
自以为写出了质量很高但实际质量很低的代码
低质量的特征包括但不限于:
文件关系混乱
注释过期、不明确或者没有
文档过期、不明确或者没有
架构乱设计
过度设计
不检查用户输入的错误情况
不检查API或者函数返回的errorcode或者exception
没有单元测试等自动化测试过程
编译起来很难
到处复制代码,公用的部分不整理成内部库
著作权归作者所有。
商业转载请联系作者获得授权,非商业转载请注明出处。
作者:匿名用户
链接:http://www.zhihu.com/question/24665029/answer/28567915
来源:知乎
我很久很久以前曾经经历的一个公司特别傻比,人力资源和svn管理员不懂程序。作为
服务端的主程序,一直以来都是代码精简,效率最优,没有冗余为毕生追求。突然我无
意中发现了一个秘密,我的绩效居然不如几个公司刚招进来的新手。我每天刻苦努力,
优化代码,让公司的服务端7*24稳稳跑在服务器上,还一心研究各种热切换,换来绩效
考核好久没有优秀了。这究竟是为什么呢,后来好心人告诉我,人力资源说了,他们看
了svn的日志记录,其他人工作很努力,每天提交很多代码,每天新编写很多很多代码
文件,贡献了好多好多的劳动成果。而看看你这个号称技术排第一的家伙,根本就没干
活嘛,3个月下来,你一共就写了这么十来个cpp和.h,每天改来改去就这么几个文件,
还好多天差不多行数的位置回改,根本就一直在偷懒。我听完以后以后直接口吐鲜血,
无语至极了。然后我就申请离职了,在那个公司剩下不多的日子了,我为了心安理得的
拿最后2个月的工资,为了让老板知道其实我一直很努力,我在最后一个月更新了几千
个文件,让服务端的内核稳定度下降了好几个档次,内存消耗上涨了好多个数量级,并
且把容易读,容易后续扩展的部分,全部用c++最高级的模版进行扩展,同时删掉了我
以前用汇编写的优化代码。嗯,我想以后这个公司估计会倒闭,我走了以后,再也没有
人能维护这份代码了,估计3个月以后,我自己也搞不定了。然后我就离职了,离职之
前老板大夸我厚道,最后一个月还给公司做了这么多的事情,别人走都是删代码,我居
然还毫无保留的为公司做出贡献。然后我走了不到半年,公司就呵呵了。我想那是我此
生唯一写垃圾代码写的心安理得的一次机会了。 |
d******e 发帖数: 2265 | 2 微软谷歌都是这样。
数bugs数目。那么升级的简单方法就是狂checkin.在疯狂的修bug.
【在 z****e 的大作中提到】 : 写出低质量的代码的特征包括但不限于: : 心安理得地写出低质量的代码 : 最终写出了低质量代码却不在之后改善它 : 自以为写出了质量很高但实际质量很低的代码 : 低质量的特征包括但不限于: : 文件关系混乱 : 注释过期、不明确或者没有 : 文档过期、不明确或者没有 : 架构乱设计 : 过度设计
|
g*****g 发帖数: 34805 | 3 有 严格的 code review, 你想 checkin 一堆垃圾也得有垃圾队友才行。当然以行数论
贡献是不科学。
【在 d******e 的大作中提到】 : 微软谷歌都是这样。 : 数bugs数目。那么升级的简单方法就是狂checkin.在疯狂的修bug.
|
c******n 发帖数: 16666 | 4 看到人说淘宝那边
有人进去优化了代码 结果发现人家都是增加行数 他是减少行数 被鄙视了
【在 d******e 的大作中提到】 : 微软谷歌都是这样。 : 数bugs数目。那么升级的简单方法就是狂checkin.在疯狂的修bug.
|
d******e 发帖数: 2265 | 5 code review哪里有那么神。
你丢一万行代码过去大家也就是随便看看。
【在 g*****g 的大作中提到】 : 有 严格的 code review, 你想 checkin 一堆垃圾也得有垃圾队友才行。当然以行数论 : 贡献是不科学。
|
g*****g 发帖数: 34805 | 6 不关我屁事的代码我是不会认真看的,日后我可能得擦屁股的还是得认真看。你给我来
一万行我直接打回去让分着 PR.
【在 d******e 的大作中提到】 : code review哪里有那么神。 : 你丢一万行代码过去大家也就是随便看看。
|
d******e 发帖数: 2265 | 7 我相信淘宝,ebay, amzn这种电商的软件都是一滩屎。
做系统软件的严谨点。
就这也架不住改bug时copy paste.
【在 c******n 的大作中提到】 : 看到人说淘宝那边 : 有人进去优化了代码 结果发现人家都是增加行数 他是减少行数 被鄙视了
|
d******e 发帖数: 2265 | 8 扯淡不是这么扯的。软家一堆65,67你敢让人回去?
不要混了。另外,多少大feature都是一个branch,做完测好,提交一个cr.
【在 g*****g 的大作中提到】 : 不关我屁事的代码我是不会认真看的,日后我可能得擦屁股的还是得认真看。你给我来 : 一万行我直接打回去让分着 PR.
|
g*****g 发帖数: 34805 | 9 我老不混啥软家这种垃圾公司。技术上的东西就事论事,从来没人敢拿资历压我。
【在 d******e 的大作中提到】 : 扯淡不是这么扯的。软家一堆65,67你敢让人回去? : 不要混了。另外,多少大feature都是一个branch,做完测好,提交一个cr.
|
p**r 发帖数: 5853 | 10 我说实话,我一直都心安理得的写低质量代码,
有时候我看看我自己写的,真是觉得狗屎都不如。
但是没办法,单位时间出货,能赚钱是最重要的。
看看公司那些其他部门的那些逼人,
一个10分钟就可以解决的问题,
非要飞来飞去的,浪费10来个人的机票酒店+薪水扯蛋研究,
你写再低质量的代码你也就心安理得了。 |
|
|
S*******e 发帖数: 525 | 11 同感!现在在的一个小组里(咱公司本身很大),我们从不做codereview,代码咱随便
写,别人的代码经理让咱把关,咱也一目十行的看几眼。
经理想帮咱做校长,director也帮咱写材料,他娘X的,这还得SVP亲自把关,VP同意都
没用。SVP昨天有个townhall,听了咱就死了这心了。想想,就是“你写再低质量的代
码你也就心安理得了”。
【在 p**r 的大作中提到】 : 我说实话,我一直都心安理得的写低质量代码, : 有时候我看看我自己写的,真是觉得狗屎都不如。 : 但是没办法,单位时间出货,能赚钱是最重要的。 : 看看公司那些其他部门的那些逼人, : 一个10分钟就可以解决的问题, : 非要飞来飞去的,浪费10来个人的机票酒店+薪水扯蛋研究, : 你写再低质量的代码你也就心安理得了。
|
z*y 发帖数: 1311 | 12 low pay
no salary increase
no promotion
not my company
... |
c*********e 发帖数: 16335 | 13 你真以为程序员傻啊?如果公司不给高工资,还要你天天出活,你会怎么想? 一个公
司,管理层如果瞎搞,轻视程序员,那程序员还从心里尊敬管理层?尊重是相互的,没
人是傻子。
【在 z****e 的大作中提到】 : 写出低质量的代码的特征包括但不限于: : 心安理得地写出低质量的代码 : 最终写出了低质量代码却不在之后改善它 : 自以为写出了质量很高但实际质量很低的代码 : 低质量的特征包括但不限于: : 文件关系混乱 : 注释过期、不明确或者没有 : 文档过期、不明确或者没有 : 架构乱设计 : 过度设计
|
ET 发帖数: 10701 | 14 读过这个。说淘宝hr就这么考核的。
【在 c******n 的大作中提到】 : 看到人说淘宝那边 : 有人进去优化了代码 结果发现人家都是增加行数 他是减少行数 被鄙视了
|
a*****g 发帖数: 19398 | 15 唉。
【在 z****e 的大作中提到】 : 写出低质量的代码的特征包括但不限于: : 心安理得地写出低质量的代码 : 最终写出了低质量代码却不在之后改善它 : 自以为写出了质量很高但实际质量很低的代码 : 低质量的特征包括但不限于: : 文件关系混乱 : 注释过期、不明确或者没有 : 文档过期、不明确或者没有 : 架构乱设计 : 过度设计
|