読者です 読者をやめる 読者になる 読者になる

No more Death March

あるSEのチラシの裏 C# WPF

C# リファクタリング 早期リターン

int型の引数が5以下だったらTrueを返すとして、

        public bool Hoge(int value)
        {
            if(value < 5)
            {
                return true;
            }else{
                return false;
            }
        }

こうだったのを

        public bool Hoge(int value)
        {
            if (value < 5) return true;
            return false;
        }

こうする。

ソースコードが5行から2行へ、
ごくごくシンプルな例ですが、複雑な条件が組み合わさる場合時は、
結果が確定している時点でさっさとメソッドから抜け出すようにすると修正時のリスクが少し軽減出来る。