ASP.NET MVC 5 Validation - Property are different

Question

Hi I thought this was a simple one.

To reduce the problem for the sake of clarity I have a three properties in my view model, user name, password and confirm password.

Obviously we want to make sure the passwords match but I also want to make sure that they do not match the username.

There are lots of hacks out there for previous version of MVC but I can't believe there is not a more subtle way of dealing with this obvious comparison in MVC5

    [Required]
    [DataType(DataType.Text)]
    [Display(Name = "User name")]
    public string UserName { get; set; }

    [Required]
    [StringLength(100, ErrorMessage = "The {0} must be at least {2} characters long.", MinimumLength = 6)]
    [DataType(DataType.Password)]
    [Display(Name = "Password")]
    [NotEqualTo("UserName", ErrorMessage = "The password can not be the same as the User name.")]
    public string Password { get; set; }

    [Required]
    [DataType(DataType.Password)]
    [Display(Name = "Confirm password")]
    [Compare("Password", ErrorMessage = "The password and confirmation password do not match.")]
    public string ConfirmPassword { get; set; }

So what's the problem? Go ahead and implement this attribute, shouldn't take the whole day — Vsevolod Goloviznin
– Vsevolod Goloviznin, Commented Dec 12, 2014 at 19:06
You can also check for the RemoteValidation, as example: stackoverflow.com/questions/10163683/… — Rafael Merlin
– Rafael Merlin, Commented Dec 12, 2014 at 19:17
It's not a problem Vsevolod it was a question, can it be done in a more subtle way. — JasonLCook
– JasonLCook, Commented Dec 15, 2014 at 9:38
Instead of adding third party dependencies and mixing up attribute & fluent validation, I would have chosen to implement your approach in the question for consistency. — mcy
– mcy, Commented Jul 18, 2018 at 13:28

jerone · Accepted Answer · 2018-07-18 12:58:49Z

3

I'm a big fan of using the FluentValidation library when dealing with any non-standard validations. It even contains a NotEqual() method for doing exactly what you're talking about.

public class RegisterViewModelValidator: AbstractValidator<RegisterViewModel>  
{
    public RegisterViewModelValidator()
    {
        RuleFor(m => m.Password).NotEqual(m => m.UserName).WithMessage("Password Cannot Match Username");
    }
}

You may want to go a step further and change it not allow the password to CONTAIN the username.

RuleFor(m => m.Password).Must((vm, pwd) => !pwd.Contains(vm.UserName)).WithMessage("Password Cannot Match Username");

While you can mix FluentValidation and the standard attributes as well, MVC will get mad if they both apply the same validation (for example, including the [Required] attribute and also adding the NotEmpty() rule in Fluent will throw an error).

edited Jul 18, 2018 at 12:58

jerone

17.1k4 gold badges41 silver badges58 bronze badges

answered Dec 12, 2014 at 19:18

zaparker

4853 silver badges14 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

ASP.NET MVC 5 Validation - Property are different

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related