How to handle the pylint message: Warning: Method could be a function

PythonOopPylint

Python Problem Overview


I have a python class and ran pylint against it. One message it gave was:

Warning: Method could be a function

Is this telling me that it would be better to move this method out of the class because it doesn't use any instance variables?

In C# I would make this a static method. What's the most pythonic thing to do here?

Python Solutions


Solution 1 - Python

Moving it to a function is common, if it doesn't touch the class at all.

If it manipulates class attributes, use the classmethod decorator:

@classmethod
def spam(cls, ...):
   # cls is the class, you can use it to get class attributes

classmethod and staticmethod (which is the same as the former, except that the method doesn't get a reference to the class from its first parameter) have been introduced quite recently. It means that some Python programmers are used to avoid static and class methods.

Some hardcore Python programmers will tell you that these decorators just complicate things; some other people (usually former C# or Java programmers) will tell you that using a function isn't object-oriented enough. I think it's just a matter of preference.

Attributions

All content for this solution is sourced from the original question on Stackoverflow.

The content on this page is licensed under the Attribution-ShareAlike 4.0 International (CC BY-SA 4.0) license.

Content TypeOriginal AuthorOriginal Content on Stackoverflow
QuestionbignumView Question on Stackoverflow
Solution 1 - PythonBastien LéonardView Answer on Stackoverflow