【16.2.6】自定义公式计算出现短路现象

收藏

本帖最后由 Richard.Huang 于 2023-12-25 11:55 编辑

产品：SpreadJS
版本：V16.2.6
问题编号：SJS-22025

自定义的异步公式：MyFormula，这个异步公式的evaluateAsync中会请求一个“注册”函数：RegFn，该函数返回一个Promise，但是不会立刻为公式提供运算结果：

MyFormula.evaluateAsync = async function(ctx){    const key = `${ctx.id}-${ctx.row}-${ctx.col}`;//公式唯一标记    const req = {key,param:{/*其他请求参数*/}}    const res = await RegFn(req)          context.setAsyncResult(res);}
const reqMap = {}const promiseMap = {}
RegFn函数定义：function RegFn(req){ return new Promise((resolve, reject) => {       reqMap[req.key] = req //收集所有公式的请求参数       promiseMap[req.key] = {resolve, reject}  //保存resolve，以便后面调完后端以后，把结果提交给公式 });

}
手动执行所有公式：async function exe(){    const res = await api.xxx.post("url", reqMap)  //请求后端，一次性执行所有公式， res.data.forEach((v)=>{       promiseMap[v.key].resolve(v.value)  //后端返回的结果中包含公式的唯一key，用之前保存好的promiseMap获取resolve，将结果返回给公式 })}

以上内容，已经实现，达到的预期效果，但是出现了如下问题：

如果单元格中只使用1-2个MyFormula公式，是没问题的，但是如果超过2个MyFormula公式，第3个MyFormula公式就不会运算了，表现为在第二个MyFormula公式处短路。
比如：MyFormula() - MyFormula() + MyFormula() - MyFormula()  .....
上述表达式中，前面2个MyFormula公式会触发MyFormula.evaluateAsync，但是第3个及之后的MyFormula公式，则不会触发MyFormula.evaluateAsync。
反复执行exe()函数多次后，最终所有的公式都会触发，但是这个不是预期效果，不可能让用户一遍遍执行exe以获取最终结果。

需求：所有公式编辑完成以后，都执行一次，即调用一次MyFormula.evaluateAsync，不要出现短路现象。

PS：上次咨询过：如何避免几百上千个公式瞬间调用后端几百上千次？没有得到解决方案，这个是我能想到的解决方案，可以实现一次请求，计算所有公式的结果，然后返回给公式使用。

Richard.Huang · Richard.Huang

您好，我按照您的描述在SpreadJSV16.2.6中进行多个自定义异步函数的计算，但是并没有复现您所描述的问题，仅仅只是多个自定义异步函数组合计算时会因为延迟的原因计算有些慢。您也可以在这篇学习指南的demo上进行测试：https://demo.grapecity.com.cn/sp ... ync-function/purejs
asyncfunction.gif460587768.png

WilliamChang · WilliamChang

你好，猜测原因是MyFormula 的默认值不是数字，这就导致了MyFormula() - MyFormula() + MyFormula() - MyFormula() ..... 前两个结点计算之后出现了#VALUE!，然后后面的就不需要evaluateAsync了。然后前两个结点的结果设置之后再继续计算这个公式。

情况和这个很像 https://gcdn.grapecity.com.cn/fo ... read&tid=175545

Richard.Huang · Richard.Huang

WilliamChang 发表于 2023-12-22 14:19
你好，猜测原因是MyFormula 的默认值不是数字，这就导致了MyFormula() - MyFormula() + MyFormula() - MyFo ...

Universail · Universail

WilliamChang 发表于 2023-12-22 14:19
你好，猜测原因是MyFormula 的默认值不是数字，这就导致了MyFormula() - MyFormula() + MyFormula() - MyFo ...

是同样的意思。但是默认值我一直设置的都是0：

   public defaultValue() {
      return 0;
   }

Universail · Universail

Richard.Huang 发表于 2023-12-22 13:59
您好，我按照您的描述在SpreadJSV16.2.6中进行多个自定义异步函数的计算，但是并没有复现您所描述的问题， ...

测试的方法不对，异步函数里“只注册，不计算”，如果直接计算，那是没问题的，但是我需要把所有公式的请求收集起来，一次性提交给后端（不能一个公式一个公式提交，这会造成很大并发）。

所以当多个公式运算的时候，由于短路效果，导致后面的公式不能注册，所以后端请求的结果里就没有对应的数据了

Universail · Universail

Richard.Huang 发表于 2023-12-22 13:59
您好，我按照您的描述在SpreadJSV16.2.6中进行多个自定义异步函数的计算，但是并没有复现您所描述的问题， ...

window.onload = function () {
var spread = new GC.Spread.Sheets.Workbook(document.getElementById("ss"), {sheetCount: 2});
initSpread(spread);
};
const reqMap = {}
const pMap = {}
function RegFn(key){
return new Promise((resolve)=>{
pMap[key] = {resolve}
})
}
function Execute1(){
for (let k in pMap){
pMap[k].resolve(100)
}
}
function initSpread(spread) {
const asyncExe = function(){}
asyncExe.prototype = new GC.Spread.CalcEngine.Functions.AsyncFunction("AEXE", 1, 1);
asyncExe.prototype.defaultValue = function () {
return 0
};
asyncExe.prototype.evaluateAsync =async function (context) {
await Execute1()
context.setAsyncResult("这个运算结果不是重点，只是为了触发函数");
};
var asyncSum = function () {
};
asyncSum.prototype = new GC.Spread.CalcEngine.Functions.AsyncFunction("ASUM", 1, 255);
asyncSum.prototype.defaultValue = function () {
return 0
};
asyncSum.prototype.evaluateAsync =async function (context) {
// use setTimeout to simulate server side evaluation
// in read world it maybe an ajax post to server for evaluation
const key = `${context.id}-${context.row}-${context.col}`;
alert('注册异步任务：'+context.id)//这里可以观察到单元格C2中第三个公式没有执行
const res = await RegFn(key)
context.setAsyncResult(res);
alert("此时才返回结果")
};
var GetNumberFromServer = function () {
};
GetNumberFromServer.prototype = new GC.Spread.CalcEngine.Functions.AsyncFunction("GETNUMBERFROMSERVER", 1, 2);
GetNumberFromServer.prototype.evaluate = function (context, arg1, arg2) {
setTimeout(function () {
var value = Math.random() + 1;
context.setAsyncResult(value);
}, 500);
};
GC.Spread.CalcEngine.Functions.defineGlobalCustomFunction("GETNUMBERFROMSERVER", new GetNumberFromServer());
var GetTimeFromServer = function () {
};
GetTimeFromServer.prototype = new GC.Spread.CalcEngine.Functions.AsyncFunction("GETTIMEFROMSERVER");
GetTimeFromServer.prototype.evaluate = function (context) {
setTimeout(function () {
var date = new Date();
context.setAsyncResult(date);
}, 500);
};
GetTimeFromServer.prototype.evaluateMode = function () {
return 2;
};
GetTimeFromServer.prototype.interval = function () {
return 1000;
};
GC.Spread.CalcEngine.Functions.defineGlobalCustomFunction("GETTIMEFROMSERVER", new GetTimeFromServer());
var sheet = spread.sheets[0];
sheet.suspendPaint();
sheet.options.allowCellOverflow = true;
sheet.setArray(0, 0, [[5, 15]]);
sheet.addCustomFunction(new asyncSum());
sheet.addCustomFunction(new asyncExe());
sheet.setText(1, 1, 'ASUM(A1,B1)');
sheet.setText(2, 1, 'SUM(A1,B1)');
sheet.setFormula(1, 2, "ASUM(A1,B1)+ASUM(A1,B1)+ASUM(A1,B1)");
sheet.getCell(1, 2).foreColor("green");
sheet.setFormula(2, 2, "SUM(A1,B1)");
sheet.setValue(4, 0, "Edit the formula of C2 or referenced cell' value to see how async function works.");
sheet.setValue(8, 0, 'CHANGEVALUE');
sheet.setValue(8, 1, 'FORMULA');
sheet.setValue(8, 2, 'RESULT');
sheet.setValue(8, 3, 'COMMENTS');
sheet.setValue(9, 3, 'On A10 changed');
sheet.setValue(10, 3, 'On A10 changed');
sheet.setValue(11, 3, 'Evaluate once');
sheet.setValue(12, 3, 'Every 2 seconds');
sheet.setValue(9, 0, 1);
sheet.setValue(9, 1, '=GetNumberFromServer(A10)');
sheet.setValue(10, 1, '=Refresh(GetNumberFromServer(A10), 0)');
sheet.setValue(11, 1, '=Refresh(GetNumberFromServer(A10), 1)');
sheet.setValue(12, 1, '=Refresh(GetNumberFromServer(A10), 2, 2000)');
sheet.setFormula(9, 2, '=GetNumberFromServer(A10)');
sheet.setFormula(10, 2, '=Refresh(GetNumberFromServer(A10), 0)');
sheet.setFormula(11, 2, '=Refresh(GetNumberFromServer(A10), 1)');
sheet.setFormula(12, 2, '=Refresh(GetNumberFromServer(A10), 2, 2000)');
sheet.getCell(12, 2).foreColor("green");
sheet.setColumnWidth(0, 100);
sheet.setColumnWidth(1, 300);
sheet.setColumnWidth(2, 200);
sheet.setColumnWidth(3, 200);
sheet.setValue(15, 1, "=Refresh(now(), 2, 1000)");
sheet.setValue(15, 3, "Every 1 second");
sheet.setFormula(15, 2, "=Refresh(now(), 2, 1000)");
sheet.getCell(15, 2).foreColor("green");
sheet.setValue(18, 1, "=GetTimeFromServer()");
sheet.setValue(18, 3, "Every 1 second");
sheet.setFormula(18, 2, "=GetTimeFromServer()");
sheet.getCell(18, 2).foreColor("green");
sheet.getCell(18, 2).hAlign(GC.Spread.Sheets.HorizontalAlign.right);
sheet.resumePaint();
}

复制代码

Universail · Universail

Richard.Huang 发表于 2023-12-22 13:59
您好，我按照您的描述在SpreadJSV16.2.6中进行多个自定义异步函数的计算，但是并没有复现您所描述的问题， ...

用我提供的代码测试，可以重现问题，本来想做个按钮来触发异步执行，但是不会做，所以你就用AEXE这个公式来触发执行吧

Richard.Huang · Richard.Huang

本帖最后由 Richard.Huang 于 2023-12-25 11:54 编辑

Universail 发表于 2023-12-22 20:18
用我提供的代码测试，可以重现问题，本来想做个按钮来触发异步执行，但是不会做，所以你就用AEXE这个公式 ...

感谢您提供的测试代码，我按照您的代码确实复现了您所描述的问题。对此我们将进行深度调研，后续有相关调研结果，我会第一时间在本贴中进行回复。问题编号：SJS-22025

Universail · Universail

Richard.Huang 发表于 2023-12-25 11:40
感谢您提供的测试代码，我按照您的代码确实复现了您所描述的问题。对此我们将进行深度调研，后续有相关调 ...

好的，等你好消息

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

[已处理] 【16.2.6】自定义公式计算出现短路现象

31 个回复

SpreadJS 认证

SpreadJS 高级认证