资深同事写的代码不够优化, 有拖后腿的潜力, 怎么说比较合适?

2014-07-15 10:28:51 +08:00
 raynix
8772 次点击
所在节点    程序员
66 条回复
williamx
2014-07-15 16:31:38 +08:00
刚入行的时候写代码性能是第一优先级;不知道什么时候开始性能成了最后考虑的问题了。话说现在时间都花在了诸如:这个名字应该叫什么、这行代码放在这个函数是否合适、这个文件应该放在哪个目录下,这样的事情上了。
jp5490
2014-07-15 16:51:26 +08:00
@williamx 取名字什么 最讨厌了。。。。。。
Pixeller
2014-07-15 16:56:13 +08:00
依项目而定, 同时估计下优化能得到的效率或损失, 如果已达到n2增长的bug级别 最好还是趁早说, 如果不是 请别钻牛角尖-.-!, 关键是要求好自己, 对以后自己的项目和团队都有好处.
最后我只能说大部分不关心效率的80%的时间其实都是在写ui和相关逻辑, 优化待到项目完成才进行的我会说请不要搞笑么? 数w行的小项目, 看uml都能头大别说看代码.
zhangdawei
2014-07-15 17:02:39 +08:00
性能优化是迭代出来的,
可维护性优先级更高吧。
guoxx_
2014-07-15 17:42:44 +08:00
不是所有的烂代码都能用后期优化当借口的。
以我自己的经验来说,一个有可能比较长的有序数组的插入操作,如果实现成n的复杂度,那是不能接受的,实现成对数复杂度才可以(手机回复,不方便打公式)。这类代码属于写的时候就该想到的事情。
还有关于后期优化,我见过的代码里就真有我上面说的问题的,还是一个很著名的产品。作为library或者framework的提供者,你可能没有机会对你的代码进行详尽的优化,所以建议写的时候多留点心。
另外针对楼主的问题,@Livid的做法很好,条件不允许的话code review也是一个很好的办法。
raynix
2014-07-15 17:46:22 +08:00
@guoxx_ 所言极是.
a591826944
2014-07-15 17:47:56 +08:00
@williamx 这叫架构。。兄弟你上道儿啦
cmingxu
2014-07-15 17:53:40 +08:00
先活下来再考虑性能优化, 代码简洁吧;

写代码重在团队协作; 还有一点就是尽量不要给项目内引入只有少数人熟悉的技术, 语言; 这绝逼是万恶之源。
cmingxu
2014-07-15 17:55:09 +08:00
@williamx 严重同意
vjnjc
2014-07-15 23:17:51 +08:00
@Livid Hello,我看到你提及APM,请问你们使用什么监控软件的APM的啊?
vjnjc
2014-07-15 23:19:24 +08:00
关于楼主的问题我也考虑过,性能问题不是最优先的,我感觉代码的可阅读性很重要。毕竟机器性能和人力比起来我感觉人力比较宝贵。
akira
2014-07-15 23:25:30 +08:00
既然是资深了,他自己心里也应该知道什么需要优化 什么不需要优化了吧
em70
2014-07-15 23:58:44 +08:00
@lazyphp 99%的互联网项目死掉都是因为只有100个用户的时候去考虑一亿用户的事情,快速迭代,先上线跑,不断改进才是正道,QQ就经历过4次大的重构,早期就一个服务器跑,然后两个服务器数据和逻辑分离,然后再上集群,如果马化腾2000年就做个上亿用户的构架,QQ早就因资金链断裂死了
zjgood
2014-07-16 00:16:27 +08:00
图样图森破。
飞鸟尽,良弓藏。
mikuazusa
2014-07-16 00:17:55 +08:00
让测试人员去证明质量就行了。
mikuazusa
2014-07-16 00:19:56 +08:00
@Livid 我们部门已经把这个“APM系统”放到小米电视上去挂在每个部门的办公区显眼处让每个人出入都可以看到最实时的数据。确实很不错。
raynix
2014-07-16 07:53:27 +08:00
@em70 代码优化恰好能延迟硬件方面的扩张吧.
hekailiang
2014-07-16 08:03:11 +08:00
可以考虑先设计benchmark test,先拿到性能测试的基准数据。
2ex
2014-07-16 08:45:16 +08:00
我们这儿资深的都是不优化代码的,用最简单的方式方法完成一个项目,达到需求方的要求就可以了。
时间紧,项目多,如果每个项目都力求完美,那就是给自己挖了个坑。
cai314494687
2014-07-16 09:52:57 +08:00
code review 是个好方法

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

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

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

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

© 2021 V2EX