分析团队中的六则糟糕代码，腾讯前端开发面试经验

/处理右上角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’

})

let sign = handler.sign(data, ‘qwerqwerqwerqwerqwer’)

…

小编13年上海交大毕业，曾经在小公司待过，也去过华为、OPPO等大厂，18年进入阿里一直到现在。

深知大多数初中级前端工程师，想要提升技能，往往是自己摸索成长或者是报班学习，但自己不成体系的自学效果低效又漫长，而且极易碰到天花板技术停滞不前！
因此收集整理了一份《2024年Web前端开发全套学习资料》送给大家，初衷也很简单，就是希望能够帮助到想自学提升又不知道该从何学起的朋友，同时减轻大家的负担。

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

如果你觉得这些内容对你有帮助，可以添加下面V无偿领取！（备注：前端）

打开全栈工匠技能包-1小时轻松掌握SSR

两小时精通jq+bs插件开发

生产环境下如歌部署Node.js

开源分享：【大厂前端面试题解析+核心总结学习笔记+真实项目实战+最新讲解视频】

网易内部VUE自定义插件库NPM集成

谁说前端不用懂安全，XSS跨站脚本的危害

webpack的loader到底是什么样的？两小时带你写一个自己loader

3ZWVjOWJiZGVmYjJiMjNjNzExNzgzZWM4MzIwZV9oZC5qcGc?x-oss-process=image/format,png)

谁说前端不用懂安全，XSS跨站脚本的危害

webpack的loader到底是什么样的？两小时带你写一个自己loader