Dynamically adding class methods to a class
class-method, metaprogramming, python
Solution
Use a closure to preserve the value of `kind`:
for type_tuple in FEED_TYPES:
kind, name = type_tuple
def make_notify(kind):
def notify(self, **kwargs):
print "notifying %s" % kind
self.create(kind, **kwargs)
return notify
notify = make_notify(kind)
notify.__name__ = "notify_%s" % kind
setattr(cls, notify.__name__, classmethod(notify))
By the way, don't use `type` as a variable name since it shadows the builtin of the same name.
A more elegant way to modify `Feed` is to create a class decorator. This makes it clearer that you have code modifying the original definition of `Feed`.
FEED_TYPES = [
('fan_mail', 'Fan Mail'),
('review', 'Review'),
('tip', 'Tip'),
('fan_user', 'Fan User'),
('fan_song', 'Fan Song'),
('fan_album', 'Fan Album'),
('played_song', 'Played Song'),
('played_album', 'Played Album'),
('played_radio', 'Played Radio'),
('new_event', 'New Event'),
]
def add_feed_types(cls):
for type_tuple in FEED_TYPES:
kind, name = type_tuple
def make_notify(kind):
def notify(self, **kwargs):
print "notifying %s" % kind
self.create(kind, **kwargs)
return notify
notify = make_notify(kind)
notify.__name__ = "notify_%s" % kind
setattr(cls, notify.__name__, classmethod(notify))
return cls
@add_feed_types
class Feed:
@classmethod
def do_create(cls, **kwargs):
print kwargs
@classmethod
def create(cls, kind, **kwargs):
kwargs['feed_type'] = kind
cls.do_create(**kwargs)
Feed.create("FanMail", to_profile="Gerson", from_profile="Felipe")
Feed.notify_fan_mail(to_profile="Gerson2", from_profile="Felipe2")
Problem
I have the following snippet: ``` FEED_TYPES = [ ('fan_mail', 'Fan Mail'), ('review', 'Review'), ('tip', 'Tip'), ('fan_user', 'Fan User'), ('fan_song', 'Fan Song'), ('fan_album', 'Fan Album'), ('played_song', 'Played Song'), ('played_album', 'Played Album'), ('played_radio', 'Played Radio'), ('new_event', 'New Event'), ] class Feed: @classmethod def do_create(cls, **kwargs): print kwargs @classmethod def create(cls, type, **kwargs): kwargs['feed_type'] = type cls.do_create(**kwargs) for type_tuple in FEED_TYPES: type, name = type_tuple def notify(self, **kwargs): print "notifying %s" % type self.create(type, **kwargs) notify.__name__ = "notify_%s" % type setattr(Feed, notify.__name__, classmethod(notify)) Feed.create("FanMail", to_profile="Gerson", from_profile="Felipe") Feed.notify_fan_mail(to_profile="Gerson2", from_profile="Felipe2") ``` The idea is to dynamically create one class method (like notify_fan_mail) for each feed type. It works almost great, the only problem is that the print statement always prints "notifying new_event", regardless of the method I call (same for notify_new_mail, notify_review, etc.). I realize it's because it's using the last value assigned to type. My question is: how can I dynamically create methods that would use the correct value for type? Also, if I have this exact code in a Python file, is that the correct way to add methods to the Feed class, or is there a more elegant way?