python3で複数のインスタンスをfor文を使ってリストに入れたい

前提・実現したいこと

item_1~itemn_100
という感じで作成したクラスのインスタンスをfor in range()文を使ってitems[]というリストにすべて入れたいと思っています。

該当のソースコード

下記のような要領で実装できるのかなと思ったのですが。。。

python3
1# Itemクラスからitem_0~item_100という変数名のインスタンスを作成
2item_0 = Item('hoge')
3item_1 = Item('fuga')
4.
5.
6.
7item_100 = Item('hogehoge')
8
9
10#itemsリストを用意
11items = []
12# for分でインスタンスをリストに挿入していく
13for i in range(101):
14    items.append(item_ + i)

当たり前と言えばそうなのですが、
item_
までで評価されてしまうようで、下記のエラーが出てしまいます。

NameError: name 'item_' is not defined

手書きでリストに一つずつインスタンスを挿入していくのはいかにもプログラミングらしくないしヒューマンエラーも起こりやすいので、なんとかスマートに実装したいのですが、どうするのがベストでしょうか？

行動規範の内容に同意します

回答2件

ベストアンサー

item_1~itemn_100

という感じで作成したクラスのインスタンス

インスタンスを作るために変数に入れる必要はありません。変数に入れるとしてもインスタンスの数だけ変数を用意する必要もありません。

よって、一番直感的なのはリテラルでベタ書きすることです。

python
1items = [Item('hoge'),
2         Item('fuga'),
3         # ..., 
4         Item('hogehoge')]

コンストラクタに渡している引数のリストみたいなものを持っているのであれば、ループも活かせます。

python
1data = ['hoge', 'fuga', "# ...", 'hogehoge']
2items = [Item(arg) for arg in data]

投稿2020/02/25 10:14

hayataka2049

総合スコア30939

yujico

2020/02/25 10:24 編集

早速ご回答いただきありがとうございます。 Itemクラスは、質問では簡易にするためにコンストラクタに渡す引数を1つにしていますが、実は6程度あり、その場合、おそらく後者だと ``` data = ['hoge1','hoge2',...,hoge6, 'fuga1','fuga2'...fuga6,'hogehoge6'] ``` のようにすべてのデータをごそっとdataリストに入れると言うことでしょうか？だとすると、今回のケースだと前者の方がいいかもしれないですね。ありがとうございます！

hayataka2049

2020/02/25 10:25

data = [('hoge1', 'hoge2', ..., 'hoge6'), ...] のようにして [Item(*args) for args in data] としてください。内側はtupleじゃなくてlistでもいいし、引数名と対応付けたいので辞書にするといった判断もありですが、「引数の値を入れた何かのリスト」のような形式にしないととても扱いづらくなります。*は引数リストのアンパックです。

hayataka2049

2020/02/25 10:26

あるいは data1 = ['hoge1', 'fuga1', ...] data2 = ['hoge2', 'fuga2', ...] ... のような形でデータが得られるなら、zipを使って上の形式に変換することが可能です。

yujico

2020/02/25 11:29

なるほど、そのようにすると、後者でも視認性を担保できそうですね。どちらが適切か比較して決めようと思います。ありがとうございます！

hayataka2049

2020/02/25 11:42

100*6のデータは、そもそもソースコードに埋め込まないで外部のCSVやJSONなどのテキストファイルから読み込むことも検討されるべきものだと思います。参考までに。

yujico

2020/02/25 11:47

確かにそうですよねw 今回作成しているプログラムが完全に自分専用で、なるべく短時間で作ってしまいたかったという理由がありまして。(にしてもテキストファイルからの読み込みの方が逆に簡単だったかなとも思いますが) 参考にさせていただきます。