首页
题库
公司真题
专项练习
面试题库
在线编程
面试
面试经验
AI 模拟面试
简历
求职
学习
基础学习课
实战项目课
求职辅导课
专栏&文章
竞赛
搜索
我要招人
发布职位
发布职位、邀约牛人
更多企业解决方案
在线笔面试、雇主品牌宣传
登录
/
注册
聪敏的卡尔在开会
郑州大学 前端工程师
发布于北京
关注
已关注
取消关注
@程序员鱼皮:
这些代码,差点把我气出内伤
大家好,我是鱼皮,一个正在烂代码的泥潭里面摸爬滚打的程序员。先问大家一个小问题:你觉得看别人代码累,还是自己写代码累?我相信有很多朋友会说,当然是自己写代码累了,要思考逻辑、要动手敲键盘,身心俱疲啊;但是,如果你需要经常阅读别人的代码,尤其是烂代码,答案就不一定了。因为自己写代码,逻辑是自己来梳理的、代码自己是熟悉的;但是看别人的代码,你就要去理解别人的代码逻辑,再加上烂代码的加持,可能很简单的逻辑,你都得看半天才能懂。所以很多大佬在面对前人的 “屎山” 项目时,宁愿自己重写,也不去读烂代码。举个夸张的例子:让你实现 1 + 1 的求和自己写:let sum = 1 + 1;某烂代码可能是:let a = {};let b = {};a.toString = function() { return 1;};b.valueOf = function() { return "1";};let sum = a + b;把简单的逻辑搞复杂,便是绝大多数程序员的拿手好戏(当然也包括我)。因为我从大学就开始带团队做项目了嘛,所以经常会审查团队同学的代码,做好二次校验。包括现在虽然开公司了,前端 / 后端同学的代码,也都会在我这过一遍才会发布上线。总之算是看了很多代码,其中有一些真的是让我哭笑不得。下面给大家分享一些代码片段出来,希望大家 不要学习 。1、过于抽象的命名还记得咱们刚学编程的时候,变量的命名都是用的 abcdefg。自学时这么写完全没问题,但是在实际项目中,如果还用过于抽象的命名,那就不太合适了。比如下面这段,大家能看懂是什么意思么?const [l, setL] = useState<boolean>(false);const [d, setD] = useState<any>();但如果我稍微完善下命名呢:const [loading, setLoading] = useState<boolean>(false);const [data, setData] = useState<any>();很多同学应该立刻能看懂了,一个是 “加载中” 的变量,一个是 “存储数据” 的变量。最好的代码应该是不用写注释的,因为 代码即注释 。如果你能把命名做到 “见名知义”,看代码的人会极度舒适。2、有深度的代码比如下面这段:if (condition1) { // 逻辑 A if (condition2) { // 逻辑 B if (condition3) { // 逻辑 C if (condition4) { // 逻辑 D } } }}这里的深度有 2 重含义,一重是字面意思:代码一层嵌一层、深不见底;另外一重是指真的 “很有深度” —— 指让人看不懂。阅读这段代码的感觉就像是你在一座巨大的迷宫里,每次转弯都要判断下是左还是右,最后你只会迷失方向。如何改进呢?最简单直接的方法就是使用早返回策略(early return):if (!condition1) // 逻辑 A return;if (!condition2) // 逻辑 B return;if (!condition3) // 逻辑 C return;if (!condition4) // 逻辑 D return;这样,你的代码就清晰了很多,阅读这种代码的感觉就像是走在了一条直路上,前方的路一目了然。当然,还可以将一些逻辑抽象成独立函数来简化代码,或者使用设计模式来优化。怎么判断一段代码是否过于复杂、应该优化了呢?这里提到一个概念: 圈复杂度 ,这是一种量化代码复杂程度的概念。通常你代码中的 if else 分支越多,圈复杂度就越高,代码就越复杂。企业中一般建议圈复杂度不要超过 10 - 15,我个人的编码习惯是一般不会在代码中出现 3 层以上的嵌套(除非必要)。3、冗余代码这是我遇到最多的问题!分为 3 种情况:能用一行代码搞定,偏偏要写 10 行能用一个变量或函数搞定,偏偏每次都是复制粘贴重复写没有用到的代码,又不舍得删除举个例子,下面这段前端代码,大家觉得有什么问题:这是我们鱼聪明 AI 前端开发过程的真实代码<Spin spinning={!(currDownloadUrl || originPictureUrl || pictureUrl)}> {type === DRAW_APP ? ( drawImg(image) ) : ( drawImg(currDownloadUrl || originPictureUrl || pictureUrl) )}</Spin>第一眼看到这段代码时,我就发现了,判断 spinning(旋转)的代码逻辑比较复杂,包含了两个 || 逻辑。而下面的 drawImg 函数的参数中,又包含了这段一模一样的逻辑。这段判断,其实就是冗余代码,完全没必要写两遍!我调整过后的代码如下:// 要展示的图片地址const showPictureUrl = currDownloadUrl || originPictureUrl || pictureUrl;直接定义一个通用变量,写上清晰的注释,其他地方要使用时就无需关注内部判断逻辑,看注释就行了。这就是所谓的 DRY 原则(Don't Repeat Yourself) ,尽量避免代码冗余。如果你在多处写下相同的代码,那么当需要修改这段代码时,你就需要在所有这些地方都修改,漏一个地方就是一个 Bug。还有,关于项目中没有用到的函数和变量,如果你近期没有使用计划,不妨就删除或者注释掉,别舍不得。利用好 Git 版本控制系统,只要你的代码提交信息写得好,要用到被删掉的历史代码时,去提交记录里找即可。注意,如果是学习时写代码,多保留一些也是合理的。但企业项目中,项目代码精简凝练些会更好。这里打个比方,写代码就像是我在公司里堆东西,一开始总觉得多一点没关系,反正有空间。但是,冗余代码就像是杂物,会越堆越多,迟早有一天,会影响到你,就像我们公司现在一样(右边有一堆杂物):大概就分享到这里,请大家在评论区 @ 出你写烂代码的同事吧 🐶 ~
点赞 135
评论 17
全部评论
推荐
最新
楼层
还没有回复哦~
相关推荐
11-25 10:19
门头沟学院 商业技术工程师
阿里云VS腾讯VS华为,决赛圈
bg:24届华科本,计算机专业。又到了一年一度offer pk的时刻了,想到了我去年也是同时拿到了阿里云、腾讯、华为的管培生offer,最终选择了阿里云,给大家分享一下我的决策过程,希望能够帮到你们。这几个offer本质上都是偏销售类型的,因为我当时很明确自己想做这种技术复合人才,一方面是我的性格比较适合,另一方面我觉得发展空间更大,所以我投递的基本都是这种岗位,我称之为懂技术的销售!幸运的是,我拿到了行业中比较顶级的3个销售管培offer,分别是:阿里云,商业技术工程师,今年叫阿里云管培生(商业技术方向)腾讯,销售管培,广告方向华为,华为云客户经理机智的我采用了逐一淘汰机制...
投递腾讯等公司10个岗位 >
点赞
评论
收藏
分享
11-20 11:25
重庆三峡学院 新媒体运营
快手行业运营实习面经
自我介绍优缺点何时到岗实习的持续时间对公司的看法一二面问的问题大差不差,放心冲吧
查看5道真题和解析
非技术面试记录
点赞
评论
收藏
分享
10-30 10:33
河北民族师范学院 体育教师/老师
求助啊😭😭怎么就是找不到一个工作啊
点赞
评论
收藏
分享
10-25 16:26
门头沟学院 电力电子工程师
双非硕0offer
已经开始25年高考的第一轮复习了
牛客717484937号:
双飞硕没实习挺要命的
点赞
评论
收藏
分享
11-20 13:35
广东白云学院 数据运营
校招入职后第一次后悔选错offer
秋招的时候一直纠结两个offer,最后选择了一家互联网大厂 大厂薪资不如另外一个科技公司,离家也远 当时想着毕业第一份工作不能只图钱 奔着大厂的团队氛围、高福利、大title来的 实际上每天加班,遇到pua老板,工资也低 现在非常懊恼希望可以重回去年秋招时刻,重新选择结果会不会不一样,我知道不能美化没有走过的那条路,但是后悔都情绪一直在
立马来offer:
去哪个大厂了啊,其实你选了小公司,遇到不开心的也会后悔没有选择大平台。别美化小公司了,至少大厂有平台,有二次选择的能力
如果再来一次,你还会选择这个工作吗?
点赞
评论
收藏
分享
点赞成功,聊一聊 >
点赞
收藏
评论
分享
回复帖子
提到的真题
返回内容
全站热榜
1
...
从露宿街头到百万级种子轮融资——我的大学时代经历了什么
4.2W
2
...
字节20多面终究一场空
1.7W
3
...
C++选手秋招总结
1.6W
4
...
不装了!牛客就是OFFER判官
1.3W
5
...
华为电话oc了
9533
6
...
泡出来啦
9145
7
...
秋招圆满结束
8642
8
...
总结一下鼠鼠找实习的历程
8025
9
...
阿里云管培生开奖了
7672
10
...
爱信等
7282
正在热议
#
25届秋招总结
#
334509次浏览
3166人参与
#
上班苦还是上学苦呢?
#
75251次浏览
669人参与
#
阿里云管培生offer
#
37650次浏览
427人参与
#
地方国企笔面经互助
#
4646次浏览
12人参与
#
如果有时光机,你最想去到哪个年纪?
#
22444次浏览
433人参与
#
选完offer后,你后悔学本专业吗
#
22154次浏览
159人参与
#
百度开奖
#
187763次浏览
1175人参与
#
我的实习求职记录
#
6074765次浏览
83564人参与
#
如何一边实习一边秋招
#
997895次浏览
12670人参与
#
找工作时遇到的神仙HR
#
554036次浏览
3805人参与
#
入职第一天,你准备什么时候下班
#
21705次浏览
144人参与
#
招聘要求与实际实习内容不符怎么办
#
10860次浏览
277人参与
#
bilibili求职进展汇总
#
33438次浏览
358人参与
#
许愿池
#
215047次浏览
2535人参与
#
学历or实习经历,哪个更重要
#
54283次浏览
426人参与
#
你今年的平均薪资是多少?
#
71219次浏览
345人参与
#
实习工作,你找得还顺利吗?
#
248168次浏览
2918人参与
#
通信硬件薪资爆料
#
610697次浏览
5224人参与
#
海康威视求职进展汇总
#
401184次浏览
3408人参与
#
携程求职进展汇总
#
136084次浏览
932人参与
#
正在实习的你,几点下班
#
53620次浏览
396人参与
#
工作两年想退休了
#
53177次浏览
673人参与
牛客网
牛客企业服务