Как корректно разделить проект на слои?

Question

[Devil Devil]

Доброго времени суток!
Имеется проект в котором всего один класс модели
и такие слои :
DAL- класс модель,контекст данных, интерфейс репозитория, реализация методов репозитория (оперции CRUD)
BBl - как корректно продублировать репозиторий?примеры в интернете более сложные ,т.к. там наличие нескольких моделей и связей между ними
UI ну и здесь ,контроллеры,представления

Answer 1

[Алексей Немиро]

Берем проект, смотрим на класс модели, затем смотрим на слои. Берем лист бумаги и шариковую ручку с синими чернилами. Каллиграфическим почерком пишем:

"Уважаемый В,

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

С надеждой, _подпись_ /_расшифровка_/

PS: Люблю сгущенку."

---

Если серьезно, то ваш проект - вам виднее :-)

Можно изначально все проанализировать, взвесить, написать подробный план и потом попытаться сделать. Но для этого нужно обладать исчерпывающей информацией о том, что нужно делать. Это отдельная задача, которая потребует отдельного времени. В идеале времени должно быть не меньше, чем при разработке плана отправки первого космонавта на Марс.

Либо делать и вносить корректировки в процессе. Этот вариант используется, если не известно, что нужно сделать и как это делать, и если в процессе все может перевернуться с ног на голову, в общем сплошная импровизация. При наличии достаточного количества опыта, в плане временных затрат, этот вариант может быть экономичней.

Во всех случаях, без хорошей реализации, не имеет значения, был план или все придумалось и эволюционировало в процессе. Космический корабль все равно упадет в океан, не добравшись даже до верхних слоев атмосферы Земли, в лучшем случае безнадежно зависнет где-то там :-) Вариант с постепенным развитием будет лучшим, если смотреть на вопрос с позиции обучения и получения опыта. Стройте как вам удобно для достижения стоящих целей. Вписывать проект в неподходящий шаблон, в некоторых случаях, может быть даже хуже хаоса и неразберихи. Когда сделаете, посмотрите, в какой шаблон более ли менее вписались, сделаете выводы, насколько это хорошо или плохо для конкретной задачи и как это можно сделать лучше/проще/быстрее, как выписаться полностью (если этот вариант окажется приемлемым) и уменьшить количество коллизий. Если в процессе у вас возникают какие-то сомнения, то всегда можно задать узкий вопрос, наверняка вам ответят.

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

Comments

[Devil Devil]

ВСЯ СУТЬ В ТОМ ЧТО НЕ ЗНАЮ ИМЕННО КАК МЕТОДЫ РЕПОЗИТОРИЯ ДАЛ ПЕРЕДУБЛИРОВАТЬ В БЛЛ

[Алексей Немиро]

Alina Barkova: С чем у вас проблемы? Со ссылками, интерфейсами, наследованием? Что вы используете? Покажите упрощенный код.

[Devil Devil]

Алексей Немиро: спасибо, уже разобралась, использовала automapper. Не могли бы помочь с такой проблемой Трехслойная архитектура приложения, слой бизнес логики, как вынести сортировку из контроллера в отдельный метод в бизнесс логике?

[Алексей Немиро]

Alina Barkova: вопрос видел, но на него тоже нет ответа :-) Точнее ответ очевиден - сделайте нужные методы и передайте туда все, что требуется для поиска и сортировки. Но по идее вы уже должны были это сделать. Или вы боитесь делать методы? :-) В общем, вопрос требует уточнения, непонятно с чем именно у вас возникли проблемы. Попробуйте для начала разбить метод контроллера на небольшие части внутри контроллера, чтобы понимать, как это работает.

public ViewResult Index(string sortOrder, string currentFilter, string searchString, int? page)
{
  return AnyMethod(sortOrder, currentFilter, searchString, page);
}

