Python3: インスタンスを他クラスに渡して処理することは避けるべきですか？

Question

前提・実現したいこと

具体的な内容については書くことができないので少々分かりにくいかもしれませんが，最後まで読んでいただけると幸いです．

・入力ファイルを読み込む．
・読み込んだ結果を変数に格納（ここでは10個の変数があるとします）．
・10個の変数をもとに出力ファイルを作成する．
というのがやりたいことです．

当初のプログラム

def load_file(file):
    """
    ファイルを読み込んで，いくつか処理を加えて変数val_1, ..., val_10を返す．
    """
    return val_1, ..., val_10

def make_file(arg_1, arg_2, ..., arg_10):
    """
    受け取った変数をもとにファイルを作成．
    """

if __name__ == "__main__":
    val_1, ..., val_10 = load_file(file_name)
    make_file(val_1, ..., val_10)

初めはこのようにしていたのですが，
・引数（と戻り値）が多すぎて扱いにくい
・val_1, ..., val_10をまとめて利用する場面が他にもあり，そのたびに全てを渡すのは面倒かつミスが起こりやすい
と感じたため，次のようにval_1, ..., val_10をクラスにまとめることにしました．

（実際にはval_1, ..., val_10のように連番の変数ではなく，データ型もサイズも異なる変数なので，これらをvaluesといったリストに格納することは考えておりません．）

修正後のプログラム

class Values:
    def __init__(self):
        """
        self.val_1, ..., self.val_10の初期化
        """

class FileCreator:
    def __init__(self, values):
        self.values = values

    def load_file(self, file):
        """
        上と同様にファイルを読み込んでself.values.val_1, ..., self.values.val_10に値を入れる．
        """

    def make_file(self):
        """
        やることは上と同じだが，self.valuesの変数をもとにファイルを作成するので引数が必要ない．
        """

if __name__ == "__main__":
    values = Values()  # このvaluesは他の場面でも利用する．
    file_creator = FileCreator(values)
    file_creator.load_file(file_name)
    file_creator.make_file()

疑問

FileCreatorクラスが直接Valuesクラスのインスタンスを取り込んでいる以上，Valuesクラスの変更がFileCreatorクラスにもろに影響するという状況があると思います．そのため，このような書き方はコードの修正のしやすさという面でよろしくないのでしょうか．
（例えば，val_1という変数をval_aに変更した場合，FileCreatorクラス内の全てのself.values.val_1をself.values.val_aに変える作業が発生します．）

また，このような状況ではどのようにすべきでしょうか．
自分としては，Valuesクラスのインスタンスを渡さず，FileCreatorのメンバ変数としてval_1, ..., val_10を定義してあげて，以下のような関数をFileCreator内で定義すればよいかと考えました．こうすればValuesクラス内の変数名に変更があっても，この部分のみの修正で済むと思います．

def make_values(self):
    values = Value()
    values.val_1 = self.val_1
    ...
    values.val_10 = self.val_10
    return values

クラスの扱いに関してよくわからないこと多いので，根本的に使い方が間違っていましたらご指摘いただけると幸いです．

大変長くなってしまい申し訳ありません．
お手数おかけしますが，何卒よろしくお願いします．

Answer 1

Valuesクラスにload(path),save(path)なりの関数を持たせる、つまりファイル操作させるのもひとつの自然な考えかと思います。

Comments

[hirame19168]

回答ありがとうございます．
クラスを分けない方がいいかもしれませんね......
検討してみます．

Answer 2

Valuesの属性を直接読むのではなく、getter的な関数を用意して、FileCreatorからはその関数を呼ぶようにするのはどうでしょうか。

関数名を何らかの意味に対応した名前にしておけば、変更があってもValuesの中だけで閉じられると思います。

Comments

[hirame19168]

回答ありがとうございます．
たしかにそういう方法もありますね．参考にさせていただきます．

Answer 3

使い方がよく見えないのですが継承させるのがスマートな気がします。
適宜改変してみてください。

class Values(object):
    def declare_variable(self, values):
        self.val_1 = values[0]
        self.val_2 = values[1]
        self.val_3 = values[2]
        ...

class FileCreator(Values):
    def __init__(self, values):
        super().declare_variable(values)

    def load_file(self, file):
        # これは他のクラスでは使わない値のようなので
        # 親クラスではなくここで宣言する

        files = open(file) # すいませんここ適当です

        # 変数(リスト)に何かの値を追加するで正しいですか？
        self.val_1.append(files[0])
        self.val_2.append(files[1])
        self.val_3.append(files[2])
        ...

    # load_fileで読み込んだ分もprintされる
    def print_file(self):
        print(self.val_1)
        print(self.val_2)
        print(self.val_3)
        ...

