从入行做页面仔的时候开始就有点代码洁癖,喜欢比较整齐的码代。后来开始写 JS
了慢慢也开始去看一些 Good/Bad
样例,让自己的代码尽量简单易读不复杂且美观。
但是遇到的人越多越发现这是个问题,总会觉得其他人写的代码很随意,按排出去的任务都会自己过一遍,然后帮忙修改或者指出优化方案。
但是很多情况下功能模块会是同级的或者职级更高的同事编写,直接指出问题或者说优化方案,会让对方觉得尴尬/难堪,但是自己看到这些代码又会很难受。
即使要求团队开启了 ESlint
但是一些业务代码处理方式就特别让人难受。大块的无效代码,但是你改写吧浪费时间,它也不是不能跑,就是看着难受。
因为工作重心的问题,也没有办法组织 review
,所以应该怎样避免自己这种主观的“优化”思维方式的呢。
2006 年我开始工作,那会儿没有 GitHub、Google、StackOverflow,也没什么最佳实践之类的东西。大部分时候,每个人都有一个或多个文本文件,里面抄着各种将将 能工作 的代码。然后需要某个功能的时候,就翻一翻,找出一段程序贴上。程序员之间的交流,基本也就是互抄代码片段。
这些代码大部分时候能工作;有时候不能工作,摸索着改改也就行了。公司发展的很快,业务越来越复杂,渐渐的问题越来越多,修复问题的时候,很多代码根本看不懂,我就只好自己写。然后我发现我的代码清晰很多,维护成本很低、复用简单。后来我就开始大规模重构,几乎很少抄旧代码。
重点:然后我就秃了,眉毛也没了。(感兴趣的同学可以点开我的头像)
早先我的观点跟你一样,一定要把所有代码都弄好。最近几年我不断回顾过往,观点渐渐改变:
最后,code review 是必须的,否则一切皆扯淡。