我应该更改为扩展方法（在ListItemCollection上）

Question

（我希望这不会被视为“太像讨论”，但我们会看到的。）

我有几个小时来重构长期存在的代码。 我注意到ListItemCollection上有很多静态方法：

public static ListItem ListItem_InsertIfNotPresent(ListItemCollection lic, ...)
public static ListItemCollection ListItems_Selected(ListItemCollection lic)
public static string[] ListItemValues(ListItemCollection lic)
public static void ListItems_SortByText(ListItemCollection lic)
....

因此，它们看起来很适合重构。 ListItemCollections由许多CLR函数返回，因此我无法派生/子类。 我在玩：

1. 实施为扩展方法。 代码当前没有任何扩展方法。 不必为必须using ListItemCollectionExtensions添加到我的所有源文件而生气，也不必冒将来可能会添加冲突的.NET CLR的风险。 但是调用语法很好。
2. 添加一个ListItemCollectionHelper类以将其放入。代码将必须为var licx = new ListItemCollectionHelper(dropDownList.Items); 在任何licx.SortByText();之前licx.SortByText(); 类型调用，当我只需要调用一个方法时，这有点混乱。
3. 保持原样，并使用命名静态变量！

我看过类似.Net中使用扩展方法的最佳实践是什么的文章？ 和其他，但他们并没有真正建议这种情况。 特别有趣的是您对第一和第二的评论。

实际上，我不必担心我的代码的“其他用户”，但是良好的实践永远不会有什么坏处。 对还是错，我不想使用LINQ或lambda解决方案。

Answer 1

在您的示例中，除非您在其他地方需要代码，否则我将保留原样，因为它正在运行，并且仅重构的重构可能会导致错误。 但是，如果我从头开始，我将使用这些扩展方法。 它使代码更易于重用，并希望防止有人在下次需要相同模式时复制此代码。

我更喜欢扩展方法而不是Helper类，因为通过Intellisense通常更容易找到方法。 以我的经验，任何以“ Helper”一词结尾的课程在某个时候都会变成一团糟。 这就变成了不适合任何地方的代码的地方。 Helper类从不会从坏开始，但是在一个更大的团队中，您很快就会在其中找到没有业务的方法。

Answer 2

我会继续进行扩展。

1. 它不会破坏任何内容（您仍可以像Helper模式中的常规方法那样使用，因此您现有的代码仍将起作用）
2. 当您重构现有代码或编写新代码时，可以使用更清洁的扩展方法语法。