回答編集履歴
2
の→が
answer
CHANGED
|
@@ -22,7 +22,7 @@
|
|
|
22
22
|
その他
|
|
23
23
|
設計には個々の好みがあるので、確実にこうした方がいい部分だけ列挙します。
|
|
24
24
|
|
|
25
|
-
`Directory`クラスは実ファイルと密結合しているし、各々のクラス
|
|
25
|
+
`Directory`クラスは実ファイルと密結合しているし、各々のクラス同士が少し結合度が高いかなと思います。
|
|
26
26
|
Dependency Injectionなどで少し結合度を下げた方が良いと思います。
|
|
27
27
|
|
|
28
28
|
Pythonじゃなくて「Javaっぽいな」というのが印象です、少なくともPEP8を守った方がよいかなと思います。`dict`などのビルトインクラスの名前の上書きしている部分があるので、`flake8`などの静的解析ツールを導入したほうがよいですよ。
|
1
修飾子を追加
answer
CHANGED
|
@@ -1,4 +1,4 @@
|
|
|
1
|
-
①に関しては
|
|
1
|
+
①に関して、一番簡単な方法は以下の通りです。
|
|
2
2
|
|
|
3
3
|
```python
|
|
4
4
|
class Report:
|
|
@@ -22,7 +22,7 @@
|
|
|
22
22
|
その他
|
|
23
23
|
設計には個々の好みがあるので、確実にこうした方がいい部分だけ列挙します。
|
|
24
24
|
|
|
25
|
-
`Directory`クラスは実ファイルと密結合しているし、各々のクラスの結合度が高いかなと思います。
|
|
25
|
+
`Directory`クラスは実ファイルと密結合しているし、各々のクラスの少しだけ結合度が高いかなと思います。
|
|
26
|
-
Dependency Injectionなどで結合度を下げた方が良いと思います。
|
|
26
|
+
Dependency Injectionなどで少し結合度を下げた方が良いと思います。
|
|
27
27
|
|
|
28
28
|
Pythonじゃなくて「Javaっぽいな」というのが印象です、少なくともPEP8を守った方がよいかなと思います。`dict`などのビルトインクラスの名前の上書きしている部分があるので、`flake8`などの静的解析ツールを導入したほうがよいですよ。
|