MNG [Moon Nation Game] Token

Reentrancy in MoonNation._transfer(address,address,uint256) (#1010-1068):
External calls:
- swapAndLiquify(contractTokenBalance) (#1047)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1102-1108)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1047)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#1059)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#951)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1148)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1157)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1149)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1168)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1178)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1159)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1169)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1180)
- _rOwned[_burn] = _rOwned[_burn].add(calculateBurnFee(amount).mul(_getRate())) (#1062)
- _rOwned[_marketingAndDev] = _rOwned[_marketingAndDev].add(calculateMarketingAndDevFee(amount).mul(_getRate())) (#1064)
- _tokenTransfer(from,to,amount,takeFee) (#1059)
- _rTotal = _rTotal.sub(rFee) (#915)
- _tokenTransfer(from,to,amount,takeFee) (#1059)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#953)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1177)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1167)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1158)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1179)
- _tOwned[_burn] = _tOwned[_burn].add(calculateBurnFee(amount)) (#1061)
- _tOwned[_marketingAndDev] = _tOwned[_marketingAndDev].add(calculateMarketingAndDevFee(amount)) (#1063)
Apply the check-effects-interactions pattern.

Additional information: link

Unable to verify that contract auditor is trusted: Certik, Quantstamp, Hacken, Solidity, Paladinsec, Openzeppelin, Verichains

Contract ownership is not renounced (belongs to a wallet)

Not a direct threat, but may indicate unreliable intentions of developer. Widespread names (e.g. Elon, King, Moon, Doge) are common among meme-tokens and scams. The allow to gain free hype and attract unexperienced investors.

MoonNation.addLiquidity(uint256,uint256) (#1111-1124) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
Ensure that all the return values of the function calls are used.

Additional information: link

MoonNation.constructor(address).marketingAndDev (#742) lacks a zero-check on :
- _marketingAndDev = marketingAndDev (#744)
Check that the address is not zero.

Additional information: link

MoonNation.allowance(address,address).owner (#788) shadows:
- Ownable.owner() (#412-414) (function)
MoonNation._approve(address,address,uint256).owner (#1002) shadows:
- Ownable.owner() (#412-414) (function)
Rename the local variables that shadow another component.

Additional information: link

MoonNation.setTaxFeePercent(uint256) (#877-879) should emit an event for:
- _taxFee = taxFee (#878)
MoonNation.setLiquidityFeePercent(uint256) (#881-883) should emit an event for:
- _liquidityFee = liquidityFee (#882)
MoonNation.setBurnFeePercent(uint256) (#885-887) should emit an event for:
- _burnFee = burnFee (#886)
MoonNation.setMaxTxPercent(uint256) (#889-893) should emit an event for:
- _maxTxAmount = _tTotal.mul(maxTxPercent).div(10 ** 2) (#890-892)
MoonNation.setMaxWalletAmount(uint256) (#895-897) should emit an event for:
- _maxWalletToken = setMaxWallet (#896)
MoonNation.setMarketingAndDevFeePercent(uint256) (#899-901) should emit an event for:
- marketingAndDevelopment = mFee (#900)
Emit an event for critical parameter changes.

Additional information: link

Reentrancy in MoonNation._transfer(address,address,uint256) (#1010-1068):
External calls:
- swapAndLiquify(contractTokenBalance) (#1047)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1102-1108)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1047)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#1059)
- _burnFee = _previousBurnFee (#994)
- _burnFee = 0 (#987)
- _tokenTransfer(from,to,amount,takeFee) (#1059)
- _liquidityFee = _previousLiquidityFee (#993)
- _liquidityFee = 0 (#986)
- _tokenTransfer(from,to,amount,takeFee) (#1059)
- _tFeeTotal = _tFeeTotal.add(tFee) (#916)
- _tokenTransfer(from,to,amount,takeFee) (#1059)
- _taxFee = _previousTaxFee (#992)
- _taxFee = 0 (#985)
- _tokenTransfer(from,to,amount,takeFee) (#1059)
- marketingAndDevelopment = _previousMarketingAndDev (#995)
- marketingAndDevelopment = 0 (#988)
Reentrancy in MoonNation.constructor(address) (#742-760):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#747-748)
State variables written after the call(s):
- _isExcludedFromFee[owner()] = true (#754)
- _isExcludedFromFee[address(this)] = true (#755)
- _isExcludedFromFee[_marketingAndDev] = true (#756)
- _isExcludedFromFee[_burn] = true (#757)
- uniswapV2Router = _uniswapV2Router (#751)
Reentrancy in MoonNation.swapAndLiquify(uint256) (#1070-1091):
External calls:
- swapTokensForEth(half) (#1082)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1102-1108)
- addLiquidity(otherHalf,newBalance) (#1088)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1088)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (#1088)
- _allowances[owner][spender] = amount (#1006)
Reentrancy in MoonNation.transferFrom(address,address,uint256) (#797-801):
External calls:
- _transfer(sender,recipient,amount) (#798)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1102-1108)
External calls sending eth:
- _transfer(sender,recipient,amount) (#798)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#799)
- _allowances[owner][spender] = amount (#1006)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in MoonNation._transfer(address,address,uint256) (#1010-1068):
External calls:
- swapAndLiquify(contractTokenBalance) (#1047)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1102-1108)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1047)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (#1152)
- _tokenTransfer(from,to,amount,takeFee) (#1059)
- Transfer(sender,recipient,tTransferAmount) (#1172)
- _tokenTransfer(from,to,amount,takeFee) (#1059)
- Transfer(sender,recipient,tTransferAmount) (#1162)
- _tokenTransfer(from,to,amount,takeFee) (#1059)
- Transfer(sender,recipient,tTransferAmount) (#1183)
- _tokenTransfer(from,to,amount,takeFee) (#1059)
- Transfer(from,_burn,calculateBurnFee(amount)) (#1065)
- Transfer(from,_marketingAndDev,calculateMarketingAndDevFee(amount)) (#1066)
Reentrancy in MoonNation.constructor(address) (#742-760):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#747-748)
Event emitted after the call(s):
- Transfer(address(0),_msgSender(),_tTotal) (#759)
Reentrancy in MoonNation.swapAndLiquify(uint256) (#1070-1091):
External calls:
- swapTokensForEth(half) (#1082)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1102-1108)
- addLiquidity(otherHalf,newBalance) (#1088)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1088)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1007)
- addLiquidity(otherHalf,newBalance) (#1088)
- SwapAndLiquify(half,newBalance,otherHalf) (#1090)
Reentrancy in MoonNation.transferFrom(address,address,uint256) (#797-801):
External calls:
- _transfer(sender,recipient,amount) (#798)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1102-1108)
External calls sending eth:
- _transfer(sender,recipient,amount) (#798)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1116-1123)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1007)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#799)
Apply the check-effects-interactions pattern.

Additional information: link

Ownable.unlock() (#459-464) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp > _lockTime,Contract is locked until 7 days) (#461)
Avoid relying on block.timestamp.

Additional information: link

Address.isContract(address) (#264-273) uses assembly
- INLINE ASM (#271)
Address._functionCallWithValue(address,bytes,uint256,string) (#357-378) uses assembly
- INLINE ASM (#370-373)
Do not use evm assembly.

Additional information: link

Address._functionCallWithValue(address,bytes,uint256,string) (#357-378) is never used and should be removed
Address.functionCall(address,bytes) (#317-319) is never used and should be removed
Address.functionCall(address,bytes,string) (#327-329) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#342-344) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#352-355) is never used and should be removed
Address.isContract(address) (#264-273) is never used and should be removed
Address.sendValue(address,uint256) (#291-297) is never used and should be removed
Context._msgData() (#236-239) is never used and should be removed
SafeMath.mod(uint256,uint256) (#209-211) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#225-228) is never used and should be removed
Remove unused functions.

Additional information: link

MoonNation._rTotal (#698) is set pre-construction with a non-constant function or state variable:
- (MAX - (MAX % _tTotal))
MoonNation._previousTaxFee (#706) is set pre-construction with a non-constant function or state variable:
- _taxFee
MoonNation._previousLiquidityFee (#709) is set pre-construction with a non-constant function or state variable:
- _liquidityFee
MoonNation._previousBurnFee (#712) is set pre-construction with a non-constant function or state variable:
- _burnFee
MoonNation._previousMarketingAndDev (#715) is set pre-construction with a non-constant function or state variable:
- marketingAndDevelopment
Remove any initialization of state variables via non-constant state variables or function calls. If variables must be set upon contract deployment, locate initialization in the constructor instead.

Additional information: link

Low level call in Address.sendValue(address,uint256) (#291-297):
- (success) = recipient.call{value: amount}() (#295)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#357-378):
- (success,returndata) = target.call{value: weiValue}(data) (#361)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence

Additional information: link

Function IUniswapV2Pair.DOMAIN_SEPARATOR() (#503) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#504) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#521) is not in mixedCase
Function IUniswapV2Router01.WETH() (#543) is not in mixedCase
Parameter MoonNation.setSwapAndLiquifyEnabled(bool)._enabled (#903) is not in mixedCase
Parameter MoonNation.calculateTaxFee(uint256)._amount (#956) is not in mixedCase
Parameter MoonNation.calculateLiquidityFee(uint256)._amount (#962) is not in mixedCase
Parameter MoonNation.calculateBurnFee(uint256)._amount (#968) is not in mixedCase
Parameter MoonNation.calculateMarketingAndDevFee(uint256)._amount (#975) is not in mixedCase
Variable MoonNation._marketingAndDev (#693) is not in mixedCase
Constant MoonNation._burn (#694) is not in UPPER_CASE_WITH_UNDERSCORES
Variable MoonNation._taxFee (#705) is not in mixedCase
Variable MoonNation._liquidityFee (#708) is not in mixedCase
Variable MoonNation._burnFee (#711) is not in mixedCase
Variable MoonNation._maxTxAmount (#724) is not in mixedCase
Variable MoonNation._maxWalletToken (#726) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (#237)" inContext (#231-240)
Remove redundant statements if they congest code but offer no value.

Additional information: link

Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#548) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#549)
Variable MoonNation._transferToExcluded(address,address,uint256).rTransferAmount (#1156) is too similar to MoonNation._transferFromExcluded(address,address,uint256).tTransferAmount (#1166)
Variable MoonNation._transferBothExcluded(address,address,uint256).rTransferAmount (#1176) is too similar to MoonNation._transferStandard(address,address,uint256).tTransferAmount (#1147)
Variable MoonNation._transferFromExcluded(address,address,uint256).rTransferAmount (#1166) is too similar to MoonNation._transferToExcluded(address,address,uint256).tTransferAmount (#1156)
Variable MoonNation.reflectionFromToken(uint256,bool).rTransferAmount (#836) is too similar to MoonNation._transferStandard(address,address,uint256).tTransferAmount (#1147)
Variable MoonNation._transferBothExcluded(address,address,uint256).rTransferAmount (#1176) is too similar to MoonNation._transferBothExcluded(address,address,uint256).tTransferAmount (#1176)
Variable MoonNation._transferFromExcluded(address,address,uint256).rTransferAmount (#1166) is too similar to MoonNation._getValues(uint256).tTransferAmount (#922)
Variable MoonNation._transferToExcluded(address,address,uint256).rTransferAmount (#1156) is too similar to MoonNation._transferToExcluded(address,address,uint256).tTransferAmount (#1156)
Variable MoonNation._transferToExcluded(address,address,uint256).rTransferAmount (#1156) is too similar to MoonNation._getValues(uint256).tTransferAmount (#922)
Variable MoonNation._getValues(uint256).rTransferAmount (#925) is too similar to MoonNation._transferStandard(address,address,uint256).tTransferAmount (#1147)
Variable MoonNation._transferFromExcluded(address,address,uint256).rTransferAmount (#1166) is too similar to MoonNation._transferStandard(address,address,uint256).tTransferAmount (#1147)
Variable MoonNation._transferBothExcluded(address,address,uint256).rTransferAmount (#1176) is too similar to MoonNation._transferFromExcluded(address,address,uint256).tTransferAmount (#1166)
Variable MoonNation._transferToExcluded(address,address,uint256).rTransferAmount (#1156) is too similar to MoonNation._transferStandard(address,address,uint256).tTransferAmount (#1147)
Variable MoonNation._transferStandard(address,address,uint256).rTransferAmount (#1147) is too similar to MoonNation._transferStandard(address,address,uint256).tTransferAmount (#1147)
Variable MoonNation._transferBothExcluded(address,address,uint256).rTransferAmount (#1176) is too similar to MoonNation._transferToExcluded(address,address,uint256).tTransferAmount (#1156)
Variable MoonNation._transferFromExcluded(address,address,uint256).rTransferAmount (#1166) is too similar to MoonNation._transferFromExcluded(address,address,uint256).tTransferAmount (#1166)
Variable MoonNation._transferBothExcluded(address,address,uint256).rTransferAmount (#1176) is too similar to MoonNation._getValues(uint256).tTransferAmount (#922)
Variable MoonNation._transferStandard(address,address,uint256).rTransferAmount (#1147) is too similar to MoonNation._getValues(uint256).tTransferAmount (#922)
Variable MoonNation._getValues(uint256).rTransferAmount (#925) is too similar to MoonNation._transferToExcluded(address,address,uint256).tTransferAmount (#1156)
Variable MoonNation._transferStandard(address,address,uint256).rTransferAmount (#1147) is too similar to MoonNation._transferBothExcluded(address,address,uint256).tTransferAmount (#1176)
Variable MoonNation._getValues(uint256).rTransferAmount (#925) is too similar to MoonNation._getValues(uint256).tTransferAmount (#922)
Variable MoonNation.reflectionFromToken(uint256,bool).rTransferAmount (#836) is too similar to MoonNation._transferFromExcluded(address,address,uint256).tTransferAmount (#1166)
Variable MoonNation._getValues(uint256).rTransferAmount (#925) is too similar to MoonNation._transferBothExcluded(address,address,uint256).tTransferAmount (#1176)
Variable MoonNation._transferFromExcluded(address,address,uint256).rTransferAmount (#1166) is too similar to MoonNation._transferBothExcluded(address,address,uint256).tTransferAmount (#1176)
Variable MoonNation.reflectionFromToken(uint256,bool).rTransferAmount (#836) is too similar to MoonNation._transferToExcluded(address,address,uint256).tTransferAmount (#1156)
Variable MoonNation._transferStandard(address,address,uint256).rTransferAmount (#1147) is too similar to MoonNation._transferFromExcluded(address,address,uint256).tTransferAmount (#1166)
Variable MoonNation.reflectionFromToken(uint256,bool).rTransferAmount (#836) is too similar to MoonNation._getValues(uint256).tTransferAmount (#922)
Variable MoonNation.reflectionFromToken(uint256,bool).rTransferAmount (#836) is too similar to MoonNation._transferBothExcluded(address,address,uint256).tTransferAmount (#1176)
Variable MoonNation._getValues(uint256).rTransferAmount (#925) is too similar to MoonNation._transferFromExcluded(address,address,uint256).tTransferAmount (#1166)
Variable MoonNation._transferStandard(address,address,uint256).rTransferAmount (#1147) is too similar to MoonNation._transferToExcluded(address,address,uint256).tTransferAmount (#1156)
Variable MoonNation._transferToExcluded(address,address,uint256).rTransferAmount (#1156) is too similar to MoonNation._transferBothExcluded(address,address,uint256).tTransferAmount (#1176)
Prevent variables from having similar names.

Additional information: link

MoonNation.slitherConstructorVariables() (#681-1190) uses literals with too many digits:
- _tTotal = 384400000 * 10 ** 9 (#697)
MoonNation.slitherConstructorVariables() (#681-1190) uses literals with too many digits:
- _maxTxAmount = 3000000 * 10 ** 9 (#724)
MoonNation.slitherConstructorVariables() (#681-1190) uses literals with too many digits:
- numTokensSellToAddToLiquidity = 1000000 * 10 ** 9 (#725)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

MoonNation._decimals (#703) should be constant
MoonNation._name (#701) should be constant
MoonNation._symbol (#702) should be constant
MoonNation._tTotal (#697) should be constant
MoonNation.numTokensSellToAddToLiquidity (#725) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#431-434)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#440-444)
geUnlockTime() should be declared external:
- Ownable.geUnlockTime() (#446-448)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (#451-456)
unlock() should be declared external:
- Ownable.unlock() (#459-464)
name() should be declared external:
- MoonNation.name() (#762-764)
symbol() should be declared external:
- MoonNation.symbol() (#766-768)
decimals() should be declared external:
- MoonNation.decimals() (#770-772)
totalSupply() should be declared external:
- MoonNation.totalSupply() (#774-776)
transfer(address,uint256) should be declared external:
- MoonNation.transfer(address,uint256) (#783-786)
allowance(address,address) should be declared external:
- MoonNation.allowance(address,address) (#788-790)
approve(address,uint256) should be declared external:
- MoonNation.approve(address,uint256) (#792-795)
transferFrom(address,address,uint256) should be declared external:
- MoonNation.transferFrom(address,address,uint256) (#797-801)
increaseAllowance(address,uint256) should be declared external:
- MoonNation.increaseAllowance(address,uint256) (#803-806)
decreaseAllowance(address,uint256) should be declared external:
- MoonNation.decreaseAllowance(address,uint256) (#808-811)
isExcludedFromReward(address) should be declared external:
- MoonNation.isExcludedFromReward(address) (#813-815)
totalFees() should be declared external:
- MoonNation.totalFees() (#817-819)
deliver(uint256) should be declared external:
- MoonNation.deliver(uint256) (#821-828)
reflectionFromToken(uint256,bool) should be declared external:
- MoonNation.reflectionFromToken(uint256,bool) (#830-839)
excludeFromReward(address) should be declared external:
- MoonNation.excludeFromReward(address) (#847-854)
excludeFromFee(address) should be declared external:
- MoonNation.excludeFromFee(address) (#869-871)
includeInFee(address) should be declared external:
- MoonNation.includeInFee(address) (#873-875)
setMaxWalletAmount(uint256) should be declared external:
- MoonNation.setMaxWalletAmount(uint256) (#895-897)
setSwapAndLiquifyEnabled(bool) should be declared external:
- MoonNation.setSwapAndLiquifyEnabled(bool) (#903-906)
isExcludedFromFee(address) should be declared external:
- MoonNation.isExcludedFromFee(address) (#998-1000)
Use the external attribute for functions never called from the contract.

Additional information: link

MoonNation.includeInReward(address) (#856-867) has costly operations inside a loop:
- _excluded.pop() (#863)
Use a local variable to hold the loop computation result.

Additional information: link

Token is deployed only at one blockchain

Contract has 10% buy tax and 10% sell tax.
Taxes are low but contract ownership is not renounced. Token has a high risk of becoming a honeypot.

Average 30d number of PancakeSwap swaps is low.

Unable to find code repository for the project

Token is not listed at Mobula.Finance

Additional information: link