.NET C＃中MVC的最佳实践

Question

I am novice to MVC framework. I am using entityframework DBcontext to perform Database related operation. I am doing this way Entityframework-->dbcontext-->Model-->Controller--> View . I am directly binding Model to view. eg there is Table name UserProfile and i have created Model with same name and colum as property of Model. I am puling collection of record from Userprofile and bind this collection directly to View. Here is Model

[Table("UserProfile")]
public class UserProfile
{
    [Key]
    [DatabaseGeneratedAttribute(DatabaseGeneratedOption.Identity)]
    [DisplayName("ID")]

    public long UserId { get; set; }
    private string _UserName;
    public string UserName
    {
        get { return _UserName; }
        set { _UserName = value; }

    }
    public string Thalassemic
    {
        get;
        set;
    }

    [Display(Name = "Your First Name")]
    [Required(ErrorMessage = "First Name is Required.")]
    [StringLength(30, ErrorMessage = "First Name must be {2}-{1} to long", MinimumLength = 2)]
    [RegularExpression("^[a-zA-z ]+$", ErrorMessage = "First name must contain only characters")]
    public string FirstName { get; set; }

    [Display(Name = "Your Last Name")]
    [Required(ErrorMessage = "Last Name is Required.")]
    [RegularExpression("^[a-zA-z ]+$", ErrorMessage = "Only characters are allowed.")]
    [StringLength(20, ErrorMessage = "First Name must be {2}-{1} to long", MinimumLength = 2)]
    public string LastName { get; set; }

    [Required(ErrorMessage = "Email Address is Required.")]
    [StringLength(250, ErrorMessage = "{0} must be {2}-{1} to long", MinimumLength = 4)]
    [Display(Name = "Email Address")]
    [DataType(DataType.EmailAddress, ErrorMessage = "Please enter a valid email-address.")]
    [RegularExpression("^([a-zA-Z0-9_\\-\\.]+)@((\\[[0-9]{1,3}\\.[0-9]{1,3}\\.[0-9]{1,3}\\.)|(([a-zA-Z0-9\\-]+\\.)+))([a-zA-Z]{2,4}|[0-9]{1,3})(\\]?)$", ErrorMessage = "Please enter a valid email-address.")]
    //todo: re-think about updating email address as we are user email as login id and this can't change
    public string Email { get; set; }}

Should I use this DTO object to bind data with view

public class UserProfileDTO
{
    public long UserId { get; set; }
    private string _UserName;
    public string UserName
    {
        get { return _UserName; }
        set { _UserName = value; }
    }
    public string Thalassemic
    {   get;
        set;
    }
    public string FirstName { get; set; }
    public string LastName { get; set; }
    public string Email { get; set; }
}

based on above I want to know that is it right method? if not then What is best practice?

Answer 1

Using database POCO classes as a model is generally discouraged in MVC, in place of view models that contain only the data that the view needs. Imagine you have a User record with 30+ fields, but you only want to display the Username , you could simply create a UsernameViewModel :

public class UsernameViewModel
{
    public string Username { get; set; }
}

It's much cleaner because you're only sending to the client the fields that you need.

Answer 2

As @mat said, it's not recommended to use POCO classes directly in the views. It is advised against it because the user should never be able to modify the POCO values, also, what happens when you want to show more than one POCO to the user? (I hope your answer is not "Modify the POCO to meet the users request").

Also, if your application is n-tier, it's strongly advised to use some DTOs to transfer the data between the layers.

As a suggestion and benefits of using a DTO class you can check the msdn link here or, as a demo part, there is a good tutorial on what to use depending on your requirements on codeproject here .

Read more on stackoverflow about your question here