Дублирование значений в базе данных при создании зависимого объекта

@Esdas · Регистрация: 03.02.2021

Студворк — интернет-сервис помощи студентам

Добрый день!
Столкнулся с проблемой дублирования записей в базе данных при сохранении объекта с вложенной структурой
Например: у меня есть 2 таблицы - менеджеры и покупатели
У каждого покупателя должен быть менеджер

Начальные данные:
Менеджеры:
Менеджер 1 Id:1
Менеджер 2 Id:2

Покупатели:
Покупатель 1 Id:1 (Менеджер 1 Id:1)
Покупатель 2 Id:2 (Менеджер 1 Id:1)
Покупатель 3 Id:3 (Менеджер 2 Id:2)

При редактировании привязки менеджера к покупателю, создается новый менеджер с теми же данными
Данные после попытки привязки другого менеджера к покупателю 2

Менеджеры:
Менеджер 1 Id:1
Менеджер 2 Id:2
Менеджер 2 Id:3

Покупатели:
Покупатель 1 Id:1 (Менеджер 1 Id:1)
Покупатель 2 Id:2 (Менеджер 2 Id:3)
Покупатель 3 Id:3 (Менеджер 2 Id:2)

Если ссылка на объект не изменяется, то новый не создается

Где может быть ошибка?

Исходный sql код:

SQL
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
CREATE TABLE [dbo].[Managers] (
    [Id]   INT            IDENTITY (1, 1) NOT NULL,
    [Name] NVARCHAR (MAX) NULL,
    CONSTRAINT [PK_dbo.Managers] PRIMARY KEY CLUSTERED ([Id] ASC)
);
 
 
 
 
 
CREATE TABLE [dbo].[Customers] (
    [Id]         INT            IDENTITY (1, 1) NOT NULL,
    [Name]       NVARCHAR (MAX) NULL,
    [STATUS]     INT            NOT NULL,
    [Manager_Id] INT            NULL,
    CONSTRAINT [PK_dbo.Customers] PRIMARY KEY CLUSTERED ([Id] ASC),
    CONSTRAINT [FK_dbo.Customers_dbo.Managers_Manager_Id] FOREIGN KEY ([Manager_Id]) REFERENCES [dbo].[Managers] ([Id])
);
 
 
GO
CREATE NONCLUSTERED INDEX [IX_Manager_Id]
    ON [dbo].[Customers]([Manager_Id] ASC);

В C# выполняется только метод SaveChanges()

@kolorotur · 03.08.2022, 14:51

Сообщение от Esdas

Где может быть ошибка?

Вот здесь:

Сообщение от Esdas

создается новый менеджер с теми же данными

Зачем два одинаковых объекта?

@Esdas · 03.08.2022, 14:56 **[ТС]**

Так в этом и проблема
Я привязываю к покупателю другого менеджера по ссылке, но в итоге создается менеджер с теми же данными но с другим Id

@kolorotur · 03.08.2022, 15:17

Сообщение от Esdas

Я привязываю к покупателю другого менеджера по ссылке

Так не привязывайте другого менеджера. Откуда берется этот другой менеджер, зачем второй менеджер с такими же данными, если есть оригинальный, кошерный менеджер? Его и используйте.

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

@Esdas · 03.08.2022, 15:39 **[ТС]**

Разобрался
Тому виной оказался legacy код, который я сразу не (исправил/заметил)
Был реализован менеджер БД через Singleton
Но внутри его функций создавались экземпляры этого класса (Не спрашивайте меня зачем. Эти функции были перенесены с другого класса)
Ко всему прочему исправилась еще одна проблема: не работал SelectedItem в ComboBox-е

Спасибо за помощь)

@nicolas2008 · 06.08.2022, 19:52

Esdas, бегите оттуда...

Новые блоги и статьи Все статьи Все блоги /
Программный контроль заполнения реквизита табличной части документа Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать контроль заполнения реквизита табличной части. . .	wmic не является внутренней или внешней командой Maks 02.04.2026 Решение: DISM / Online / Add-Capability / CapabilityName:WMIC~~~~ Отсюда: https:/ / winitpro. ru/ index. php/ 2025/ 02/ 14/ komanda-wmic-ne-naydena/	Программная установка даты и запрет ее изменения Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: при создании документов установить период списания автоматически. . .	Вывод данных в справочнике через динамический список Maks 01.04.2026 Реализация из решения ниже выполнена на примере нетипового справочника "Спецтехника" разработанного в конфигурации КА2. Задача: вывести данные из ТЧ нетипового документа. . .
Функция заполнения текстового поля в реквизите формы документа Maks 01.04.2026 Алгоритм из решения ниже реализован на нетиповом документе "ВыдачаОборудованияНаСпецтехнику" разработанного в конфигурации КА2, в дополнении к предыдущему решению. На форме документа создается. . .	К слову об оптимизации kumehtar 01.04.2026 Вспоминаю начало 2000-х, университет, когда я писал на Delphi. Тогда среди программистов на форумах активно обсуждали аккуратную работу с памятью: нужно было следить за переменными, вовремя. . .	Идея фильтра интернета (сервер = слой+фильтр). Hrethgir 31.03.2026 Суть идеи заключается в том, чтобы запустить свой сервер, о чём я если честно мечтал давно и давно приобрёл книгу как это сделать. Но не было причин его запускать. Очумелые учёные напечатали на. . .	Модель здравосоХранения 6. ESG-повестка и устойчивое развитие; углублённый анализ кадрового бренда anaschu 31.03.2026 В прикрепленном документе раздумья о том, как можно поменять модель в будущем

@Esdas 0 / 0 / 0 Регистрация: 03.02.2021 Сообщений: 4
	03.08.2022, 14:56 [ТС]
	Так в этом и проблема Я привязываю к покупателю другого менеджера по ссылке, но в итоге создается менеджер с теми же данными но с другим Id 0

@Esdas 0 / 0 / 0 Регистрация: 03.02.2021 Сообщений: 4
	03.08.2022, 15:39 [ТС]
	Разобрался Тому виной оказался legacy код, который я сразу не (исправил/заметил) Был реализован менеджер БД через Singleton Но внутри его функций создавались экземпляры этого класса (Не спрашивайте меня зачем. Эти функции были перенесены с другого класса) Ко всему прочему исправилась еще одна проблема: не работал SelectedItem в ComboBox-е Спасибо за помощь) 0

@nicolas2008 1152 / 860 / 263 Регистрация: 30.04.2009 Сообщений: 3,603
	06.08.2022, 19:52
	Esdas, бегите оттуда... 0