質問編集履歴
2
メソッド変数→メンバ変数に修正しました。
test
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
クラス設計時のprivateメ
|
|
1
|
+
クラス設計時のprivateメンバ変数について
|
test
CHANGED
|
@@ -1,4 +1,4 @@
|
|
|
1
|
-
クラス設計時に、メ
|
|
1
|
+
クラス設計時に、メンバ変数を用意すると思いますが、
|
|
2
2
|
|
|
3
3
|
いわゆる作業箱的な変数を定義するって、お作法的に良いでしょうか、良くないでしょうか？
|
|
4
4
|
|
|
@@ -20,7 +20,7 @@
|
|
|
20
20
|
|
|
21
21
|
メソッド１で取得したページオブジェクトを、メソッド２の引数として渡しています。
|
|
22
22
|
|
|
23
|
-
このページオブジェクトって、クラスのメ
|
|
23
|
+
このページオブジェクトって、クラスのメンバ変数としてキープしておいてはいけないんでしょうか？
|
|
24
24
|
|
|
25
25
|
そうするとメソッド２でわざわざ引数渡しする必要がなくなります。
|
|
26
26
|
|
|
@@ -28,7 +28,7 @@
|
|
|
28
28
|
|
|
29
29
|
Webページのクローリングの際には、ページオブジェクトはあちこちで
|
|
30
30
|
|
|
31
|
-
生まれるんだし、それを作業用変数のようにメ
|
|
31
|
+
生まれるんだし、それを作業用変数のようにメンバ変数にキープしておけば、
|
|
32
32
|
|
|
33
33
|
あちこちのメソッドで参照できて便利ではないか？
|
|
34
34
|
|
|
@@ -44,7 +44,7 @@
|
|
|
44
44
|
|
|
45
45
|
|
|
46
46
|
|
|
47
|
-
1. 作業箱的なprivateメ
|
|
47
|
+
1. 作業箱的なprivateメンバ変数」をそもそも定義すべきではない
|
|
48
48
|
|
|
49
49
|
2. こういったことを解決するため、なんらかのデザインパターンが用意されており、
|
|
50
50
|
|
1
聞きたい点を明確にしてみました。
test
CHANGED
|
File without changes
|
test
CHANGED
|
@@ -40,15 +40,25 @@
|
|
|
40
40
|
|
|
41
41
|
|
|
42
42
|
|
|
43
|
-
つまり、「作業箱的なprivateメソッド変数」をそもそも定義すべきではない、という
|
|
44
|
-
|
|
45
|
-
解
|
|
43
|
+
つまり、以下のいずれの見解があっていますでしょうか。
|
|
46
44
|
|
|
47
45
|
|
|
48
46
|
|
|
49
|
-
も
|
|
47
|
+
1. 作業箱的なprivateメソッド変数」をそもそも定義すべきではない
|
|
50
48
|
|
|
49
|
+
2. こういったことを解決するため、なんらかのデザインパターンが用意されており、
|
|
50
|
+
|
|
51
|
-
上記サンプルクラスはそれを適用しているだけ
|
|
51
|
+
上記サンプルクラスはそれを適用しているだけ。
|
|
52
|
+
|
|
53
|
+
3. クローリングクラスの属性は「サイトURL」であり、そのサイトURLの分析のために用意する
|
|
54
|
+
|
|
55
|
+
ページオブジェクト(BeautifulSoupライブラリのインスタンス)は
|
|
56
|
+
|
|
57
|
+
厳密にはクローリングクラスの「属性」ではないため、外部から挿入すべき
|
|
58
|
+
|
|
59
|
+
(でもそうだとすると、そのページオブジェクトを取得するメソッドがこのクラス内にあるのが・・・)
|
|
60
|
+
|
|
61
|
+
|
|
52
62
|
|
|
53
63
|
|
|
54
64
|
|