code recommend

[noonhorse]

mp-store/src/diff.js

Lines 13 to 43 in be26f98

	function diffValues (left, right, path, patchs) {
	// filter functin and null
	if (typeof left === 'function' || left === null) {
	patchs.push(new Patch(REPLACE, path, right, left))
	} else if (Array.isArray(left)) {
	// diff array
	if (Array.isArray(right)) {
	walkArray(left, right, path, patchs)
	} else {
	patchs.push(new Patch(REPLACE, path, right, left))
	}
	} else if (typeof left === 'object') {
	// diff object
	if (
	right !== null &&
	typeof right === 'object' &&
	!Array.isArray(right)
	) {
	// filter Date object
	if (left instanceof Date || right instanceof Date) {
	patchs.push(new Patch(REPLACE, path, right, left))
	} else {
	walkObject(left, right, path, patchs)
	}
	} else {
	patchs.push(new Patch(REPLACE, path, right, left))
	}
	} else {
	patchs.push(new Patch(REPLACE, path, right, left))
	}
	}

---

这几行代码没看明白，你是不是这样实现更好。
这里面对array 和 object 考虑不够，
目的是diff ,建议细致思考 Array lenght 或 Object.keys lenght 不一样问题，

const ArrayType = '[object Array]'
const ObjectType = '[object Object]'

function type(obj) {
    return Object.prototype.toString.call(obj)
}

function diffValues (left, right, path, patchs) {
    if((type(left) === ArrayType && type(right) === ArrayType) || type(right) === ObjectType){
        walkObject(left, right, path, patchs)
    } else {
        patchs.push(new Patch(REPLACE, path, right, left))
    }
}

[imtaotao]

感谢您的建议，这边是这样考虑衡量的

1. 虽然使用 toString.call 的方式判断类型，在 es5 的规范下是准确的，但在 es6 下类型的判断也不是绝对准确，所以相对于 typeof 这方面的优势存在的不明显，更严重的问题是使用 toString.call 的判断比 typeof 性能损坏更加严重，在大量的数据 diff 情况下，这些损耗可能会累加到一个可观的地步。关于这方面的 demo，在 redux 源码中有体现。

2. 关于 array 和 普通的 object 要进行区分的原因是，当 array 的长度超过一定的限定后，需要做取舍，也就是直接替换（这块需要优化），但在 object 中不需要，这两种数据结构的使用场景是完全不同的。第二个是 array 中的 item 删减后，我们不能用 undefined 或者 null 来替换，这样会带来模板渲染上的问题，所以需要单独进行处理。再一个是对于 array 的遍历，使用 while 循环比 for in 要更好

3. 关于 if else 这种结构，并不是越精简越好，核心的逻辑，需要保证的是各种情况的逻辑尽可能的清晰易懂，这对于 bug 的调试和后续的更改扩展带来的好处是显而易见的。这里对所有的基础类型和常用的 object 类型进行了处理就好，比如 symbol 类型就不做处理，这点已经在文档中着重说明了。

所以在你给出的 demo 中，没有区分这两种结构，而且当 left 和 right 类型不一致时，又有递归，这会导致逻辑上的复杂度提升。如果有不当或者细节不完善的地方，欢迎讨论，谢谢啦~ (^▽^)