ユーザー定義クラスについて

Question

コード
```class Person:
    def __init__(self,name,age):
        self.name=name
        self.age=age
    def print_younger_person_name(p1,p2):
        if p1>p2:
            print('小林花子')
        elif p1==p2:
            print('高橋太郎')

a=Person("高橋太郎",19)
b=Person("小林花子",18)
a.print_younger_person_name()
Traceback (most recent call last):
  File "<pyshell#44>", line 1, in <module>
    a.print_younger_person_name()
TypeError: Person.print_younger_person_name() missing 1 required positional argument: 'p2'


引数で受け取る2つのpersonオブジェクトのうち、年齢の若い方を出力する。ただし、同じ年齢の場合はp1の名前を出力するという処理なのですが上手くいきません。
どこが間違っているのでしょうか。

Comments

[tatsu99]

インデントがでたらめなので、ソースの意図が判りません。
ソースのアップ方法
以下のようにすると、きちんとソースが表示されます。

</>をクリックします。
```ここに言語を入力
コード
```

と表示されるので
```①
②
```

①の箇所にPython3と入力し
②の箇所にソースをコピペして貼り付けます。
そうすると、きちんと表示されます。
質問は編集可能です。ソースをきちんと表示してください。

[shin000]

これでできていますでしょうか

[tatsu99]

修整、ありがとうございました。
回答欄に記入しました。

Answer 1

tatsu99さんから正しい回答がついていますから、そちらをBAにしていただければいいですが、悩んだあとが見えるので、ちょっと解説を。

class Person:
    def __init__(self,name,age):
        self.name=name
        self.age=age
    def print_younger_person_name(p1,p2):
        if p1>p2:
            print('小林花子')
        elif p1==p2:
            print('高橋太郎')

a=Person("高橋太郎",19)
b=Person("小林花子",18)
a.print_younger_person_name()

でているエラーですが、「メソッド(関数)に必要な引数が1つ(p2)足りません」とあります。
定義では「def print_younger_person_name(p1,p2)」このように引数を 2つ必要としていますが、実際の呼び出しは「a.print_younger_person_name()」このように 引数がありません。ただpythonでメソッドを呼ぶ場合は、暗黙で第一引数にインスタンスそのもの(この場合はa)が渡されるので、1つ足りないということになるのです。
ちなみに、メソッド定義で 第一引数に使われるselfは、言語仕様的には実は名前は何でもよくて、決まりとしてselfを使いましょうということになっています。なので、p1でもエラーにはなりません。

なので、単に回答をするのであれば、

a.print_younger_person_name(b)

とするということになります。 これで、p2にbが対応するようになります。

次に、比較の「p1>p2」ですが、このようにクラスのインスタンスを直接比較できるようにするには、特殊メソッド(__gt__や__lt__など)の定義ぎ必要です。 簡単にやるのであれば、クラスの属性を使ったtatsu99さんの回答のようにすればいいでしょう。

さらに、メソッド定義ですが、先にも書いたように第一引数はselfとするのが通例です。

ということで、別解としてはこんな感じでしょうか。
aのメソッドを使っているので、aをさらに渡す必要はないかなと思います。

class Person:
    def __init__(self,name,age):
        self.name=name
        self.age=age
    def print_younger_person_name(self,p2):
        if self.age < p2.age:
            print(self.name)
        elif self.age > p2.age:
            print(p2.name)
        else:
            print("same age")

a=Person("高橋太郎",19)
b=Person("小林花子",18)
a.print_younger_person_name(b)

Comments

[shin000]

メソッド定義をよく理解していませんでした。
細かい説明ありがとうございます。

Answer 2

以下のようにしてください。

class Person:
  def __init__(self,name,age):
    self.name=name
    self.age=age
  def print_younger_person_name(self,p1,p2):
    if p1.age > p2.age:
      print(p2.name)
    else:
      print(p1.name)

a=Person("高橋太郎",19)
b=Person("小林花子",18)
a.print_younger_person_name(a,b)

実行結果
小林花子