1.4版本 HttpResult.cs 的 RedirectUrl 可以这样改写

285900537

1.4版本 HttpResult.cs 的 RedirectUrl 可以这样改写：

[C#] 纯文本查看 复制代码

return Header == null || Header.Count <= 0
    ? string.Empty
    : Header.AllKeys.AsEnumerable().FirstOrDefault(key => key.ToLower() == "location").SafeString();

其中SafeString() 的方法是

[C#] 纯文本查看 复制代码

[DebuggerStepThrough]
public static string SafeString(this object @object)
{
    return @object as string;
}

用于安全返回 System.String 的值

站长苏飞

首先感谢你的热心不过你这样写有几点不妥

1. 这个RedirectUrl本身就是属性提取，为什么要二次提取？这点感觉是不是有点多此一举了。
2.这样写如果location不存在的时候就无法获取最后一次访问的地址的。

[C#] 纯文本查看 复制代码

return Header == null || Header.Count <= 0
    ? string.Empty
    : Header.AllKeys.AsEnumerable().FirstOrDefault(key => key.ToLower() == "location").SafeString();

这样只有在Location存在并且 Allowautoredirect =false时才会有值。
3.？ 号表达式其实是我个人一直不喜欢的，因为很容易把简单的问题复杂化。破坏了代码的结构性

也可能是你有其他的用意，可以说一下交流交流

285900537

其实这段就是对 RedirectUrl 的 Getter 的改写。

无论是 Location 还是 location，其实只有一个字母的区别，用两个 If 去判断会让代码不美观。

这段表达式能够获得 location 和 Location 这两个键的值。当然，用两个 If 能做到优先获得 location，未果后才去拿 Location 的值。

SafeString 安全返回 System.String，（如果 header 内既没有 Location 也没有 location，则返回 null）。

如果不要 null，一定要 String.Empty 的话，可以这样写

[C#] 纯文本查看 复制代码

 
 return Header == null || Header.Count <= 0
    ? string.Empty
    : Header.AllKeys.AsEnumerable().Any(key => key.ToLower() == "location")
        ? Header.AllKeys.AsEnumerable().FirstOrDefault(key => key.ToLower() == "location").SafeString()
         : string.Empty;

当然，这也没有什么讲究的，还是看个人喜好。比如我更偏向用表达式来处理，会选用上面这段代码。站长更偏向于采用效率高一些的 If（在这里 If 的效率比表达式略高一些）来处理，也有人会用三元表达式来弄，比如：

[C#] 纯文本查看 复制代码

 
return Header == null || Header.Count <= 0
    ? string.Empty
    : Header.AllKeys.Contains("location")
        ? Header["location"].SafeString()
        : Header.AllKeys.Contains("Location")
            ? Header["Location"].SafeString()
            : string.Empty;