循环里边使用sheet.resumePaint();报错

收藏

本帖最后由 Richard.Huang 于 2024-2-22 11:35 编辑

产品：SpreadJS
版本：V16.2.6

运行出现这个错误提示： image.png649566344.png

我的代码是如下：

for(let index in sheets) {
let sheet = spread.getSheet(index);
sheet.suspendPaint();
// do something
sheet.resumePaint();
}

复制代码

我认为是否因为写在循环里边导致的，不知是否是这样导致？

我把spread.suspendpaint()包在for外边可否解决？

前端小白 · 前端小白

这要看你for-in中的sheets是什么了, 如果是json里的sheets节点。
那么示例代码中的index实际是sheet name，而不是sheet index。
然后导致spread.getSheet(sheetName)拿不出sheet来，对null进行操作然后报错

如果sheets是spread.sheets，那么index就是sheets array的下标，理论没啥问题，但是为啥不直接用for循环？或者forEach？

另，最好在循环外使用spread.suspendPaint，这样总体代码只重绘一次。而示例代码中，有多少sheet，就会重绘多少次。

晓名 · 晓名

前端小白发表于 2024-2-18 18:16
这要看你for-in中的sheets是什么了, 如果是json里的sheets节点。
那么示例代码中的index实际是sheet name ...

for-in就是sheet下标index

前端小白 · 前端小白

那咱先不考虑性能，用forEach他不香？都不用spread再拿一遍
除非调用getSheet的spread和你for-in的sheets的spread都不是一个

如果这，你就要查业务了。

spread.sheets.forEach((sheet, index) => {
sheet.suspendPaint();
//do what you want
sheet.resumePaint();
})

复制代码

Joestar.Xu · Joestar.Xu

您好，请参考上述回复看能否解决您的问题，如仍不能，请您上传一个可以复现您问题的Demo，我们这边根据您的实际代码调研一下。

晓名 · 晓名

前端小白发表于 2024-2-18 18:42
那咱先不考虑性能，用forEach他不香？都不用spread再拿一遍
除非调用getSheet的spread和你for-in的sheets ...

你可能没看懂我的意思：代码可以简化如下：

for (let i = 0; i < n; i++) {
let sheet = spread.getSheet(index);
sheet.suspendpaint();
// ...
sheet.resumepaint();
}

复制代码

是不是因为这样会出现那问题，我觉得可能是代码写的不对引起的

晓名 · 晓名

Joestar.Xu 发表于 2024-2-19 09:29
您好，请参考上述回复看能否解决您的问题，如仍不能，请您上传一个可以复现您问题的Demo，我们这边根据您的 ...

我这个也只是线上时不时出现，我就想直接把停止/重新绘制放在for外边

前端小白 · 前端小白

晓名发表于 2024-2-19 10:07
我这个也只是线上时不时出现，我就想直接把停止/重新绘制放在for外边

放外面肯定是推荐的。至于时不时出现这种问题，应该不是挂起画法的逻辑导致的，就我所知，SpreadJS的画法都是同步的，其执行逻辑是稳定的。如果还是时不时出现，那确实需要排查业务代码了。

晓名 · 晓名

前端小白发表于 2024-2-19 10:11
放外面肯定是推荐的。至于时不时出现这种问题，应该不是挂起画法的逻辑导致的，就我所知，SpreadJS的画法 ...

好的

Joestar.Xu · Joestar.Xu

请问楼主的问题解决了吗？

		自动登录	找回密码
密码			立即注册

[已处理] 循环里边使用sheet.resumePaint();报错

11 个回复

评分

评分

SpreadJS 认证

SpreadJS 高级认证