Function ordering does not follow the Solidity style guide
Summary
According to the Solidity style guide, functions should be laid out in the following order: constructor(), receive(), fallback(), external, public, internal, private. The functions listed below do not follow this layout. (If at least one function was found to be out of order, the contract and one function is listed. For the listed contracts, all functions should be examined for proper ordering.)
Vulnerability Details
There is 1 instance of this issue.
| File Link | Instance Count | Instance Link |
|---|---|---|
| Beedle.sol | 1 | 9 |
Impact
Decreased readability and maintainability.
Tools Used
baudit: a custom static code analysis tool; manual review
Recommendations
To increase the readability and to increase standardization, order functions per the Solidity style guide.
Prize pool breakdown
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.