c# - 使用带有单元测试的 ViewModel 的 CRUD 编辑操作

Question

首先:我找到了this post , 但我并不完全理解，所以请不要将其锁定为重复。

我正在尝试使用 ViewModel 执行编辑操作。

我的问题是出于某种原因添加 新行到表格而不是编辑 它。在我进行测试之前，这一切都曾经有效。

我相信我错过了一些愚蠢的事情，但我不知道我做错了什么。

如果有任何不同，我将使用code-first

我的 View 模型:

public class CreateViewModel
{
    public string Title { get; set; }

    [Display(Name = "Author")]
    public int AuthorId { get; set; }
    public DateTime? PublicationDate { get; set; }
    public float? Edition { get; set; }
    public SelectList Authors { get; set; }
}

我的 Controller 功能:

    // GET: Books/Edit/5
    public ViewResult Edit(int? id)
    {
        if (id == null)
        {
            return View("Error");
        }
        Book book = db.Books.FirstOrDefault(a => a.Id == id);

        var vm = new CreateViewModel()
        {
            AuthorId = book.AuthorId,
            Authors = new SelectList(db.Authors, "Id", "Name"),
            PublicationDate = book.PublicationDate,
            Title = book.Title,
            Edition = book.Edition
        };

        if (book == null)
        {
            return View("Error");
        }
        return View(vm);
    }

    // POST: Books/Edit/5
    // To protect from overposting attacks, please enable the specific properties you want to bind to, for 
    // more details see https://go.microsoft.com/fwlink/?LinkId=317598.
    [HttpPost]
    [ValidateAntiForgeryToken]
    public ActionResult Edit(CreateViewModel vm)
    {
        if (ModelState.IsValid)
        {
            var repo = new EFLibraryRepository();
            repo.Save(new Book(){
                AuthorId = vm.AuthorId,
                PublicationDate = vm.PublicationDate,
                Title = vm.Title,
                Edition = vm.Edition
            });
            return RedirectToAction("Index");
        }
        return View("Edit", vm);
    }

我的模拟仓库:

public class EFLibraryRepository : ILibraryRepository
{
    AuthorAndBookDbModel db = new AuthorAndBookDbModel();
    public IQueryable<Author> Authors { get { return db.Authors; } }

    public IQueryable<Book> Books { get { return db.Books; } }

    public void Delete(Book book)
    {
        db.Books.Remove(book);
        db.SaveChanges();
    }

    public void Delete(Author author)
    {
        db.Authors.Remove(author);
        db.SaveChanges();
    }

    public Book Save(Book book)
    {
        if (book.Id == 0)
        {
            db.Books.Add(book);
        }
        else
        {
            db.Entry(book).State = System.Data.Entity.EntityState.Modified;
        }
        db.SaveChanges();
        return book;
    }
    public Author Save(Author author)
    {
        if (author.Id == 0)
        {
            db.Authors.Add(author);
        }
        else
        {
            db.Entry(author).State = System.Data.Entity.EntityState.Modified;
        }
        db.SaveChanges();
        return author;
    }
}

ILibraryRepository:

public interface ILibraryRepository
{
    IQueryable<Book> Books { get; }
    IQueryable<Author> Authors { get; }

    Book Save(Book book);
    Author Save(Author author);

    void Delete(Book book);
    void Delete(Author author);
}

Answer 1

在您的 POST 方法中，您永远不会设置 Book 的 Id 属性的值，因此它始终是 0(int 的默认值)所以反过来，你总是执行代码来添加一个新的Book。

您首先需要在 View 模型中为 id 包含一个属性，以便它的值将绑定(bind)在 POST 方法中。

public class CreateViewModel
{
    public int? Id { get; set; } // add this
    ....

请注意，假设您使用默认路由，您不需要在 View 中为它包含隐藏输入(它的值将从表单 action 属性中的路由值绑定(bind))。

然后在POST方法中，根据 View 模型设置Book的Id

repo.Save(new Book() {
    Id = vm.Id, // add
    AuthorId = vm.AuthorId,
    ....

但是，编辑现有记录时的正确做法是根据Id从存储库中获取原始数据模型并更新其属性，例如

Book book = db.Books.FirstOrDefault(a => a.Id == vm.Id);
book.AuthorId = vm.AuthorId;
....
repo.Save(book);

而不是创建一个新的 Book 实例。这种方法的一些好处包括

1. 您的数据模型通常会包含不应包含在其中的属性 View (例如，指示记录日期的属性创建，以及由谁创建)。创建数据模型的新实例和保存它意味着这些属性将被覆盖并设置为他们的默认值。
2. 你可以做并发检查，例如你可以检查TIMESTAMP 值，如果它们不同，您就知道另一个用户同时修改了记录(您可能需要不同的行动方针，而不仅仅是覆盖以前的使用变化)