Entity Framework 6, не обновляется внешний ключ в таблице. Как исправить?

Question

[SMARTi]

Создал простенькую объектную модель с продуктами и категориями:

public class Product
    {
        public int Id { get; set; }
        public string Name { get; set; }
        public virtual Category Category { get; set; }
    }

    public class Category
    {
        public int Id { get; set; }
        public string Name { get; set; }
    }

VS помогла сгенирировать контролер с использованием odata ProductController, он содержит метод для обновления данных:

// PUT odata/Product(5)
        public IHttpActionResult Put([FromODataUri] int key, Product product)
        {
            if (!ModelState.IsValid)
            {
                return BadRequest(ModelState);
            }

            if (key != product.Id)
            {
                return BadRequest();
            }
            
            db.Entry(product).State = EntityState.Modified;

            try
            {
                db.SaveChanges();
            }
            catch (DbUpdateConcurrencyException)
            {
                if (!ProductExists(key))
                {
                    return NotFound();
                }
                else
                {
                    throw;
                }
            }

            return Updated(product);
        }

В этот метод приходит объект Product с измененными Category и Name,
но в итоге в базе в таблице Product Name изменяется, а Category_Id нет, без каких-либо ошибок.

Answer 1

[Станислав Бояринцев]

Чтобы избежать проблем при работе с отключенными сущностями (Disconnected Entities) вам нужно использовать не только Navigation Properties но и связи на основе внешнего ключа ( foreign key associations), для этого сделайте вашу модель такой:

public class Product
    {
        public int Id { get; set; }
        public string Name { get; set; }
        public virtual Category Category { get; set; }
        public int CategoryId { get; set; }
    }

Подробнее можно прочитать в книге "Programming Entity Framework" Julia Lerman, 85 страница раздел "Working with Relationships with and Without Foreign Keys"

Comments

[SMARTi]

Нет, это не помогает

[Станислав Бояринцев]

SMARTi: обновил

[SMARTi]

masterL: я добавил свойство CategoryId в класс Product, после миграции в БД в таблице Product соответственно добавилось поле CategoryId, теперь в таблице по сути два поля для связи с категориями и Category_Id и CategoryId, разве это правильно?

[Станислав Бояринцев]

Нет не правильно. Попробуйте пересоздать БД, а не делать миграцию.

[SMARTi]

masterL: Пересоздание базы не помогло, в новой таблице Product также оказываются оба поля и Category_Id и CategoryId. Еще заметил странность, при добавление нового объекта Product, в его Category находится уже существующая категория, то после сохранения генерируется новая запись в Category с которой и связывается новый объект Product.

[Станислав Бояринцев]

А если новое поле в модели назвать не CategoryId а Category_Id ?

[SMARTi]

masterL: Если назвать Category_id то в таблице появляется поле Categoy_Id1. По сути решением проблемы оказалось помимо добавления свойства CategoryId в класс Product, необходимо было в класс Category добавить такое свойство public virtual ICollection Products { get; set; }
Но все же создается ощущение, что написал костыль...

[Станислав Бояринцев]

Нет это не костыль, это вполне стандартный способ через коде фёрст задать связь "Один ко Многим", то есть на Одну категорию может ссылаться множество Продуктов, просто я забыл, что в Category тоже нужно сделать ссылку на Product.

[SMARTi]

masterL: Пока занимался редактированием записей, забыл про добавление новых, а там творятся еще большие непонятки... В этой строчке: db.Products.Add(product); объект продукт такой как надо для создания нового, Id и CategoryId = 0, а Category содержит ссылку на правильный объект с Id = 2. Но после выполнения db.SaveChanges(); В таблице Category появляется новая запись, и именно на нее начинает ссылаться новый продукт, получается с каждым добавлением нового продукта, создается новая категория...

Answer 2

[asdz]

Вам просто нужно присвоить объект свойству навигации, либо, как сказал masterL , перейти от независимых сопоставлений к сопоставления на основе внешнего ключа, и тогда, вы просто можете задать значение ключа для изменения свойства. Здесь, описаны варианты работы с внешними ключами.

Comments

[SMARTi]

Большое спасибо за ссылку, она очень помогла.

Answer 3

[issa05]

Вам все правильно написали, создаете поле CategoryId, при редактировании этому полю присваиваете новое значение - id другой категории и сохраняете изменения.