Почему не работает связь много-ко-многим?

Question

[TeVeTed]

Сделал 2 модели таблиц Colleges и Specialities, между которыми должна быть связь много-ко-многим (как бы и техникум обучает по нескольким специальностям, и какой-то специальности можно обучиться в нескольких техникумах)
Вот модели:

public class College
    {

        public int CollegeId { get; set; }

        public string Name { get; set; }

        public string Locality { get; set; }

        public int LocalityTypeId { get; set; }
        public virtual LocalityType LocalityType { get; set; }

        public int? UniversityId { get; set; }
        public virtual University University { get; set; }

        public int AreaId { get; set; }
        public virtual Area Area { get; set; }

        public string Address { get; set; }

        public string Phone { get; set; }

        public string Site { get; set; }

        public string Email { get; set; }

        public int? StatusId { get; set; }
        public virtual Status Status { get; set; }

        public int? AccrLevel { get; set; }

        public Director DirectorId { get; set; }

        public bool? Full { get; set; }

        public bool? Demo { get; set; }

        public List<Speciality> Specialities { get; set; }
    }

public class Speciality
    {

        public string DirectionCode { get; set; }

        public virtual Direction Direction { get; set; }

        [Key]
        public string SpecialityCode { get; set; }

        public string Name { get; set; }

        public List <College> Colleges { get; set; }
    }

И промежуточная модель, реализующая связь много-ко-многим:

public class CollegeSpeciality
    {

        public int CollegeSpecialityId { get; set; }

        public int CollegeId { get; set; }
        public virtual College College { get; set; }

        public string SpecialityCode { get; set; }
        public virtual Speciality Speciality { get; set; }

        public List<Speciality> Specialities { get; set; }

        public List<College> Colleges { get; set; }
    }

Я уверен, что напортачил в промежуточной модели, но может накосячил и в связующих
Помогите, кто знает)

Answer 1

[Sanostee]

В Вашем случае явно создавать таблицу связей не нужно, она будет создана автоматически

public class College
    {
        public int? AccrLevel { get; set; }

        public string Address { get; set; }

        public virtual Area Area { get; set; }

        public int AreaId { get; set; }

        public bool? Demo { get; set; }

        public virtual Director Director { get; set; }

        public int DirectorId { get; set; }

        public string Email { get; set; }

        public bool? Full { get; set; }

        [Key]
        public int CollegeId { get; set; }

        public string Locality { get; set; }

        public virtual LocalityType LocalityType { get; set; }

        public int LocalityTypeId { get; set; }

        public string Name { get; set; }

        public string Phone { get; set; }

        public string Site { get; set; }

        public virtual ICollection<Speciality> Specialities { get; set; }

        public virtual Status Status { get; set; }

        public int? StatusId { get; set; }

        public virtual University University { get; set; }

        public int? UniversityId { get; set; }
    }

    public class Speciality
    {
        public virtual ICollection<College> Colleges { get; set; }

        [ForeignKey("DirectionCode")]
        public virtual Direction Direction { get; set; }

        public string DirectionCode { get; set; }

        public string Name { get; set; }

        [Key]
        public string SpecialityCode { get; set; }
    }

Подробнее: www.entityframeworktutorial.net/code-first/configu...

Comments

[TeVeTed]

Спасибо!) Теперь возник другой вопрос - помогите, если в курсе)
Вы, я видел ответили на другой мой вопрос Как работать со связью много-ко-многим в контроллере?, поэтому наверняка должны знать - я там использовал следующий код ".Where(p => region == null || p.Area.Region.Name == region)", как мне написать подобное и для моей коллекции, чтобы вместо p.Area.Region.Name вызывать элемент ICollection?

[Sanostee]

Какое свойство ICollection? Specialities?

[Sanostee]

Какой запрос нужно выполнить? Опишите словами

[TeVeTed]

Sanostee: нужно отбирать те техникумы, если в их коллекции встречается элемент, равный локальной переменной specialities

[TeVeTed]

Sanostee: да, свойство у ICollection - Specialities

[Sanostee]

Не очень понял) Помимо свойства Specialities у сущности College у Вас есть еще локальная коллекция specialities? Тогда, что-то вроде этого:

.Where(c => c.Specialities.Any(s => specialities.Count(a => a.SpecialityCode == s.SpecialityCode) >0))

[Sanostee]

Данный запрос выберет только те техникумы, которые содержат хотя бы одну специальность из локальной коллекции specialities. Сравнение выполняется по свойству SpecialityCode класса Speciality.

[Sanostee]

Можно заменить
specialities.Count(a => a.SpecialityCode == s.SpecialityCode) >0
на
specialities.Any(a => a.SpecialityCode == s.SpecialityCode)

[TeVeTed]

Sanostee: не знал, как правильно выразится) "public ViewResult List(string region, string area, string localityType, string speciality, int page = 1)" Вот - сравнивать элемент коллекции нужно с speciality)

[Sanostee]

Тогда

.Where(c => c.Specialities.Any(s => s.SpecialityCode == speciality))

[Sanostee]

Если в конструкторе College инициализация коллекции Specialities не выполняется, тогда стоит проверить, что ссылка на коллекцию не равна null:

.Where(c => c.Specialities != null && c.Specialities.Any(s => s.SpecialityCode == speciality))

[Sanostee]

Вместо SpecialityCode используйте, что требуется (Name?), в зависимости от того, что ожидается в переменной speciality.

[TeVeTed]

Sanostee: Классно, спасибо)

