Submission Details
Severity:
`eggFindThreshold` in `EggHuntGame` contract should be constant to save gas.
Summary
The eggFindThreshold value is never changed, so it should be marked as constant to save gas.
Vulnerability Details
In the EggHuntGame contract, eggFindThreshold is declared as a state variable. Storing this as a state variable unnecessarily consumes gas.
uint256 public eggFindThreshold = 20; // Default is a 20% chance
Impact
Making the variable constant instead of a state variable saves gas.
Tools Used
manual code review.
Recommendations
uint256 public CONSTANT EGG_FIND_THRESHOLD = 20;
Updates
Lead Judging Commences
m3dython 3 months ago
Submission Judgement Published Reason: Out of scope
Assigned finding tags:
Gas optimization
Strategy to save gas and minimize transaction costs
Rewards breakdown
nSLOC
129This 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.