同じクラス内にあるメソッドから違うメソッドを呼び出したい

Question

やりたいこと

一つのクラス内にあるメソッド内で、別メソッドを呼び出したいのですが、できません。どのような記述をすれば良いのでしょうか。

環境

• Windows10
• Python v3.8.6

コード

import datetime

class Component():
    def __init__(self):
        pass
    def MakeTimeSerial(self,string):
        date_string, time_string = string.split("-")
        year,month,day = date_string.split("/")
        hour,minute,second = time_string.split(":")
        d = datetime(int(year),int(month),int(day),int(hour),int(minute),int(second))
        return int(time.mktime(d.timetuple())

    def Output(self):
        val = "2021/3/4-23:34:21"
        res = Component.MakeTimeSerial(val)
        return res

test = Component()
print(test.Output())

出力されるエラー

TypeError : MakeTimeSerial() missing 1 required positional argument: 'string'

Answer 1

res = self.MakeTimeSerial(val)

--
メソッド MakeTimeSerial は、定義を見るとインスタンスメソッドになっていて、呼び出しを見るとクラスメソッドになっているから動きません。

回答は、定義に合せてインスタンスメソッドとして呼んでいます。

ただ、MakeTimeSerial を見ると、インスタンス変数にアクセスしていないので、クラスメソッドとして定義しなおせば、元の呼び出し方でも動くようになります。

インスタンスメソッドとクラスメソッドの定義のしかたや使いかたについてはリファレンスなどのドキュメントを参照ください。

Comments

[BuhKeil]

ありがとうございました。

[TakaiY]

ちょっと追記します。

[BuhKeil]

遅くなりました。ありがとうございました。

Answer 2

~~res = Component.MakeTimeSerial(self, val)~~として、他にもエラーあったので修正してみました。下記でいかがでしょうか。

from datetime import datetime #修正
import time #修正

class Component():
    def __init__(self):
        pass
    def MakeTimeSerial(self,string):
        date_string, time_string = string.split("-")
        year,month,day = date_string.split("/")
        hour,minute,second = time_string.split(":")
        d = datetime(int(year),int(month),int(day),int(hour),int(minute),int(second))
        return int(time.mktime(d.timetuple())) #修正

    def Output(self):
        val = "2021/3/4-23:34:21"
        res = Component.MakeTimeSerial(self, val) #修正
        return res

test = Component()
print(test.Output())
# 1614868461

Comments

[BuhKeil]

コメントありがとうございます。

from datetime import datetimeの行、
import time の行、
return int ....の行は、私の記述ミスです。ご指摘ありがとうございます。

質問があります。先にご指摘頂いたTakaiYさまの
res = self.MakeTimeSerial(val)という記述と、
jeanbiegoさまの
res = Component.MakeTimeSerial(self, val)
との記述はどちらも正しいのでしょうか？

[jeanbiego]

いや、なんかうっかりしてました。私の回答は誤りですので気になさらないでください。