Вы избавляетесь от DataContext
, но возвращаете что-то, что все еще зависит от него.
Параметры:
- Не выбрасывайте
DataContext
. Я знаю, это звучит странно, но руководство от команды LINQ to SQL (ну, Мэтт Уоррен) указало, что в большинстве дел (то есть, если вы не делаете ничего необычного) удаление не требуется
- Вызов
ToList()
внутри блока using
метода Get()
.
Обратите внимание, что использование выражения запроса только с вырожденным запросом является достаточно бессмысленным. (Если это в вашем собственном коде, даже неявное Select(s => s)
на самом деле не будет полезным.)
Я бы предложил изменить ваш метод на:
public IList<StockCategory> GetAllStockCategories()
{
using (leDataContext db = new leDataContext())
{
return db.StockCategories.ToList();
}
}