MVVM - 验证真的必须如此繁琐吗?

时间:2013-09-18 10:00:04

标签: c# wpf validation mvvm

在我的应用程序中,我有大量的表单,大部分都有自己绑定的模型!当然,数据验证很重要,但是没有比为所有模型实现IDataErrorInfo更好的解决方案,然后编写所有属性的代码来验证它们吗?

我已经创建了验证帮助程序,删除了很多实际的验证代码,但我仍然忍不住觉得我错过了一两招!我可以补充一点,这是我使用MVVM的第一个应用程序,所以我相信我有很多关于这个主题的知识!

编辑:

这是我不喜欢的典型模型的代码(让我解释一下):

    string IDataErrorInfo.Error
    {
        get
        {
            return null;
        }
    }

    string IDataErrorInfo.this[string propertyName]
    {
        get
        {
            return GetValidationError(propertyName);
        }
    }

    #endregion

    #region Validation

    string GetValidationError(String propertyName)
    {
        string error = null;

        switch (propertyName)
        {
            case "carer_title":
                error = ValidateCarerTitle();
                break;
            case "carer_forenames":
                error = ValidateCarerForenames();
                break;
            case "carer_surname":
                error = ValidateCarerSurname();
                break;
            case "carer_mobile_phone":
                error = ValidateCarerMobile();
                break;
            case "carer_email":
                error = ValidateCarerEmail();
                break;
            case "partner_title":
                error = ValidatePartnerTitle();
                break;
            case "partner_forenames":
                error = ValidatePartnerForenames();
                break;
            case "partner_surname":
                error = ValidatePartnerSurname();
                break;
            case "partner_mobile_phone":
                error = ValidatePartnerMobile();
                break;
            case "partner_email":
                error = ValidatePartnerEmail();
                break;
        }

        return error;
    }

    private string ValidateCarerTitle()
    {
        if (String.IsNullOrEmpty(carer_title))
        {
            return "Please enter the carer's title";
        }
        else
        {
            if (!ValidationHelpers.isLettersOnly(carer_title))
                return "Only letters are valid";
        }

        return null;
    }

    private string ValidateCarerForenames()
    {
        if (String.IsNullOrEmpty(carer_forenames))
        {
            return "Please enter the carer's forename(s)";
        }
        else
        {
            if (!ValidationHelpers.isLettersSpacesHyphensOnly(carer_forenames))
                return "Only letters, spaces and dashes are valid";
        }

        return null;
    }

    private string ValidateCarerSurname()
    {
        if (String.IsNullOrEmpty(carer_surname))
        {
            return "Please enter the carer's surname";
        }
        else
        {
            if (!ValidationHelpers.isLettersSpacesHyphensOnly(carer_surname))
                return "Only letters, spaces and dashes are valid";
        }

        return null;
    }

    private string ValidateCarerMobile()
    {
        if (String.IsNullOrEmpty(carer_mobile_phone))
        {
            return "Please enter a valid mobile number";
        }
        else
        {
            if (!ValidationHelpers.isNumericWithSpaces(carer_mobile_phone))
                return "Only numbers and spaces are valid";
        }

        return null;
    }

    private string ValidateCarerEmail()
    {
        if (String.IsNullOrWhiteSpace(carer_email))
        {
            return "Please enter a valid email address";
        }
        else
        {
            if (!ValidationHelpers.isEmailAddress(carer_email))
                return "The email address entered is not valid";
        }
        return null;
    }

    private string ValidatePartnerTitle()
    {
        if (String.IsNullOrEmpty(partner_title))
        {
            return "Please enter the partner's title";
        }
        else
        {
            if (!ValidationHelpers.isLettersOnly(partner_title))
                return "Only letters are valid";
        }

        return null;
    }

    private string ValidatePartnerForenames()
    {
        if (String.IsNullOrEmpty(partner_forenames))
        {
            return "Please enter the partner's forename(s)";
        }
        else
        {
            if (!ValidationHelpers.isLettersSpacesHyphensOnly(partner_forenames))
                return "Only letters, spaces and dashes are valid";
        }

        return null;
    }

    private string ValidatePartnerSurname()
    {
        if (String.IsNullOrEmpty(partner_surname))
        {
            return "Please enter the partner's surname";
        }
        else
        {
            if (!ValidationHelpers.isLettersSpacesHyphensOnly(partner_surname))
                return "Only letters, spaces and dashes are valid";
        }

        return null;
    }

    private string ValidatePartnerMobile()
    {
        if (String.IsNullOrEmpty(partner_mobile_phone))
        {
            return "Please enter a valid mobile number";
        }
        else
        {
            if (!ValidationHelpers.isNumericWithSpaces(partner_mobile_phone))
                return "Only numbers and spaces are valid";
        }

        return null;
    }

    private string ValidatePartnerEmail()
    {
        if (String.IsNullOrWhiteSpace(partner_email))
        {
            return "Please enter a valid email address";
        }
        else
        {
            if (!ValidationHelpers.isEmailAddress(partner_email))
                return "The email address entered is not valid";
        }
        return null;
    }

    #endregion

