あるオブジェクトの要素を別のオブジェクトにセットするときのメソッド名や作法

Question

2種類のオブジェクト a b があって、a の特定のプロパティを b にセットするとき、どうするのが自然でしょうか？
私の現在の案は、まず a に b を渡すメソッド a.set_by_b を用意して、その中で b の b.set_a_props を呼び出し、b にセットしたいプロパティを与えるということです。

class A():
    def __init__(self, x, y, z):
        self.x, self.y, self.z = x, y, z
    def set_by_b(self, b):
        b.set_a_props(self.y, self.z)

class B():
    def __init__(self, o):
        self.o = o
        self.y = 0
        self.z = 0
    def set_a_props(self, y, z):
        self.y = y
        self.z = z

def main():
    a = A(100, 200, 300)
    b = B(95)
    a.set_by_b(b)

if __name__ == '__main__':
    main()

便宜上 a のメソッド名を set_by_b としましたが、実際こういう設計の場合で一般的に使われるメソッド名があるのならそれを使いたいです。
また、この設計はおかしいとか、こうする方が自然という意見がございましたらご指摘いただけたらと存じます。

Answer 1

a,bそれぞれの関係性によると思います。

オブジェクト指向的には、値を渡すというのはあまり良くありません。
aの要素（サンプルの場合y,z）が変化してもbの要素は変更しませんし、逆にbの要素が変更されてもaの要素は変更されません。

これを防ぐために、bは要素としてaを持ち、bのメソッドでaの要素が必要な場合はaのgetterメソッド（またはプロパティ）から都度値を取得する方法があります。

またはaがほとんどbを作るだけに存在する一時的なオブジェクトだとするなら、set_by_bとするのではなく、make_bやcreate_bなどのメソッド(サンプルの場合、引数にoが入る)でオブジェクトを作るという手もあります。

そして、aから一つずつ要素を取り出してbにセットする方法などもあります。

他にも言語によっていろいろなパターンがあると思いますが、実際どのような関係を持ちどのように使われるかが大事でソースコードが一番読みやすくなる方法を選ぶ必要があります。

”場合による”という応えになっていない回答ですが正直なところです。

Comments

[y_waiwai]

tetatailのバグですねー

[iwamoto_takaaki]

コメントもらわなきゃ気が付きませんでした。
ありがとうございます。

あぁ～。書き直そうかな削除依頼しようかな。

[y_waiwai]

編集ボタン押したら出てきますよｗ

[iwamoto_takaaki]

サポートありがとうございます。

Answer 2

標準のクラスやライブラリでどうやってるのかを見ていって、極力それにあわせるようにする、ということで。

言語が指定されてませんが、言語により作法が違う場合もあったりするので、標準のものにあわせるのがいいかと。

Comments

[noc]

ありがとうございます。標準のクラスやライブラリの読み方がよくわかっておらず、そもそもあるオブジェクトの要素の一部を別のオブジェクトにセットするパターンが使われている場所も使われているかどうかも解っていないのですが、そういう状況に心当たりがあるのなら教えていただきたいという趣旨でございます。

Answer 3

set_by_b は b によってセットするという意味なので目的語が無く意味不明になっています。
それでも b を使って何かをセットするんだろうと読んでいくと b をセットするのでびっくりします。

またこのコードを読む限り b は a のサブセットになっているので a には x y z プロパティは不要で b を保持または b を継承していればいいように見えます。

Comments

[noc]

すいません、実際 b は a のサブセットではない、全然別種のものとお考え下さい。
別種のオブジェクトからあるプロパティの値を別のオブジェクトにセットするのにどういうスタイルをとるべきかという話です。

[Zuishin]

この仕様を生かすならメソッド名は私なら set_position_to のように目的語をはっきりさせます。
しかしそれよりも x y z をひとまとめにしたクラスを作って a と b にプロパティとして持たせます。
b が a のサブセットでしかないなら回答本文の通りです。

Answer 4

初めの設計が一般的でないとみなして、下記の設計に修正しました。
a は b にセットするための要素を返すゲッターを持ち、b にすぐ受け渡すようにしました。
標準的な設計というものに疎いので、これが正しい設計なのかおかしな設計なのかどうか判別がつきません。
ご意見がございましたならおっしゃってください。

class A():
    def __init__(self, x, y, z):
        self.x, self.y, self.z = x, y, z
    def get_props_for_b(self):
        return (self.y, self.z)

class B():
    def __init__(self, o):
        self.o = o
        self.y = 0
        self.z = 0
    def set_a_props(self, y, z):
        self.y = y
        self.z = z

def main():
    a = A(100, 200, 300)
    b = B(95)
    b.set_a_props(*a.get_props_for_b())

if __name__ == '__main__':
    main()