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这里使用 a 似乎没有必要，因为您确切知道可以执行多少个连接 (4)，并且除非它已被证明是一个瓶颈，否则只会使代码混乱。

你可以通过使用来使它更简单String.Format()。

Answer 3

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