使用switch语句来识别正确的属性然后必须为每个属性编写唯一的验证函数的想法感觉太多(不是在工作方面,而是在所需的代码量方面)。也许这是一个优雅的解决方案,但它只是感觉不像一个!

注意:我将根据其中一个答案中的建议将我的验证助手转换为扩展名(谢谢Sheridan)

解决方案:

所以,按照我接受的答案,这是我最初实现它的工作的原因(很明显我会改进部件 - 但我只是想让它先行,因为我没有使用lambda的经验在实现这个之前的表达式或反射。)

Validtion Dictionary类(显示主要功能):

    private Dictionary<string, _propertyValidators> _validators;
    private delegate string _propertyValidators(Type valueType, object propertyValue);


    public ValidationDictionary()
    {
        _validators = new Dictionary<string, _propertyValidators>();
    }

    public void Add<T>(Expression<Func<string>> property, params Func<T, string>[] args)
    {
        // Acquire the name of the property (which will be used as the key)
        string propertyName = ((MemberExpression)(property.Body)).Member.Name;

        _propertyValidators propertyValidators = (valueType, propertyValue) =>
        {
            string error = null;
            T value = (T)propertyValue;

            for (int i = 0; i < args.Count() && error == null; i++)
            {
                error = args[i].Invoke(value);
            }

            return error;
        };

        _validators.Add(propertyName, propertyValidators);
    }

    public Delegate GetValidator(string Key)
    {
        _propertyValidators propertyValidator = null;
        _validators.TryGetValue(Key, out propertyValidator);
        return propertyValidator;
    }

模型实施:

public FosterCarerModel()
    {
        _validationDictionary = new ValidationDictionary();
        _validationDictionary.Add<string>( () => carer_title, IsRequired);
    }

    public string IsRequired(string value)
    { 
        string error = null;

        if(!String.IsNullOrEmpty(value))
        {
            error = "Validation Dictionary Is Working";
        }

        return error;
    }

IDataErrorInfo实现(它是模型实现的一部分):

string IDataErrorInfo.this[string propertyName]
    {
        get
        {
            Delegate temp = _validationDictionary.GetValidator(propertyName);

            if (temp != null)
            {
                string propertyValue = (string)this.GetType().GetProperty(propertyName).GetValue(this, null);
                return (string)temp.DynamicInvoke(typeof(string), propertyValue);
            }                

            return null;
        }
    }

忽略我的slapdash命名约定,并在编码的地方,我很高兴让这个工作!特别感谢nmclean当然,也感谢所有为这个问题做出贡献的人,所有回复都非常有帮助,但经过一番考虑,我决定采用这种方法!

4 个答案:

答案 0 :(得分:6)