[TeVeTed]

Sanostee: Но теперь появляется при указании speciality ошибка "Значение не может быть неопределенным"

[Sanostee]

В запросе проверяете c.Specialities != null

[Sanostee]

Полное сообщение об ошибке напишите

[TeVeTed]

Sanostee: Значение не может быть неопределенным.
Имя параметра: source

Описание: Необработанное исключение при выполнении текущего веб-запроса. Изучите трассировку стека для получения дополнительных сведений о данной ошибке и о вызвавшем ее фрагменте кода.

Сведения об исключении: System.ArgumentNullException: Значение не может быть неопределенным.
Имя параметра: source

Ошибка источника:

Строка 29: Colleges = repository.Colleges
Строка 30: .Where(p => region == null || p.Area.Region.Name == region)
Строка 31: .Where(p => speciality == null || p.Specialities.Any(s => s.Name == speciality))
Строка 32: .Where(p => localityType == null || p.LocalityType.Name == localityType)
Строка 33: .Where(p => area == null || p.Area.Name == area)

[Sanostee]

Попробуйте переписать так:
.Where(p => speciality == null || (p.Specialities != null && p.Specialities.Any(s => s.Name == speciality)))

[TeVeTed]

Sanostee: не помогло(

[Sanostee]

Сложно заочно диагностировать причину. Посмотрите внимательно, запустите проект в режиме отладки...

[Sanostee]

В крайнем случае если есть желание и TeamViewer, могу зайти вместе посмотрим.

[TeVeTed]

Sanostee: TeamViewer к сожалению, нет, но могу дать ссылку на проект https://www.dropbox.com/s/c6nzh25v7i01v8t/UkraineC...

[Sanostee]

Сейчас гляну

[Sanostee]

Нет у меня Sql Server local к сожалению, не удалось запустить.

Попробуйте исправить эту строку, добавив проверку college.Specialities !=null

PagingInfo = new PagingInfo
{
.....
.Where(college => college.Specialities !=null && college.Specialities.Any(s => s.Name == speciality))
.....
},

[Sanostee]

+ у Вас ошибки в атрибутах модели

[Sanostee]

У сущности College должно быть так:

public virtual Director Director { get; set; }

public Director DirectorId { get; set; }

[Sanostee]

неее, вот так)

public virtual Director Director { get; set; }

public int DirectorId { get; set; }

[Sanostee]

У Direction вот так:

public string BranchCode { get; set; }

[ForeignKey("BranchCode")]
public virtual Branch Branch { get; set; }

[Sanostee]

У Director вместо:

[ForeignKey("CollegeId")]
public int DirectorId { get; set; }

public College CollegeId { get; set; }

должно быть так:

public int CollegeId { get; set; }

[ForeignKey("CollegeId")]
public virtual College College { get; set; }

[TeVeTed]

Sanostee: при отладке показывает, что в Specialities значение null, но должна быть коллекция значений

[Sanostee]

У Вас ошибки в модели данных, так что не удивительно. Сейчас отредактирую и скину все файлы с классами модели.

[TeVeTed]

Sanostee: У Director и College отношение должно быть один-к-одному, я поэтому и делал так...

[Sanostee]

Да, Вы правы, я выше слегка напутал... Поправил классы модели, надеюсь ничего не пропустил. https://onedrive.live.com/redir?resid=DB3E6982525C...

[Sanostee]

На мой взгляд, для начала знакомства с EF и Linq, Вы выбрали слегка сложноватую модель данных.

[TeVeTed]

Sanostee: Это дипломный проект для техникума)
Куратор дал тему, а когда я полез в дебри и осознал что знаний у меня маловато здесь, изменить тему было уже нельзя
Как-то так)

[TeVeTed]

Sanostee: Да, и теперь ругается на неопределенность связи между College и Director

Исключение типа "System.InvalidOperationException" возникло в EntityFramework.dll, но не было обработано в коде пользователя

Дополнительные сведения: Unable to determine the principal end of an association between the types 'UkraineColleges.Domain.Entities.Director' and 'UkraineColleges.Domain.Entities.College'. The principal end of this association must be explicitly configured using either the relationship fluent API or data annotations.

[TeVeTed]

Sanostee: вернул обратно отношение College и Director, но теперь что-то вообще непонятное стало происходить
Invalid object name 'dbo.SpecialityColleges'.

Описание: Необработанное исключение при выполнении текущего веб-запроса. Изучите трассировку стека для получения дополнительных сведений о данной ошибке и о вызвавшем ее фрагменте кода.

Сведения об исключении: System.Data.SqlClient.SqlException: Invalid object name 'dbo.SpecialityColleges'.

Ошибка источника:

Строка 42: repository.Colleges.Count() :
Строка 43: repository.Colleges.Where(college => college.Area.Region.Name == region)
Строка 44: .Where(college => college.Specialities != null && college.Specialities.Any(s => s.Name == speciality))
Строка 45: .Where(college => college.Area.Name == area)
Строка 46: .Where(college => college.LocalityType.Name == localityType).Count()

[Sanostee]

В моем варианте удалите свойства College и CollegeId у класса Director

[TeVeTed]

Sanostee: теперь проблема в другом Invalid object name 'dbo.SpecialityColleges'. Не могу только понять, где устанавливать имя связующей таблицы, чтобы указать имя той, которая уже есть в базе, могу переименовать ту, что в базе, но тогда придется делать миграцию, а у меня с миграцией уже был печальный опыт...