分析团队中的六则糟糕代码

startDay = 8

endDay = 16

break

case ‘16~31’:  //16~31天过生日

startDay = 16

endDay = 31

break

case ‘31~999’:  //31天以后天过生日

startDay = 31

endDay = 999

break

}

写了20多行，其实就是一句话能搞定的事情，基本功太差：

let [startDay, endDay] = query.birth.split(‘~’).map(it => +it)

案例三

---

使用 /** ... */ 作为多行注释。包含描述、指定所有参数和返回值的类型和值。

/**

* 函数说明

* @关键字

*/

使用 // 作为单行注释。在评论对象上面另起一行使用单行注释。在注释前插入空行。

某前端同学的 angular 代码：

/处理右上角btn操作/

handleWithBtn(btn) {

switch (btn) {

case ‘export’: { // 批量认证

Debug.log(‘导出’)

break

}

}

}

上面的注释既不规范，也是多余的，当起了一个好的名字之后，代码就已经非常明确了。

案例四

---

逻辑互斥的 if 语句一定要配合 else 或 return 使用，把概率高的写在前面。

某后端同学的 js 代码：

if (productClass === ‘Card’ && action === BUYCARD) {

seneca.sendSms(smsData, params)

}

if (productClass === ‘Card’ && action === TURNCARD) {

seneca.patchStatus(productId)

}

if (productClass === ‘Card’ && action === REPLACE) {

seneca.changeStatus(‘crm’, ‘Card’)

}

if (productClass === ‘Lesson’) {

seneca.changeStatus(‘course’, ‘Lesson’)

}

…

上面每个判断都要执行一次，完全没有必要，这种情况下要么使用 switch 要么 if 配合 else 或 return 使用。

案例五

---

保持函数简短，一个好的函数适合展现在一个幻灯片(slide)上，这样如果在一个比较大房间中，也便于最后一排的人阅读。每一个函数的代码应该限制在 15 行左右，另外为了避免 if 语句过度嵌套, 应该提前将函数值返回.

某前端同学为了去除 params 对象中的 value 为 null，‘’，undefined 的 key 写的代码：

getUrlParam(sUrl, sKey) {

const param = sUrl.split(‘#’)[0].split(‘?’)[1]

if (param) {

if (sKey) {       // 指定参数名称

const strs = param.split(‘&’)

const arrs = new Array()  // 如果存在多个同名参数，则返回数组

for (let i = 0, len = strs.length; i < len; i++) {

const tmp = strs[i].split(‘=’)

if (tmp[0] === sKey) {

arrs.push(tmp[1])

}

}

if (arrs.length === 1) {// 返回该参数的值或者空字符串

return arrs[0]

} else if (arrs.length === 0) {

return ‘’

} else {

return arrs

}

} else {// 不指定参数名称，返回全部的参数对象 或者 {}

if (param === undefined || param === ‘’) {

return {}

} else {

const strs = param.split(‘&’)

const arrObj = new Object()

for (let i = 0, len = strs.length; i < len; i++) {

const tmp = strs[i].split(‘=’)

if (!(tmp[0] in arrObj)) {

arrObj[tmp[0]] = []

}

arrObj[tmp[0]].push(tmp[1])

}

return arrObj

}

}

} else {

return ‘’

}

}

这种函数可维护性极差，自己写的过个星期也读不懂什么意思了，出现错误很难定位。下面是改造后的：

function filterParams(obj) {

const keys = Object.keys(obj)

keys.forEach(key => {

const value = obj[key]

if (isObject(value)) filterParams(value)

if (isEmpty(value)) delete obj[key]

})

return obj

}

function isEmpty(input) {

return [‘’, undefined, null].includes(input)

}

function isObject(input) {

return input !== null && (!Array.isArray(input)) && typeof input === ‘object’

}

明显清晰很多，可读性很强，逻辑也很健壮。如果你觉得一个 15 行以内的函数搞不定某个事情，就把它拆分成多个小于 15 行的函数。

案例六

---

配置要写在配置文件里面统一管理，常量也要定义在单独的文件里面，常量名全部大写。

某后端同学写的 js 代码：

let client = new TopClient({

‘appkey’: ‘12345678’,

‘appsecret’: ‘asdfasdfasdfasdfasdfasdf’,

‘REST_URL’: ‘http://gw.api.taobao.com/router/rest’
自我介绍一下，小编13年上海交大毕业，曾经在小公司待过，也去过华为、OPPO等大厂，18年进入阿里一直到现在。

深知大多数前端工程师，想要提升技能，往往是自己摸索成长或者是报班学习，但对于培训机构动则几千的学费，着实压力不小。自己不成体系的自学效果低效又漫长，而且极易碰到天花板技术停滞不前！

因此收集整理了一份《2024年Web前端开发全套学习资料》，初衷也很简单，就是希望能够帮助到想自学提升又不知道该从何学起的朋友，同时减轻大家的负担。

既有适合小白学习的零基础资料，也有适合3年以上经验的小伙伴深入学习提升的进阶课程，基本涵盖了95%以上前端开发知识点，真正体系化！

由于文件比较大，这里只是将部分目录截图出来，每个节点里面都包含大厂面经、学习笔记、源码讲义、实战项目、讲解视频，并且会持续更新！

如果你觉得这些内容对你有帮助，可以扫码获取！！（备注：前端）

最后

资料过多，篇幅有限，需要文中全部资料可以点击这里免费获取前端面试资料PDF完整版！

自古成功在尝试。不尝试永远都不会成功。勇敢的尝试是成功的一半。

由于文件比较大，这里只是将部分目录截图出来，每个节点里面都包含大厂面经、学习笔记、源码讲义、实战项目、讲解视频，并且会持续更新！

如果你觉得这些内容对你有帮助，可以扫码获取！！（备注：前端）

[外链图片转存中…(img-cYuXTe1p-1713696234921)]

最后

[外链图片转存中…(img-qXwnhApz-1713696234921)]

[外链图片转存中…(img-JQU2QBWd-1713696234921)]

资料过多，篇幅有限，需要文中全部资料可以点击这里免费获取前端面试资料PDF完整版！

自古成功在尝试。不尝试永远都不会成功。勇敢的尝试是成功的一半。