private ViewResult AnyMethod(string sortOrder, string currentFilter, string searchString, int? page)
{
  ViewBag.CurrentSort = sortOrder;
  ViewBag.NameSortParm = String.IsNullOrEmpty(sortOrder) ? "name_desc" : "";
  ViewBag.DateSortParm = sortOrder == "Date" ? "date_desc" : "Date";

  if (searchString != null)
  {
    page = 1;
  }
  else
  {
    searchString = currentFilter;
  }

  ViewBag.CurrentFilter = searchString;

  var students = GetStudents();

  if (!String.IsNullOrEmpty(searchString))
  {
    students = SearchStudents(students, searchString);
  }

  // и далее можно сортировку аналогично вынести,
  // либо сгруппировать все, как нужно
  switch (sortOrder)
  {
    case "name_desc":
      students = students.OrderByDescending(s => s.LastName);
      break;

    case "Date":
      students = students.OrderBy(s => s.EnrollmentDate);
      break;

    case "date_desc":
      students = students.OrderByDescending(s => s.EnrollmentDate);
      break;

    default:  // Name ascending 
      students = students.OrderBy(s => s.LastName);
      break;
  }

  int pageSize = 3;
  int pageNumber = (page ?? 1);

  return View(students.ToPagedList(pageNumber, pageSize));
}

// NOTE: не могу быть уверен в типе (IQueryable<Students>), вам виднее, что у вас там
private IQueryable<Students> GetStudents()
{
  return from s in db.Students select s
}

private IQueryable<Students> SearchStudents(IQueryable<Students> students, string searchString)
{
  return students.Where(s => s.LastName.Contains(searchString) ||  s.FirstMidName.Contains(searchString));
}

Вы можете создавать сколько угодно методов и классов. Ограничения только в здравом смысле :-)

PS: Вместо ViewBag лучше использовать класс(ы) модели.

[Devil Devil]

Алексей Немиро: Благодарю, а если эти все методы вынести в другом классе на уровне бизнесс логиик, как их вызвать в контроллере?

[Алексей Немиро]

Alina Barkova: создается экземпляр класса и вызываются нужные методы.

public ViewResult Index(string sortOrder, string currentFilter, string searchString, int? page)
{
  // создаем экземпляр класса ИмяКласса (имя может быть любым)
  var bl = new ИмяКласса();
  // вызываем метод GetStudents у созданного экземпляра ИмяКласса
  var students = bl.GetStudents();
  // ...
}

Единственное, нужно обратить внимание на модификаторы доступа:
private - доступ только внутри класса;
public - доступ на уровне всего приложения;
internal - доступ внутри сборки;
protected - доступ внутри класса плюс для всех наследников класса.

Вам скорее всего будет достаточно использовать private и public. Можно и в public все делать, но это не очень хорошо, лучше ограничивать, чтобы не засорять "внешний мир" не нужными публичными методами/свойствами (членами).

Чтобы методы бизнес-логики были доступны в коде контроллера, эти методы должны иметь модификатор доступа public.

public class ИмяКласса
{

  public IQueryable<Students> GetStudents()
  {
    return from s in db.Students select s
  }

}

Ссылку на db, наверное будет лучше передать через конструктор (если понадобится - понадобится, если бизнес-логика не имеет прямого доступа к объектам базы (Data Access Layer)):

public class ИмяКласса
{

  // вместо DatabaseContext использовать тот тип, который у вас имеет переменная db
  private DatabaseContext Database;

  // DatabaseContext - аналогично, поменять тип на свой
  public ИмяКласса(DatabaseContext db)
  {
    this.Database = db;
  }

  public IQueryable<Students> GetStudents()
  {
    return from s in this.Database.Students select s
  }

  // любые другие методы

}

В контроллере:

public ViewResult Index(string sortOrder, string currentFilter, string searchString, int? page)
{
  var bl = new ИмяКласса(db); // передаем в класс ссылку на текущий экземпляр db
  // получаем студентов из метода GetStudents экземпляра ИмяКласса
  var students = bl.GetStudents();
  // ...
  // остальной код
}