[C#] 次からはカラーでお願いします-ウンコード・マニア

ayuina 2012-08-17 08:11:20

[C#] 次からはカラーでお願いします

投稿者からのアピールポイント

コードレビューで回ってきた印刷物、A4の2枚割付の両面印刷で20枚くらいあったんじゃないかな。
これはそのうちの1メソッド。こんなメソッドがいくつかあるクラスがいくつかあった。
俺の敗因は大半がコメントであることにしばらく気づかなかったこと
気づいてからまずやったことはハサミでちょきちょきしてコメント部分の除去。3ページくらいに収まった。（3枚じゃなくて）

ウンコードっていうよりウンコメントでした。

void Method1()
{
    なんか処理がしばらく続く
/*
void Method2()
{
    なんか処理がしばらく続く
*/
    なんか処理がしばらく続く
/*
    なんか処理がしばらく続く
}
*/
    なんか処理がしばらく続く
}

使い方ヒント: 「これは臭う」という行を見付けたら、各行の smell をクリックしてマーキングしておきましょう(要Twitter OAuth認証)

コメント(7)

#1 math_neko 2012-08-17 08:55:51

このコメントアウトの仕方は反則

#2 InternalServerE 2012-08-17 13:18:39

ソースレビューを紙で回覧て、ずいぶんレトロな職場ですなぁ。
IT社会がペーパーレス社会だなんて、果たして誰が言ったのだろう。

#3 hako584 2012-09-28 16:49:03

これの一番ウンコなのは、紙に印刷してコードレビューしてる所じゃないですかね

#4 y_a_m_a_s_a_n 2012-09-28 16:54:20

紙に印刷すること自体がウンコとまでは言えません(俯瞰できる量が増えるので)が、電子データが見れないならウンコ

#5 InternalServerE 2012-09-28 19:54:04

自分は、広く俯瞰したいからこそ電子データの直接レビューが好きなんだけどなあ。

#6 b1396819 2013-01-16 15:28:40

//を使おう（推奨）

#7 migimatsu 2017-02-01 06:02:54

紙レビューは対面でやるときとかは好きな方ですが、これは見た瞬間に「書き直してこい」でレビュー終了でしょう

コメント投稿には、twitter認証が必要です。

Twitter認証

« C#一覧に戻る

このウンコードに臭った人は、こちらのウンコードにも臭ってます

[Java] フルHD推奨。

画面の左側がわりと無駄な気がします。

public int hoge(){
    int result = 0;
...

鑑賞する »

[Java] スベテカンスウ

何でもやってくれる関数って、便利ですよね...

class Processor
{
	public bool process...

鑑賞する »

[その他] HOW TO ABC..

実際にあった某システムの超重要なマスター...

create table item_master (
 A varchar(2...

鑑賞する »