c# – 避免泛型函数中的范围问题

我来自 PHP和 Javascript的Wild Wild West,您可以从函数返回任何内容.虽然我不喜欢缺乏问责制,但我在努力保持代码“完美”方面也遇到了新的挑战.

我制作了这个通用函数来从列表中选择一个随机元素

public static T PickRandom<T>(this IList<T> list) {
    Random random = new Random();
    int rnd = random.Next(list.Count);
    return list[rnd];
}

但我想保护自己不要在0值列表中使用它.显然我不能从T以外的函数返回任何东西,例如false或-1.我当然可以这样做

if(myList.Count > 0)
   foo = Utilites.PickRandom(myList);

然而,在C#中有很多疯狂的事情我不知道,对于这个应用程序,我正在创建我非常,经常必须从列表中选择一个可以在其Count中不断递减的随机元素.有没有更好的办法？

解决方法

你有的选择

return default(T)

这将是一个模棱两可的行为,因为这可能是列表的有效元素.

或者你可以像你说的那样返回-1之类的内容,但这与你的代码完全相关.

或者你可以返回null,但这只能在T是可以为空的类型时才能完成.

在以前的所有情况下,如果调用者不知道这种情况,应用程序可能会继续使用无效值,从而导致未知后果.

所以最好的选择可能是抛出异常：

throw new InvalidOperationException();

通过这种方法,您可以快速失败并确保在调用者的意图之外不会发生任何意外情况.

支持此选项的一个原因.以Linq的扩展方法为例.如果在空列表上调用First(),Single()或Last(),则会收到InvalidOperationException,并显示消息“Sequence contains no elements”.给你的类一个类似于框架类的行为总是一件好事.

由于阿列克谢·列文科夫在问题中的评论,我正在添加旁注.随机生成并不是最好的方法.看看this question.

第二方注意.您正在声明您的函数作为IList< T>的扩展方法. (你通过在第一个参数之前使用它来做到这一点)但是你就像一个静态助手方法一样调用它.扩展方法是一种语法糖,而不是这样做：

foo = Utilites.PickRandom(myList);

让你这样做：

foo = myList.PickRandom();

有关扩展方法的更多信息,请参见here.