c# - この C# コードは匂いがします..これを書くためのよりクリーンな方法があるはずです

Question

これをきれいにするアイデアはありますか？

 public string FullName
    {
        get
        {
            var fullName = new StringBuilder();
            if (FirstName.Length > 0)
                fullName.Append(FirstName);
            if (LastName.Length > 0)
            {
                if (fullName.Length > 0)
                    fullName.Append(" " + LastName);
                else
                    fullName.Append(LastName);
            }
            return fullName.ToString();
        }
    }

Answer 1

FullName = string.format("{0} {1}",FirstName,LastName).Trim()

Answer 2

まず、StringBuilder実行できる連結の数が正確にわかっているため (4)、here を使用する必要はないように思われます。また、ボトルネックであることが示されていない限り、コードを乱雑にするだけです。

代わりに使用することで、はるかに簡単にすることができますString.Format()。

Answer 3

public string FullName
{
    get
    {
        return string.IsNullOrEmpty(FirstName) ? LastName 
            : string.IsNullOrEmpty(LastName) ? FirstName : FirstName + " " + LastName;
    }
}