Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Severity: low
Invalid

Storage variables set on deployment only, should be immutable

Summary

Since the organizer and entranceFee and predictionFee and scoreBoard is set on deployment and shouldn't be changed, they should be immutable. This also saves gas.

Recommendations

- address public organizer;
- address[] public players;
- uint256 public entranceFee;
- uint256 public predictionFee;
- ScoreBoard public scoreBoard;
+ address public immutable organizer;
+ uint256 public immutable entranceFee;
+ uint256 public immutable predictionFee;
+ ScoreBoard public immutable scoreBoard;
+ address[] public players;
Updates

Lead Judging Commences

NightHawK Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.