这个c#代码闻起来......似乎应该有一个更清晰的方式来写这个

时间:2012-04-06 03:08:15

标签: c# string

任何人都有任何想法要清理它吗?

 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();
        }
    }

3 个答案:

答案 0 :(得分:9)

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

答案 1 :(得分:1)

首先,在这里使用StringBuilder似乎是不必要的,因为你确切知道可以执行多少个连接(4),除非它被证明是一个瓶颈,否则只会使代码混乱。

您可以使用String.Format()来简化它。

答案 2 :(得分:1)

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