Статический бизнес-уровень и методы расширения - PullRequest
Новая
       36

Статический бизнес-уровень и методы расширения

1 голос
/ 10 апреля 2011

Я только что открыл методы расширения, и я их так сильно люблю, что боюсь за их хорошее использование ...

У меня есть 3-уровневое приложение Asp Net.Мой DAL основан на Entity Framework, мой пользовательский интерфейс - aspx-страницы, и моя бизнес-логика раньше была общими классами с LINQ для запросов сущностей, подобных этому: 

public List<Site> GetAll()
{
     return db.Sites.Include(s=> s.City).ToList();
}

, которые необходимо создать и использовать следующим образом:

CSite objsite = new CSite();
List<Site> sites = objsite.GetAll();

Затем я обнаружил Iqueryable, поэтому таким образом я могу «повторно использовать» свои запросы следующим образом: 

public ObjectQuery<Site> GetAll()
{
     return db.Sites.Include(s=> s.City);
}

public IQueryable<Site> Filter(IQueryable<Site> query, string filterWord)
{
     return (from s in query
             s.Name.Contains(word)
             select s);
}

List<Site> sites = objsite.Filter(objsite.GetAll(),"filter word").ToList();

Метод Filter () просто применяет предложение where к GetAll() iqueryable, и это здорово, пока я не обнаружил методы расширения, поэтому я могу справиться с этим так: 

public static IQueryable<Site> Filter(this IQueryable<Site> query, string word)
{
      return (from s in query
             s.Name.Contains(word)
             select s);
}

И это даже лучше, потому что теперь у меня есть intellisense для моих запросов, таких как: 

List<Site> sites = objsite.GetAll().Filter("filter word").ToList();

Теперь, это та часть, где я боюсь, и из-за трех вещей:

  1. Как вы думаете, это хороший подход к n-уровневому приложению?Это хороший шаблон проектирования или просто ленивое решение?

  2. Учитывая требование, чтобы методы расширения были статическими методами в статических классах, мой бизнес-уровень был бы полностью статичным, если бы этохороший подход ?, долженЯ положил методы без расширения под тот же статический класс?(например, GetAll () или AddNew ())

  3. Быть приложением Asp Net - хорошо ли иметь все эти статичные вещи?

Спасибомного ребят!

1 Ответ

2 голосов
/ 10 апреля 2011

Очень распространено использование пользовательских методов расширения поверх IQueryable. Я думаю, что это хороший способ повторно использовать части запроса. Использование методов расширения должно быть сделано с осторожностью. Метод расширения является просто синтаксическим сахаром вокруг статического метода. У моего друга есть хорошая история о его коллеге, который недавно открыл методы расширения и начал использовать код вроде: 

var order = 1.GetOrder(); // Extension on integer - this is really bad example

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

Но все же вы должны использовать расширения на IQueryalbe. Вы, вероятно, обнаружите, что ваш бизнес-класс - это просто оболочка вокруг context.Query, и в этом случае это дополнительный слой, который не нужен.

Также имейте в виду, что все это работает в многоуровневой архитектуре, но если ваш n-уровень означает реальное физическое разделение (каждый уровень на отдельном сервере), то это другая история, потому что вам нужно что-то, что позволило бы вам создать запрос linq на одном уровне и его выполнение на другом уровне (на другом сервере, поэтому запрос и результат должны передаваться по сети) - одним из таких инструментов являются службы данных WCF.

...