在验证方法的输入时,我通常会检查参数是否为空,如果是,我会抛出 ArgumentNullException。我对列表中的每个参数都执行此操作,因此最终得到如下代码:
public User CreateUser(string userName, string password,
string Email, string emailAlerts,
string channelDescription)
{
if (string.IsNullOrEmpty(userName))
throw new ArgumentNullException("Username can't be null");
if (string.IsNullOrEmpty(Email))
throw new ArgumentNullException("Email can't be null");
//etc, etc, etc
}
这个可以吗?我为什么要这样做?如果我简单地将所有检查分组并返回空值而不是抛出异常,可以吗?解决这种情况的最佳做法是什么?
PS:我想改变这一点,因为使用很长的方法,这样做会变得非常乏味。
Ideas?
用这样的东西创建一个 ArgChecker 类
ArgChecker.ThrowOnStringNullOrEmpty(userName, "Username");
其中 ThrowOnStringNullOrEmpty 是
public static void ThrowOnStringNullOrEmpty(string arg, string name)
{
if (string.IsNullOrEmpty(arg))
throw new ArgumentNullException(name + " can't be null");
}
您还可以尝试使用 params arg 处理参数列表,例如:
public static void ThrowOnAnyStringNullOrEmpty(params string[] argAndNames)
{
for (int i = 0; i < argAndName.Length; i+=2) {
ThrowOnStringNullOrEmpty(argAndNames[i], argAndNames[i+1]);
}
}
像这样打电话
ArgChecker.ThrowOnAnyStringNullOrEmpty(userName, "Username", Email, "email");
本文内容由网友自发贡献,版权归原作者所有,本站不承担相应法律责任。如您发现有涉嫌抄袭侵权的内容,请联系:hwhale#tublm.com(使用前将#替换为@)