org.sonar.l10n.py.rules.python.S2201.html Maven / Gradle / Ivy

Go to download

Show more of this group Show more artifacts with this name
Show all versions of python-checks Show documentation

There is a newer version: 4.23.0.17664

Why is this an issue?
When the call to a function doesn’t have any side effects, what is the point of making the call if the results are ignored? In such case, either
the function call is useless and should be dropped or the source code doesn’t behave as expected.
This rule raises an issue when a builtin function or methods which has no side effects is called and its result is not used.
Noncompliant code example
myvar = "this is a multiline"
"message from {}".format(sender)  # Noncompliant. The formatted string is not used because the concatenation is not done properly.

Compliant solution
myvar = ("this is a multiline"
"message from {}".format(sender))

Exceptions
No issue will be raised when the function or method call is in a try...except body. This usually indicates that an exception is
expected, and this exception is the side-effect.
def tryExcept():
    d = {}
    try:
        d[1]
    except IndexError as e:
        pass

    try:
        divmod(1, 0)
    except ZeroDivisionError as e:
        pass

Resources

   Python documentation - Built-in Functions