Steadefi

Steadefi
DeFiHardhatFoundryOracle
35,000 USDC
View results
Submission Details
Severity: low
Invalid

Name mapping parameters

Summary

Using names for parameters in mappings increases code readability

Tools Used

Manual Review

Recommendations

Add named parameters in mappings:

// Mapping of token to Chainlink USD price feed
- mapping(address => address) public feeds;
+ mapping(address token => address priceFeed) public feeds;
Updates

Lead Judging Commences

hans Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

INFO: Bad naming

Support

FAQs

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