回答編集履歴
4
修正
test
CHANGED
|
@@ -40,6 +40,10 @@
|
|
|
40
40
|
|
|
41
41
|
|
|
42
42
|
|
|
43
|
+
**訂正：strcmp = 辞書順 は誤り。詳しくはa_saitohさんの回答をご覧ください。**
|
|
44
|
+
|
|
45
|
+
|
|
46
|
+
|
|
43
47
|
---
|
|
44
48
|
|
|
45
49
|
すごく頭でっかちですけど、ここまで書けばかなり誤解は減るでしょう。
|
3
コメントを受けて
test
CHANGED
|
@@ -91,3 +91,135 @@
|
|
|
91
91
|
|
|
92
92
|
|
|
93
93
|
C99以降では、stdbool.hに定義されているtrue/falseを使えば良いかと思います。
|
|
94
|
+
|
|
95
|
+
|
|
96
|
+
|
|
97
|
+
コメントを受けて
|
|
98
|
+
|
|
99
|
+
---
|
|
100
|
+
|
|
101
|
+
> おそらく関数の定義を自分が誤解していたので関数は辞書順に見て前か後ろ(または同じ)かという実装の問題ではないかと思います
|
|
102
|
+
|
|
103
|
+
|
|
104
|
+
|
|
105
|
+
私もそんな気がします。
|
|
106
|
+
|
|
107
|
+
|
|
108
|
+
|
|
109
|
+
---
|
|
110
|
+
|
|
111
|
+
> ```C
|
|
112
|
+
|
|
113
|
+
bool strBefore(char *s, char *t) {
|
|
114
|
+
|
|
115
|
+
if (strcmp(s, t) <0)
|
|
116
|
+
|
|
117
|
+
return true;
|
|
118
|
+
|
|
119
|
+
else
|
|
120
|
+
|
|
121
|
+
return false;
|
|
122
|
+
|
|
123
|
+
}
|
|
124
|
+
|
|
125
|
+
bool strBefore2(char *s, char *t) {
|
|
126
|
+
|
|
127
|
+
return strcmp("1", "2")<0?true:false;
|
|
128
|
+
|
|
129
|
+
}
|
|
130
|
+
|
|
131
|
+
> ```
|
|
132
|
+
|
|
133
|
+
|
|
134
|
+
|
|
135
|
+
次のように書いた方が簡潔かと思います。
|
|
136
|
+
|
|
137
|
+
```C
|
|
138
|
+
|
|
139
|
+
bool strBefore3(char *s, char *t) {
|
|
140
|
+
|
|
141
|
+
return strcmp(s, t) < 0;
|
|
142
|
+
|
|
143
|
+
}
|
|
144
|
+
|
|
145
|
+
```
|
|
146
|
+
|
|
147
|
+
|
|
148
|
+
|
|
149
|
+
> 変数名には 「より前」の翻訳で出てきたbeforeを使い
|
|
150
|
+
|
|
151
|
+
|
|
152
|
+
|
|
153
|
+
cmpよりは彩りがある言葉ですね。
|
|
154
|
+
|
|
155
|
+
しかし`s.before(t)`という形式で記述出来ないので、コメントは添えた方が丁寧に思います。
|
|
156
|
+
|
|
157
|
+
|
|
158
|
+
|
|
159
|
+
> bool型に変えられる部分は変えてみました。
|
|
160
|
+
|
|
161
|
+
|
|
162
|
+
|
|
163
|
+
falseが0、trueが非0というのは信頼してコードを書いていいかと。
|
|
164
|
+
|
|
165
|
+
厳密には規格書に当たらないといけないですし、それに反する実装も存在し得ますが、
|
|
166
|
+
|
|
167
|
+
私にはそれを判断するだけの知見が現状無いです。すみません。
|
|
168
|
+
|
|
169
|
+
|
|
170
|
+
|
|
171
|
+
> マジックナンバーはあまりよくないらしいですが、この関数では、多くの場合実装を変更する人は、strcmp()の仕様を既に理解しているか、知らなくても理解してから変更すると思われるので、「if (strcmp(s, t) <= 0)」の0をローカルだからと言って短い名前の変数にしても、strcmp()の返り値がわかっている人が見た場合、逆にコードの理解の妨げになるかと思ってつけませんでした。
|
|
172
|
+
|
|
173
|
+
|
|
174
|
+
|
|
175
|
+
その判断で正しいように思います。
|
|
176
|
+
|
|
177
|
+
返り値を三つ以上に分岐させたい場合は、列挙体の利用も視野に入れると良いでしょう。
|
|
178
|
+
|
|
179
|
+
|
|
180
|
+
|
|
181
|
+
---
|
|
182
|
+
|
|
183
|
+
> 余談ですがC言語の名前のつけ方でよりよく使われるのは次の二つのどちらでしょうか、またそれ以上に使われる名前の付け方はあるでしょうか。最もオーソドックスなやり方を覚えておきたいんです
|
|
184
|
+
|
|
185
|
+
xxxYyy(文字の区切りに大文字)、xxx_yyy(文字の区切りに_)
|
|
186
|
+
|
|
187
|
+
|
|
188
|
+
|
|
189
|
+
xxx_yyyが一般的ですね。[『C言語 命名規則』でググると](https://www.google.co.jp/search?q=C%E8%A8%80%E8%AA%9E+%E5%91%BD%E5%90%8D%E8%A6%8F%E5%89%87)色々な意見が見られます。
|
|
190
|
+
|
|
191
|
+
|
|
192
|
+
|
|
193
|
+
|名称|Cでの用途|別名|
|
|
194
|
+
|
|
195
|
+
|:--|:--|:--|
|
|
196
|
+
|
|
197
|
+
|PascalCase||UpperCamelCase|
|
|
198
|
+
|
|
199
|
+
|camelCase||lowerCamelCase|
|
|
200
|
+
|
|
201
|
+
|snake_case|変数、関数|snail_case|
|
|
202
|
+
|
|
203
|
+
|UPPER_SNAKE_CASE|マクロ|UPPER_SNAIL_CASE|
|
|
204
|
+
|
|
205
|
+
|
|
206
|
+
|
|
207
|
+
他にも**プレフィックス**(アプリケーションハンガリアン)を知っておくと彩りが増えます。
|
|
208
|
+
|
|
209
|
+
今見てもピンとこないかもしれませんが： [間違ったコードは間違って見えるようにする](http://local.joelonsoftware.com/wiki/%E9%96%93%E9%81%95%E3%81%A3%E3%81%9F%E3%82%B3%E3%83%BC%E3%83%89%E3%81%AF%E9%96%93%E9%81%95%E3%81%A3%E3%81%A6%E8%A6%8B%E3%81%88%E3%82%8B%E3%82%88%E3%81%86%E3%81%AB%E3%81%99%E3%82%8B)
|
|
210
|
+
|
|
211
|
+
|
|
212
|
+
|
|
213
|
+
あとは、**動詞形と名詞形の使い分け**ですね。
|
|
214
|
+
|
|
215
|
+
関数に動詞、変数に名詞、配列には複数形名詞を付けると見通しが良くなります。
|
|
216
|
+
|
|
217
|
+
また、論理値を返す関数名をis, can, hasなどで始める人が多いです。
|
|
218
|
+
|
|
219
|
+
|
|
220
|
+
|
|
221
|
+
命名規則は状況によって変わり得ます。
|
|
222
|
+
|
|
223
|
+
チームで開発をしているのならチームの規則に合わせるべきです。
|
|
224
|
+
|
|
225
|
+
個人で開発をする際には、充分なメリットが説明できれば特殊な規則に従っても良いでしょう。
|
2
追記
test
CHANGED
|
@@ -80,6 +80,8 @@
|
|
|
80
80
|
|
|
81
81
|
不正な入力に対して脆弱(というかスルー)ですが、それはまた別の話です。
|
|
82
82
|
|
|
83
|
+
あと面倒なので桁あふれの可能性も無視してます。
|
|
84
|
+
|
|
83
85
|
|
|
84
86
|
|
|
85
87
|
---
|
1
修正
test
CHANGED
|
@@ -30,7 +30,7 @@
|
|
|
30
30
|
|
|
31
31
|
これに関しましては、名前に意味を含めた方が良いかと思います。
|
|
32
32
|
|
|
33
|
-
なぜなら、今後の拡張によっては、**その
|
|
33
|
+
なぜなら、今後の拡張によっては、**その複数を実装する場合も考えられるからです。**
|
|
34
34
|
|
|
35
35
|
|
|
36
36
|
|