tomatosum tomatosum 2017-02-14 22:06:52

[C#] 一行に詰め込まれた無駄な処理 このエントリーをはてなブックマークに追加

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

int unk = 0;って書けば良い処理。

Constantsクラスは実際の変数名、プロパティ名に合わせています。

public class Constants
{
    private const int ZERO = 0;
    public static int Zero
    {
        get{ return ZERO ;}
    }
}

public class Unk
{
    public Unk()
    {
        int unk = int.Parse(Constants.Zero.ToString());
    }
}

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

コメント(2)

#1 liftblockn5 liftblockn5 2017-02-15 20:30:35  

なんでこんな遠回りしちゃったの...

#2 KudouJ KudouJ 2017-02-17 13:37:55  

いやまぁ、文化として好きじゃないけど、定数は全部 Constants クラスに定義しろって規約かも知れないから ZERO までは目を瞑る。

何でわざわざ getter まで作ったの…w

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

Twitter認証

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

[Java] 連番

このエントリーをはてなブックマークに追加

もはや人間が読むものではない。

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[Java] Let's Rock

このエントリーをはてなブックマークに追加

ただのスペル違いですよ

static void mainRoop(){
    while(True)...

鑑賞する »

[C#] 君はコピペが得意なフレンズなんだね!

このエントリーをはてなブックマークに追加

コピペ量産コード クエリーの構築もEn...

class HogeRepst
{
    public void Load...

鑑賞する »