Entity Framework автоматически заполняет неправильные объекты

Question

Я разрабатываю пример приложения Music Store с ASP.NET MVC, Entity Framework и WCF.

Это многоуровневое приложение, имеющее общий слой для сущностей.

IЯ написал метод семени, который работает нормально, но когда я пытаюсь добавить Album к Cart, еще один дубликат Album также добавляется к таблице Album плюс также добавляются дубликаты Artists.Это происходит, когда я добавляю оператор Album = album при заполнении объекта Cart.Я добавил это утверждение, поскольку, когда я создаю свое представление, оно генерирует нуль для item.Album.Title entry

Это мой код:

public static void AddToCart(Album album, string ShoppingCartID)
{
            using (MusicStoreEntities db = new MusicStoreEntities())
            {
                // Get the matching cart and album instances
                var cartItem = db.Carts.SingleOrDefault(
                    c => c.CartId == ShoppingCartID
                         && c.AlbumId == album.AlbumId);

                if (cartItem == null)
                {
                    // Create a new cart item if no cart item exists
                    cartItem = new Cart
                    {
                        AlbumId = album.AlbumId,
                        CartId = ShoppingCartID,
                        Count = 1,
                        DateCreated = DateTime.Now,
                        Album = album
                    };

                    db.Carts.Add(cartItem);
                }
                else
                {
                    // If the item does exist in the cart, then add one to the quantity
                    cartItem.Count++;
                }

                // Save changes
                db.SaveChanges();
            }
}

Классы моделей:

namespace MusicStore.Core
{
    [Serializable]
    [DataContract]
    //[Bind(Exclude = "AlbumId")]
    public class Album
    {
        [DataMember]
        [ScaffoldColumn(false)]
        public int AlbumId { get; set; }

        [DataMember]
        [DisplayName("Genre")]
        public int GenreId { get; set; }

        [DataMember]
        [DisplayName("Artist")]
        public int ArtistId { get; set; }

        [DataMember]
        [Required(ErrorMessage = "An Album Title is required")]
        [StringLength(160)]
        public string Title { get; set; }

        [DataMember]
        [Required(ErrorMessage = "Price is required")]
        [Range(0.01, 100.00, ErrorMessage = "Price must be between 0.01 and 100.00")]
        public decimal Price { get; set; }

        [DataMember]
        [DisplayName("Album Art URL")]
        [StringLength(1024)]
        public string AlbumArtUrl { get; set; }

        [DataMember]
        public virtual Genre Genre { get; set; }

        [DataMember]
        public virtual Artist Artist { get; set; }
        public virtual List<OrderDetail> OrderDetails { get; set; }
    }
}

namespace MusicStore.Core
{
    [Serializable]
    [DataContract]
    public class Cart
    {
        [Key]
        [DataMember]
        public int RecordId { get; set; }
        [DataMember]
        public string CartId { get; set; }
        [DataMember]
        public int AlbumId { get; set; }
        [DataMember]
        public int Count { get; set; }
        [DataMember]
        public System.DateTime DateCreated { get; set; }
        [DataMember]
        public virtual Album Album { get; set; }
    }
}

Метод посева:

namespace MusicStore.Data
{
    internal class CommonDBInitializer : CreateDatabaseIfNotExists<MusicStoreEntities>
    {
        protected override void Seed(MusicStoreEntities context)
        {
            var genres1 = new List<Genre>
            {
                new Genre
                {
                    Name = "Rock",
                    Description = "Rock",
                    Albums = new List<Album>()
                },
                new Genre
                {
                    Name = "Jazz",
                    Description = "Jazz",
                    Albums = new List<Album>()
                },
                new Genre
                {
                    Name = "Metal",
                    Description = "Metal",
                    Albums = new List<Album>()
                },
                new Genre
                {
                    Name = "Alternative",
                    Description = "Alternative",
                    Albums = new List<Album>()
                },
                new Genre
                {
                    Name = "Disco",
                    Description = "Disco",
                    Albums = new List<Album>()
                },
                new Genre
                {
                    Name = "Blues",
                    Description = "Blues",
                    Albums = new List<Album>()
                },
                new Genre
                {
                    Name = "Latin",
                    Description = "Latin",
                    Albums = new List<Album>()
                },
                new Genre
                {
                    Name = "Reggae",
                    Description = "Reggae",
                    Albums = new List<Album>()
                },
                new Genre
                {
                    Name = "Pop",
                    Description = "Pop",
                    Albums = new List<Album>()
                },
                new Genre
                {
                    Name = "Classical",
                    Description = "Classical",
                    Albums = new List<Album>()
                },
            };
            genres1.ForEach(d => context.Genres.Add(d));
            context.SaveChanges();

            var artist1 = new List<Artist>
            {
                new Artist
                {
                    Name = "Aaron Copland & London Symphony Orchestra",

                },
                 new Artist
                {
                    Name = "Barry Wordsworth & BBC Concert Orchestra",

                },

            };

            artist1.ForEach(d => context.Artists.Add(d));
            context.SaveChanges();

            var album1 = new List<Album>
            {
                new Album { Title = "The Best Of Men At Work",  Price = 8.99M,  AlbumArtUrl = "/Content/Images/placeholder.gif",Genre = genres1.FirstOrDefault(d => d.GenreId == 1), Artist  = artist1.FirstOrDefault(d => d.ArtistId == 1) },
               new Album { Title = "A Copland Celebration, Vol. I",  Price = 8.99M, AlbumArtUrl = "/Content/Images/placeholder.gif",Genre = genres1.FirstOrDefault(d => d.GenreId == 2),Artist  = artist1.FirstOrDefault(d => d.ArtistId == 2) },

            };

            album1.ForEach(s => context.Albums.Add(s));

            context.SaveChanges();

        }
    }

Контроллер:

namespace MusicStore.Web.Controllers
{
    public class ShoppingCartController : Controller
    {
        MusicShoppingCartMgr.Cart serviceref1 = new MusicShoppingCartMgr.Cart();
        MusicShoppingCartMgr.iShoppingCart servicemethodref1 = new iShoppingCartClient();
        //
        // GET: /ShoppingCart/
        public ActionResult Index()
        {
            var cart = ShoppingCart.GetCart(this.HttpContext);

            // Set up our ViewModel
            var viewModel = new ShoppingCartViewModel
            {
                CartItems = cart.GetCartItems(cart.ShoppingCartId),
                CartTotal = cart.GetTotal(cart.ShoppingCartId)
            };

            // Return the view
            return View(viewModel);
        }
        //
        // GET: /Store/AddToCart/5

