クラスを使ったリストの並び替えについて

Question

Humanというクラスを作り、そのクラスを使い、humansというリストを作りました。
humansリストからageに沿って並び替えをしたいというプログラムです。
並び替えのため、sorted関数を使いました。

class Human():
    def __init__(self,age,name,gender):
        self.age = age
        self.name = name
        self.gender = gender
    
human1 = Human(22,"mori","男")
human2 = Human(20,"moriyama","男")
human3 = Human(21,"moji","女")
humans = [human1,human2,human3]
result = sorted(humans, key=lambda t:t[0])
print(result)

エラーメッセージは以下の通りです。

Traceback (most recent call last):
  File "/home/ec2-user/environment/class_test.py", line 11, in <module>
    humanss = sorted(humans, key=lambda t:t[0])
  File "/home/ec2-user/environment/class_test.py", line 11, in <lambda>
    humanss = sorted(humans, key=lambda t:t[0])
TypeError: 'Human' object does not support indexing

どうしてエラーになるのかを教えてください。また、sorted関数を使うときにkey=lambdaを使うとよいと知りましたが、key=lambdaのメカニズムは、tとしてhumansを引数にとり、humans[0]として、一つ目の要素、今回で言えばageを取り出すという理解でよいのでしょうか。
よろしくお願いします。

Answer 1

Humanクラスはlistやtupleなどではないので、[0]といった形式で属性にアクセスすることは出来ません。

エラーを出さないためには、こう書く必要があります。

class Human():
    def __init__(self,age,name,gender):
        self.age = age
        self.name = name
        self.gender = gender

human1 = Human(22,"mori","男")
human2 = Human(20,"moriyama","男")
human3 = Human(21,"moji","女")
humans = [human1,human2,human3]
result = sorted(humans, key=lambda t:t.age)
print(result)

key=lambdaのメカニズム

キーワード引数keyに、ソート対象の要素から値を取り出すような関数を渡す必要があります。今回はlambda t:t.ageを渡せば用を達します。

Comments

[bbettbn]

ありがとうございます。
エラーはなくなりました。
ただ、

[<__main__.Human object at 0x7f74b8cd5128>, <__main__.Human object at 0x7f74b8cd5160>, <__main__.Human object at 0x7f74b8cd50b8>]

のように出力されます。何が原因でしょうか。

[hayataka2049]

「__main__という名前空間で定義されているHumanクラスのオブジェクトですよ」というような意味の表示で、それで正常な表示です。
実際、human1などをそれぞれprintしてもそう見えるはずで。
適当なフォーマットで見えてほしければ、__repr__メソッドを定義してあげてください（検索すればその辺の話は出てくると思います）

[bbettbn]

def __repr__(self):
return repr((self.age,self.name,self.gender))
とすることで文字と数値で出力されました。ありがとうございました。
もしこのクラスにプロパティを利用するならば、以下のようなコードになるかと思いましたが、エラーでした。何が原因でしょうか。

```
class Human():
def __init__(self,age,name,gender):
self.__data=["age","name","gender"]
@property
def age(self):
return self.__data[age]
@property
def name(self):
return self.__data[name]
@property
def gender(self):
return self.__data[gender]

def __repr__(self):
return repr((self.age,self.name,self.gender))

human1 = Human(22,"mori","男")
human2 = Human(20,"moriyama","男")
human3 = Human(21,"moji","女")
humans = [human1,human2,human3]
result = sorted(humans, key=lambda t:t.age)
print(result)

```

[hayataka2049]

__init__は受け取った引数をネグレクトして"age", "name", "gender"の文字列のリストなんかをself.__dataに代入していますし、プロパティ使うにしてもself.__data[age]とかはage等がメソッド内では未定義だろうし、そもそもリストにアクセスするなら整数渡してあげないといけないし

[hayataka2049]

