`init` method returns a value¶

ID: py/explicit-return-in-init
Kind: problem
Security severity: 
Severity: error
Precision: very-high
Tags:
   - quality
   - reliability
   - correctness
Query suites:
   - python-code-quality.qls
   - python-security-and-quality.qls

Click to see the query in the CodeQL repository

The __init__ method of a class is used to initialize new objects, not create them. As such, it should not return any value. Returning None is correct in the sense that no runtime error will occur, but it suggests that the returned value is meaningful, which it is not.

Recommendation¶

Convert the return expr statement to a plain return statement, or omit it altogether if it is at the end of the method.

Example¶

In this example, the __init__ method attempts to return the newly created object. This is an error and the return method should be removed.

class ExplicitReturnInInit(object):
    def __init__(self, i):
        self.i = i
        return self

References¶

Python: The init method.

GeezerVison

__init__ method returns a value¶

Recommendation¶

Example¶

References¶

`init` method returns a value¶