        public ActionResult AddToCart(int id)
        {
            var addedAlbum = servicemethodref1.GetAlbum(id);

            // Add it to the shopping cart
            var cart = ShoppingCart.GetCart(this.HttpContext);

            cart.AddToCart(addedAlbum, cart.ShoppingCartId);

            // Go back to the main store page for more shopping
            return RedirectToAction("Index");
        }

    }
}

Вид:

@model MusicStore.Web.ViewModels.ShoppingCartViewModel
@{
    ViewBag.Title = "Shopping Cart";
}
<script src="/Scripts/jquery-1.10.2.min.js" type="text/javascript"></script>

<h3>
    <em>Review</em> your cart:
</h3>
<p class="button">
    @Html.ActionLink("Checkout >>", "AddressAndPayment", "Checkout")
</p>
<div id="update-message">
</div>
<table>
    <tr>
        <th>
            Album Name
        </th>
        <th>
            Price (each)
        </th>
        <th>
            Quantity
        </th>
        <th></th>
    </tr>
    @foreach (var item in Model.CartItems)
    {
        <tr id="row-@item.RecordId">
            <td>
                @Html.ActionLink(item.Album.Title, "Details", "Store", new { id = item.AlbumId }, null)
            </td>
            <td>
                @item.Album.Price
            </td>
            <td id="item-count-@item.RecordId">
                @item.Count
            </td>
            <td>
                <a href="#" class="RemoveLink" data-id="@item.RecordId">Remove from cart</a>
            </td>
        </tr>
    }
    <tr>
        <td>
            Total
        </td>
        <td></td>
        <td></td>
        <td id="cart-total">
            @Model.CartTotal
        </td>
    </tr>
</table>

Вопрос : почему он автоматически заполняется кактакой необычный способ, и как это исправить?

Answer 1

У вас есть дублирование, потому что в вашем методе AddToCart вы создали новый контекст MusicStoreEntities.Этот контекст ничего не знает о сущности Album, которую вы передаете через параметр album.

Итак, когда вы сделали этот код:

cartItem = new Cart
{
    // [...]
    Album = album
};

и назвали db.Carts.Add(cartItem);, cartItem будет в состоянии Added.EF также проверит граф объектов cartItem и обнаружит, что новый контекст не отслеживал album (помните, что этот экземпляр получен из параметра вашего метода), а затем по умолчанию он также переведет эту сущность в состояние Added.EF будет делать то же самое с графом объектов album, и вы также получите исполнителя этого альбома в состоянии Added.При вызове db.SaveChanges(); каждая сущность с состоянием Added будет рассматриваться как генерирующая новую вставку в базу данных, поэтому у вас есть дублирование.

Чтобы устранить проблему, вы можете позвонить db.Attach(album) и позволить контексту позаботиться о сущности, как показано ниже:

using (MusicStoreEntities db = new MusicStoreEntities())
{
     db.Albums.Attach(album);

    // [...]
}

Или использовать album экземпляр, вы можетепросто установите свойство внешнего ключа AlbumId и избавьтесь от присвоения свойства Album.Таким образом, вы просто получите это при создании нового элемента корзины:

cartItem = new Cart
{
    AlbumId = album.AlbumId,
    CartId = ShoppingCartID,
    Count = 1,
    DateCreated = DateTime.Now
};