Merge pull request #1177 from Meituan-Dianping/hucq/code-style

Hucq/code style

Author: hucq

src/core/observer/index.js

@@ -147,8 +147,6 @@ export function defineReactive (
     return
   }
 
-  // TODO: 先试验标记一下 keyPath
-
   // cater for pre-defined getter/setters
   const getter = property && property.get
   const setter = property && property.set
@@ -222,8 +220,10 @@ export function set (target: Array<any> | Object, key: any, val: any): any {
     return val
   }
   defineReactive(ob.value, key, val)
-  // Vue.set 添加对象属性，渲染时候把val传给小程序渲染
-  target.__keyPath = target.__keyPath ? target.__keyPath : {}
+  // Vue.set 添加对象属性，渲染时候把 val 传给小程序渲染
+  if (!target.__keyPath) {
+    target.__keyPath = {}
+  }
   target.__keyPath[key] = true
   ob.dep.notify()
   return val
@@ -252,8 +252,10 @@ export function del (target: Array<any> | Object, key: any) {
   if (!ob) {
     return
   }
-  target.__keyPath = target.__keyPath ? target.__keyPath : {}
-  // Vue.del 删除对象属性，渲染时候把这个属性设置为undefined
+  if (!target.__keyPath) {
+    target.__keyPath = {}
+  }
+  // Vue.del 删除对象属性，渲染时候把这个属性设置为 undefined
   target.__keyPath[key] = 'del'
   ob.dep.notify()
 }

src/platforms/mp/runtime/diff-data.js

@@ -18,11 +18,12 @@ function getDeepData (keyList, viewData) {
     }
   }
 }
+
 function compareAndSetDeepData (key, newData, vm, data) {
   // 比较引用类型数据
   try {
     const keyList = key.split('.')
-     //page.__viewData__老版小程序不存在，使用mpvue里绑的data比对
+    // page.__viewData__老版小程序不存在，使用mpvue里绑的data比对
     const oldData = getDeepData(keyList, vm.$root.$mp.page.data)
     if (oldData === null || JSON.stringify(oldData) !== JSON.stringify(newData)) {
       data[key] = newData
@@ -106,7 +107,7 @@ export function diffData (vm, data) {
   })
   // console.log(rootKey)
 
-    // 值类型变量不考虑优化，还是直接更新
+  // 值类型变量不考虑优化，还是直接更新
   const __keyPathOnThis = vmData.__keyPath || vm.__keyPath || {}
   delete vm.__keyPath
   delete vmData.__keyPath
@@ -115,11 +116,11 @@ export function diffData (vm, data) {
     // 第二次赋值才进行缩减操作
     Object.keys(vmData).forEach((vmDataItemKey) => {
       if (vmData[vmDataItemKey] instanceof Object) {
-          // 引用类型
+        // 引用类型
         if (vmDataItemKey === '__keyPath') { return }
         minifyDeepData(rootKey, vmDataItemKey, vmData[vmDataItemKey], data, vm._mpValueSet, vm)
-      } else if(vmData[vmDataItemKey] !== undefined){
-          // _data上的值属性只有要更新的时候才赋值
+      } else if (vmData[vmDataItemKey] !== undefined) {
+        // _data上的值属性只有要更新的时候才赋值
         if (__keyPathOnThis[vmDataItemKey] === true) {
           data[rootKey + '.' + vmDataItemKey] = vmData[vmDataItemKey]
         }
@@ -131,7 +132,7 @@ export function diffData (vm, data) {
         // 引用类型
         if (vmPropsItemKey === '__keyPath') { return }
         minifyDeepData(rootKey, vmPropsItemKey, vmProps[vmPropsItemKey], data, vm._mpValueSet, vm)
-      } else if(vmProps[vmPropsItemKey] !== undefined){
+      } else if (vmProps[vmPropsItemKey] !== undefined) {
         data[rootKey + '.' + vmPropsItemKey] = vmProps[vmPropsItemKey]
       }
       // _props上的值属性只有要更新的时候才赋值
@@ -146,11 +147,11 @@ export function diffData (vm, data) {
     Object.keys(vmComputedWatchers).forEach((computedItemKey) => {
       data[rootKey + '.' + computedItemKey] = vm[computedItemKey]
     })
-      // 更新的时候要删除$root.0:{},否则会覆盖原正确数据
+    // 更新的时候要删除$root.0:{},否则会覆盖原正确数据
     delete data[rootKey]
   }
   if (vm._mpValueSet === undefined) {
-      // 第一次设置数据成功后，标记位置true,再更新到这个节点如果没有keyPath数组认为不需要更新
+    // 第一次设置数据成功后，标记位置true,再更新到这个节点如果没有keyPath数组认为不需要更新
     vm._mpValueSet = 'done'
   }
   if (Vue.config.devtools) {

src/platforms/mp/runtime/render.js

@@ -1,6 +1,5 @@
 // 节流方法，性能优化
 import { getComKey } from '../util/index'
-
 import { diffData } from './diff-data'
 
 // 全局的命名约定，为了节省编译的包大小一律采取形象的缩写，说明如下。
@@ -131,7 +130,6 @@ function getPage (vm) {
 }
 
 // 优化js变量动态变化时候引起全量更新
-
 // 优化每次 setData 都传递大量新数据
 export function updateDataToMP () {
   const page = getPage(this)
@@ -140,9 +138,7 @@ export function updateDataToMP () {
   }
 
   const data = formatVmData(this)
-
   diffData(this, data)
-
   throttleSetData(page.setData.bind(page), data)
 }