我使用extension方法来减少我必须编写的验证文本的数量。如果您不熟悉它们,请查看MSDN上的Extension Methods (C# Programming Guide)页面,了解extension方法。我有几十个这样可以验证每种情况。举个例子:

if (propertyName == "Title" && !Title.ValidateMaximumLength(255)) error = 
    propertyName.GetMaximumLengthError(255);

Validation.cs课程中:

public static bool ValidateMaximumLength(this string input, int characterCount)
{
    return input.IsNullOrEmpty() ? true : input.Length <= characterCount;
}

public static string GetMaximumLengthError(this string input, int characterCount, 
    bool isInputAdjusted)
{
    if (isInputAdjusted) return input.GetMaximumLengthError(characterCount);
    string error = "The {0} field requires a value with a maximum of {1} in it.";
    return string.Format(error, input, characterCount.Pluralize("character"));
}

请注意Pluralize是另一种extension方法,如果输入值不等于1,则只在输入参数的末尾添加“s”。另一种方法可能是:

public static bool ValidateValueBetween(this int input, int minimumValue, int 
    maximumValue)
{
    return input >= minimumValue && input <= maximumValue;
}

public static string GetValueBetweenError(this string input, int minimumValue, int 
    maximumValue)
{
    string error = "The {0} field value must be between {1} and {2}.";
    return string.Format(error, input.ToSpacedString().ToLower(), minimumValue, 
        maximumValue);
}

当然,实现您需要的所有方法需要一段时间,但之后您将节省大量时间,并且您将获得所有错误消息一致的额外好处。

答案 1 :(得分:5)

我个人喜欢FluentValidation方法。

这将使用基于表达式的规则替换您的切换表,例如:

            RuleFor(x => x.Username)
                .Length(3, 8)
                .WithMessage("Must be between 3-8 characters.");

            RuleFor(x => x.Password)
                .Matches(@"^\w*(?=\w*\d)(?=\w*[a-z])(?=\w*[A-Z])\w*$")
                .WithMessage("Must contain lower, upper and numeric chars.");

            RuleFor(x => x.Email)
                .EmailAddress()
                .WithMessage("A valid email address is required.");

            RuleFor(x => x.DateOfBirth)
                .Must(BeAValidDateOfBirth)
                .WithMessage("Must be within 100 years of today.");

来自http://stevenhollidge.blogspot.co.uk/2012/04/silverlight-5-validation.html

有关此http://fluentvalidation.codeplex.com/的更多信息 - 尽管那里的文档主要基于web-MVC。对于Wpf,还有一些博客文章,如http://blogsprajeesh.blogspot.co.uk/2009/11/fluent-validation-wpf-implementation.html

答案 2 :(得分:4)

你是对的。 switch语句太多了。将IDEI(和INotifyDataErrorInfo)逻辑隔离到基类中要容易得多。

实现此目的的一种简单方法是公开一个方法来设置属性的错误,并清除属性的错误。虽然您必须为每个属性编写验证代码,但这很容易实现。

public string SomeProperty { get { return _someProperty; }
    set
    {
        _someProperty = value;
        if(string.IsNullOrWhiteSpace(value))
            SetError("SomeProperty", "You must enter a value or something kthx");
        else
            ClearError("SomeProperty");
    } 

在基类中,您保留一个只包含这些错误值的词典

protected void SetError(string propertyName, string error)
{
    _errors[propertyName] = error;
{

并按需提供,例如

string IDataErrorInfo.Error
{
    get
    {
        return string.Join(Environment.NewLine, _errors.Values);
    }
}

当您将这种模式与数据注释,一点点反射以及4.5中的某些功能相结合时,这种模式可以变得更加强大,以避免完全验证。

有几个使用CallerMemberNameAttributesimply and cleanly implement INotifyPropertyChanged in a base class的示例。如果您具有要设置的属性的名称,并使用反射(如果您担心执行而在第一次调用后将其缓存)以获取属性上的任何数据注释,则可以执行所有验证检查并将结果存储为全部在基类内。这会将派生类的属性简化为以下内容:

[NotNullOrWhiteSpace, NotADirtyWord, NotViagraSpam]
public string SomeProperty{ 
    get {return _lol;} 
    set{ _lol = value; PropertyChanged(); } }

从根本上简化了整个验证流程,仅用于少量工作。

答案 3 :(得分:3)

我看起来像这样:

new ValidationDictionary() {
    {() => carer_title,
        ValidationHelpers.Required(() => "Please enter the carer's title"),
        ValidationHelpers.LettersOnly(() => "Only letters are valid")}
}

ValidationDictionary是字符串字典 - &gt;代表。它重载Add以接受lambda表达式,该表达式转换为键的属性名称字符串,以及params委托数组,它们合并为值的一个委托。代理接受一些信息,如属性类型和值,并返回错误消息或null

在这种情况下,RequiredLettersOnly是高阶函数,它们生成在无效时返回给定字符串的委托。字符串本身作为代理传递,因此它们可以是动态的。

IDataErrorInfo是通过在字典中查找属性名称并调用委托来获取错误消息来实现的。