defの関数名に変数を使って、for文など変数を使って綺麗な形で呼び出したい。

Question

class test:
    def test_1(self):
        print(1)

    def test_2(self):
        print(2)

    def test_3(self):
        print(3)

    def test_4(self):
        print(4)



if __name__ == '__main__':
    T = test()
    T.test_1()
    T.test_2()
    T.test_3()
    T.test_4()

このT.test_i()とかいちいち変数名を書いている部分をfor文で呼び出すのに、

if __name__ == '__main__':
    T = test()
    for i in range(4):
        trgfunc = "T.test_" + str(i+1) + "()"
        eval(trgfunc)

としました。これで一応動くのですが、trgfunc = "T.test_" + str(i+1) + "()"
とか、いまいち綺麗じゃないのですが、こんなものでしょうか？
変数を使っての関数の呼び出し方、もう少し綺麗なやり方はないでしょうか？

Comments

[quickquip]

実際のコードでもtest_で始まる名前なのか? 何をするコードのなのか（なんのためのclassなのか）?
あたりが書かれてないと一般論として”それはよくない"という回答が集まりやすくなってしまいます。
XY問題とはなんですか https://ja.meta.stackoverflow.com/questions/2701/

Answer 1

関数名に変数を使うというのは考え方からして間違ってます（にもう一票）。

これは本当に質問者さんの手元のコードは test_1 みたいなことになっているのでしょうか?

class test:
    def test(self, i):
        print(i)

とするのがよかったのであって、そうせずに関数名に1とか2とかが入っている合理的な理由がほんの少しでも存在するとは思えません。
（質問に書かれていない全然違う事情があるとしても、その事情の方を質問するべきだろうと思えます）

---

(追記)

目的が曖昧なのですが、inspectモジュールでメソッドを列挙させる方がメソッドを追加した時に実行する方の変更が要らなくて便利だと思うんですよね。

if __name__ == '__main__':
    import inspect

    t = Test()
    for name, method in inspect.getmembers(t, inspect.ismethod):
        if name.startswith('test_'):
            method()

Comments

[Aki1000]

知りたいのは、関数に変数を使用して呼び出す方法でした。
たくさん関数があって、かつ、増える予定で、いちいちその名前を作って呼び出すのはちょっと……。
それで、for文で呼び出したら数行で済むじゃ無いか、と。

[quickquip]

なぜたくさんの関数を全部呼び出さなきゃいけないなんてことが発生するのか? という方に目に行きます。
たくさんメソッドがあって全部呼び出したいというシチュエーションはユニットテストなどではありえるでしょうが、それはユニットテスト用のライブラリを使いましょう、という方が自然です。

> たくさん関数があって、かつ、増える予定で、いちいちその名前を作って呼び出すのはちょっと……。
は理解できますが「for文で関数名を作って呼び出す」というのは解決策としてちょっと……と感じました。

典型的なXY問題の質問という印象です。

Answer 2

メソッド名を合成するための基底クラスを作ってみました。

class Method:
    def call2(self, first, second, *args, **kwargs):
        type(self).__dict__[f'{first}{second}'](self, *args, **kwargs)

このクラスを基底クラスとすると以下のことができます。

>>> class test(Method):
...     def test_1(self):
...         print(1)
...     def test_2(self):
...         print(2)
...     def test_3(self):
...         print(3)
...     def test_4(self):
...         print(4)
...
>>> T = test()
>>>
>>> T.call2("test_", 1)
1
>>>
>>> for i in range(4):
...     T.call2("test_", i+1)
...
1
2
3
4

Answer 3

そもそも関数名に変数を使うというのは考え方からして間違ってます
関数のリストでも作ればどうでしょう

Comments

[xail2222]

質問者ではないですが、関数のリストはどうやってつくるのですか？
classのメソッドではないならリストにする方法はわかるのですが

[Aki1000]

当時、関数のリストをつくるの意味が分かっていませんでした。

pythonでは関数そのものをリスト変数としてリストの中に格納できる。
関数をリストに代入して、一番目二番目と呼び出せば今となってはこれが一番良い気がします。

Answer 4

ネット調べたら書いてあった。

if __name__ == '__main__':
    T = test()
    for i in range(1,5):
        getattr(T,'test_' + str(i))()

リンク

（追記）
呼び出す関数をリストにすると言っても
クラスなので、インスタンスの実体を作るたびにリストに格納して呼び出すのは
非効率な感じがします。

また関数で分けずに一まとめにして、パラメーターを使って処理を分ける
と言うより、処理を分けた方が明瞭化されると思うし

呼び出す関数を振り分ける関数を一つ追加する。というのは
関数の名前で処理を示している場合に、関数を指すパラメータは何がいいのかということもあるし

と、私の拙い知識で考えてると、確かに実際の用途によって何が最適かは変わってくるかとは思いますが
クラスの関数の呼び出しはgetattrでもいいんじゃないかと思いました。