MoonFarm is a DeFi and Centralized Finance (CeFi) hybrid yield farming aggregator. It provides the best of both worlds, offering all the cost-effectiveness, easy access, transparency and rapidity of DeFi yield farming, in addition to offering the security associated with a centralized financial ecosystem
Moon Farm optimises Yield Farming interest (APY/ APR) and provides users with the abilities to choose what farms they want to use or the system will automatically determine the best farming strategies using the yield optimization engine.
Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
Different versions of Solidity is used:
- Version used: ['>=0.6.0<0.8.0', '^0.6.0']
- >=0.6.0<0.8.0 (#3)
- >=0.6.0<0.8.0 (#28)
- >=0.6.0<0.8.0 (#106)
- >=0.6.0<0.8.0 (#268)
- ^0.6.0 (#574)
Use one Solidity version.
Additional information: link
Context._msgData() (#20-23) is never used and should be removed
ERC20._burn(address,uint256) (#513-521) is never used and should be removed
ERC20._setupDecimals(uint8) (#551-553) is never used and should be removed
SafeMath.div(uint256,uint256) (#206-208) is never used and should be removed
SafeMath.div(uint256,uint256,string) (#222-228) is never used and should be removed
SafeMath.mod(uint256,uint256) (#242-244) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#258-261) is never used and should be removed
SafeMath.mul(uint256,uint256) (#180-192) is never used and should be removed
SafeMath.sub(uint256,uint256) (#149-151) is never used and should be removed
Remove unused functions.
Additional information: link
Pragma version>=0.6.0<0.8.0 (#3) is too complex
Pragma version>=0.6.0<0.8.0 (#28) is too complex
Pragma version>=0.6.0<0.8.0 (#106) is too complex
Pragma version>=0.6.0<0.8.0 (#268) is too complex
Pragma version^0.6.0 (#574) allows old versions
solc-0.6.1 is not recommended for deployment
Deploy with any of the following Solidity versions: 0.5.16 - 0.5.17, 0.6.11 - 0.6.12, 0.7.5 - 0.7.6 Use a simple pragma version that allows any of these versions. Consider using the latest version of Solidity for testing.
Additional information: link
Redundant expression "this (#21)" inContext (#15-24)
Remove redundant statements if they congest code but offer no value.
Additional information: link
MFO.constructor() (#578-580) uses literals with too many digits:
- _mint(msg.sender,100000000 * (10 ** uint256(decimals()))) (#579)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
name() should be declared external:
- ERC20.name() (#328-330)
symbol() should be declared external:
- ERC20.symbol() (#336-338)
totalSupply() should be declared external:
- ERC20.totalSupply() (#360-362)
balanceOf(address) should be declared external:
- ERC20.balanceOf(address) (#367-369)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (#379-382)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (#387-389)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (#398-401)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (#416-420)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (#434-437)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (#453-456)
Use the external attribute for functions never called from the contract.
Additional information: link
Unable to find Discord account