Sparkn

CodeFox Inc.
DeFiFoundryProxy
15,000 USDC
View results
Submission Details
Severity: low

Magic number is used

To improve the code’s readability and facilitate refactoring, consider using BASIS_POINTS constant instead of magic number at the Distributor._distribute function L135:

60: // a constant value of 10,000 (basis points) = 100%
61: uint256 private constant BASIS_POINTS = 10000;
...
135: if (totalPercentage != (10000 - COMMISSION_FEE)) {

https://github.com/Cyfrin/2023-08-sparkn/blob/0f139b2dc53905700dd29a01451b330f829653e9/src/Distributor.sol#L135

Support

FAQs

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