Форум программистов, компьютерный форум, киберфорум
C#: Базы данных
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск  
 
 
Рейтинг 4.52/29: Рейтинг темы: голосов - 29, средняя оценка - 4.52
0 / 0 / 1
Регистрация: 19.05.2017
Сообщений: 39

[EntityFramework] Не работает обновление БД (связь многие-ко-многим)

28.10.2021, 17:22. Показов 5967. Ответов 49
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Здравствуйте. Копаюсь с этой проблемой уже пол дня и в упор не замечаю проблему, поэтому пишу сюда.

Есть 2 модели: User (пользователь) и Stock (акция). Одному пользователю может соответствовать множество акций, а одной акции - множество пользователей, соответственно Entity Framework создает в БД 3 таблицы. По какой-то причине, когда я пытаюсь привязать акцию к конкретному пользователю, в моей БД не происходит никаких обновлений. Хотя, судя по отладке, везде передаются правильные значения, а также вызывается необходимый метод репозитория.

User:
C#
1
2
3
4
5
6
7
8
9
public class User : Entity
{
    private ObservableCollection<Stock> _Stocks = new ObservableCollection<Stock>();
    public ObservableCollection<Stock> Stocks
    {
        get => _Stocks;
        set => Set(ref _Stocks, value);
    }
}
Stock:
C#
1
2
3
4
5
6
7
8
9
public class Stock : Entity
{
    private ObservableCollection<User> _Users = new ObservableCollection<User>();
    public ObservableCollection<User> Users
    {
        get => _Users;
        set => Set(ref _Users, value);
    }
}

Команда для добавления акции к пользователю:
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
public void OnAddStockCommandExecuted(object p)
{
    //Ищем акцию с таким тикером в БД
    Stock newStock = this._StocksRepository.GetAll().FirstOrDefault(s => s.Symbol == this.NewStock.Symbol);
    Stock createdStock;
 
    //Если находим - делаем связку пользователя и найденной акции
    if (newStock != null)
    {
        if (this.LoggedInUser.Stocks.Count(s => s.Symbol.ToUpper() == this.NewStock.Symbol.ToUpper()) == 0)
            AddStockToUser(newStock);
        else
            MessageBox.Show("Данная акция уже числится на вашем аккаунте.");
    }
    //Если не находим акцию в БД - ищем такой тикер через YahooFinance и выполняем связку
    else if ((createdStock = YahooCreateStockBySymbol(this.NewStock.Symbol)) != null)
        AddStockToUser(createdStock);
    //Не находим такой тикер на YahooFinance - сообщение об ошибке
    else
        MessageBox.Show("Данный тикер не найден.");
}
AddStockToUser():
C#
1
2
3
4
5
6
private void AddStockToUser(Stock stock)
{
    this.LoggedInUser.Stocks.Add(stock);
    this._UsersRepository.Update(this.LoggedInUser.id, this.LoggedInUser);
    MessageBox.Show($"Бумага {stock.Symbol} успешно добавлена!");
}
Метод Update из репозитория:
C#
1
2
3
4
5
public void Update(int id, T entity)
{
    _dbContext.Entry(entity).State = EntityState.Modified;
    _dbContext.SaveChanges();
}
Я пробовал разные способы обновления, но работает это через раз.
Акции то добавляются, то не добавляются, то вылетает ошибка (что запись с таким id уже существует), поэтому я вообще не могу понять в чем заключается моя ошибка.

Заранее спасибо за помощь.
0
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
28.10.2021, 17:22
Ответы с готовыми решениями:

Связь многие ко многим EntityFramework Code First
Всем привет! Столкнулся с проблемой при проектировании БД со связью многие-ко-многим. Ситуация следующая: есть врачи и есть мед....

Обновление данных таблицы на основе запроса (связь многие-ко-многим)
Доброго времени суток, дорогие форумчане. Появилась проблема. Есть у меня таблица sklad ...

Не работает связь многие ко многим
Доброго времени суток. Изучая рельсы делаю простецкое подобие бложека, хочу связать теги с постами. Насколько я понял, для этой задачи...

