ここだけ全員焼肉店(焼肉の夏、日本の夏) 793店舗目
↓
1-
覧
板
20
338
:
さとりん
[sage]
2021/05/04(火) 01:35:38.84 ID:ffv1VO+no
// Aの場合
if(xの値が○○でyの値が××の時)
※Aは業務的なワード
以下略
AAS
339
:
エグゼリカ
[sage]
2021/05/04(火) 01:36:58.97 ID:atKvCFSm0
コメントはメソッド頭の機能説明と分かりにくいところの補足説明がありゃいいわ
is〇〇みたいな判定系はそれすらいらんと思う
340
:
さとりん
[sage]
2021/05/04(火) 01:38:38.85 ID:ffv1VO+no
そういえばcheck○○とかウンコメソッドあったな。
問題があれば業務例外スローするメソッドでこの名前は付けた事があったけど。
341
:
エグゼリカ
[sage]
2021/05/04(火) 01:39:19.19 ID:atKvCFSm0
変数も、そいつが業務的に何らかの意味がある値ならValueObject化しちゃえばコメントでの説明いらんのよな
342
:
さとりん
[sage]
2021/05/04(火) 01:39:41.01 ID:ffv1VO+no
前案件の嫌いだったポイント
業務的な単位Aを数える変数:count
業務的な単位Bを数える変数:count2
以下略
AAS
343
:
さとりん
[sage]
2021/05/04(火) 01:42:49.06 ID:ffv1VO+no
>>341
わからんくもないんだけど、半ば風習として付けたくなるんだよな。
C89/90の仕事してたとき、先頭で変数宣言する必要があって、
int i; // ループカウンタ
とか書いてたわ。
344
:
さとりん
[sage]
2021/05/04(火) 01:43:34.79 ID:ffv1VO+no
まぁほかがその実装だし、自明である証明は出来ないし、
そこでバトルする時間も無駄なので数文字くらいつけてやるよって気持ちではあった
345
:
エグゼリカ
[sage]
2021/05/04(火) 01:44:58.56 ID:atKvCFSm0
>>342
なるべくAもBもリストに一括で取ってきたいところだねぇ
346
:
エグゼリカ
[sage]
2021/05/04(火) 01:46:47.42 ID:atKvCFSm0
>>343
Cはまぁ、そりゃしょうがないわな
、/| 、/|
/ 」 | / | |
以下略
AAS
347
:
さとりん
[sage]
2021/05/04(火) 01:47:51.76 ID:ffv1VO+no
てかね、現場でどうしても迷うのが、
新しい記法とか使うかは迷う。
Java8のStreamAPIとか。Java8を新しいって言いたくはないけど、
レベルの低い現場だと最先端な書き方になってしまうから
1002Res/1939.58 KB
↑[8]
前[4]
次[6]
書[5]
板[3]
1-[1]
l20
ここだけ全員焼肉店(焼肉の夏、日本の夏) 793店舗目-パー速VIP http://ex14.vip2ch.com/test/read.cgi/part4vip/1596710681/
VIPサービス増築中!
携帯うpろだ
|
隙間うpろだ
Powered By
VIPservice