'staticmethod' object is not callable

Question

I have this code:

class A(object):     @staticmethod     def open():         return 123     @staticmethod     def proccess():         return 456      switch = {         1: open,         2: proccess,            }  obj = A.switch[1]() 

When I run this I keep getting the error:

TypeError: 'staticmethod' object is not callable 

how to resolve it?

Answer 1

You are storing unbound staticmethod objects in a dictionary. Such objects (as well as classmethod objects, functions and property objects) are only bound through the descriptor protocol, by accessing the name as an attribute on the class or an instance. Directly accessing the staticmethod objects in the class body is not an attribute access.

Either create the dictionary after creating the class (so you access them as attributes), or bind explicitly, or extract the original function before storing them in the dictionary.

Note that 'binding' for staticmethod objects merely means that the context is merely ignored; a bound staticmethod returns the underlying function unchanged.

So your options are to unindent the dictionary and trigger the descriptor protocol by using attributes:

class A(object):     @staticmethod     def open():         return 123     @staticmethod     def proccess():         return 456  A.switch = {     1: A.open,     2: A.proccess,    } 

or to bind explicitly, passing in a dummy context (which will be ignored anyway):

class A(object):     @staticmethod     def open():         return 123     @staticmethod     def proccess():         return 456      switch = {         1: open.__get__(object),         2: proccess.__get__(object),        } 

or access the underlying function directly with the __func__ attribute:

class A(object):     @staticmethod     def open():         return 123     @staticmethod     def proccess():         return 456      switch = {         1: open.__func__,         2: proccess.__func__,        } 

However, if all you are trying to do is provide a namespace for a bunch of functions, then you should not use a class object in the first place. Put the functions in a module. That way you don't have to use staticmethod decorators in the first place and don't have to unwrap them again.