-1 loop per iteration by only doing rollDist when needed

This commit is contained in:
Ean Milligan 2025-08-06 14:30:41 -04:00
parent 1bb8c1a308
commit 587d5aa19d
3 changed files with 4 additions and 4 deletions

View File

@ -170,7 +170,7 @@ export const runCmd = (rollRequest: QueuedRoll): SolvedRoll => {
returnMsg.line3 = line3;
// Reduce rollDist maps into a single map
returnMsg.rollDistributions = reduceRollDistMaps(tempRollDists);
if (rollRequest.modifiers.rollDist) returnMsg.rollDistributions = reduceRollDistMaps(tempRollDists);
} catch (e) {
// Fill in the return block
const solverError = e as Error;

View File

@ -278,7 +278,7 @@ Please click on "<@${botId}> *Click to see attachment*" above this message to se
details: returnMsg.line3,
},
counts: rollRequest.modifiers.count ? returnMsg.counts : null,
rollDistributions: returnMsg.rollDistributions.entries().toArray(),
rollDistributions: rollRequest.modifiers.rollDist ? returnMsg.rollDistributions.entries().toArray() : null,
},
}),
{

View File

@ -201,7 +201,7 @@ export const tokenizeMath = (
const formattedRoll = formatRoll(executedRoll, modifiers);
mathConf[i] = formattedRoll.solvedStep;
countDetails.push(formattedRoll.countDetails);
rollDists.push(formattedRoll.rollDistributions);
if (modifiers.rollDist) rollDists.push(formattedRoll.rollDistributions);
}
}
@ -318,7 +318,7 @@ export const tokenizeMath = (
const formattedRoll = formatRoll(executedRoll, modifiers);
mathConf[rollGroupIdx] = formattedRoll.solvedStep;
countDetails.push(formattedRoll.countDetails);
rollDists.push(formattedRoll.rollDistributions);
if (modifiers.rollDist) rollDists.push(formattedRoll.rollDistributions);
});
}