Dieser C# Code riecht ... scheint so, als sollte es eine sauberere Möglichkeit geben, dies zu schreiben

Question

Wer hat irgendwelche Ideen, dies zu bereinigen?

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

Zunächst einmal, indem ein StringBuilder hier scheint nicht notwendig, da Sie genau wissen, wie viele Verkettungen ausgeführt werden können (4) und, es sei denn, es ist ein Engpass erwiesen wurde, nur den Code Krempel dient .

Sie könnten es viel einfacher machen, indem Sie stattdessen String.Format() verwenden.

Answer 3

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