class Hokanobamen(Values):
    def __init__(self, values):
        super().declare_variable(values)

    # load_fileの影響を受けてないのでvaluesの値だけがそのまま反映される
    def print_val(self):
        print(self.val_1)
        print(self.val_2)
        print(self.val_3)
        ...


if __name__ == "__main__":
    file_creator = FileCreator(values)
    file_creator.load_file(file_name)
    file_creator.print_file()
    hokanobamen = Hokanobamen(values)
    hokanobamen.print_val()

コメントを受けて追記

だとしたらクラスは1つあれば大丈夫そうですね。
ファイル読み書き関連は全部適当に書いてます。
ModelViewControlを心がけると読みやすと思います。

class RetouchFile(object):
    def __init__(self, values):
        self.val_1 = values[0]
        self.val_2 = values[1]
        self.val_3 = values[2]
    def load_file(self, file):
        self.load_data = open(file, 'r')
    def edit_data(self):
        self.val_1.append(self.load_data[0])
        self.val_2.append(self.load_data[1])
        self.val_3.append(self.load_data[2])
    def write_data(self):
        data = []
        data.append(self.val_1)
        data.append(self.val_2)
        data.append(self.val_3)
        write_data = open(file, 'w')
        write_data.write(data)

if __name__ == '__main__':
    values = ...  # どこか別のところからvaluesは持ってくる予定でしょうか
    retouch_file = RetouchFile(values)
    retouch_file.load_file(file)
    retouch_file.edit_data()
    retouch_file.write_data()

Comments

[hirame19168]

回答ありがとうございます．
質問が抽象的で大変申し訳ありませんでした．
このプログラムの場合，Valuesクラスのインスタンス（というかメンバ変数val_1~val_10）をFileCreatorやHokanobamenが，それぞれで固有に保持しているという考えで合っていますでしょうか．
質問には書いていなかったのですが，一つのvalues = Values()というインスタンスを各処理に回したいと思っています（FileCreatorに渡すのもHokanobamenに渡すのもvaluesという一つのインスタンス）．
そういった場合の対処法をご教授いただければ幸いです．

[shirai]

それぞれ固有に保持しているというところは正しいです。
if __name__ == '__main__'内の最終行に
print(file_creator.val_1)
print(hokanobamen.val_1)
を追加してみてください。
fileで追加した分だけ差分が出るはずです。

インスタンスとはFileCreaterやHokanobamenではなく、file_createrやhokanobamenをさします。

各処理に回すというのはFileConectorのインスタンスメソッドで変化した
valuesの値をHokanobamenでも使いたいということですか？

[hirame19168]

> 各処理に回すというのはFileConectorのインスタンスメソッドで変化した
valuesの値をHokanobamenでも使いたいということですか？

そうです！個別にvaluesを保持している必要はなくて，
file_creatorでload_file()してvaluesに値を入れる→このvalues内の値ををhokanobamenで処理→file_creatorに戻してmake_file()
といったことができたらよいと考えています．

[shirai]

回答追記しました。

[hirame19168]

ありがとうございます．
この方法で進めていこうと思います．
何度も回答していただき本当にありがとうございました．

Answer 4

リストにできない、ではなく、それをリストにするにはどうすればいいかを考えればいいのでは

それさえできればあとは簡単でしょ。

Comments

[hirame19168]

回答ありがとうございます．
val_1~val_10は配列がほとんどなのですが，それらのサイズが異なります．それらをvaluesというリストに格納したときに
values[0][0], values[1][0][0]
といった具合にインデックスの書き方が変わってしまうのを避けたいというのがリストにしたくない理由でした．もう一度リストで上手く扱えないか考えてみたいと思います．

[y_waiwai]

クラスなり配列なりサイズの違うものをリストに入れて処理するってのはふつーに行われてることですし、何がダメなのかよくわからないですね

[hirame19168]

ダメというわけではないのですが，リストに入れたときにvalues[0]~values[10]それぞれ何なのかが分かりにくくなるのを避けたいんです（こっちの理由の方がメインです．というか上で述べた理由は大した問題ではないですね，すみません）．
val_1~val_10はリストで回して使うようなものではなくて場面ごとに使いたい変数で，よくあるゲームの例で言えばname, hp, weaponといった感じです．それをリストに入れてしまうと0番目がnameで...といった具合に知っておかなければならないですよね．ただ変数自体はまとめておきたいので，クラスに入れたいという話です．
「クラスやら配列やらをリストに入れて処理する」ことについては理解しているつもりですが，このような考えでクラスを利用するのは間違っていますでしょうか．

[y_waiwai]

どうやら、今回の案件はそれぞれクラスを作ってそれで駆動するのがいいきがしますが。

＃そんでそのインスタンスでリストを作ると