为何要对ConfigurationManager.AppSettings[key]调用.ToString()方法？

这是个特别接地气的问题——我刚入行写C#的时候，看到同事写这个也满脸问号：明明AppSettings["key"]本身就是字符串类型，多这一步ToString()到底图啥？结合自己踩过的坑和圈内的讨论，我总结了几个可能的原因：

• 历史遗留的习惯惯性
  早年间的.NET Framework版本里，部分开发者可能混淆了其他配置相关的API（比如某些自定义配置节的索引器返回object），或者从更古老的代码库继承了这个写法。虽然从.NET Framework 2.0开始，AppSettings的索引器就明确返回string了，但很多老习惯就这么延续下来，没人特意去改。

• 对“防御性编程”的误解
  有些开发者可能觉得“多调用一次ToString总能更安全”，但实际上这完全是反效果：如果AppSettings["key"]不存在（返回null），直接调用.ToString()会立刻抛出NullReferenceException；而直接使用原字符串的话，你还能优雅地处理null情况，比如用ConfigurationManager.AppSettings["key"] ?? "默认值"来避免崩溃。

• 代码生成工具的“锅”
  不少早期的Visual Studio模板、第三方代码生成工具，会自动给配置取值代码加上.ToString()。很多开发者直接复用生成的代码，根本没注意到这一步是多余的，久而久之就成了“约定俗成”的写法。

• 团队规范的惯性传承
  有些团队在制定代码规范时，可能出于统一风格的考虑（比如要求所有从外部源获取的值都调用ToString()），即使后来发现这个场景下不需要，也因为没人愿意打破规范而一直保留着。

明确结论： #

从技术层面来说，这个写法毫无必要，甚至有风险。ConfigurationManager.AppSettings["key"]本身就是强类型的string，调用.ToString()不会带来任何好处，反而会在值为null时引发不必要的异常。

最佳实践推荐： #

直接使用ConfigurationManager.AppSettings["key"]，并主动处理null场景；如果是在.NET Core/.NET 5+环境下，建议使用更现代的IConfiguration接口，它的索引器同样返回string，也不需要额外调用ToString()。

内容的提问来源于stack exchange，提问作者Eleanor Holley