pythonのクラス定義の引数の使い方について

Question

python3.7を利用しています。一番下の書き方が正解だと思っています。しかし、上記２つのような書き方を目にしました。特に二番目の書き方だと以下のようなエラーがでます。python3.7では、上２つの書き方はできなくなったのでしょうか？(なぜエラーがでるのかよくわかっていません) こういうときには上２つのような書き方が必要になる、などご教示いただけませんでしょうか？

TypeError: __init__() takes 1 positional argument but 2 were given

以下は、クラスを呼ぶ場所

def _exec(request, _id):
    # こっちだとエラーはでない
    a_id = TEST()
　　 # 以下だとエラーがでる
    b_id = TEST(_id)

class TEST():
    def __init__(self, _id):
        from logging import getLogger
        logger = getLogger(__name__)
        logger.info("_id of this is %s", _id)
        self._id = _id

class TEST(object):
    def __init__(self, _id):
        from logging import getLogger
        logger = getLogger(__name__)
        logger.info("_id of this  is %s", _id)
        self._id = _id

class TEST:
    def __init__(self, _id):
        from logging import getLogger
        logger = getLogger(__name__)
        logger.info("_id of this  is %s", _id)
        self._id = _id

Comments

[hayataka2049]

classはどこに行ったのでしょうか・・・

[sequelanonymous]

すみません、追記しました。

[hayataka2049]

TEST:
じゃなくて
class TEST:
なのでは？　という話です・・・

[tachikoma]

TESTのコンストラクタではloggerとidのどちらを引数に取るのでしょうか・・・。

[sequelanonymous]

修正しました。転記ミスです。

[sequelanonymous]

> TESTのコンストラクタではloggerとidのどちらを引数に取るのでしょうか・・・。
すみません、こちらも転記ミスをしていました。修正しました。

[t_obara]

余計なコメントになりますが、この質問こそ、「必要最小限の再現コードで確認すべき」が当てはまる内容ですね。まずは解決して良かったと思います。hayataka2049氏のおっしゃる通り、一度解決とすべきかと。

Answer 1

class TEST():
class TEST(object):
class TEST:

python3では「すべて同じ」です。この丸かっこの中には継承するクラスを書きます。なければ省略できます。

• 一番上

丸かっこだけ書いて中身を省略しているのでこの丸かっこはないのと同じ。

• 真ん中

objectは暗黙的にすべてのクラスの基底になるため、明示的に書いても書かなくても同じ。

• 一番下

丸かっこまで省略しても問題ない

TypeError: __init__() takes 1 positional argument but 2 were given

このエラーになったのなら、質問文で気にしているところとはまったく関係ない箇所でバグがあります。

    # こっちだとエラーはでない　←　出ないとおかしい
    a_id = TEST()
　　 # 以下だとエラーがでる　←　少なくとも引数の数はこれで合うと思うんですが・・・
    b_id = TEST(_id)

Comments

[sequelanonymous]

ありがとうございます。上記、全て同じという理解はしていましたが、自信がありませんでした。
また、転記ミスが再度あったため、修正しました。
> # こっちだとエラーはでない　←　出ないとおかしい
a_id = TEST()
　　 # 以下だとエラーがでる　←　少なくとも引数の数はこれで合うと思うんですが・・・
b_id = TEST(_id)

再度確認しましたが、a_idだとエラーがでず、b_idだとエラーが質問記載のエラーがでてしまいます。
エラーログも、__init__の箇所でとまっています。やはり、わからずという状況です。

[hayataka2049]

質問文の_exec, TESTの定義をコピペして適当な引数を与えて呼んでみましたが、再現しないです。この時点できつい状況なのですが、もう少し情報があれば問題解決に結びつくヒント程度は与えられる可能性があります。
そもそも
TypeError: __init__() takes 1 positional argument but 2 were given
は本当に「質問文に書かれたTEST」の__init__が吐いているのか、というあたりから正直疑問です。tracebackぜんぶ見れれば手っ取り早いんですが、手打ちで転記してるんですよね・・・

[hayataka2049]

＞再度確認しましたが、a_idだとエラーがでず、b_idだとエラーが質問記載のエラーがでてしまいます。
def __init__(self):
みたいな引数フォーマットで定義された「なにか」を呼んでしまっているのでは？　それがなにかまではわかりませんが。

[sequelanonymous]

> TypeError: __init__() takes 1 positional argument but 2 were given
は本当に「質問文に書かれたTEST」の__init__が吐いているのか、というあたりから正直疑問です。

上記が正解でした。model定義の方でも同じクラス名を利用しておりました。そして、そちらを読みにいっていたように思います。当コードが記載されているモジュール内でmodel定義から読んだクラスを別のメソッド内で利用しています。model定義側ではコンストラクタは利用していないのでエラーのでかたがおかしい気はしています。

[hayataka2049]

そのmodelがなにか継承してるんじゃないでしょうか。
同名クラスとか書くなしという・・・

[sequelanonymous]

はい、おっしゃる通りだと思います。。。
命名を変更しようと思うのですが、正直悩みます。同じようなデータをDBに保管してそこからデータ引っ張てきて利用するか、クラス内の変数にいれて使うかの違いなのですが、例えば、TEST_clsというような命名はよくあるものでしょうか？
そもそも、どっちかに統一したほうがいいという思ったりもしますが。

[hayataka2049]

同名のクラスを2つ作るというのはさすがに設計としてどうかと思いますが、この質問の本筋と逸れるのでここで追求する気はありません。
とりあえず、エラーがどこから出ているのか確定させちゃいましょう。
＞そのmodelがなにか継承してるんじゃないでしょうか。

[sequelanonymous]

わかりました。おそらく読みに行っていたのは、以下です。同じ関数名です。
db.Modelの中のコンストラクタを読みに行ってしまっている？！のかと思いました。

from flask_sqlalchemy import SQLAlchemy
db = SQLAlchemy()

class TEST(db.Model):
...
...

[hayataka2049]

なるほど。では、こちらは解決扱いにして、設計については必要であれば別途質問を立てると良いかと思います。

[sequelanonymous]

わかりました、お付き合い頂きありがとうございました。

Answer 2

sample.pyという名前で下記の内容を書き込んで試しましたが、エラーが再現しないんですよね・・・。

from logging import getLogger

class TEST:
    def __init__(self, _id):
        logger = getLogger(__name__)
        logger.warning("_id of this is %s", _id)
        self._id = _id

if __name__ == "__main__":
    test = TEST(0)

実行方法@Python 3.7.0

$ python sample.py
_id of this is 0

Comments

[sequelanonymous]

ご確認ありがとうございます。同じクラス名のモデル定義からデータをとってくる関数があったためそちらを読みにいっていたようです。エラーのでかたがおかしいですが。

Answer 3

思っているのと違うファイルを読んでいるために、違う定義のTESTを参照しているとか。

Comments

[sequelanonymous]

上記が正解でした。model定義の方でも同じクラス名を利用しておりました。そして、そちらを読みにいっていたように思います。にしてもエラーのでかたがおかしいように思いますが。model定義でコンストラクタは利用していないので。