if(currency.equal("CND")) if (string.IsNullOrEmpty(member.LastName)) { return $"{member.FirstName}".Trim(); } else { return $"{member.LastName} {member.FirstName}".Trim(); } else if (string.IsNullOrEmpty(member.LastName)) { return $"{member.FirstName}".Trim(); } else { return $"{member.FirstName} {member.LastName}".Trim(); }我需要簡化這個(gè)更短的陳述，但我不確定如何？我是新手，在這些東西我可能需要一些幫助.任何建議？
3 回答
開滿天機(jī)
TA貢獻(xiàn)1786條經(jīng)驗(yàn) 獲得超13個(gè)贊
這并不一定簡化了它，它只是使它成為一個(gè)單行
return $"{(!member.LastName.IsNullOrEmpty() ? member.LastName : "")}{member.FirstName}".Trim();
但是，為了更好的清晰度和可讀性，該塊是完全可以的...if/else
if (string.IsNullOrEmpty(member.LastName))
{
return $"{member.FirstName}".Trim();
}
else
{
return $"{member.LastName} {member.FirstName}".Trim();
}
我肯定更喜歡塊而不是使用單行字符串插值if...else
藍(lán)山帝景
TA貢獻(xiàn)1843條經(jīng)驗(yàn) 獲得超7個(gè)贊
您可以移動(dòng)一些代碼以刪除重復(fù)項(xiàng)。如果缺少姓氏，則名字是唯一要打印的姓氏，因此請(qǐng)先進(jìn)行測(cè)試。然后測(cè)試按首選順序打印出全名的條件。
if (string.IsNullOrEmpty(member.LastName))
{
return member.FirstName.Trim();
}
else
{
return currency.equal("CND")
? $"{member.LastName} {member.FirstName}".Trim()
: $"{member.FirstName} {member.LastName}".Trim();
}
您也可以將其重寫為嵌套的三元操作，盡管它實(shí)際上并不短，并且它是否更具可讀性取決于閱讀它的人...
return (string.IsNullOrEmpty(member.LastName)
? member.FirstName
: currency.equal("CND")
? $"{member.LastName} {member.FirstName}"
: $"{member.FirstName} {member.LastName}").Trim();
慕田峪4524236
TA貢獻(xiàn)1875條經(jīng)驗(yàn) 獲得超5個(gè)贊
這絕對(duì)是過度設(shè)計(jì)的。不是更短，但意圖更清晰，易于擴(kuò)展。
public static string GetDisplayName(Member member, string currency)
{
return string.Join(" ", GetDisplayNameParts(member, currency));
}
public static IEnumerable<string> GetDisplayNameParts(Member member, string currency)
{
switch (currency)
{
case "CND":
yield return member.LastName ?? ""
yield return member.FirstName ?? ""
yield break;
default:
yield return member.FirstName ?? ""
yield return member.LastName ?? ""
yield break;
}
}
- 3 回答
- 0 關(guān)注
- 136 瀏覽
添加回答
舉報(bào)
0/150
提交
取消