review(3)

1.commit 规范

调整期：

                        aging: 130
                    },
                    {
                        name: '云主机申请服务',
                        frequency: 123,
                        score: 2,
                        successRate: 50,
                        aging: 130
                    },
                    {
                        name: '云主机申请服务',
                        frequency: 123,
                        score: 2,
                        successRate: 50,
                        aging: 130
                    }
​

调整后：

​

自己造的假数据最好不要commit，等完成接口对接后再提交。commit以独立且完整的内容为宜，比如一个组件的新增、修改，或者一个模块的相同部分修改，或者一个缺陷的修复

调整前:

 <div class="botom-table">
        <div class="title">服务清单</div>
        <bk-table style="margin-top: 15px;"
            :data="tableData"
        >
​

调整后:

 <div class="botom-table">
        <div class="title">服务清单</div>
        <bk-table
            style="margin-top: 15px;"
            :data="tableData"
        >
​

换行格式统一下，第一行也要换；右括号放在同一行末尾

3

调整前:

getServeReport(serviceType) {
    this.isLoading = true
    this.$api.Operation.getServeReport(serviceType.index).then(res => {
        this.isLoading = false
​

调增后:

getServeReport(serviceType) {
    this.isLoading = true
    this.$api.Operation.getServeReport(serviceType.index).then(res => {
    ...
    }.finally{this.isLoading = false}

重置loading放在finally，否则接口报错时不会重置

4

调整前:

initDateTimeRange: [new Date(), new Date()],
statusList: [
    {
        name: '成功',
        index: 'success'
​

调整后:

initDateTimeRange: [new Date(), new Date()],
statusList: [
    {
        name: '成功',
        key: 'success'

尽量避免index直接声明,用key