DeFiFoundry
20,000 USDC
View results
Submission Details
Severity: low
Invalid

Move structs to a seperate file

Summary

The staking contract defines structs directly within the main contract file. While this approach is functional, it can lead to a less organized and maintainable codebase, especially as the project scales.

Impact

As the project scales, there might be a need to reuse the structs defined in the staking contract. Seperating the structs from the main contract makes it easier to inherit just the struct file where needed without having to inherit the entire staking contract file. Leading to less deployment cost for future contracts

Tools Used

Manual Review

Recommendations

Move struct definitions to a separate file, and import them into the necessary contracts. This will enhance code organization and reusability, making it easier to maintain and update the project as it grows.

Updates

Lead Judging Commences

inallhonesty Lead Judge 10 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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