discord-bots/TheArtificer
discord-bots
/
TheArtificer
mirror of https://github.com/Burn-E99/TheArtificer.git
1
1
Fork 0
Code Issues 1 Packages Projects Releases Wiki Activity

-3 loops per iteration by only incrementing loop count when math is actually being handled

This commit is contained in:
Ean Milligan 2025-08-06 14:40:46 -04:00
parent 6bf671f82d
commit 4a5e33c9a0
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/artigen/math/mathSolver.ts
View File

@ -77,8 +77,8 @@ export const mathSolver = (conf: MathConf[], wrapDetails = false): SolvedStep =>
for (let i = 1; i < conf.length; i++) { for (let i = 1; i < conf.length; i++) {
loopCountCheck('mathSolver.ts - checking for implicit multiplication'); loopCountCheck('mathSolver.ts - checking for implicit multiplication');
const prevConfAsStr = <string> conf[i - 1]; const prevConfAsStr = <string>conf[i - 1];
const curConfAsStr = <string> conf[i]; const curConfAsStr = <string>conf[i];
if (!signs.includes(curConfAsStr) && !signs.includes(prevConfAsStr)) { if (!signs.includes(curConfAsStr) && !signs.includes(prevConfAsStr)) {
// Both previous and current conf are operators, slip in the "*" // Both previous and current conf are operators, slip in the "*"
conf.splice(i, 0, '*'); conf.splice(i, 0, '*');
@ -98,11 +98,10 @@ export const mathSolver = (conf: MathConf[], wrapDetails = false): SolvedStep =>
loggingEnabled && log(LT.LOG, `Evaluating roll ${JSON.stringify(conf)} | Evaluating ${JSON.stringify(curOps)}`); loggingEnabled && log(LT.LOG, `Evaluating roll ${JSON.stringify(conf)} | Evaluating ${JSON.stringify(curOps)}`);
// Iterate thru all operators/operands in the conf // Iterate thru all operators/operands in the conf
for (let i = 0; i < conf.length; i++) { for (let i = 0; i < conf.length; i++) {
loopCountCheck('mathSolver.ts - evaluating roll');
loggingEnabled && log(LT.LOG, `Evaluating roll ${JSON.stringify(conf)} | Evaluating ${JSON.stringify(curOps)} | Checking ${JSON.stringify(conf[i])}`); loggingEnabled && log(LT.LOG, `Evaluating roll ${JSON.stringify(conf)} | Evaluating ${JSON.stringify(curOps)} | Checking ${JSON.stringify(conf[i])}`);
// Check if the current index is in the active tier of operators // Check if the current index is in the active tier of operators
if (curOps.includes(conf[i].toString())) { if (curOps.includes(conf[i].toString())) {
loopCountCheck('mathSolver.ts - evaluating roll');
// Grab the operands from before and after the operator // Grab the operands from before and after the operator
const operand1 = conf[i - 1]; const operand1 = conf[i - 1];
const operand2 = conf[i + 1]; const operand2 = conf[i + 1];
@ -198,7 +197,7 @@ export const mathSolver = (conf: MathConf[], wrapDetails = false): SolvedStep =>
stepSolve.details = conf[0].toString(); stepSolve.details = conf[0].toString();
} else { } else {
// Else fully populate the stepSolve with what was computed // Else fully populate the stepSolve with what was computed
const tempConf = <SolvedStep> conf[0]; const tempConf = <SolvedStep>conf[0];
stepSolve.total = tempConf.total; stepSolve.total = tempConf.total;
stepSolve.details = tempConf.details; stepSolve.details = tempConf.details;
stepSolve.containsCrit = tempConf.containsCrit; stepSolve.containsCrit = tempConf.containsCrit;