Submission Details
Severity:
Gas Optimization Opportunities in the `RockPaperScissors::_determineWinner()` function
Description: The winner determination logic could be optimized to reduce gas costs.
Impact: Higher than necessary gas costs for turn processing
Proof of Concept:
// Current implementation
if (game.moveA == game.moveB) {
// Tie
} else if ((game.moveA == Move.Rock && game.moveB == Move.Scissors) ||
(game.moveA == Move.Paper && game.moveB == Move.Rock) ||
(game.moveA == Move.Scissors && game.moveB == Move.Paper)) {
// Player A wins
}
Recommended Mitigation: Use bitwise operations or mathematical comparisons instead of multiple conditionals.
Updates
Appeal created
m3dython about 2 months ago
Submission Judgement Published Reason: Non-acceptable severity
Assigned finding tags:
Informational
Code suggestions or observations that do not pose a direct security risk.
Gas Optimization
Rewards breakdown
nSLOC
349This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.