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

fix (5)-5 so it does +-5 instead of *-5

This commit is contained in:
Ean Milligan 2025-05-03 21:41:23 -04:00
parent 058695415e
commit 504df7c80f
2 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -72,6 +72,7 @@ export const mathSolver = (conf: MathConf[], wrapDetails = false): SolvedStep =>
// Look for any implicit multiplication that may have been missed // Look for any implicit multiplication that may have been missed
// Start at index 1 as there will never be implicit multiplication before the first element // Start at index 1 as there will never be implicit multiplication before the first element
loggingEnabled && log(LT.LOG, `Checking for missing implicit multiplication ${JSON.stringify(conf)}`);
for (let i = 1; i < conf.length; i++) { for (let i = 1; i < conf.length; i++) {
loopCountCheck(); loopCountCheck();

8
src/artigen/math/mathTokenizer.ts
View File

@ -16,7 +16,9 @@ import { legalMathOperators } from 'artigen/utils/legalMath.ts';
import { loggingEnabled } from 'artigen/utils/logFlag.ts'; import { loggingEnabled } from 'artigen/utils/logFlag.ts';
import { assertParenBalance } from 'artigen/utils/parenBalance.ts'; import { assertParenBalance } from 'artigen/utils/parenBalance.ts';
const operators = ['(', ')', '^', '*', '/', '%', '+', '-']; // minusOps are operators that will cause a negative sign to collapse into a number (in cases like + - 1)
const minusOps = ['(', '^', '*', '/', '%', '+', '-'];
const allOps = [...minusOps, ')'];
export const tokenizeMath = (cmd: string, modifiers: RollModifiers, previousResults: number[]): [ReturnData[], CountDetails[]] => { export const tokenizeMath = (cmd: string, modifiers: RollModifiers, previousResults: number[]): [ReturnData[], CountDetails[]] => {
const countDetails: CountDetails[] = []; const countDetails: CountDetails[] = [];
@ -132,7 +134,7 @@ export const tokenizeMath = (cmd: string, modifiers: RollModifiers, previousResu
} else { } else {
throw new Error(`IllegalVariable_${curMathConfStr}`); throw new Error(`IllegalVariable_${curMathConfStr}`);
} }
} else if (![...operators, ...legalMathOperators].includes(curMathConfStr)) { } else if (![...allOps, ...legalMathOperators].includes(curMathConfStr)) {
// If nothing else has handled it by now, try it as a roll // If nothing else has handled it by now, try it as a roll
const formattedRoll = generateFormattedRoll(curMathConfStr, modifiers); const formattedRoll = generateFormattedRoll(curMathConfStr, modifiers);
mathConf[i] = formattedRoll.solvedStep; mathConf[i] = formattedRoll.solvedStep;
@ -140,7 +142,7 @@ export const tokenizeMath = (cmd: string, modifiers: RollModifiers, previousResu
} }
// Identify if we are in a state where the current number is a negative number // Identify if we are in a state where the current number is a negative number
if (mathConf[i - 1] === '-' && ((!mathConf[i - 2] && mathConf[i - 2] !== 0) || operators.includes(<string> mathConf[i - 2]))) { if (mathConf[i - 1] === '-' && ((!mathConf[i - 2] && mathConf[i - 2] !== 0) || minusOps.includes(<string> mathConf[i - 2]))) {
if (typeof mathConf[i] === 'string') { if (typeof mathConf[i] === 'string') {
// Current item is a mathOp, need to insert a "-1 *" before it // Current item is a mathOp, need to insert a "-1 *" before it
mathConf.splice(i - 1, 1, ...[parseFloat('-1'), '*']); mathConf.splice(i - 1, 1, ...[parseFloat('-1'), '*']);