Constant in conditional expression or statement¶
ID: py/constant-conditional-expression Kind: problem Severity: warning Precision: very-high Tags: - maintainability - useless-code - external/cwe/cwe-561 - external/cwe/cwe-570 - external/cwe/cwe-571 Query suites: - python-security-and-quality.qls
Using a constant value as a test in a conditional statement renders the statement pointless as only one branch will be run regardless of any other factors.
If the conditional statement is required for debugging or similar then use a variable instead. Otherwise, remove the conditional statement and any associated dead code.
In the first example the
if statement will always be executed and therefore can be removed. The contents of the statement should be kept though.
In the second example the statement
l = 100 is never executed because
1 > 100 is always false. However, it is likely that the intention was
l > 100 (the number ‘1’ being misread as the letter ‘l’) and that the test should be corrected, rather than deleted.
if True: print "True is true!" def limit(l): if l < -100: l = -100 if 1 > 100: l = 100 return l