コードから「プログラミングの基本的な考え方に慣れていない人が書いたような臭い」がするので、とりあえず力に余ることはしないで、簡単な書き方をして慣れていけば良いのではないでしょうか

[bbettbn]

回答ありがとうございます。
おっしゃる通り、まだプログラミングを始めたばかりです。classの考え方には特に慣れていません。
ただ、課題として出されているので、何としても完成させたいと思っています。
もう少し考えてみるので、またお願いします。

[hayataka2049]

課題ですか。頑張ってくださいね

[bbettbn]

きちんと出力されますが、これはpropetyになってますか？
見づらいでしょうが、よろしくお願いします。
class Human():
def __init__(self,age,name,gender):
self.__data={"age":age,"name":name,"gender":gender}

@property
def age(self):
return self.__data["age"]
@property
def name(self):
return self.__data["name"]
@property
def gender(self):
return self.__data["gender"]

def __repr__(self):
return repr((self.age,self.name,self.gender))

human1 = Human(22,"mori","男")
human2 = Human(20,"moriyama","男")
human3 = Human(21,"moji","女")
humans = [human1,human2,human3]
result = sorted(humans, reverse=True, key=lambda t:t.age)
print(result)

[hayataka2049]

基本的にはそれでよさそうです。
プロパティはsetterとdeleterを書いた方が気分的には安心できます。
あと、__repr__の返す文字列は"Human(22,'mori','男')"のような、コードとして評価すると元のオブジェクトに戻るようなフォーマットにするのが一般的な作法とされています。

[bbettbn]

ありがとうございます。セッター関数も追加しました。
参考書片手に書いただけですので、意味は分かりません。
セッター関数を使うと、ゲッター関数だけのときに比べてどのようなメリットがあるのでしょうか。
また、基本的にプロパティを使うときは、辞書を初期化メソッドで定義してから使うことが多いのでしょうか。
フォーマットのついてのご指摘ありがとうございます。
どうしたらその位置でHumanが出力されますか。

class Human():
def __init__(self,age,name,gender):
self.__data={"age":age,"name":name,"gender":gender}

@property
def age(self):
return self.__data["age"]
@age.setter
def age(self,value):
self.__data["age"]=value

@property
def name(self):
return self.__data["name"]
@name.setter
def name(self,value):
self.__data["name"]=value
@property
def gender(self):
return self.__data["gender"]
@gender.setter
def gender(self,value):
self.__data["gender"]=value

def __repr__(self):
return repr((self.age,self.name,self.gender))

human1 = Human(22,"mori","男")
human2 = Human(20,"moriyama","男")
human3 = Human(21,"moji","女")
humans = [human1,human2,human3]
result = sorted(humans, reverse=True, key=lambda t:t.age)
print(result)

[hayataka2049]

setter、deleterがないと、後からその属性に代入、削除できません。できなくても良いなら書かなくて良いので、「気分的には安心できます」と書きました。ちょっとわかりづらかったですね
フォーマットに関しては、
return "Human({0}, '{1}', '{2}')".format(self.age,self.name,self.gender)
みたいな感じでどうですか

[bbettbn]

そうなんですね。
フォーマットの件もありがとうございました。
今回、回答を継続して頂いたおかげでとても勉強になりました！
本当に助かりました。
またよろしくお願いします。

Answer 2

単に値を格納することが主目的なら、collections.namedtupleが便利です。

from collections import namedtuple

Human = namedtuple('Human', ['age', 'name', 'gender'])

human1 = Human(22, "mori", "男")
human2 = Human(20, "moriyama", "男")
human3 = Human(21, "moji", "女")
humans = [human1, human2, human3]

result = sorted(humans, key=lambda h: h[0])   # operators.itemgetterを使っても良い
print(result)

ただし今後クラスを拡張していく場合は、普通のクラスを用いた方が良いでしょう。
Python3.7以降ならdataclassesを使ってみるのもアリです。

Comments

[bbettbn]

そのモジュールは知りませんでした。
ありがとうございました！