【v17】内存泄漏问题

产品：SpreadJS
版本：17.1.8
调研编号：SJS-27082
Last Review：2024-11-08
当前进展：已在17.1.9版本修复。

场景：用户在vue项目中目录树来回切换会触发多次fromJSON

问题：spread.destroy后，有些数据依旧会存在内存当中，导致用户切换多次后，内存累增

在demo中执行以下代码即可浮现：

1. // 模拟渲染第一篇文章
   
2. window.fn(window.excel1);
   
3. 
   
4. // 模拟切换第二篇文章
   
5. window.sheet = null;
   
6. window.spread.destroy();
   
7. window.spread = null;
   
8. window.fn(window.excel2)
   
9. 
   
10. // 模拟切换至非excel页面
    
11. window.sheet = null;
    
12. window.spread.destroy();
    
13. window.spread = null;

复制代码

您好，我需要多次测试下，有结果会及时通知您。

您好，重新修改了您的html文件，且将版本更新为最新版17.1.8。
代码调整如下：

1. function loadSpread1() {
   
2.         return new Promise((resolve, reject) => {
   
3.           window.fn(window.excel1);
   
4.           console.log('加载1')
   
5.         });
   
6.       }
   
7. 
   
8.       function loadSpread2(i) {
   
9.         return new Promise((resolve, reject) => {
   
10.           window.sheet = null;
    
11.           window.spread.destroy();
    
12.           window.spread = null;
    
13.           console.log('销毁2', i)
    
14.           window.fn(window.excel2)
    
15.           console.log('加载2', i)
    
16.         });
    
17.       }
    
18. 
    
19.       function loadSpread3() {
    
20.         return new Promise((resolve, reject) => {
    
21.           window.sheet = null;
    
22.           window.spread.destroy();
    
23.           window.spread = null;
    
24.           console.log('销毁3')
    
25. 
    
26.         });
    
27.       }
    
28. 
    
29. 
    
30.       let list = [], num = 500
    
31.       for (let i = 0; i < num; i++) {
    
32.         if (i == 0) {
    
33.           list.push(loadSpread1())
    
34.         } else if (i == num - 1) {
    
35.           list.push(loadSpread3())
    
36.         }
    
37.         else {
    
38.           list.push(loadSpread2(i))
    
39.         }
    
40.       }
    
41. 
    
42.       Promise.all(list).then(() => {
    
43.         console.log("finish");
    
44.       });
    
45. 
    
46.     }

复制代码

在此基础上，修改num的值分别为10，100，500 ，观察memory变化。 分别对应memory大概为100M , 300M , 1000M 。
我将深入调研此问题。调研编号：SJS-27082

您好，根据楼上的demo ，调研结果如下：


1、在fromJSON中使用incrementalLoad，这将使fromJSON流程变为异步，一旦fromJSON流程变为异步模式，即表示。
     1）当使用destroys 销毁spread对象时，spread仍在fromJSON过程中   
     2）当加载一个json时，可能之前的fromJSON过程还没有完成。所以，需要在测试demo中，注释掉incrementalLoad相关代码，否则将会报错。
      当然，这个问题在用户实际来回点击过程中，有可能出现（还未加载完成，切换另一个页面），有可能不出现。

2、在您的代码中，初始化了Designer对象，但是销毁了spread ，这是一个不正确的用法。
  1） 可以销毁designer对象。designer.destroy()
  2)    如果初始化的是spread对象 ，可以用spread.destroy()


经过上面修改后，html文件如下： index_update (1).html (4.07 KB, 下载次数: 2)

2024-10-30 14:57 上传

点击文件名下载附件

将此html文件，放到楼上给出的demo中，然后在浏览器中执行，发现修改num值为10，100，200 ，内存大小分别为：100M, 180M ， 243M 。我将继续调研此问题

Ellia.Duan 发表于 2024-10-30 15:24
您好，根据楼上的demo ，调研结果如下：

您好，由于文档内容过大，因此我们需要才用incrementalLoad去进行加载，否则会导致用户加载时间变长，因此期望在使用incrementalLoad的基础上修复此问题，感谢。

收到反馈，我继续调研此问题。

您好，关于内存累加的问题将在17.1.9版本修复，我将此贴移至bug反馈区。
以及您提到的incrementalLoad异步加载问题，可以在callback时执行designer.destroy()。

您好！该问题已在17.1.9版本修复，17.1.9版本也已发布，您可以升级版本后使用。

您好，17.1.9版本修复后，修改num值，内存情况如下图所示：