質問編集履歴
1
書式改善
title
CHANGED
|
File without changes
|
body
CHANGED
|
@@ -29,10 +29,10 @@
|
|
|
29
29
|
|
|
30
30
|
初めはこのようにしていたのですが，
|
|
31
31
|
・引数（と戻り値）が多すぎて扱いにくい
|
|
32
|
-
・val_1, ..., val_10をまとめて利用する場面が他にもあり，そのたびに全てを渡すのは面倒かつミスが起こりやすい
|
|
32
|
+
・`val_1, ..., val_10`をまとめて利用する場面が他にもあり，そのたびに全てを渡すのは面倒かつミスが起こりやすい
|
|
33
|
-
と感じたため，次のようにval_1, ..., val_10をクラスにまとめることにしました．
|
|
33
|
+
と感じたため，次のように`val_1, ..., val_10`をクラスにまとめることにしました．
|
|
34
34
|
|
|
35
|
-
（実際にはval_1, ..., val_10のように連番の変数ではなく，データ型もサイズも異なる変数なので，これらをvaluesといったリストに格納することは考えておりません．）
|
|
35
|
+
（実際には`val_1, ..., val_10`のように連番の変数ではなく，データ型もサイズも異なる変数なので，これらを`values`といったリストに格納することは考えておりません．）
|
|
36
36
|
|
|
37
37
|
### 修正後のプログラム
|
|
38
38
|
|
|
@@ -66,11 +66,11 @@
|
|
|
66
66
|
|
|
67
67
|
### 疑問
|
|
68
68
|
|
|
69
|
-
FileCreatorクラスが直接Valuesクラスのインスタンスを取り込んでいる以上，Valuesクラスの変更がFileCreatorクラスにもろに影響するという状況があると思います．そのため，このような書き方はコードの修正のしやすさという面でよろしくないのでしょうか．
|
|
69
|
+
`FileCreator`クラスが直接`Values`クラスのインスタンスを取り込んでいる以上，`Values`クラスの変更が`FileCreator`クラスにもろに影響するという状況があると思います．そのため，このような書き方はコードの修正のしやすさという面でよろしくないのでしょうか．
|
|
70
|
-
（例えば，val_1という変数をval_aに変更した場合，FileCreatorクラス内の全てのself.values.val_1をself.values.val_aに変える作業が発生します．）
|
|
70
|
+
（例えば，`val_1`という変数を`val_a`に変更した場合，`FileCreator`クラス内の全ての`self.values.val_1`を`self.values.val_a`に変える作業が発生します．）
|
|
71
71
|
|
|
72
72
|
また，このような状況ではどのようにすべきでしょうか．
|
|
73
|
-
自分としては，Valuesクラスのインスタンスを渡さず，FileCreatorのメンバ変数としてval_1, ..., val_10を定義してあげて，以下のような関数をFileCreator内で定義すればよいかと考えました．こうすればValuesクラス内の変数名に変更があっても，この部分のみの修正で済むと思います．
|
|
73
|
+
自分としては，`Values`クラスのインスタンスを渡さず，`FileCreator`のメンバ変数として`val_1, ..., val_10`を定義してあげて，以下のような関数を`FileCreator`内で定義すればよいかと考えました．こうすれば`Values`クラス内の変数名に変更があっても，この部分のみの修正で済むと思います．
|
|
74
74
|
```python
|
|
75
75
|
def make_values(self):
|
|
76
76
|
values = Value()
|