質問編集履歴
5
typo
test
CHANGED
File without changes
|
test
CHANGED
@@ -1,4 +1,4 @@
|
|
1
|
-
すいません。まず、この話題はC++言語限定とさせて下さい。広げると収
|
1
|
+
すいません。まず、この話題はC++言語限定とさせて下さい。広げると収拾がつかなくなる予感がしますので。
|
2
2
|
|
3
3
|
|
4
4
|
|
@@ -132,7 +132,7 @@
|
|
132
132
|
|
133
133
|
######ところで、何故にC++限定だったのか?
|
134
134
|
|
135
|
-
最初に「この話題はC++言語限定とさせて下さい。広げると収
|
135
|
+
最初に「この話題はC++言語限定とさせて下さい。広げると収拾がつかなくなる予感がしますので。」と記載させて頂きました。いまいち明確に表現できてなかったのですが、今回の議論を通じてやっと言語化できました。
|
136
136
|
|
137
137
|
|
138
138
|
|
4
微修正
test
CHANGED
File without changes
|
test
CHANGED
@@ -62,7 +62,7 @@
|
|
62
62
|
|
63
63
|
|
64
64
|
|
65
|
-
【検討のポイント】
|
65
|
+
**【検討のポイント】**
|
66
66
|
|
67
67
|
私はシンプルイズベスト信者なので、メリットのない複雑さは悪と考えます。
|
68
68
|
|
@@ -80,7 +80,7 @@
|
|
80
80
|
|
81
81
|
|
82
82
|
|
83
|
-
【アクセサを設けるメリットのまとめ】
|
83
|
+
**【アクセサを設けるメリットのまとめ】**
|
84
84
|
|
85
85
|
皆さんの意見のうち、【検討のポイント】的にメリットが明確なものを選択させて頂き、私が理解した内容で纏めてみました。(多少違っているかも知れません。その時はごめんなさい。)
|
86
86
|
|
@@ -116,7 +116,7 @@
|
|
116
116
|
|
117
117
|
|
118
118
|
|
119
|
-
【私的な方針のまとめ】
|
119
|
+
**【私的な方針のまとめ】**
|
120
120
|
|
121
121
|
私自身が担当しているプロジェクトは小規模・少数精鋭型が多いです。
|
122
122
|
|
3
結果報告追記
test
CHANGED
File without changes
|
test
CHANGED
@@ -48,10 +48,102 @@
|
|
48
48
|
|
49
49
|
|
50
50
|
|
51
|
-
|
51
|
+
##結果報告
|
52
|
+
|
53
|
+
この議論を通じて、やっと自分なりの方針を明確にできました。
|
54
|
+
|
55
|
+
皆さん、ありがとうございます。
|
52
56
|
|
53
57
|
|
54
58
|
|
55
|
-
|
59
|
+
ベストアンサーは、C++らしい方針を最も明確に記載して頂けたcatsforepawさんにさせて頂きました。
|
56
60
|
|
61
|
+
また、「ブレーク貼れるようにアクセサ」という使い方に気づかせて頂けたこともありがたいです。
|
62
|
+
|
63
|
+
|
64
|
+
|
65
|
+
【検討のポイント】
|
66
|
+
|
67
|
+
私はシンプルイズベスト信者なので、メリットのない複雑さは悪と考えます。
|
68
|
+
|
69
|
+
なので、メリットもないのにアクセサを設けるのは悪なわけです。
|
70
|
+
|
71
|
+
ということは、アクセサを設けるメリットは何か?がポイントになります。
|
72
|
+
|
73
|
+
|
74
|
+
|
75
|
+
なお、アクセスされたタイミングで何らかのアクションを行うようなケースはメリットもなにも必須事項なので検討不要ですね。
|
76
|
+
|
77
|
+
また、そのような機能の追加も含め、何らかの変更が予想されるケースでは、アクセサを設けるメリットが明確ですので、こちらも検討不要です。
|
78
|
+
|
79
|
+
変更が予想されない時でも、アクセサ化するメリットがあるのか?が検討のポイントとなります。
|
80
|
+
|
81
|
+
|
82
|
+
|
83
|
+
【アクセサを設けるメリットのまとめ】
|
84
|
+
|
85
|
+
皆さんの意見のうち、【検討のポイント】的にメリットが明確なものを選択させて頂き、私が理解した内容で纏めてみました。(多少違っているかも知れません。その時はごめんなさい。)
|
86
|
+
|
87
|
+
|
88
|
+
|
89
|
+
**raccy**さん
|
90
|
+
|
91
|
+
メンバ変数へのアクセスは原則Read Onlyとするので、常にアクセサを使う。
|
92
|
+
|
93
|
+
|
94
|
+
|
95
|
+
私的な見解:
|
96
|
+
|
97
|
+
この基準(コーディング規約かも?)により、多少不慣れな人がいても比較的安全に開発を進めることができると言うメリットがあると感じます。
|
98
|
+
|
99
|
+
大勢の人がメンテナンスするようなプロジェクトで特に有用ですね。
|
100
|
+
|
101
|
+
|
102
|
+
|
103
|
+
**catsforepaw**さんと**yohhoy**さん
|
104
|
+
|
105
|
+
変更が予想されないとはいえ可能性は0ではないので、アクセス時の処理変更の影響範囲が大きい場合、その影響を軽減するためにアクセサを設ける。
|
106
|
+
|
57
|
-
|
107
|
+
また、多くの場所からアクセスされる時はアクセサを設けるとデバッグしやすい。
|
108
|
+
|
109
|
+
|
110
|
+
|
111
|
+
私的な見解:
|
112
|
+
|
113
|
+
逆に、変更しても影響範囲が狭い場合はアクセサを設けるメリットが小さいのでpublicのままでも十分ということです。
|
114
|
+
|
115
|
+
そして、C++の場合は比較的熟練したプログラマが担当するケースも少なくないと思います。そのようなケースでは密結合部分についてpublicとして開示した方が、見通しも良くなりポインタ経由アクセス等、使えるテクニックも増えるのでより好ましいと感じます。
|
116
|
+
|
117
|
+
|
118
|
+
|
119
|
+
【私的な方針のまとめ】
|
120
|
+
|
121
|
+
私自身が担当しているプロジェクトは小規模・少数精鋭型が多いです。
|
122
|
+
|
123
|
+
そこで、下記方針を選択することにしました。
|
124
|
+
|
125
|
+
- メンバ変数が少しの場所(数カ所程度?)からしかアクセスされないものについては、publicとする。
|
126
|
+
|
127
|
+
- 外部に公開する部分も含め、これに該当しない場合はアクセサ化する。
|
128
|
+
|
129
|
+
あくまでも方針なのでそれなりに例外もあるとは思いますが、これで罪悪感を感じず(笑)にpublic定義できます。
|
130
|
+
|
131
|
+
|
132
|
+
|
133
|
+
######ところで、何故にC++限定だったのか?
|
134
|
+
|
135
|
+
最初に「この話題はC++言語限定とさせて下さい。広げると収集がつかなくなる予感がしますので。」と記載させて頂きました。いまいち明確に表現できてなかったのですが、今回の議論を通じてやっと言語化できました。
|
136
|
+
|
137
|
+
|
138
|
+
|
139
|
+
C++言語でのプログラミングでは、可読性やメンテナンス性も当然熟慮しますが、高速性もかなり重視しており、多少(場合によっては大きく)可読性/メンナナンス性を劣化させてでも速度を取る言語です。ポインタはその典型例と思います。更に参照とかconst参照とか右辺値参照とかもうパニック・レベルですね。まして、テンプレート・メタ・プログラミングと来た日にはorz。
|
140
|
+
|
141
|
+
|
142
|
+
|
143
|
+
この辺がスピードを犠牲にして可読性やメンテナンス性に注力しているJavaやC#のような言語と大きく使い方が異なる部分と思います。
|
144
|
+
|
145
|
+
その結果、アクセサを定義する/しないの判断基準もそれなりに異なっていることが予想できるため、C++での方針を形作りたい自分にとっては、言語を限定した方がよいということだったのです。
|
146
|
+
|
147
|
+
|
148
|
+
|
149
|
+
それでは、最後にもう一度、Stripeさんも含め、ご回答頂いた皆さんありがとうございました。
|
2
getMember\(\)にconst修飾漏れ追加
test
CHANGED
File without changes
|
test
CHANGED
@@ -18,7 +18,7 @@
|
|
18
18
|
|
19
19
|
public:
|
20
20
|
|
21
|
-
int getMember() {return mMember;}
|
21
|
+
int getMember() const {return mMember;}
|
22
22
|
|
23
23
|
void setMember(int iMember) {mMember=iMember;}
|
24
24
|
|
1
追記
test
CHANGED
File without changes
|
test
CHANGED
@@ -45,3 +45,13 @@
|
|
45
45
|
しかし、構造体(データを保持することが主目的)として設計したクラスであれば良いのですが、オブジェクト指向的に設計したクラスでは抵抗を感じてしまい、躊躇してます。
|
46
46
|
|
47
47
|
そこで、参考にさせて頂きたく、皆さんのご意見をお聞かせ下さい。
|
48
|
+
|
49
|
+
|
50
|
+
|
51
|
+
---
|
52
|
+
|
53
|
+
|
54
|
+
|
55
|
+
【追記】
|
56
|
+
|
57
|
+
もう少し多くの方のご意見を聞きたいので、今しばらく未解決のままにさせて下さい。
|