Question

为什么我收到RangeError: Maximum call stack exceeded错误？我试图通过文本解析来找到数学并解决它。它一直在工作，直到我开始实现括号'。我试图找到错误，但我无法弄明白。

我的代码：

var alg = {
        calc: function(eq, solveFor) {
            var out;
            var sideOne = eq.substring(0, eq.indexOf('='))
            var sideTwo = eq.substring(eq.indexOf('=') + 1)
            if (sideOne === solveFor) {
                alg.simplify(sideTwo);
            }
            if (sideTwo === solveFor) {
                alg.simplify(sideOne);
            }
        },
        simplify: function(eq) {
            str = $.trim(eq);
            if (str == undefined) {
                console.error('Error: null string')
            } else {
                var charMatch = /^[\d\*\/\+\-\^\(\) ]+$/
                if (charMatch.exec(str) === null) {
                    console.error('Error: Invalid char/expression')
                } else {
                    alg.parMath('not');
                    alg.expRoot(solve);
                    alg.multDiv(solve);
                    alg.addSubtr(solve);
                }
            }
        },
        fromPar: function(par) {
            alg.parMath(par);
            alg.expRoot(solve);
            alg.multDiv(solve);
            alg.addSubtr(solve);
        },
        parMath: function(source) {
            var reP = /\(([\d\*\/\+\-\^\(\) ]+)\)/
            var exP = reP.exec(str)
            if (source === 'par') {
                str = str.replace(exP[0], solve)
            }
            if (exP !== null) {
                use = 'par'
                solve = exP[1]
            } else {
                use = 'not'
                solve = str;
            }
        },
        expRoot: function() {
            var fracCon = /(\d+)\/(\d+)/
            var reER = /(\d+)(\^)(\d+(\/\d)?)(?!\/)/
            var exER = reER.exec(solve)
            if (exER !== null) {
                var exFC = fracCon.exec(exER[3])
                if (exFC !== null) {
                    var rep = Math.pow(parseFloat(exER[1]),(parseFloat(exFC[1]) / parseFloat(exFC[2])))
                } else {
                    var rep = Math.pow(parseFloat(exER[1]),parseFloat(exER[3]))
                }
                solve = solve.replace(exER[0], rep)
                if (reER.exec(solve) !== null) {
                    alg.expRoot();
                }
            }
        },
        multDiv: function() {
            var reMD = /(\d+(?:\.\d+)?) *([\*|\/]) *(\d+(?:\.\d+)?)/
            var exMD = reMD.exec(solve);
            if (exMD !== null) {
                if (exMD[2] === "*") {
                    var rep = parseFloat(exMD[1]) * parseFloat(exMD[3]);
                    var rep = Math.round(rep * 1000000) / 1000000;
                } else {
                    var rep = parseFloat(exMD[1]) / parseFloat(exMD[3]);
                    var rep = Math.round(rep * 1000000) / 1000000;
                }
                if (use !== 'par') {
                    solve = solve.replace(exMD[0], rep);
                }
                if (reMD.exec(solve) !== null) {
                    alg.multDiv();
                }
            }
        },
        addSubtr: function() {
            var reAS = /(\d+(?:\.\d+)?) *([\+|\-]) *(\d+(?:\.\d+)?)/
            var exAS = reAS.exec(solve); //Getting RangeError here
            if (exAS !== null) {
                if (exAS[2] === "+") {
                    var rep = parseFloat(exAS[1]) + parseFloat(exAS[3])
                    var rep = Math.round(rep * 1000000) / 1000000
                } else {
                    var rep = parseFloat(exAS[1]) - parseFloat(exAS[3])
                    var rep = Math.round(rep * 1000000) / 1000000
                }
                if (use !== 'par') {
                    str = str.replace(exAS[0], rep)
                }
                if (exAS !== null) {
                    alg.addSubtr(solve);
                } else {
                    if (use == 'not') {
                        out = solve;
                    } else {
                        alg.fromPar('par')
                    }
                }
            } else {
                if (use == 'not') {
                    out = solve;
                } else {
                    alg.fromPar('par')
                }
            }
        }
    
    };
    
    console.log(alg.calc('x=(1+1)', "x"));

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>

我在addSubtr功能开始时收到错误（由评论标记）。任何人都可以帮我找到如何解决错误？

Answer 1

问题是你的代码进入无限循环。这是逻辑的相关部分

addSubtr: function() {
    /* ommitted */
    var exAS = reAS.exec(solve); //Getting RangeError here
    if (exAS !== null) {
        /* ommitted - logic here*/
        if (exAS !== null) {
          alg.addSubtr(solve);
          /* ommitted */
        }
    }
}

通过正则表达式解析exAS，可以获得solve的值。
如果这返回非空值，则表示您已获得匹配
那只母鸡你进入if条件并做一些逻辑
还在那里，还有另一个if语句，用于检查正则表达式是否匹配任何内容。现在，根据定义，这将是正确的 - 可以很容易地看到删除了大量代码 - 检查相同的条件两次。没有什么可以改变两个if之间的结果。
由于条件检查通过，您使用相同的输入再次递归调用相同的函数。

因为输入是相同的，逻辑将工作相同，所以再次执行步骤1-5并再次调用该函数。

这会导致无限递归。好吧，实际上有一个限制，那就是JavaScript的堆栈大小，这就是你得到错误的原因。它有点误导，因为它是在调用堆栈大小上运行的正则表达式，而不是对addSubtr的递归调用，否则它会更清楚发生了什么。< / p>

关于如何修复它 - 你需要重构逻辑，这样你就不会进入无限循环。我不确定你的情况最好的方法是什么，但我建议你自己动手 - 无论如何，这将是一个有用的练习。这里有一些指示

在我提出的4.点中，我提到有一个基本无用的检查。我认为它应该是有用的。 - 您可能希望内部if位于外部if之外。现在看来，两者是等价的，所以内部if可以被删除 - 也许内部exAS的条件不正确 - 可能是你有时只想做递归调用，而不是每次都这样。 - 也许应该有一些改变solve或http://localhost或两者的东西。因此，条件将（可能）在第二次检查时产生不同的结果，或者函数在递归调用时会产生不同的结果（这会使递归调用变得有用）或两者兼而有之。

Answer 2

你的主要问题是你有一个无限循环

[][]{}

这是一项无用的检查有两个原因。一个是因为你不喜欢exAS是在addSubtr中定义的。因此，每次调用此功能时，您都要重置该值您的其他问题是您创建具有相同名称的变量，并依赖于静态变量/ void函数。如果我，我不能帮助你锻炼正确的方法来构建这个功能和所有其他功能。

功能工作，现在是RangeError

2 个答案: