23

I have a registration form and the user must enter the square footage of their house. I would like this value to be only an integer. Is there a way to validate this value using attributes asp.net mvc?

1

3 Answers 3

74

Realise this has already been answered, but Stefanvds' answer is uneccessarily complicated. Just use MVCs built in validation attributes:

[DisplayName("Square Feet")]
[Required(ErrorMessage = "Square Feet is Required")]
[Range(0, int.MaxValue, ErrorMessage = "Square Feet must be a positive number")]
public int SquareFeet { get; set; }
Sign up to request clarification or add additional context in comments.

3 Comments

For Range, system has a ErrorMessage "The field XXX must be between XXX and XXX." So the ErrorMessage is optional
Yes, you are correct. But the custom ErrorMessage is a lot more user friendly than: "The field Square Feet must be between 0 and 2147483647". ;)
At least in MVC2 this does not work if the data passed in is not a valid integer. The framework will pass in the default value which is 0 and the validation will succeed, but later fail when the framework goes to actually instantiate the model and sees it cannot convert the data into an integer. It puts up the default error message at that point.
27

yes, it is, but you will have to make a flat version of the object you are wanting to create, because the validation with attributes only runs AFTER MVC has converted your data into the model. which, when your value is an int, will fail to validate if the user did not enter an int, and you will get a MVC error message in stead of your errormessage.

can you post the object you are wanting to make?

with a flat version i mean all datetimes and ints are stings in the flat version.

then i use this:

    [DisplayName("Square meters")]
    [PosNumberNoZero(ErrorMessage = "need a positive number, bigger than 0")]
    public string squaremeters { get; set; }

in the same file

public class PosNumberNoZeroAttribute : ValidationAttribute {
    public override bool IsValid(object value) {
        if (value == null) {
            return true;
        }
        int getal;
        if (int.TryParse(value.ToString(), out getal)) {

            if (getal == 0)
                return false;

            if (getal > 0)
                return true;
        }
        return false;

    }
}

if my modelstate is valid then, i use AutoMapper to convert my FlatModel into my Model, which is just 2 lines of code.

edit: if 0 is a valid number:

public class PosNumberAttribute : ValidationAttribute {
    public override bool IsValid(object value) {
        if (value == null) {
            return true;
        }
        int getal;
        if (int.TryParse(value.ToString(), out getal)) {

            if (getal >= 0)
                return true;
        }
        return false;
    }
}

2 Comments

sorry. for the trouble I should have known that. Just had a brain fart
Why not use decimal.TryParse or something? In that way, you can support more number types than just integers
10

I usually use the range attribute like this:

Positive int:

[Range(0,int.MaxValue)]
public int Id { get; set; }

Negative int:

[Range(int.MinValue,-1)]
public int Id { get; set; }

Any int:

[Range(int.MinValue,int.MaxValue)]
public int Id { get; set; }

1 Comment

Zero - is not a positive integer.

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.