別クラスのリストからmainの変数にアクセス

Question

前提・実現したいこと

①Mainでlistを定義する
②MethodでMainのlistに値を格納
③mainでprint

selfの正しい使い方イマイチ理解できていません。
詳しい方教えて下さい。

発生している問題・エラーメッセージ

TypeError: append() missing 1 required positional argument: 'self'

該当のソースコード

import method

class Main():
    def __init__(self):
        self.list_x = []

def main():
    Method.append()
    print(Main.list_x)

if __name__ == '__main__':
    Main = Main()
    Method = method.Method
    main()

import main

class Method():
    def append(self):
        self.list_x.append("x1")
        self.list_x.append("x2")

実行環境

Python 3.8.5

Answer 1

問題点が2つあります。

1. クラスオブジェクトの上書き

このプログラムで最初に実行される

Main = Main()

は、同じ名前なので、MainクラスをMainクラスのインスタンスで上書きしてしまいます。
Main()は二度と使えなくなります。今回はこのように変えます。(Methodも同様に変えています)

mainInstance = Main()

2. MethodクラスのインスタンスからMainクラスのインスタンスへの参照がない

MethodクラスのインスタンスからMainクラスのインスタンス内にあるlist_xを書き換えるためには、MethodクラスのインスタンスからMainクラスのインスタンスへの参照がないといけません。ようはMethodクラスのインスタンスメンバ変数にMainクラスのインスタンスを代入しないといけません。

このためには、Methodクラスにも__init__()を用意します。

class Method():
    def __init__(self, mainObject):
        self.main = mainObject

あとは、この参照を使って、list_xを書き換えればいいだけです。

    def append(self):
        self.main.list_x.append("x1")
        self.main.list_x.append("x2")

正解例

import method

class Main():
    def __init__(self):
        self.list_x = []

def main():
    methodInstance.append()
    print(mainInstance.list_x)

if __name__ == '__main__':
    mainInstance = Main()
    methodInstance = method.Method(mainInstance)
    main()

import main

class Method():
    def __init__(self, mainObject):
        self.main = mainObject

    def append(self):
        self.main.list_x.append("x1")
        self.main.list_x.append("x2")

Comments

[katsu96]

ありがとうございます。
selfの理解も深まり、実装することができました。

1点質問ですが、今回用意した「list_x」を数十個用意し、Method.appendで条件に応じて「list_x」に追加するという機能を実装しようとした際、「list_x」を定義する場所はmainじゃない方が理想(綺麗)でしょうか？
mainのdef __init__(self)に何行もあるのが綺麗じゃないような気がしているので、一般的に？どういうのが適切なのか知っていたら教えていただきたいです。

[退会済みユーザー]

「綺麗」「汚い」「一般的」というのは結構主観なので、他人に説明できるものではないですが、個人的な好みで言えば、MainとかMethodとかいうクラス名がもう汚く一般的ではないと思います。

クラスはオブジェクト指向発祥の言葉ですが、オブジェクト指向ではデータに現実に模した何かの意味を与え、その役割に適した名前のメンバ属性やメソッドを持つように仕向けたと思っています。そういう意味では、Mainという曖昧模糊な名前にどんな属性やメソッドがあったとしても、もう正しいのかどうか分からない、というのが私の評価です。Methodについては、明らかにメソッドじゃないオブジェクトなのでNGですね。それらの中身に何が入っていたとしても、何をしてたとしても、なんでもいいのではないかと思ってしまいます。

今回のように、目的が具体的に決まっていないことに対しては、名前と役割の評価が出来ず、それに伴い、その細部である形の評価ができません。役割が決まっていることについては、これがいいと歴史的に決まった適した形があるものもあり、そういうものはデザインパターン、もしくは単にパターンと呼ばれています。そういうモノに当てはまるケースでは、多くの場合、パターンに従っている方が良いとされます。そうでない場合は、「好み」であり、どちらとも言えない場合が多いと個人的に思っています。

ただパターンにないケースでも、具体的に問題が発生している場合は、その個々のケースではそこに問題があるわけですけどね。起きてもない問題について、いろいろ危惧する必要は特にないと思うということです。