unsafeskyANAпропущено...
Тогда объясните зачем выделенный ниже фрагмент кода:
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
if (!clients.TryGetValue(client.ID, out existingClient)) // client don't exist
{
clients.Add(client.ID, client);
company.Clients.Add(client);
}
else
{
clients[client.ID].Companies.Add(company);
company.Clients.Add(existingClient);
}
Добавление компании к уже существующему клиентуЯ же Вам чётко выделил фрагмент:
clients[client.ID] , - этот код ничего никуда не добавляет, он достаёт из словаря значение по ключу.
Зачем, если Вы уже достали его методом
TryGetValue ?
unsafeskyANAИ кстати Вы дублируете
company.Clients.Add в ветках if-а.
Да, но в случае когда клиента не существует добавляется клиент который был считан из БД, когда же клиент существует, он добавляется в словарь к уже существующей компании.Я прекрасно понял, что должен делать Ваш код, намекаю на то, что его можно упростить :)
unsafeskyANAИ ещё у Вас сплошные баги. Вы сначала найдите и клиента и компанию, или создайте новые, если их нет в словарях, а потом уже связывайте друг с другом.
В чём именно заключаются баги? Разница лишь в том что в моей реализации сначала считываются все данные, а потом уже проверяется существует ли такой клиент/компания. Скорее это не баг, а не рациональное использовании ресурсов. Ваша реализация гораздо боле практичная. Спасибо.Вы бы для начала внимательно посмотрели на свой код, а потом уже утверждали, что "Разница лишь в том.." :)
Рассмотрим случай, когда и клиент и компания уже существуют в словарях. НЕ существующими я буду называть объекты, что созданы через new, а не найдены в словаре.
Что делает Ваш код?
1.
1.
2.
clients[client.ID].Companies.Add(company);
company.Clients.Add(existingClient);
Существующему клиенту, добавляет ссылку на НЕ существующую компанию, а ей зачем-то добавляет ссылку на существующего клиента.
2.
1.
2.
client.Companies.Add(existingCompany);
companies[company.ID].Clients.Add(client);
НЕ существующему клиенту добавляется ссылка на существующую компанию, а потом ей же добавляется ссылка на НЕ существующего клиента.
В результате задание не выполнено.
