分享一下平常开发经常出现问题,增加代码质量的十个小点:
特别是网络请求或者其他异步操作中,await 记得包裹 try catch,可以给用户一个友好提示,同时可以考虑 catch 中需要做什么兜底处理,必要时进行上传日志。
- try {
- this.loading = this.$loading({
- lock: true,
- text: '加载中...',
- spinner: 'el-icon-loading',
- background: 'rgba(0, 0, 0, 0.7)',
- });
- const info = await resDistributeService({ taskTicketId: this.id });
- ...
- } catch (e) {
- this.$message({
- type: 'error',
- message: e.msg || e.message || '失败',
- });
- } finally {
- this.loading.close();
- }
可以结合 finally,处理 loading 等。
前端经常使用 !v ,来判断 v 是不是有值。
- if(!v){
- return
- }
- doSomething()
但如果 0 是 v 的有效值 ,此时本该处理,但会提前结束,最终引发错误。此时需要显示的判断是否是 null 或者 undefined 。
- if(v === null || v=== undefined){
- return
- }
- doSomething()
由于 js 中的对象是引用,因此赋默认值的时候最好通过函数,每次都返回一个新对象。
bad:
- const defaultCondition = {
- name: '',
- conditionList: [
- {
- conditionCode: '',
- conditionValue: null,
- },
- ],
- }
- export default {
- data() {
- return {
- condition: {...defaultCondition},
- };
- },
- methods: {
- closeDialog() {
- this.condition = {...defaultCondition};
- this.configId = null;
- this.$refs.form.resetFields();
- },
- },
- };
good:
- const getDefaultCondition = () => ({
- name: '',
- conditionList: [
- {
- conditionCode: '',
- conditionValue: null,
- },
- ],
- })
- export default {
- data() {
- return {
- condition: getDefaultCondition(),
- };
- },
- methods: {
- closeDialog() {
- this.condition = getDefaultCondition();
- this.configId = null;
- this.$refs.form.resetFields();
- },
- },
- };
将接口的定义放到统一文件中,未来变动改动起来会比较方便,如果各个 url 都写死在页面中以后就很麻烦了。
- // service.js
- import request from 'utils/request';
-
- const service = new (request('/api/m/mallorder/exp/compensation/customer'))();
-
- export const listService = (params) => {
- return service.post('/queryRuleList', params);
- };
-
- export const listDataKey = 'ruleVOList';
- export const idKey = 'ruleId';
-
- export const dialogEnumService = () => {
- return service.get('/info');
- };
-
- export const saveService = (params) => {
- return service.post('/saveRule', params);
- };
-
- export const detailService = (params) => {
- return service.get('/detail', params);
- };
此外,网络请求一般都会在 npm 包的基础上自己再包一层,一方面可以注入共用参数,另一方面可以对返回数据进行统一的错误处理。
如果定义一个函数需要 3 个以上的参数
- function(a,b,c,d){
-
- }
此时可以考虑采用对象解构,改为
- function({a=1,b,c,d}={}){
-
- }
好处是未来需要扩展参数的时候,不需要太担心其他地方调用时候传参是否会引起问题。
当然,如果参数过多也需要思考一下当前函数是否承载了太多的功能,进行一下功能上的拆分。
当我们已经定义了一个函数,比如去初始一些变量。
- function initOptions(){
- a = xxx
- b = xxx
- }
此时我们需要做另一件无关的事 【A】,虽然它和 initOptions 调用的时机一致,但最好不要直接放到 initOptions 中,而是新建一个函数单独调用。
不然未来如果其他地方也要调 initOptions,但此时可能并不需要做【A】这件事情就会引起 bug。
由于 js 语言的灵活性,函数传入的参数很可能不符合预期,必要时我们需要进行判断并且进行兜底处理,不可完全信任调用方。
团队合作中,该函数在未来极大可能会被其他人调用。
- function doSomeThing(params1, params2) {
- if(params1 === null){
- return;
- }
- if(params2){
- ...
- }
- // 再去做我们的事情
- }
如果后边的流程强依赖于 params,我们可以直接 return,必要时也可以上报日志或者 throw Error。
js 中没有整数类型,即 java 中的 int、long 这些,所有数字都遵循 IEEE 754 标准,即 java 中的 double 类型,详细的可参考 浮点数详解。
可以精确表示的最大整数是 9007199254740991,共 16 位,超过这个数精度可能会丢失,对于新接口,可以问一下后端相应数字字段的最大值会是多少。
对于浮点数的处理,除了众所周知的 0.1 + 0.2 === 0.3 的值为false 外,当我们对数字进行运算的时候也需要注意。
常见的将 9.04 元转为 904 分:

img
我们需要对结果进行取整处理。

img
可选链操作符,参考 MDN ,用的比较多。
和后端定的数组或者对象,后端有时候返回来的很可能是 null 甚至没有该字段,因此前端可以用可选链操作符用于数组、对象、函数,防止出现错误直接阻断后续流程。
- let nestedProp = obj.first?.second; // 等效于 obj.first && obj.fisrt.second
- //后续流程
但不要过度使用可选链,如果某些地方理论上不会出问题,比如 let test = obj.first?.second,如果 second 一定能取到,我们直接 let test = obj.first.second 即可。
不然未来如果这里由于某种原因出了问题导致 obj.first 是 null,但我们使用了可选链,所以 obj.first?.second 也不会报错,我们就永远不会知道这里出现问题了。
当然也需要权衡下,不加可选链造成js Error 会不会影响业务逻辑。
修改或者使用对象、数组时,时刻切记它们为引用,一处修改会造成处处修改。
以上的点应该算已经融入血液中了,平常开发和帮同事过 pr 的时候会格外注意,和业务逻辑没有关系,但可以提升代码质量。还有 Vue 一些常见的点也总结了一下,在语雀建了一个文档,未来有其他想法也会再更新一下,感兴趣的同学可以收藏一下,前端实践沉淀。
