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

You don't need to set totalFees to 0 because it will be automatically set to 0 anyway

Summary

Variables that aren't set to anything in solidity will be initialized to 0. You are wasting gas during deployment by setting totalFees to 0 when it is already 0.

Vulnerability Details

uint64 public totalFees = 0;

Impact

You are making your contract more expensive to deploy.

Tools Used

Manual review

Recommendations

Delete =0:

uint64 public totalFees;
Updates

Lead Judging Commences

Hamiltonite Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Gas optimizations

Support

FAQs

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