無礼なコードレビュー

無礼なコードレビューと派生の与太話
11

事の発端

ぐるぐる系SQL @bleis

「このコードのレビューしてくれ。設計には目をつぶって、コーディングスタイルとか中心に頼む」→レビューする→「名前とかスタイルとかは我々にとって些末なことだ」→今ココ

2013-10-24 17:58:37
ぐるぐる系SQL @bleis

些末なことって言うならちゃんとしてもらえませんかねぇせめて!

2013-10-24 18:01:01
ぐるぐる系SQL @bleis

おちつけ、おちつくんだ俺

2013-10-24 18:02:42
もう疲れちゃって 全然動けなくてェ...(💉x5) @kamekoopa

と言うかコーディングスタイルなんてフォーマッタかければ有無を言わせず終了なわけですし、設計をレビューせずに何をレビューしたらいいのか。

2013-10-24 18:02:53
がくぞ @gakuzzzz

コーディングスタイルってOOPスタイルとかFPスタイルとかそーいう

2013-10-24 18:06:28
なぎせ ゆうき @nagise

コーディングスタイルってSemicolonlessスタイルってそーいう(違う

2013-10-24 18:07:18
もう疲れちゃって 全然動けなくてェ...(💉x5) @kamekoopa

コーディングスタイルって正座しながらコーディングするとかエビ反りでコーディングするとかそーいう

2013-10-24 18:07:19
A-pZ|ぢつにんさん @alpha_pz

コーディングスタイルとは、例えば正座しながらコーディングすることや、全裸+ネクタイでコーディングすることなのか。

2013-10-24 18:08:10
Takashi Miyamoto @tmiya_

@bleis そもそも「設計には目をつぶって、コーディングスタイルとか中心に頼む」という時点で大きく間違ってますよね。。。でも管理職な人たちには設計よりスタイルを統一する方が重要というのもありがち。。。

2013-10-24 18:09:51
ぐるぐる系SQL @bleis

そういうことだったのかー!!!

2013-10-24 18:07:58
ぐるぐる系SQL @bleis

じゃぁエビ反りスタイルで頼む

2013-10-24 18:08:17
ぐるぐる系SQL @bleis

一生エビ反ってろ!!!

2013-10-24 18:09:11

無礼なコードレビュー

ぐるぐる系SQL @bleis

無礼だって言われたー。コード批判したら無礼だって言われたー。

2013-10-25 10:19:36
ぐるぐる系SQL @bleis

クソなコードをクソと言って何が悪い、って返したい

2013-10-25 10:20:10
なぎせ ゆうき @nagise

「この部分はnew ArrayList<>();とするべき。new Vector();というのはJava1.1で知識が止まっているとしか思えず、件のコードを記述した担当者の今までの開発箇所は全面的に再検査する必要があると思われる」 #無礼なコードレビュー

2013-10-25 10:48:36
なぎせ ゆうき @nagise

あれれ。 #無礼なコードレビュー で大喜利が始まるかと思ったんだけど誰も後に続いてくれないぞ…

2013-10-25 11:11:08
Shin Tanimoto / CERO-METAL @cero_t

スペルミス。英語力が非常に低く見えるので気をつけること #無礼なコードレビュー

2013-10-25 11:14:22
Shin Tanimoto / CERO-METAL @cero_t

このようなコメントは馬鹿に見えるので控えること #無礼なコードレビュー

2013-10-25 11:15:16
Shin Tanimoto / CERO-METAL @cero_t

余計な一言、って感じだな。

2013-10-25 11:15:33
co1row.jp @co1rowjp

コントローラからDBにアクセスしない。MVCくらい理解して欲しい #無礼なコードレビュー

2013-10-25 11:20:58
椎路ちひろ @ChihiroShiiji

Thread.sleep()で長い時間待たせて誤魔化さずにきちんと同期をとること。競合についてちゃんと理解していないのではないかと疑われる。 #無礼なコードレビュー

2013-10-25 11:22:20
co1row.jp @co1rowjp

本当に無礼なコードレビューてどんなんだろ・・指摘事態が的を射ていれば無礼とは感じられない。オマケに人格攻撃とか始めたらアレだけど。

2013-10-25 11:24:00
1 ・・ 4 次へ