public string GetUserDisplay(User u) =>
u?.IsActive == true ? (u.Name ?? "").Trim() is var n && n.Length > 0
? n + (u.Role > 0 ? $" ({(Role)u.Role})" : "") : "Unknown" : "Inactive";
پس فردا یه باگ میخوری ولی نمیدونی از کدوم قسمتش ممکنه باشه چون بسیار خفن و پیچیده نوشتی ولی دیگ یه سال دیگ عقل امروزت که کدتو نوشتی رو نداری برمیگردی میگی این شتو کی نوشته بعد گیت لنز رو نگاه میکنی میبینی عه ایمیل خودته !
ولی میتونستی اینطوری بنویسیش
public string GetUserDisplay(User user)
{
if (user is null || !user.IsActive)
return "Inactive";
var name = user.Name?.Trim();
if (string.IsNullOrEmpty(name))
return "Unknown";
if (user.Role > 0)
return $"{name} ({(Role)user.Role})";
return name;
}
از نظر عملکرد کاملا مشابه هم دیگس ولی خوانایی این کد کجا اون کجا پس KISS مرد بزرگ !
@codehalics | کدهالیک