DogeKing Token (2)

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

DogeKing._tokenTransfer(address,address,uint256,bool) (#476-527) performs a multiplication on the result of a division:
-_takeTransfer(sender,_destroyAddress,tAmount.div(1000).mul(_destroyFee),currentRate) (#491-496)
DogeKing._tokenTransfer(address,address,uint256,bool) (#476-527) performs a multiplication on the result of a division:
-_rOwned[recipient] = _rOwned[recipient].add(rAmount.div(1000).mul(recipientRate)) (#523-525)
DogeKing._tokenTransfer(address,address,uint256,bool) (#476-527) performs a multiplication on the result of a division:
-_takeTransfer(sender,fund1Address,tAmount.div(1000).mul(_fund1Fee),currentRate) (#501-506)
DogeKing._tokenTransfer(address,address,uint256,bool) (#476-527) performs a multiplication on the result of a division:
-Transfer(sender,recipient,tAmount.div(1000).mul(recipientRate)) (#526)
DogeKing._tokenTransfer(address,address,uint256,bool) (#476-527) performs a multiplication on the result of a division:
-_reflectFee(rAmount.div(1000).mul(_taxFee),tAmount.div(1000).mul(_taxFee)) (#514-517)
DogeKing._takeInviterFee(address,address,uint256,uint256) (#545-575) performs a multiplication on the result of a division:
-curTAmount = tAmount.div(1000).mul(rate) (#570)
Consider ordering multiplication before division.

Additional information: link

DogeKing._tokenTransfer(address,address,uint256,bool).rate (#488) is a local variable never initialized
Initialize all the variables. If a variable is meant to be initialized to zero, explicitly set it to zero to improve code readability.

Additional information: link

DogeKing.allowance(address,address).owner (#318) shadows:
- Ownable.owner() (#91-93) (function)
DogeKing._approve(address,address,uint256).owner (#431) shadows:
- Ownable.owner() (#91-93) (function)
Rename the local variables that shadow another component.

Additional information: link

Ownable.changeOwner(address) (#103-105) should emit an event for:
- _owner = newOwner (#104)
Emit an event for critical parameter changes.

Additional information: link

Ownable.changeOwner(address).newOwner (#103) lacks a zero-check on :
- _owner = newOwner (#104)
DogeKing.changeRouter(address).router (#577) lacks a zero-check on :
- uniswapV2Pair = router (#578)
Check that the address is not zero.

Additional information: link

Variable Ownable._owner (#86) is not in mixedCase
Variable DogeKing._taxFee (#245) is not in mixedCase
Variable DogeKing._destroyFee (#249) is not in mixedCase
Variable DogeKing._inviterFee (#253) is not in mixedCase
Variable DogeKing._fund1Fee (#264) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

DogeKing.constructor(address) (#268-287) uses literals with too many digits:
- _tTotal = 100000000000 * 10 ** _decimals (#273)
DogeKing.constructor(address) (#268-287) uses literals with too many digits:
- _tTotalMin = 100000000 * 10 ** _decimals (#275)
DogeKing.slitherConstructorVariables() (#226-581) uses literals with too many digits:
- _destroyAddress = address(0x000000000000000000000000000000000000dEaD) (#250-251)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

DogeKing._tOwned (#230) is never used in DogeKing (#226-581)
Remove unused state variables.

Additional information: link

DogeKing._destroyAddress (#250-251) should be constant
DogeKing._destroyFee (#249) should be constant
DogeKing._fund1Fee (#264) should be constant
DogeKing._inviterFee (#253) should be constant
DogeKing._taxFee (#245) should be constant
DogeKing.fund1Address (#260) should be constant
DogeKing.inviterAddress (#254) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

owner() should be declared external:
- Ownable.owner() (#91-93)
changeOwner(address) should be declared external:
- Ownable.changeOwner(address) (#103-105)
name() should be declared external:
- DogeKing.name() (#289-291)
symbol() should be declared external:
- DogeKing.symbol() (#293-295)
decimals() should be declared external:
- DogeKing.decimals() (#297-299)
totalSupply() should be declared external:
- DogeKing.totalSupply() (#301-303)
transfer(address,uint256) should be declared external:
- DogeKing.transfer(address,uint256) (#309-316)
allowance(address,address) should be declared external:
- DogeKing.allowance(address,address) (#318-325)
approve(address,uint256) should be declared external:
- DogeKing.approve(address,uint256) (#327-334)
transferFrom(address,address,uint256) should be declared external:
- DogeKing.transferFrom(address,address,uint256) (#336-351)
increaseAllowance(address,uint256) should be declared external:
- DogeKing.increaseAllowance(address,uint256) (#353-364)
decreaseAllowance(address,uint256) should be declared external:
- DogeKing.decreaseAllowance(address,uint256) (#366-380)
totalFees() should be declared external:
- DogeKing.totalFees() (#382-384)
excludeFromFee(address) should be declared external:
- DogeKing.excludeFromFee(address) (#399-401)
includeInFee(address) should be declared external:
- DogeKing.includeInFee(address) (#403-405)
claimTokens() should be declared external:
- DogeKing.claimTokens() (#422-424)
isExcludedFromFee(address) should be declared external:
- DogeKing.isExcludedFromFee(address) (#426-428)
changeRouter(address) should be declared external:
- DogeKing.changeRouter(address) (#577-579)
Use the external attribute for functions never called from the contract.

Additional information: link

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

Average 30d PancakeSwap volume 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.

Average PancakeSwap trading volume, liqudity, number of swaps are extremely low. Token seems to be dead.

Number of Binance Smart Chain (BSC) token holders is low.

Token is deployed only at one blockchain

Token has only one trading pair

Contract has 10% buy tax and 9% sell tax.
Taxes are low and contract ownership is renounced.

Unable to find website, listings and other project-related information

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

Token is marked as risky (blacklisted creator, fake name, dead project, etc.)

Additional information: link

Token has no active CoinGecko listing / rank

Token has no active CoinMarketCap listing / rank