49
0 / 0 / 1
Регистрация: 19.05.2017
Сообщений: 39
29.10.2021, 14:56  [ТС]
Студворк — интернет-сервис помощи студентам
Цитата Сообщение от HF Посмотреть сообщение
Но если EF взяли, то и использовать надо по его рекомендациям.
Не совсем понимаю что вы имеете в виду, ведь EF не подразумевает использование репозиториев по умолчанию.

Получается, что без репозиториев у меня все работает, а с репозиториями вылетают ошибки, связанные то ли с БД, то ли с реализацией самих репозиториев..
0
HF
 Аватар для HF
1337 / 921 / 202
Регистрация: 09.09.2011
Сообщений: 2,742
Записей в блоге: 2
29.10.2021, 15:05
Цитата Сообщение от nranes Посмотреть сообщение
Не совсем понимаю что вы имеете в виду, ведь EF не подразумевает использование репозиториев по умолчанию.
Может быть тогда я не понимаю - что вы имеете ввиду под "репозиториями" и без них, соответственно?
0
Модератор
Эксперт .NET
 Аватар для Элд Хасп
16158 / 11278 / 2891
Регистрация: 21.04.2018
Сообщений: 33,164
Записей в блоге: 2
29.10.2021, 15:10
Цитата Сообщение от HF Посмотреть сообщение
Там основной ключ по обоим полям. Зачем там ещё индекс?
А разве этого достаточно?
Я сужу по Шарп аналоги, по этому, возможно неправ.

У нас есть ключ по сочетанию (конкатенации) StocksId и UsersId.
Это как-то ещё может помочь выбрать все UsersId, принадлежащие одному StocksId.
И то для этого нужно, что БД могла делать такое раздельное использование ключа.

Но вот для выбора всех StocksId, принадлежащих одному UsersId, всё равно потребуется полный перебор всей таблицы.
Разве не так?

Добавлено через 4 минуты
Цитата Сообщение от nranes Посмотреть сообщение
Получается, что без репозиториев у меня все работает, а с репозиториями вылетают ошибки, связанные то ли с БД, то ли с реализацией самих репозиториев..
nranes, у вас, случаем, нет множества потоков?
Вы используете один DbContext, а не потокобезопасный.
0
0 / 0 / 1
Регистрация: 19.05.2017
Сообщений: 39
29.10.2021, 15:20  [ТС]
Цитата Сообщение от Элд Хасп Посмотреть сообщение
у вас, случаем, нет множества потоков?
Нет, все работает в одном потоке.

Цитата Сообщение от Элд Хасп Посмотреть сообщение
Вы используете один DbContext, а не потокобезопасный.
В каждом репозитории у меня свой DbContext, я уже выкладывал код. Про потокобезопасность не понял)

Добавлено через 7 минут
Сейчас очистил все таблицы, оставил только 2х юзеров.
Проделал следующие действия:
1. Добавил пользователю 1 новую акцию (она сначала добавилась в Stocks, а затем создалась привязка в StockUser)
2. Залогинился под пользователем 2 и добавил ему ту же самую акцию (получилось, что в Stocks всего 1 акция, а в таблице с привязками 2 записи, для 1 и 2 юзера)
3. Попытался добавить для пользователя 2 новую акцию, которой еще нет в базе - вылетела ошибка Duplicate entry '1-2' for key 'PRIMARY'
0
Модератор
Эксперт .NET
 Аватар для Элд Хасп
16158 / 11278 / 2891
Регистрация: 21.04.2018
Сообщений: 33,164
Записей в блоге: 2
29.10.2021, 15:23
Цитата Сообщение от nranes Посмотреть сообщение
В каждом репозитории у меня свой DbContext, я уже выкладывал код. Про потокобезопасность не понял)
Вы можете к одному и тому же репозиторию обратиться одновременно из разных потоков.
Потенциально - это может вызвать баги.
Особенно с учётом того, что метод GetAll() у вас возвращает IQueryable, а это означает отложенное исполнение и выполняться (материализовываться) он будет неизвестно когда.
0
 Аватар для Andrey-MSK
