質問編集履歴

2021/01/02 06:59

投稿

siruku6

スコア1382

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -150,6 +150,24 @@
+## 上図のようなクラス設計にした理由(追記2回目)
+クラス設計の理由を書いていなかったので解説します。
+当初はCallerA, B...という複数のクラスが、各Clientを五月雨式に呼び出していました。
+しかし、各Caller間で重複する処理が出てきたことや、可読性が大幅に低下したことがきっかけで、CallerとClientの間に中間クラス(Manager)を設けることを考えつきました。
+※この中間クラスをFacadeなどと呼ぶこともあると知ったのはここ数日の話なので、もしかしたらこの呼称は間違っているかもしれません。
 ## 各クラスのソース

省略したところがわかるようにコメントなどを挿入

2021/01/02 06:58

投稿

siruku6

スコア1382

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -146,7 +146,7 @@
 ・ManagerはFacadeのつもりです
-・今回の質問とは関係ないですが、Managerは、ClientA, ClientBの処理結果を統合してCallerに返却する役割も持っています
+・今回の質問とは関係ないですが、Managerは、ClientA, ClientBの処理結果を統合してCallerに返却する他のメソッドも多数持っています
@@ -178,6 +178,8 @@
         return result
+# 以下略
 ```
@@ -222,6 +224,8 @@
         return result
+# 以下略
 ```
@@ -272,6 +276,8 @@
         return c
+# 以下略
 ```
@@ -296,6 +302,8 @@
         return a
+# 以下略
 ```
@@ -320,4 +328,6 @@
         return b
+# 以下略
-```
+```

ソースの誤記の修正

2021/01/02 04:05

投稿

siruku6

スコア1382

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -288,9 +288,9 @@
-    def return_a(self, variable_a):
+    def return_a(self, variable_a, variable_b):
-        # variable_a を使用する処理
+        # variable_a, variable_b を使用する処理
         a = ....

ソースコードや質問の背景を記載

2021/01/02 03:45

投稿

siruku6

スコア1382

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -105,3 +105,219 @@
 lambda関数とか使ったらもう少しわかりやすくなったりしないでしょうか....。
+---
+# 追記　(2021/01/02)
+ご指摘やいくつかの回答のおかげで、具体的なソースがないと回答のしようがないことがわかりましたので、ソースコードのサンプルを掲載します。
+※実際のコードではないので、そのまま実行しても動作しないかエラーになる可能性があります。
+実際のコードは、各ファイルが数百行に及び、かなり可読性も低い状態のため、大幅に省略することにしました。
+## ソースの概要
+- 以下にクラス図を示します
+ここ数日で勉強したばかりなので、間違っていたらすみません
+![クラス図](1c2a67c4a7a071f011b46c4c3204ed94.png)
+- 上記の図は以下の関係性を表しています
+・CallerA has Manager
+・CallerB has Manager
+・Manager has ClientA and ClientB
+・ManagerはFacadeのつもりです
+・今回の質問とは関係ないですが、Managerは、ClientA, ClientBの処理結果を統合してCallerに返却する役割も持っています
+## 各クラスのソース
+```python
+# CallerA
+from manager import Manager
+class CallerA():
+    def __init__(self):
+        self.manager = Manager()
+    def some_method(self):
+        self.manager.call_client('aaa', variable_a='a', variable_b='b')
+        return result
+```
+```python
+# CallerB
+from manager import Manager
+class CallerB():
+    def __init__(self):
+        self.manager = Manager()
+    def some_method_a(self):
+        self.manager.call_client('aaa', variable_a='aa', variable_b='bb')
+        return result
+    def some_method_b(self):
+        self.manager.call_client('bbb')
+        return result
+    def some_method_c(self):
+        self.manager.call_client('ccc', variable_c='c')
+        return result
+```
+```python
+# Manager
+from clients.client_a import ClientA
+from clients.client_b import ClientB
+class Manager():
+    def __init__(self):
+        self.client_a = ClientA()
+        self.client_b = ClientB()
+    def call_client(self, method, **kwargs):
+        method_dict = {
+            'aaa': self.client_a.return_a,
+            'bbb': self.client_b.return_b,
+            'ccc': self.return_c,
+        }
+        return method_dict.get(method)(**kwargs)
+    def return_c(self, variable_c):
+        # variable_c を使用する処理
+        c = ....
+        return c
+```
+```python
+# ClientA
+class ClientA():
+    def __init__(self):
+        pass
+    def return_a(self, variable_a):
+        # variable_a を使用する処理
+        a = ....
+        return a
+```
+```python
+# ClientB
+class ClientB():
+    def __init__(self):
+        pass
+    def return_b(self):
+        # 何らかの処理
+        b = ....
+        return b
+```