LKC [LuckyCoin] Token

Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)

LuckyCoin.addLiquidity(uint256,uint256) (#1049-1062) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
Ensure that an arbitrary user cannot withdraw unauthorized funds.

Additional information: link

Reentrancy in LuckyCoin._transfer(address,address,uint256) (#979-1006):
External calls:
- swapAndLiquify(contractTokenBalance) (#1001)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1040-1046)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1001)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
State variables written after the call(s):
- _tokenTransfer(from,to,amount) (#1005)
- _rOwned[address(this)] = _rOwned[address(this)].add(rLiquidity) (#929)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1114)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1105)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1106)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#870)
- _rOwned[sender] = _rOwned[sender].sub(rAmount) (#1125)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1116)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#1126)
- _rOwned[recipient] = _rOwned[recipient].add(rTransferAmount) (#872)
- _tokenTransfer(from,to,amount) (#1005)
- _rTotal = _rTotal.sub(rFee) (#884)
- _tokenTransfer(from,to,amount) (#1005)
- _tOwned[address(this)] = _tOwned[address(this)].add(tLiquidity) (#931)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#869)
- _tOwned[sender] = _tOwned[sender].sub(tAmount) (#1124)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#1115)
- _tOwned[recipient] = _tOwned[recipient].add(tTransferAmount) (#871)
Apply the check-effects-interactions pattern.

Additional information: link

Contract ownership is not renounced (belongs to a wallet)

Combination 1: Reentrancy vulnerabilities + Functions that send Ether to arbitraty destination. Usual for scams. May be justified by some complex mechanics (e.g. rebase, reflections). DYOR & manual audit are advised.

LuckyCoin.addLiquidity(uint256,uint256) (#1049-1062) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
Ensure that all the return values of the function calls are used.

Additional information: link

Redundant expression "this (#242)" inContext (#236-245)
Remove redundant statements if they congest code but offer no value.

Additional information: link

LuckyCoin.slitherConstructorVariables() (#684-1175) uses literals with too many digits:
- _tTotal = 1000000 * 10 ** 9 * 10 ** 8 (#698)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

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

Additional information: link

Variable LuckyCoin._transferFromExcluded(address,address,uint256).rTransferAmount (#1123) is too similar to LuckyCoin._getTValues(uint256).tTransferAmount (#897)
Variable LuckyCoin._transferFromExcluded(address,address,uint256).rTransferAmount (#1123) is too similar to LuckyCoin._transferFromExcluded(address,address,uint256).tTransferAmount (#1123)
Variable LuckyCoin._getValues(uint256).rTransferAmount (#890) is too similar to LuckyCoin._transferFromExcluded(address,address,uint256).tTransferAmount (#1123)
Variable LuckyCoin._transferBothExcluded(address,address,uint256).rTransferAmount (#868) is too similar to LuckyCoin._transferToExcluded(address,address,uint256).tTransferAmount (#1113)
Variable LuckyCoin._transferStandard(address,address,uint256).rTransferAmount (#1104) is too similar to LuckyCoin._transferFromExcluded(address,address,uint256).tTransferAmount (#1123)
Variable LuckyCoin._transferFromExcluded(address,address,uint256).rTransferAmount (#1123) is too similar to LuckyCoin._transferStandard(address,address,uint256).tTransferAmount (#1104)
Variable LuckyCoin.reflectionFromToken(uint256,bool).rTransferAmount (#833) is too similar to LuckyCoin._getValues(uint256).tTransferAmount (#889)
Variable LuckyCoin._transferStandard(address,address,uint256).rTransferAmount (#1104) is too similar to LuckyCoin._getTValues(uint256).tTransferAmount (#897)
Variable LuckyCoin._getValues(uint256).rTransferAmount (#890) is too similar to LuckyCoin._transferBothExcluded(address,address,uint256).tTransferAmount (#868)
Variable LuckyCoin._transferBothExcluded(address,address,uint256).rTransferAmount (#868) is too similar to LuckyCoin._transferFromExcluded(address,address,uint256).tTransferAmount (#1123)
Variable LuckyCoin._transferToExcluded(address,address,uint256).rTransferAmount (#1113) is too similar to LuckyCoin._transferToExcluded(address,address,uint256).tTransferAmount (#1113)
Variable LuckyCoin.reflectionFromToken(uint256,bool).rTransferAmount (#833) is too similar to LuckyCoin._transferBothExcluded(address,address,uint256).tTransferAmount (#868)
Variable LuckyCoin.reflectionFromToken(uint256,bool).rTransferAmount (#833) is too similar to LuckyCoin._transferToExcluded(address,address,uint256).tTransferAmount (#1113)
Variable LuckyCoin._transferStandard(address,address,uint256).rTransferAmount (#1104) is too similar to LuckyCoin._getValues(uint256).tTransferAmount (#889)
Variable LuckyCoin._transferStandard(address,address,uint256).rTransferAmount (#1104) is too similar to LuckyCoin._transferBothExcluded(address,address,uint256).tTransferAmount (#868)
Variable LuckyCoin._transferStandard(address,address,uint256).rTransferAmount (#1104) is too similar to LuckyCoin._transferStandard(address,address,uint256).tTransferAmount (#1104)
Variable LuckyCoin._transferToExcluded(address,address,uint256).rTransferAmount (#1113) is too similar to LuckyCoin._getTValues(uint256).tTransferAmount (#897)
Variable LuckyCoin._transferFromExcluded(address,address,uint256).rTransferAmount (#1123) is too similar to LuckyCoin._getValues(uint256).tTransferAmount (#889)
Variable LuckyCoin._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#905) is too similar to LuckyCoin._transferStandard(address,address,uint256).tTransferAmount (#1104)
Variable LuckyCoin._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#905) is too similar to LuckyCoin._getValues(uint256).tTransferAmount (#889)
Variable LuckyCoin._transferFromExcluded(address,address,uint256).rTransferAmount (#1123) is too similar to LuckyCoin._transferToExcluded(address,address,uint256).tTransferAmount (#1113)
Variable LuckyCoin._getValues(uint256).rTransferAmount (#890) is too similar to LuckyCoin._getTValues(uint256).tTransferAmount (#897)
Variable LuckyCoin._getValues(uint256).rTransferAmount (#890) is too similar to LuckyCoin._transferStandard(address,address,uint256).tTransferAmount (#1104)
Variable LuckyCoin._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#905) is too similar to LuckyCoin._transferBothExcluded(address,address,uint256).tTransferAmount (#868)
Variable LuckyCoin._transferStandard(address,address,uint256).rTransferAmount (#1104) is too similar to LuckyCoin._transferToExcluded(address,address,uint256).tTransferAmount (#1113)
Variable LuckyCoin.reflectionFromToken(uint256,bool).rTransferAmount (#833) is too similar to LuckyCoin._getTValues(uint256).tTransferAmount (#897)
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#551) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#552)
Variable LuckyCoin._getValues(uint256).rTransferAmount (#890) is too similar to LuckyCoin._getValues(uint256).tTransferAmount (#889)
Variable LuckyCoin._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#905) is too similar to LuckyCoin._transferFromExcluded(address,address,uint256).tTransferAmount (#1123)
Variable LuckyCoin._transferToExcluded(address,address,uint256).rTransferAmount (#1113) is too similar to LuckyCoin._transferStandard(address,address,uint256).tTransferAmount (#1104)
Variable LuckyCoin.reflectionFromToken(uint256,bool).rTransferAmount (#833) is too similar to LuckyCoin._transferStandard(address,address,uint256).tTransferAmount (#1104)
Variable LuckyCoin._transferBothExcluded(address,address,uint256).rTransferAmount (#868) is too similar to LuckyCoin._getTValues(uint256).tTransferAmount (#897)
Variable LuckyCoin._transferBothExcluded(address,address,uint256).rTransferAmount (#868) is too similar to LuckyCoin._transferStandard(address,address,uint256).tTransferAmount (#1104)
Variable LuckyCoin._transferFromExcluded(address,address,uint256).rTransferAmount (#1123) is too similar to LuckyCoin._transferBothExcluded(address,address,uint256).tTransferAmount (#868)
Variable LuckyCoin._transferBothExcluded(address,address,uint256).rTransferAmount (#868) is too similar to LuckyCoin._getValues(uint256).tTransferAmount (#889)
Variable LuckyCoin._transferToExcluded(address,address,uint256).rTransferAmount (#1113) is too similar to LuckyCoin._getValues(uint256).tTransferAmount (#889)
Variable LuckyCoin._getValues(uint256).rTransferAmount (#890) is too similar to LuckyCoin._transferToExcluded(address,address,uint256).tTransferAmount (#1113)
Variable LuckyCoin._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#905) is too similar to LuckyCoin._getTValues(uint256).tTransferAmount (#897)
Variable LuckyCoin.reflectionFromToken(uint256,bool).rTransferAmount (#833) is too similar to LuckyCoin._transferFromExcluded(address,address,uint256).tTransferAmount (#1123)
Variable LuckyCoin._transferBothExcluded(address,address,uint256).rTransferAmount (#868) is too similar to LuckyCoin._transferBothExcluded(address,address,uint256).tTransferAmount (#868)
Variable LuckyCoin._transferToExcluded(address,address,uint256).rTransferAmount (#1113) is too similar to LuckyCoin._transferFromExcluded(address,address,uint256).tTransferAmount (#1123)
Variable LuckyCoin._transferToExcluded(address,address,uint256).rTransferAmount (#1113) is too similar to LuckyCoin._transferBothExcluded(address,address,uint256).tTransferAmount (#868)
Variable LuckyCoin._getRValues(uint256,uint256,uint256,uint256).rTransferAmount (#905) is too similar to LuckyCoin._transferToExcluded(address,address,uint256).tTransferAmount (#1113)
Prevent variables from having similar names.

Additional information: link

setRouterAddress(address) should be declared external:
- LuckyCoin.setRouterAddress(address) (#1162-1167)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#436-439)
symbol() should be declared external:
- LuckyCoin.symbol() (#763-765)
decimals() should be declared external:
- LuckyCoin.decimals() (#767-769)
decreaseAllowance(address,uint256) should be declared external:
- LuckyCoin.decreaseAllowance(address,uint256) (#805-808)
excludeFromFee(address) should be declared external:
- LuckyCoin.excludeFromFee(address) (#1132-1134)
deliver(uint256) should be declared external:
- LuckyCoin.deliver(uint256) (#818-825)
approve(address,uint256) should be declared external:
- LuckyCoin.approve(address,uint256) (#789-792)
includeInFee(address) should be declared external:
- LuckyCoin.includeInFee(address) (#1136-1138)
transfer(address,uint256) should be declared external:
- LuckyCoin.transfer(address,uint256) (#780-783)
setSwapAndLiquifyEnabled(bool) should be declared external:
- LuckyCoin.setSwapAndLiquifyEnabled(bool) (#1169-1172)
totalSupply() should be declared external:
- LuckyCoin.totalSupply() (#771-773)
transferFrom(address,address,uint256) should be declared external:
- LuckyCoin.transferFrom(address,address,uint256) (#794-798)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#445-449)
geUnlockTime() should be declared external:
- Ownable.geUnlockTime() (#451-453)
allowance(address,address) should be declared external:
- LuckyCoin.allowance(address,address) (#785-787)
name() should be declared external:
- LuckyCoin.name() (#759-761)
excludeFromReward(address) should be declared external:
- LuckyCoin.excludeFromReward(address) (#844-852)
unlock() should be declared external:
- Ownable.unlock() (#464-469)
totalFees() should be declared external:
- LuckyCoin.totalFees() (#814-816)
isExcludedFromReward(address) should be declared external:
- LuckyCoin.isExcludedFromReward(address) (#810-812)
isExcludedFromFee(address) should be declared external:
- LuckyCoin.isExcludedFromFee(address) (#967-969)
increaseAllowance(address,uint256) should be declared external:
- LuckyCoin.increaseAllowance(address,uint256) (#800-803)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (#456-461)
reflectionFromToken(uint256,bool) should be declared external:
- LuckyCoin.reflectionFromToken(uint256,bool) (#827-836)
Use the external attribute for functions never called from the contract.

Additional information: link

LuckyCoin.allowance(address,address).owner (#785) shadows:
- Ownable.owner() (#417-419) (function)
LuckyCoin._approve(address,address,uint256).owner (#971) shadows:
- Ownable.owner() (#417-419) (function)
Rename the local variables that shadow another component.

Additional information: link

LuckyCoin.setChartityFeePercent(uint256) (#1152-1154) should emit an event for:
- _charityFee = charityFee (#1153)
LuckyCoin.setBurnFeePercent(uint256) (#1156-1158) should emit an event for:
- _burnFee = burnFee (#1157)
LuckyCoin.setLiquidityFeePercent(uint256) (#1148-1150) should emit an event for:
- _liquidityFee = liquidityFee (#1149)
LuckyCoin.setTaxFeePercent(uint256) (#1144-1146) should emit an event for:
- _taxFee = taxFee (#1145)
Emit an event for critical parameter changes.

Additional information: link

LuckyCoin.setcharityWallet(address).newWallet (#1140) lacks a zero-check on :
- charityWallet = newWallet (#1141)
Check that the address is not zero.

Additional information: link

Reentrancy in LuckyCoin.swapAndLiquify(uint256) (#1008-1029):
External calls:
- swapTokensForEth(half) (#1020)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1040-1046)
- addLiquidity(otherHalf,newBalance) (#1026)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1026)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (#1026)
- _allowances[owner][spender] = amount (#975)
Reentrancy in LuckyCoin.transferFrom(address,address,uint256) (#794-798):
External calls:
- _transfer(sender,recipient,amount) (#795)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1040-1046)
External calls sending eth:
- _transfer(sender,recipient,amount) (#795)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#796)
- _allowances[owner][spender] = amount (#975)
Reentrancy in LuckyCoin._transfer(address,address,uint256) (#979-1006):
External calls:
- swapAndLiquify(contractTokenBalance) (#1001)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1040-1046)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1001)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
State variables written after the call(s):
- _tokenTransfer(from,to,amount) (#1005)
- _burnFee = _previousBurnFee (#963)
- _burnFee = 0 (#957)
- _tokenTransfer(from,to,amount) (#1005)
- _charityFee = _previouscharityFee (#964)
- _charityFee = 0 (#956)
- _tokenTransfer(from,to,amount) (#1005)
- _liquidityFee = _previousLiquidityFee (#962)
- _liquidityFee = 0 (#955)
- _liquidityFee = 0 (#1088)
- _liquidityFee = _previousLiquidityFee (#1096)
- _tokenTransfer(from,to,amount) (#1005)
- _previousBurnFee = _burnFee (#951)
- _tokenTransfer(from,to,amount) (#1005)
- _previousLiquidityFee = _liquidityFee (#950)
- _tokenTransfer(from,to,amount) (#1005)
- _previousTaxFee = _taxFee (#949)
- _tokenTransfer(from,to,amount) (#1005)
- _previouscharityFee = _charityFee (#952)
- _tokenTransfer(from,to,amount) (#1005)
- _tFeeTotal = _tFeeTotal.add(tFee) (#885)
- _tokenTransfer(from,to,amount) (#1005)
- _taxFee = _previousTaxFee (#961)
- _taxFee = 0 (#954)
- _taxFee = 0 (#1087)
- _taxFee = _previousTaxFee (#1095)
Reentrancy in LuckyCoin.setRouterAddress(address) (#1162-1167):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_newPancakeRouter.factory()).createPair(address(this),_newPancakeRouter.WETH()) (#1165)
State variables written after the call(s):
- uniswapV2Router = _newPancakeRouter (#1166)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in LuckyCoin.swapAndLiquify(uint256) (#1008-1029):
External calls:
- swapTokensForEth(half) (#1020)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1040-1046)
- addLiquidity(otherHalf,newBalance) (#1026)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1026)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#976)
- addLiquidity(otherHalf,newBalance) (#1026)
- SwapAndLiquify(half,newBalance,otherHalf) (#1028)
Reentrancy in LuckyCoin._transfer(address,address,uint256) (#979-1006):
External calls:
- swapAndLiquify(contractTokenBalance) (#1001)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1040-1046)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1001)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
Event emitted after the call(s):
- Transfer(sender,recipient,tTransferAmount) (#1109)
- _tokenTransfer(from,to,amount) (#1005)
- Transfer(sender,recipient,tTransferAmount) (#1129)
- _tokenTransfer(from,to,amount) (#1005)
- Transfer(sender,recipient,tTransferAmount) (#1119)
- _tokenTransfer(from,to,amount) (#1005)
- Transfer(sender,recipient,tTransferAmount) (#875)
- _tokenTransfer(from,to,amount) (#1005)
Reentrancy in LuckyCoin.transferFrom(address,address,uint256) (#794-798):
External calls:
- _transfer(sender,recipient,amount) (#795)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1040-1046)
External calls sending eth:
- _transfer(sender,recipient,amount) (#795)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1054-1061)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#976)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#796)
Apply the check-effects-interactions pattern.

Additional information: link

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

Additional information: link

Address.isContract(address) (#269-278) uses assembly
- INLINE ASM (#276)
Address._functionCallWithValue(address,bytes,uint256,string) (#362-383) uses assembly
- INLINE ASM (#375-378)
Do not use evm assembly.

Additional information: link

LuckyCoin.includeInReward(address) (#854-865) has costly operations inside a loop:
- _excluded.pop() (#861)
Use a local variable to hold the loop computation result.

Additional information: link

Address.sendValue(address,uint256) (#296-302) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#347-349) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#230-233) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#357-360) is never used and should be removed
Context._msgData() (#241-244) is never used and should be removed
SafeMath.mod(uint256,uint256) (#214-216) is never used and should be removed
Address._functionCallWithValue(address,bytes,uint256,string) (#362-383) is never used and should be removed
Address.isContract(address) (#269-278) is never used and should be removed
Address.functionCall(address,bytes,string) (#332-334) is never used and should be removed
Address.functionCall(address,bytes) (#322-324) is never used and should be removed
Remove unused functions.

Additional information: link

LuckyCoin._rTotal (#699) is set pre-construction with a non-constant function or state variable:
- (MAX - (MAX % _tTotal))
LuckyCoin._previousLiquidityFee (#710) is set pre-construction with a non-constant function or state variable:
- _liquidityFee
LuckyCoin._previousBurnFee (#713) is set pre-construction with a non-constant function or state variable:
- _burnFee
LuckyCoin._previousTaxFee (#707) is set pre-construction with a non-constant function or state variable:
- _taxFee
LuckyCoin._previouscharityFee (#717) is set pre-construction with a non-constant function or state variable:
- _charityFee
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._functionCallWithValue(address,bytes,uint256,string) (#362-383):
- (success,returndata) = target.call{value: weiValue}(data) (#366)
Low level call in Address.sendValue(address,uint256) (#296-302):
- (success) = recipient.call{value: amount}() (#300)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence

Additional information: link

Variable LuckyCoin._taxFee (#706) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#509) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#525) is not in mixedCase
Variable LuckyCoin._burnFee (#712) is not in mixedCase
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (#508) is not in mixedCase
Variable LuckyCoin._charityFee (#715) is not in mixedCase
Parameter LuckyCoin.calculateLiquidityFee(uint256)._amount (#940) is not in mixedCase
Parameter LuckyCoin.calculateTaxFee(uint256)._amount (#934) is not in mixedCase
Function IUniswapV2Router01.WETH() (#546) is not in mixedCase
Parameter LuckyCoin.setSwapAndLiquifyEnabled(bool)._enabled (#1169) is not in mixedCase
Variable LuckyCoin._liquidityFee (#709) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Average 30d PancakeSwap liquidity is less than $100. Token is either dead or inactive.

Average 30d number of PancakeSwap swaps is less than 1. Token is either dead or inactive.

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

Token is deployed only at one blockchain

BscScan page for the token does not contain additional info: website, socials, description, etc.

Additional information: link

Unable to find token contract audit

Unable to find audit link on the website

Token is not listed at Mobula.Finance

Additional information: link

Unable to find token on CoinHunt

Additional information: link

Unable to find code repository for the project

Young tokens have high risks of scam / price dump / death

Token has no active CoinGecko listing / rank