simplifiable-if-statement / R1703ΒΆ

Message emitted:

The if statement can be replaced with %s

Description:

Used when an if statement can be replaced with 'bool(test)'.

Problematic code:

FLYING_THINGS = ["bird", "plane", "superman", "this example"]


def is_flying_animal(an_object):
    # +1: [simplifiable-if-statement]
    if isinstance(an_object, Animal) and an_object in FLYING_THINGS:
        is_flying = True
    else:
        is_flying = False
    return is_flying

Correct code:

FLYING_THINGS = ["bird", "plane", "superman", "this example"]


def is_flying_animal(an_object):
    is_flying = isinstance(an_object, Animal) and an_object.name in FLYING_THINGS
    return is_flying

Created by the refactoring checker.