制作一个不使用扩展对象的扩展方法是一个好习惯吗？

Question

在我们的 ASP.NET MVC 项目中，我们有一个 HtmlHelper 扩展方法来生成静态谷歌地图。

public static MvcHtmlString StaticMap(this HtmlHelper helper, string address, string alt, int width, int height, int zoom)
{
    var src = new Uri("http://maps.google.com/maps/api/staticmap?markers=size:mid|color:red|{0}&zoom={1}&size={2}x{3}&maptype=roadmap&sensor=false".FormatInvariant(Uri.EscapeUriString(address), zoom, width, height));
    var href = new Uri("http://maps.google.com/maps?f=q&source=s_q&hl=en&q={0}".FormatInvariant(Uri.EscapeUriString(address)));

    var img = new TagBuilder("img");
    img.MergeAttribute("src", src.ToString());
    img.MergeAttribute("alt", alt);

    var link = new TagBuilder("a") { InnerHtml = img.ToString() };
    link.MergeAttribute("href", href.ToString());

    return MvcHtmlString.Create(link.ToString());
}

对于这个新项目，我们还试图保持所有代码分析规则的开启。现在，显然，Visual Studio Code 分析表明我们应该删除 helper 参数，因为它没有被使用。

这让我想知道扩展方法是否应该始终使用扩展对象，如果不使用，那么它可能不应该是扩展方法。

有人有可以帮助我做出决定的指南或说明的链接吗？

Answer 1

如果您不使用 helper 对象，则您不会扩展任何内容，并且确实没有理由不将其设为您自己的命名空间中的常规静态方法。

Answer 2

扩展方法和静态方法之间的使用代码的区别纯粹是概念性的；一个以与实例方法相同的方式涉及特定类型的对象，而另一个则不然。

既然如此，我会问一个问题：“考虑这个操作在 HtmlHelper 对象上有意义吗？”。这又变成“考虑 HtmlHelper 对象提供此操作有意义吗？”。如果我对这个问题的回答是“是”，那么无论我是否使用 HtmlHelper 对象，我都会认为扩展方法是一种合理的方法。相反，如果我对这个问题回答“否”，我会认为扩展方法是一种不明智的方法，即使确实使用了 HtmlHelper 对象。

代码分析在分析代码使用方面比分析概念做得更好，因此这里给出了警告。它是不完美的，并且确实存在其他情况，忽略参数是合理的（保持与先前版本或接口的兼容性是最好的情况，“保留以供将来使用”是更有争议的情况）。

值得注意的是，在某些语言（例如 C++）中，您可以精确地省略参数名称，以表示“此参数在签名中，但不会被使用”。我非常喜欢这一点，因为通常不使用参数确实是一个坏兆头，所以最好有一种方法来表明您是故意这样做的。

编辑：

另一个理由。想象一下，您在一个类上有一个使用相关对象的扩展方法。现在想象一下，您意识到，通过以不再使用该对象的方式重写，可以使其更可靠、更高效，或者在某些“更好”值上“更好”。既然你已经改进了扩展方法，难道不应该允许你保留它吗？你应该被迫继续使用劣质版本吗？

Answer 3

为什么一开始就将此作为扩展方法？我想说这不是你应该做的事情。我想不出一个好的理由。