14.2循环修改行卡顿问题

收藏

本帖最后由 Clark.Pan 于 2021-12-7 18:02 编辑

给每一行做了一个“选择控件”，代码中实现方式是循环给每一行进行设置，测试时发现加载一个500行左右（识别到的行，包含空行在内）的Excel文件就会导致浏览器卡死，不使用suspendPaint();方法会更容易卡死，请问如何优化？

reanderABOptions() {
var spread1 = this.designer.getWorkbook();
spread1.suspendPaint();
var sheet = this.designer.Spread.getActiveSheet();
// 得到行数
let rowNum = sheet.getRowCount(GC.Spread.Sheets.SheetArea.viewport);
// 根据数据生成一个select，然后循环行内渲染
let rowOptBox = this.domRowSelect(false, this.economicIndicators);
let rowOptBox1 = this.domRowSelect(false, this.technicalIndicators);
for (let i = 0; i < rowNum; i++) {
sheet.setStyle(i, 0, rowOptBox);
sheet.setStyle(i, 1, rowOptBox1);
}
spread1.resumePaint();
},

复制代码

Clark.Pan · Clark.Pan

上述这两句返回的是什么内容呢？
domRowSelect方法中又了哪些操作

excelxnz · excelxnz

Clark.Pan 发表于 2021-12-7 09:33
上述这两句返回的是什么内容呢？
domRowSelect方法中又了哪些操作

// 生成行内的选择控件
domRowSelect(multiSelect, opts = []) {
let cascadeListData = {
multiSelect: multiSelect, // 是否多选
items: opts, // 选项item
};
let verticalCascadeListStyle = new GC.Spread.Sheets.Style();
verticalCascadeListStyle.cellButtons = [
{
imageType: GC.Spread.Sheets.ButtonImageType.dropdown,
command: "openList",
useButtonStyle: false,
},
];
verticalCascadeListStyle.hAlign = GC.Spread.Sheets.HorizontalAlign.center;
verticalCascadeListStyle.vAlign = GC.Spread.Sheets.VerticalAlign.center;
verticalCascadeListStyle.font = "10pt 宋体";
verticalCascadeListStyle.dropDowns = [
{
type: GC.Spread.Sheets.DropDownType.list,
option: cascadeListData,
},
];
return verticalCascadeListStyle;
}, //

复制代码

Clark.Pan · Clark.Pan

我觉得这里有可以优化的地方，如果您每一列都是同样的下拉style。您可以将style进行整列设置。例如：
image.png48913573.png

excelxnz · excelxnz

Clark.Pan 发表于 2021-12-7 11:30
我觉得这里有可以优化的地方，如果您每一列都是同样的下拉style。您可以将style进行整列设置。例如：

您好，我刚试了一下，效果没有明显提升，可能不是这里的问题。我再重新完整的描述一下：导入Excel会判断AB是否是指定设置的列（如经济指标），如果不是会插入2列并每个格子是选择控件，当切换sheet时会在spread.bind(GC.Spread.Sheets.Events.ActiveSheetChanged, this.activeSheetChanged); 这个事件中执行判断当前sheet表格是否需要添加AB列

reanderABOptions() {
var spread1 = this.designer.getWorkbook();
spread1.suspendPaint();
var sheet = this.designer.Spread.getActiveSheet();
if ("yicharu" != sheet.tag()) {
// 是否是首次打开sheet，是则插入2列，不是则不插入
sheet.addColumns(0, 2);
sheet.tag("yicharu");
}
let rowNum = sheet.getRowCount(GC.Spread.Sheets.SheetArea.viewport);
sheet.getCell(0, 0, GC.Spread.Sheets.SheetArea.colHeader).text("经济指标");
sheet.getCell(0, 1, GC.Spread.Sheets.SheetArea.colHeader).text("技术指标");
// 行内渲染
let rowOptBox = this.domRowSelect(false, this.economicIndicators);
let rowOptBox1 = this.domRowSelect(false, this.technicalIndicators);
// for (let i = 0; i < rowNum; i++) {
// sheet.setStyle(i, 0, rowOptBox);
// sheet.setStyle(i, 1, rowOptBox1);
// }
sheet.setStyle(-1, 0, rowOptBox);
sheet.setStyle(-1, 1, rowOptBox1);
spread1.resumePaint();
}

复制代码

excelxnz · excelxnz

excelxnz 发表于 2021-12-7 11:48
您好，我刚试了一下，效果没有明显提升，可能不是这里的问题。我再重新完整的描述一下：导入Excel会判断A ...

目前卡顿主要体现在了首次进入和切换sheet标签时

Clark.Pan · Clark.Pan

这块可否提供一个demo发给我，下拉的数据可以hardcode写死。我这边需要调试一下看看到底是哪里慢，有没有解决的方案。

excelxnz · excelxnz

本帖最后由 excelxnz 于 2021-12-7 14:05 编辑

您好，这是一个存在bug的demo，还请您帮忙检查一下，谢谢

Clark.Pan · Clark.Pan

您好，调查发现切换sheet时动态设置dropdown会造成公式计算，并在公式计算上花费了大量时间。至于原因目前已经移交给了开发做进一步调查，该贴暂做保留处理，有结果后我会在帖子中第一时间通知您。

Clark.Pan · Clark.Pan

您好，经调查原因是这样的，设置下拉本身不会导致公式计算，但是您代码中做了 sheet.addColumns(0,2)这样的操作，这句代码会导致相关公式引用发生位移，会使得相关的公式重新计算。如果删除这行不会阻塞太长时间。
另外，这个文件有10K+公式（1K+SUMPRODUCT），计算它们都需要大约60秒。这块没有优化的空间

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

[已处理] 14.2循环修改行卡顿问题

9 个回复

讲师达人

悬赏达人

微信认证勋章

SpreadJS 认证

SpreadJS 高级认证

元老葡萄