当我的模型属性被声明为不具有null的日期时间时,当我们检查属性的数据类型时没有问题,但是当我有一个null的日期时间属性时,下面的代码返回false
if (datatype.GetType().Equals(new DateTime().GetType()))
{}
protected override ValidationResult IsValid(object value, ValidationContext validationContext)
{
ValidationResult validationResult = ValidationResult.Success;
try
{
// Using reflection we can get a reference to the other date property, in this example the project start date
var containerType = validationContext.ObjectInstance.GetType();
var field = containerType.GetProperty(this.otherPropertyName);
var extensionValue = field.GetValue(validationContext.ObjectInstance, null);
var datatype = extensionValue.GetType();
//var otherPropertyInfo = validationContext.ObjectInstance.GetType().GetProperty(this.otherPropertyName);
if (field == null)
return new ValidationResult(String.Format("Unknown property: {0}.", otherPropertyName));
// Let's check that otherProperty is of type DateTime as we expect it to be
if (datatype.GetType().Equals(new DateTime().GetType()))
{
DateTime toValidate = (DateTime)value;
DateTime referenceProperty = (DateTime)field.GetValue(validationContext.ObjectInstance, null);
// if the end date is lower than the start date, than the validationResult will be set to false and return
// a properly formatted error message
if (toValidate.CompareTo(referenceProperty) < 1)
{
validationResult = new ValidationResult(ErrorMessageString);
}
}
else
{
validationResult = new ValidationResult("An error occurred while validating the property. OtherProperty is not of type DateTime");
}
}
catch (Exception ex)
{
// Do stuff, i.e. log the exception
// Let it go through the upper levels, something bad happened
throw ex;
}
return validationResult;
}
让我了解最佳修复方法,因为我的属性应为null null date date字段。感谢
答案 0 :(得分:2)
您的变量field
(通过调用GetProperty
返回)的类型为PropertyInfo
,因此如果您希望类型为withotu,则需要调用属性PropertyType
。< / p>
if (field.PropertyType == typeof(DateTime) || (field.PropertyType.IsGenericType && field.PropertyType == typeof(Nullable<DateTime>))) { }
您应该将类型检查转换为typeof(TypeHere)
而不是new TypeHere.GetType()
。前者不必创建一个新的实例,如果代码重复/循环多次可能会很昂贵。看看我的例子。
另请使用@ David(另一个答案)有关使用throw;
代替throw ex;
的建议。后者将重置整个捕获的异常调用堆栈。
答案 1 :(得分:1)
我不清楚为什么你要比较类型而不是值,但我想这就是重点。在这个比较中:
datatype.GetType().Equals(new DateTime().GetType())
如果true
类型为datatype
,则会返回DateTime
。因此,合理地,如果datatype
属于其他类型,例如Nullable<DateTime>
,则不将返回true。
您应该能够与该类型进行比较:
datatype.GetType().Equals(new Nullable<DateTime>().GetType())
或者两者兼而有之:
datatype.GetType().Equals(new DateTime().GetType()) ||
datatype.GetType().Equals(new Nullable<DateTime>().GetType())
附注:您的catch
块完全是多余的,实际上是删除有关异常的信息。它是一种反模式的东西,应该完全删除。如果您实际上以catch
块中的任何方式处理异常并且做想要重新抛出,只需使用throw;
代替{ {1}}。
答案 2 :(得分:1)
您无法检查空值的类型,因此在执行任何其他操作之前,您需要检查它是否为空。
if(datatype == null)
validationResult = new ValidationResult(ErrorMessageString);
else if (datatype.GetType() == typeof(DateTime))
{
if(!IsValidDate(datatype.Value))
validationResult = new ValidationResult(ErrorMessageString);
...
}