树形table在销毁时页面卡死

[TaoHuaXia]

• I have searched the issues of this repository and believe that this is not a duplicate.

Version

2.0.0-rc.8

Environment

[email protected] [email protected]

Reproduction link

https://codesandbox.io/s/trusting-leaf-hmrxh?fontsize=14&hidenavigation=1&theme=dark

Steps to reproduce

1.使用树形table展示数据（约1000条左右）
2.对组件进行销毁

What is expected?

可以迅速销毁

What is actually happening?

页面卡死大约10s以上

---

通过issue可看到类似的https://github.com/vueComponent/ant-design-vue/issues/2508，虽然issue已被关闭，但是感觉没有进行解决（包括查看了后面的合并的代码，也没有解决此问题的提交）

[TaoHuaXia]

补充：tableData从1000条数据（叶子节点）更新为空数组，也会直接卡死

[TaoHuaXia]

卡顿例子：10 * 10 * 10的数据，重置tableData就会卡死：https://codesandbox.io/s/trusting-leaf-hmrxh?fontsize=14&hidenavigation=1&theme=dark

[TaoHuaXia]

现在已经在项目中使用，所以望大佬们能给个解决方案

[TaoHuaXia]

暂时先用了element的table，等修复后再切换回来

[betgar]

@TaoHuaXia 可以试试这个https://github.com/x-extends/vxe-table

[TaoHuaXia]

@TaoHuaXia 可以试试这个https://github.com/x-extends/vxe-table

@betgar 感谢老哥，我已经先用element的table顶着了

[eeeqxxtg]

我可能定位到了性能问题：

ant-design-vue/components/_util/store/create.js

Line 5 in f589c8f

function setState(partial) {

  function setState(partial) {
    state = { ...state, ...partial };
    for (let i = 0; i < listeners.length; i++) {
      listeners[i]();
    }
  }

1000行的table，每一个row都subscribe了store。

这1000行数据，每一行的变化除了触发自己注册的事件还会同时触发其他999行注册的事件。
极端情况下，性能就被吃掉太多了。

比如卸载这1000行数据，会触发1000+999+998+...1=500500次注册事件。
就算每个注册事件耗时0.02ms，整个卸载过程也会耗时超过10s。

建议这里的store考虑使用vue的响应式设计

[eeeqxxtg]

ant-design-vue/components/vc-table/src/ExpandableTable.jsx

Line 116 in f589c8f

this.store.setState({ expandedRowKeys });

注释掉这行代码，可以极大提高楼主的卸载速度。

win10 chrome 88.0.4324.182（正式版本） （64 位）

注释之前，卸载楼主的1000条数据，耗时44s

注释之后，卸载同样的1000条数据，耗时217ms

[tangjinzhou]

update to 2.0.1

[cnguu]

update to 2.0.1

1.x版本存在同样的销毁性能问题

[betgar]

希望能够移植回到1.x

[zkwolf]

1.x应该发版到1.7.4了

[AlexZhong22c]

@zkwolf 老哥，这个bug分配了给你。现在1.x还是有问题的

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.