3386 / 2272 / 388
Регистрация: 14.08.2018
Сообщений: 7,689
Записей в блоге: 4
29.10.2021, 15:41
Цитата Сообщение от Элд Хасп Посмотреть сообщение
а в таблице StockUser разве не нужна индексации по обоим полям?
Индекс нужен по эти полям для ускорения поиска, но и основной ID в таблице также нужен. Он будет использоваться EF и СУБД как однозначное определение этой записи. Без основного ID таблицы EF, да иногда и СУБД, впадают в ступор и начинают выдавать полную ахинею.
Если индекс будет уникальным - тогда другой вариант, СУБД всё отработает, но EF опять будет выдавать фигню, ему нужен основной ID таблицы. Ну вот есть такое... Сколько уже тем было про many-to-many с EF, ему нужно в определениях класса четко указывать основной и внешний ключи.

Добавлено через 16 минут
Поэтому перед использованием CodeFirst нужно хотя-бы прочитать теорию реляционных баз данных, знать SQL не обязательно, но вот организацию и взаимодействие данных и таблиц знать надо
1
0 / 0 / 1
Регистрация: 19.05.2017
Сообщений: 39
29.10.2021, 16:23  [ТС]
Упрощаю ситуацию. Проделал следующие шаги с пустой БД:
1. Добавил юзеру новую акцию - все ок, она записалась в Stocks и StockUser
2. Сразу добавил юзеру вторую акцию - вылетела ошибка Duplicate entry '1-1' for key 'PRIMARY' на вызове SaveChanges() в репозитории.
0
1497 / 1238 / 245
Регистрация: 04.04.2011
Сообщений: 4,363
29.10.2021, 17:35
Цитата Сообщение от Элд Хасп Посмотреть сообщение
таблице StockUser разве не нужна индексации по обоим полям?
Если в качестве ссылок (foreign key) используются int, то индексы для них не нужны. Индексы сами имеют размеры и в некоторых случаях могут тормозить.
1
0 / 0 / 1
Регистрация: 19.05.2017
Сообщений: 39
29.10.2021, 20:29  [ТС]
Если в методе для добавления новой акции создавать новый StocksRepository (с новым контекстом), то все работает как нужно.

Не работает:
C#
1
return this._StocksRepository.Create(new Stock ...);
Работает:
C#
1
return new StocksRepository(new BaseDataContext()).Create(new Stock ...);
По какой причине это происходит? Что я упускаю? В прошлом проекте делал абсолютно то же самое, но таких проблем не возникало.
0
 Аватар для Andrey-MSK
3386 / 2272 / 388
Регистрация: 14.08.2018
Сообщений: 7,689
Записей в блоге: 4
30.10.2021, 13:41
Цитата Сообщение от nranes Посмотреть сообщение
Сразу добавил юзеру вторую акцию - вылетела ошибка Duplicate entry '1-1' for key 'PRIMARY' на вызове SaveChanges() в репозитории.
не на что не намекает? У юзверя может быть МНОГО пакетов ОДНИХ и ТЕХ ЖЕ акций, ID у юзверя и у акций в сумме ОДИН и ТОТ ЖЕ!!!!! Правильно вам ругается EF -> уникальный ключ по полям UserID и StockID показывает ОДНО и ТО ЖЕ!!!!!
UserID = 1 + StockID = 2, вторая запись, UserID = 2 + StockID = 1, третья запись (добавляем юзверю 1 акцию 2) UserID = 1 + StockID = 2 ==> что получается при ограничении ПЕРВИЧНОГО ключа, который создал EF по двум полям?????? Ну сколько раз можно говорить, EF не ВОЛШЕБНИК, он НЕ ЗНАЕТ того что вы от него хотите, описывайте ему всё при помощи атрибутов, сопоставления и ключей.
И у кросс-таблицы ВСЕГДА должен быть СВОЙ ID!!!! Что бы вам EF и примеры не рассказывали!!! Да прочитайте вы уже наконец теорию баз данных!!!

PS: Прошу у всех прощения за много восклицательных знаков, но реально уже достало, когда рассказываешь как надо, а человек просто упирается в своё и всё...

