How to instantiate different classes in a loop?

Asked Jun 20 '21 at 17:27

Active Jun 20 '21 at 17:27

Viewed 30 times

I wrote:

atomic_labels = ["xWant", "oWant", "xAttr", "xNeed", "xIntent", "xEffect", "xReact", "oReact", "oEffect"]

for label in atomic_labels:
    PROCESSORS["atomic_" + label.lower()] = lambda: AtomicProcessor([label, "other"])

However, it seems that it initalized all classes with the last label ([oEffect, other])

asked Jun 20 '21 at 17:27

Ahmad

8,811
11
76
141

Why do you need a `lambda:`? – OneCricketeer Jun 20 '21 at 17:28
1

You need to capture the `label` each time: `lambda label=label: AtomicProcessor([label, "other"]` – quamrana Jun 20 '21 at 17:29
@OneCricketeer if not I get `TypeError: 'AtomicProcessor' object is not callable ` – Ahmad Jun 20 '21 at 17:30
Then that is a separate issue. You've not shown `AtomicProcessor` class/function – OneCricketeer Jun 20 '21 at 17:31
@quamrana thanks, it was resolved. – Ahmad Jun 20 '21 at 17:33
This would make `PROCESSORS` a dictionary of functions, not list of classes, by the way – OneCricketeer Jun 20 '21 at 17:36
i.e. - I dont see why the lambda is needed - https://ideone.com/v3nFI2 – OneCricketeer Jun 20 '21 at 17:40
@OneCricketeer my problem was solved, but if you want to know about the class, its the link https://github.com/puraminy/pet/blob/9c5a3f2e2c35d7d11e12ac6e6ee790443d4d75ae/pet/tasks.py#L767 – Ahmad Jun 20 '21 at 20:25
Yup, again, you don't need a lambda if you actually want a _list of classes_ like your question title stated – OneCricketeer Jun 21 '21 at 20:16

How to instantiate different classes in a loop?

0 Answers0