C#MVC - 调用Reply时复制用户

时间:2013-02-08 15:57:32

标签: c# asp.net-mvc database linq

我正在尝试使用MVC创建一个简单的论坛,我无法弄清楚发布回复的用户为何会重复。

以下是回复行动:

[HttpPost]
    public ActionResult Reply(string Title, string Content,int ReplyTo)
    {
        Post masterPost = db.Posts.FirstOrDefault(p => p.PostID == ReplyTo);
        Post post = new Post();
        post.PostID = 0;
        post.CreatedOn = DateTime.Now;
        post.ModifiedOn = DateTime.Now;
        post.ReplyTo = masterPost;
        post.Forum = db.Forums.FirstOrDefault(f=>f.ForumID == masterPost.Forum.ForumID);
        post.User = (User)Session["User"];
        post.Title = Title;
        post.Content = Content;

        //if (ModelState.IsValid)
        //{
            db.Posts.Add(post);
            db.SaveChanges();
            return RedirectToAction("View", "Posts", new { id = ReplyTo });
        //}
        return View(post);
    }

这是邮政实体:

public class Post
{
    [Key]
    public int PostID { get; set; }
    public string Title { get; set; }
    [DataType(DataType.MultilineText)]
    public string Content { get; set; }
    public DateTime CreatedOn { get; set; }
    public DateTime? ModifiedOn { get; set; }
    public virtual Forum Forum { get; set; }
    public virtual User User { get; set; }
    public virtual Post ReplyTo { get; set; }
}

这是用户实体:

public class User
{
    public int UserID { get; set; }
    public string Name { get; set; }
    public string Username { get; set; }
    public string Password { get; set; }
    public string Email { get; set; }
    public DateTime RegisteredOn { get; set; }
}

每当调用ReplyTo操作时,它都会创建Post,但它也会复制存储在会话中的User(使用不同的UserID)。

此外,我评论了 ModelState.IsValid ,因为由于某种原因,它总是返回false,尽管实体已完全填充且没有DataAnnotations可能导致它无效。

修改 这是ModelState的错误: {“从'System.String'类型到'Models.Post'类型的参数转换失败,因为没有类型转换器可以在这些类型之间进行转换。”}

我做错了什么?

先谢谢。

1 个答案:

答案 0 :(得分:0)

我可以告诉你为什么ModelState无效。您不必拥有DataAnnotations来创建所需的属性。由于您无法将基元设置为null,因此它们也会计为“必需”属性,如果没有提供,则ModelState将变为无效。

在上面的示例中,我认为您没有发布CreatedOn或PostId属性(或两者)。由于您只发布了两个对您有价值的属性(标题和内容),因此最好创建一个仅包含这两个属性的简单视图模型。如果您不想这样做,即使接受两个命名参数也会更好。

以下是使用视图模型的示例。请注意,您可以将ReplyTo属性添加到视图模型中。

public class PostViewModel
{
    public int ReplyTo {get; set;}
    public string Title { get; set; }
    [DataType(DataType.MultilineText)]
    public string Content { get; set; }
 }

你可以像这样使用它:

    [HttpPost]
    public ActionResult Reply(PostViewModel viewModel)
    {
        Post masterPost = db.Posts.FirstOrDefault(p => p.PostID == viewModel.ReplyTo);
        Post post = new Post();
        post.PostID = 0;
        post.CreatedOn = DateTime.Now;
        post.ModifiedOn = DateTime.Now;
        post.ReplyTo = masterPost;
        post.Forum = db.Forums.FirstOrDefault(f=>f.ForumID == masterPost.Forum.ForumID);
        post.User = (User)Session["User"];
        post.Title = viewModel.Title;
        post.Content = viewModel.Content;

        if (ModelState.IsValid)
        {
            db.Posts.Add(post);
            db.SaveChanges();
            return RedirectToAction("View", "Posts", new { id = ReplyTo });
        }
        return View(post);
    }

对于被复制的用户,我认为其原因是User类中缺少Key属性。