Как сформировать правильный Code First?

Question

[Дмитрий Гавриленко]

Есть классы сущности

class Transaction
    {
        public int ID { get; set; }
        public Customer Customer { get; set; }
        public DateTime Date { get; set; }
        public double Price { get; set; }
        public int Discount { get; set; }
        public double Ended { get; set; }
    }
    class Customer
    {
        public int ID { get; set; }
        public string Name { get; set; }
        public string Last { get; set; }
        public string Other { get; set; }
    }

Данная модель реализована для приложения продаж. Совершается покупка, на определенную сумму, ищется покупатель и посредством

Transaction trans = new Transaction { Customer = cust, Date = DateTime.Now, Discount = discount, Price = price, Ended = ended };

данные добавляются в БД. Все работает на ура, но просмотрев БД я увидел, что


Как решить?

Comments

[veitmen]

А в чем проблема? В том, что Customer'ов создалось три? Тогда код полностью нужен. Как вы создаете объект cust.

[Дмитрий Гавриленко]

veitmen: да. проблема в том что Customer один и тот же создается.
cust создается либо с полями ФИО, без ID, если его в таблице нет. Если в таблице есть Customer, то передается экземпляр сущности этого Customer.

Answer 1

[wkololo_4ever]

Ваш экземпляр класса Customer не отслеживается EF`ом. Нужен весь код. Попробуйте использовать Attach()

Comments

[Дмитрий Гавриленко]

Code First
class DbModel : DbContext
{
static DbModel()
{
Database.SetInitializer(new DropCreateDatabaseIfModelChanges());
}
public DbModel()
: base("Data Source=main.sdf") { }
public DbSet Customers { get; set; }
public DbSet Transactions { get; set; }
}

class Transaction
{
public int ID { get; set; }
public Customer Customer { get; set; }
public DateTime Date { get; set; }
public double Price { get; set; }
public int Discount { get; set; }
public double Ended { get; set; }
}
class Customer
{
public int ID { get; set; }
public string Name { get; set; }
public string Last { get; set; }
public string Other { get; set; }

public override string ToString()
{
return String.Format("{0} {1} {2}", this.Last, this.Name, this.Other);
}
}

Добавление в данных
Transaction trans = new Transaction { Customer = tempCust, Date = DateTime.Now, Discount = discount, Price = price, Ended = ended };
using (DbModel ctx = new DbModel())
{
ctx.Transactions.Add(trans);
ctx.SaveChanges();
}
Где tempCust либо new Customer() { Last = "Фамилия", Name = "Имя", Other = "Отчество" }
либо ctx.Customers.ToList().Find((DbCust) => { ищу по ФИО }

[wkololo_4ever]

Дмитрий Гавриленко: Вы два раза создаете контексты (DbModel ctx = new DbModel()). Найдя покупателя в первом контексте и открыв новый контекст, новый контекст ничего не знает о найдем покупателе. Делайте все в одном контексте.

[wkololo_4ever]

Дмитрий Гавриленко: PS "ctx.Customers.ToList().Find((DbCust) => { ищу по ФИО }" - ад. Вы получаете с базы всех покупателей (ToList()) и после этого, в памяти, ищете нужного, среди всех полученных.

[Дмитрий Гавриленко]

wkololo_4ever: вряд ли это вообще влияет. Отдельный Customer ни как не привязан с конкретным контекстом.

[wkololo_4ever]

Дмитрий Гавриленко: что? И как тогда БД должна понять, какой Customer вам нужен?

[Дмитрий Гавриленко]

wkololo_4ever: на самом деле, все Customerы хранятся в переменных типа List. Именно из него я и ищу. А когда добавляю данные, я эти переменные обновляю и больше к контексту не притрагиваюсь пока опять не возникнет нужда добавить данные и обновить их.

[Дмитрий Гавриленко]

wkololo_4ever: тот Customer, экземпляр которого я передаю.

[wkololo_4ever]

Дмитрий Гавриленко: Добавьте такую строку ctx.Customer.Attach(tempCust), когда выбираете покупателя из существующих.

[Дмитрий Гавриленко]

wkololo_4ever: тоже самое что и было описано в самом вопросе. То же самое что и на скринах. Vladimir S предложил вариант. Он хоть как то изменяет поведение самой работы с БД. В принципе работает хуже, но все же более правильнее.

[wkololo_4ever]

Дмитрий Гавриленко Vladimir S предложил вариант, который абсолютно никак не влияет на EF и БД. Проблема в вашем коде.

[Дмитрий Гавриленко]

wkololo_4ever: влияет на запись данных и (вроде) их извлечение

[wkololo_4ever]

Дмитрий Гавриленко: окей, вы, наверное, очень хорошо знаете EF :)

[Дмитрий Гавриленко]

wkololo_4ever: нет. Но если по правде...
Сохранил я данные в отдельный List из контекста, а потом извлек нужного мне Customer из этого List<> 100% эквивалентна вашему методу получения Customer через Attach(). 100%... Не важно от куда он, важно то, что он экземпляр Customer с достоверными данными. На счет моего кода - да, не спорю, ошибка в нем, но не в добавление, а в самих сущностях.

Answer 2

[Vladimir S]

добавьте в класс Customer свойство

public virtual ICollection<Transaction> Transactions { get; set; }

для обратной связи

Comments

[Дмитрий Гавриленко]

Происходит что-то невероятное.
Если записей в Customers нет (и в Transactions естественно), то все ок!
По одной записи в каждой таблице при первой записи данных.
Если запись в Customers уже есть, то все равно создается новый Customers, но транзакции выглядят так
id|CustomerID|Price
1 1 5000
2 2 5000
3 2 1500

Это при том, что я всего два раза зачислил - 5000 и 1500 соответственно
Если тому же касту начислить еще 2000 то получится что в кастомерах появится третьй, а в транзикциях уже будет

id|CustomerID|Price
1 1 5000
2 2 5000
3 2 1500
4 3 5000
5 3 1500
6 3 2000

Даже не знаю как это поведение назвать.

[Vladimir S]

вы как кастомера создаете? поидее вы его создали раз, затем используете уже существующего из базы, а не добавляете одного и того же в базу много раз.
Связь кастомера с транзакциями прослеживается, мб попробовать явно обозначить ключевые поля аттрибутом [Key]? да и схему базу желательно переделать

[Дмитрий Гавриленко]

Vladimir S: вообще смысл в том, что главную роль тут играет Transaction. Как видно, у каждой транзакции есть кастомер. Ок. Если кастомера нет, я его ищу изначально по ФИО, то создается новый - new Customer { FIO } без ID.
В этом случае, при записи новой транзакции, я передаю информацию нового кастомера в самом транзакции. Ок.
Если кастомер есть, я его ищу изначально по ФИО, то я сохраняю его экземпляр в переменную. Сохранен он уже естественно с ФИО и с ID.
Опять же при записи новой транзакции, я указываю на этого кастомера (переменной), который существует.

[Vladimir S]

у вас связь один ко многим, один кастомер может иметь много транзакций или не иметь их совсем, но транзакция не может быть без кастомера.

пробуйте еще добавить в Transaction свойство
public int Customer_ID { get; set; }

[Дмитрий Гавриленко]

Vladimir S: без изменений.

[Дмитрий Гавриленко]

Vladimir S: вот скрин https://www.dropbox.com/s/pm6hc0qfo0v5s7p/DB3.PNG?dl=0

[Дмитрий Гавриленко]

Vladimir S: это транзакция в виде 4х операциях.

[Vladimir S]

на скрине видно, что еф не связывает модели
прочтите https://msdn.microsoft.com/en-us/data/jj679962.aspx
и попробуйте переименовать свойства моделей для автосвязки

[Дмитрий Гавриленко]

Vladimir S: может скайп?

[Vladimir S]

можно и скайп

[Дмитрий Гавриленко]

Vladimir S: rover_mc Дмитрий

Answer 3

[Кирилл]

public int CustomerID { get; set; }
public virtual Customer Customer { get; set; }