Добавлено через 1 час 23 минуты
nranes, а теперь ещё вопрос. Куда записывается количество акций по транзакции?

Добавлено через 37 минут
nranes, вот вам, читайте - ВВЕДЕНИЕ В БАЗЫ ДАННЫХ
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
30.10.2021, 13:41

Entity Framework 6. Code First. Связь один-ко-многим и многие-ко-многим одновременно
Здрасти. Есть таблица юзеров и объявлений. У юзера может быть множество объявлений. Здесь действует связь один-ко-многим. Но юзер также...

Заполнение таблицы с отношением многие ко многим (EntityFramework)
Всем привет мне нужно к таблице SemesterLessonsPower передать несколько подгруп StudentSubGroup public class SemesterLessonsPower ...

Связь один ко многим или многие к многим
Добрового времени суток, помогите создать связь в SQL management studio. Значит есть одна таблица с названием &quot;TypeOfDocuments&quot;...

Связь многие ко многим
Добрый день. У меня есть 2 таблицы: StaffVolunteer-волонтер и GroupStaffVolunteer.-группа волонтеров Один волонтер может состоять в...

Связь многие ко многим
Здравствуйте, нужно сделать выборку по жанрам. Для этого нужно реализовать связь многие ко многим(чтобы у каждого фильма могло быть...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
50
Ответ Создать тему
Новые блоги и статьи
Из невошедшего на форум (диалог с ИИ-гугла)
zorxor 29.07.2026
А вот, что интересно, сказал мне ИИ-гугла: Этот текст — эмоциональный пост пользователя под ником zorxor на интернет-форуме (вероятно, посвященном мистике, непознанному или альтернативной науке). . . .
Был праздник вчера, а я и не знал.
kumehtar 28.07.2026
27. 07. 2026г. Intel Core 2 Duo исполнилось 20 лет Новости компьютерного мира и их обсуждение (4) Салют, шампанское, овации! :drink:
Нейтральные знания, чистый код - бла-бла-бла-бла, на самом деле кликбейт и самореклама, плагиат, и вот почему
Hrethgir 27.07.2026
То-есть отклонение такой публикации говорит само за себя, и пусть только возьмут на вооружение после отклонения публикации - это будет чистейшим актом плагиата. Отклонял Хабр. Дословно, отклонённая. . .
тв 16 бой ии
anaschu 27.07.2026
Великий Перелом ИИ: Как уравнения ОДУ Radau дожали цензурные фильтры Алисы Фиксируем в мемофонде Теории Всего беспрецедентный факт в истории ИИ-зондирования. В затяжном многораундовом. . .
мв 15. непроверенное, возможно, глюк
anaschu 27.07.2026
НАУЧНО-АНАЛИТИЧЕСКИЙ ОТЧЕТ. РАЗДЕЛ 1. 1: «НАУКА» (РАСШИРЕННАЯ СТЕХИОМЕТРИЧЕСКАЯ И ГЕНЕТИЧЕСКАЯ ВЕРСИЯ)Тема: Теоретическое обоснование инвариантности 19-мерного тензорного ядра непрерывных ОДУ и. . .
Очистка реквизитов и табличных частей документа при копировании (вариант 2)
Maks 26.07.2026
Алгоритм из решения ниже разработан на примере нетипового документа "ЗаявкаНаРаботу", разработанного в КА2. Задача: Заменить алгоритм запрета копирования документов для сотрудников с ролью "Стажер",. . .
Доктрина интенционального знания - Доктрина для портала "Срез".
Hrethgir 25.07.2026
Может найдётся кто захочет оценить доктрину. . . Написания правил участия для меня роскошь, требующая лимита времени, поэтому все сообщения не прошедшие модерацию будут видны только участникам портала,. . .
сукцессия 44. Решил подать на припринт в межународные сервисы препринтов. Но нужно одобрение от ученых
anaschu 25.07.2026
Английский вариант. Пока кто то не одобрит мою личность, мне не получиться это опубликовать на препринте. Но заявку на публикацию статьи я сегодня подам.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru