改進 if 條件邏輯

Question

我想改進我的代碼邏輯。 我在邏輯中有太多的 If 條件。 我想知道是否有更好的方法來執行此邏輯。 這是代碼

if (item != null && item.Fields != null && item.Fields["WilcardPage"] != null) {
    wildCardItemUrl = item.Fields["WilcardPage"].Value;     
}

if (string.IsNullOrEmpty(wildCardItemUrl)) {

    if (item.Children.Count == 1) {
        wildCardItemUrl = Sitecore.Links.LinkManager.GetItemUrl(item);
    } else {
        var settings = Sitecore.Context.Database.GetItem(ID.Parse(settingsItem));
        if (settings != null) {
            var wildcardItem = _workContext.SitecoreContext.Cast < SiteSettings > (settings);
            wildCardItemUrl = (wildcardItem.ItemWildcardPage != null) ? MediaReleaseHelper.FixMediaReleaseUrl(wildcardItem.ItemWildcardPage.ItemUrl) : string.Empty;
        }
    }
}

Answer 1

老實說，這里沒有太多需要改進的地方，但試試這個：

wildCardItemUrl = item?.Fields?["WilcardPage"]?.Value ?? wildCardItemUrl;

if (string.IsNullOrEmpty(wildCardItemUrl)) {
  if (item?.Children.Count == 1) {
    wildCardItemUrl = Sitecore.Links.LinkManager.GetItemUrl(item);
  } else {
    var settings = Sitecore.Context.Database.GetItem(ID.Parse(settingsItem));
    if (settings != null) {
      var wildcardItem = _workContext.SitecoreContext.Cast<SiteSettings> (settings);
      wildCardItemUrl = (wildcardItem.ItemWildcardPage != null) ? MediaReleaseHelper.FixMediaReleaseUrl(wildcardItem.ItemWildcardPage.ItemUrl) : string.Empty;
    }
  }
}

Answer 2

如果：

1. Fields["..."]從不拋出和
2. wildCardItemUrl在if之前是 null

那么您可以將 if 替換為：

wildCardItemUrl = item?.Fields?["WilcardPage"]?.Value;