melnyk-a/cinema

cinema's Issues

Мертвый код

В приложении не должно быть программного кода, который никогда не может быть выполнен по каким-либо причинам. Например, программный код расположенный внутри конструкции if, условие которого никогда не выполнится.

Методы, для которых данное правило не соблюдается:

• Cinema.Data.Stub.StubFilmDataService.CloseDataGateway.

Задание 5

Задание 5 готово к проверке

Избыточность идентификаторов

Нет необходимости включать в идентификатор название типа или другое название, указывающее слишком много лишней информации (например, что переменная является массивом, или какой именно клавишей управляется класс отвечающий за команду движения вправо). Другими словами вместо string nameString стоит писать string name, или вместо int[] valuesArray можно написать int[] values, также название класса SpacebarCommand можно заменить на JumpCommand.

Поля, для идентификаторов которых данное правило нарушается:

• Cinema.Domain.FilmEventArgs.addedFilm.

Свойства, для идентификаторов которых данное правило не соблюдается:

• Cinema.Domain.FilmEventArgs.AddedFilm.

Update-Migration некорректное расположение базы

Почему, когда я вызываю Update Databese, база создается в LocalDb с заполнеными данными.

Но если базу удалить и запустить код на выполнение, то она создается на сервере с незаполненными данными.

Неиспользуемое значение переменное

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

Поля, для которых данное правило не соблюдается:

• Cinema.Domain.Models.FilmCrew.actors.

Модификатор поля readonly

Если поле не изменяет свое значение нигде кроме конструктора, стоит пометить его модификатором readonly. Если в будущем потребуется изменять значение такого поля, то модификатор можно будет убрать.

Поля, для которых данное правило не соблюдается:

• Cinema.Domain.Models.JobTitles.JobTitle.human.

Задание 1

Задание 1 готово к проверке

Задание 2

Задание 2 готово к проверке

Задание 4

Задание 4 готово к проверке

Задание 3

Задание 3 готово к проверке

Ограничение not null для записи в таблице Actors

EntityFramework автоматически добавляет поле FilmCrewDto_Id и разрешает ему устанавливать значение null.

Создается оно от сюда:
internal sealed class FilmCrewDto
{
[Required]
public ICollection ActorDtos { get; set; }
//....
}