#.net #entity-framework #.net-4.0 #entity-framework-4 #linq-to-entities
#.net #entity-framework #.net-4.0 #entity-framework-4 #привязка к сущностям
Вопрос:
Привет,
У меня есть вопрос linq к sql, подобный этому :
tmpAdList1 = (from p in context.Ads
join h in context.AdCategories on p.CategoryId equals h.Id
join l in context.Location on p.UserLocationId equals l.Id
where
(adList.S == null || adList.S.Length < 1 || p.Title.Contains(adList.S) || p.Description.Contains(adList.S)) amp;amp;
(categorylevelOrder.Length < 1 || h.LevelOrder.StartsWith(categorylevelOrder)) amp;amp;
((locationIdList != null amp;amp; lList.Contains(l.Id)) ||
(locationLevelOrder.Length < 1 || l.LevelOrder.StartsWith(locationLevelOrder))) amp;amp;
((adTypeO1 == AdType.Unknown amp;amp; adTypeO2 == AdType.Unknown amp;amp; adTypeO3 == AdType.Unknown amp;amp; adTypeO4 == AdType.Unknown amp;amp; adTypeO5 == AdType.Unknown) ||
(p.TypeOfAd == (int)adTypeO1 || p.TypeOfAd == (int)adTypeO2 || p.TypeOfAd == (int)adTypeO3 || p.TypeOfAd == (int)adTypeO4 || p.TypeOfAd == (int)adTypeO5)) amp;amp; //Check for default filters
((AdListShowType)adList.ALS.ST == AdListShowType.Both || adList.ALS.ST == p.OwnerType) amp;amp;
(p.PublishedDate.HasValue amp;amp; p.PublishedDate.Value.CompareTo(fetchAdsTo) >= 1) amp;amp;
((adOwnerType1.HasValue amp;amp; adOwnerType2.HasValue) || p.OwnerType == (int)adOwnerType1.Value) amp;amp;
p.InactivatedDate == null
orderby p.CreatedDate descending
select p).ToList();
Смотрите Edit1 для всего метода
После выполнения этого вопроса будет выполнена еще некоторая фильтрация (в контексте), но чтобы сделать это как можно быстрее, я стремлюсь извлекать как можно меньше записей с SQL server в первом вопросе.
Проблема в том, что мне нужно сравнить locationIdList, который является int[], с сущностью. Генерируемое исключение является :
Не удается сравнить элементы типа ‘System.Int32[]’. Поддерживаются только примитивные типы (такие как Int32, String и Guid) и типы сущностей.
Я загуглил проблему, и это известная проблема, однако я нашел примеры, подобные этому :
var list = new List<int> { 1, 2, 3, 5 };
var result = from s in DB.Something
where list.Contains(s.Id)
select s;
Но это вызывает то же исключение? Я также читал, что storedprocedure может решить проблему, но я не нашел, как это работает?
Есть предложения?
Наилучшие пожелания
Правка 1 : весь метод :
public List<Ad> GetAds(AdList adList, DateTime fetchAdsTo, out int totalAds)
{
AdType adTypeO1 = AdType.Unknown;
AdType adTypeO2 = AdType.Unknown;
AdType adTypeO3 = AdType.Unknown;
AdType adTypeO4 = AdType.Unknown;
AdType adTypeO5 = AdType.Unknown;
int? adOwnerType1 = null;
int? adOwnerType2 = null;
FilterModel filterModel = new FilterModel();
List<AdCategoryFilter> adCategoryFilterList;
AdsFilterValues adsFilterValues;
List<AdsFilterValueWrapper> seartchFilterValueList;
AdsFilterValueWrapper seartchFilterValue = null;
List<Ad> tmpAdList1;
List<Ad> tmpAdList2 = new List<Ad>();
int locationId = -1;
int[] locationIdList = null;
string locationLevelOrder = string.Empty;
int categoryId = -1;
string categorylevelOrder = string.Empty;
AdCategoryFilter adCategoryFilter;
AdListCompare adListCompare;
Boolean firstDropDownMatch = false;
Boolean secondDropDownMatch = false;
totalAds = 0;
int machedFilterCount;
categoryId = AdHandler.Instance.ExtractCategoryId(adList.CS);
//If there is multiple choises
//This is the last level, that means that we can check against the ID dircly
if (adList.LS.L3.Count > 0)
locationIdList = adList.LS.L3.ToArray();
else
locationId = AdHandler.Instance.ExtractLocationId(adList.LS);
switch ((AdOwnerType)adList.ALS.ST)
{
case AdOwnerType.Both:
adOwnerType1 = (int)AdOwnerType.Private;
adOwnerType2 = (int)AdOwnerType.Company;
break;
case AdOwnerType.Company:
adOwnerType1 = (int)AdOwnerType.Company;
break;
case AdOwnerType.Private:
adOwnerType1 = (int)AdOwnerType.Private;
break;
}
#region GetFilters
adCategoryFilterList = filterModel.GetCategoryFilterByCategory(categoryId);
seartchFilterValueList = FilterHandler.Instance.ConvertAdFilterToModel(adList.F, adCategoryFilterList, FilterType.Display);
#endregion
#region Set Default filters (Buy, Let, Sell, Swap, WishRent)
foreach (AdsFilterValueWrapper filterWrapper in seartchFilterValueList)
{
if ((adCategoryFilter = adCategoryFilterList.Where(c => c.Id == filterWrapper.FilterId).FirstOrDefault()) != null)
{
switch ((PublicAdFilterKey)adCategoryFilter.PublicAdFilterKey)
{
case PublicAdFilterKey.Buy:
{
if (filterWrapper.AdsFilterValues1.ValueNumber > 0)
adTypeO1 = AdType.Buy;
break;
}
case PublicAdFilterKey.Let:
{
if (filterWrapper.AdsFilterValues1.ValueNumber > 0)
adTypeO2 = AdType.Let;
break;
}
case PublicAdFilterKey.Sell:
{
if (filterWrapper.AdsFilterValues1.ValueNumber > 0)
adTypeO3 = AdType.Sell;
break;
}
case PublicAdFilterKey.Swap:
{
if (filterWrapper.AdsFilterValues1.ValueNumber > 0)
adTypeO4 = AdType.Swap;
break;
}
case PublicAdFilterKey.WishRent:
{
if (filterWrapper.AdsFilterValues1.ValueNumber > 0)
adTypeO5 = AdType.WishRent;
break;
}
}
}
}
#region Remove default filters fom filterList
adCategoryFilterList = adCategoryFilterList.Where(c => ((PublicAdFilterKey)c.PublicAdFilterKey) != PublicAdFilterKey.Buy amp;amp;
((PublicAdFilterKey)c.PublicAdFilterKey) != PublicAdFilterKey.Let amp;amp;
((PublicAdFilterKey)c.PublicAdFilterKey) != PublicAdFilterKey.Sell amp;amp;
((PublicAdFilterKey)c.PublicAdFilterKey) != PublicAdFilterKey.Swap amp;amp;
((PublicAdFilterKey)c.PublicAdFilterKey) != PublicAdFilterKey.WishRent).ToList();
#endregion
#endregion
var lList = adList.LS.L3.ToList<int>(); //new List<int> { 1, 2, 3, 5 };
using (BissEntities context = new BissEntities())
{
if (categoryId > 0)
categorylevelOrder = context.AdCategories.Where(c => c.Id.Equals(categoryId)).FirstOrDefault().LevelOrder.Trim();
if (locationId > 0)
locationLevelOrder = context.Location.Where(c => c.Id.Equals(locationId)).FirstOrDefault().LevelOrder.Trim();
tmpAdList1 = (from p in context.Ads
join h in context.AdCategories on p.CategoryId equals h.Id
join l in context.Location on p.UserLocationId equals l.Id
where
(adList.S == null || adList.S.Length < 1 || p.Title.Contains(adList.S) || p.Description.Contains(adList.S)) amp;amp;
(categorylevelOrder.Length < 1 || h.LevelOrder.StartsWith(categorylevelOrder)) amp;amp;
((locationIdList != null amp;amp; lList.Contains(l.Id)) ||
(locationLevelOrder.Length < 1 || l.LevelOrder.StartsWith(locationLevelOrder))) amp;amp;
((adTypeO1 == AdType.Unknown amp;amp; adTypeO2 == AdType.Unknown amp;amp; adTypeO3 == AdType.Unknown amp;amp; adTypeO4 == AdType.Unknown amp;amp; adTypeO5 == AdType.Unknown) ||
(p.TypeOfAd == (int)adTypeO1 || p.TypeOfAd == (int)adTypeO2 || p.TypeOfAd == (int)adTypeO3 || p.TypeOfAd == (int)adTypeO4 || p.TypeOfAd == (int)adTypeO5)) amp;amp; //Check for default filters
((AdListShowType)adList.ALS.ST == AdListShowType.Both || adList.ALS.ST == p.OwnerType) amp;amp;
(p.PublishedDate.HasValue amp;amp; p.PublishedDate.Value.CompareTo(fetchAdsTo) >= 1) amp;amp;
((adOwnerType1.HasValue amp;amp; adOwnerType2.HasValue) || p.OwnerType == (int)adOwnerType1.Value) amp;amp;
p.InactivatedDate == null
orderby p.CreatedDate descending
select p).ToList();
#region Filter collection
foreach (Ad ad in tmpAdList1)
{
machedFilterCount = 0;
adListCompare = AdListCompare.NotCompered;
if (adCategoryFilterList.Count > 0)
{
//Loop the filters that belongs to the choosen category
foreach (AdCategoryFilter existingFilter in adCategoryFilterList)
{
//Se if the ad has the proper filter If not return it
if ((adsFilterValues = ad.AdsFilterValues.Where(c => c.CategoryFilterId == existingFilter.Id).FirstOrDefault()) != null || existingFilter.PublicAdFilterKey > 0)
{
//If the filter is not a regular value filter but a filter pointed to a property on the ad
//Then extract the correct value and use it
if (existingFilter.PublicAdFilterKey > 0)
{
adsFilterValues = new AdsFilterValues();
adsFilterValues.CategoryFilterId = existingFilter.Id;
switch ((PublicAdFilterKey)existingFilter.PublicAdFilterKey)
{
case PublicAdFilterKey.Price:
{
adsFilterValues.ValueNumber = ad.Price;
break;
}
}
}
if ((seartchFilterValue = seartchFilterValueList.Where(c => c.AdsFilterValues1.CategoryFilterId == adsFilterValues.CategoryFilterId).FirstOrDefault()) != null)
{
firstDropDownMatch = false;
secondDropDownMatch = false;
adListCompare = AdListCompare.Compared;
switch ((FilterControlType)existingFilter.DisplayFilterControlType)
{
case FilterControlType.TwoDropDown:
//Check so the first dropdown value compare
//If the index is the first then any value will do
if (seartchFilterValue.FilterIndexPosition1 == FilterIndexPosition.First)
firstDropDownMatch = true;
else
{
if (adsFilterValues.ValueNumber.Value >= seartchFilterValue.AdsFilterValues1.ValueNumber.Value)
firstDropDownMatch = true;
}
if (firstDropDownMatch)
{
//Check so the second dropdown value compare
//If the index is the last then any value will do
if (seartchFilterValue.FilterIndexPosition2 == FilterIndexPosition.Last)
secondDropDownMatch = true;
else
{
if (adsFilterValues.ValueNumber.Value <= seartchFilterValue.AdsFilterValues2.ValueNumber.Value)
secondDropDownMatch = true;
}
if (secondDropDownMatch)
adListCompare = AdListCompare.Approved;
}
break;
case FilterControlType.DropDown:
//Check so the first dropdown value compare
//If the index is the first then any value will do
if (seartchFilterValue.FilterIndexPosition1 == FilterIndexPosition.First)
{
if (adsFilterValues.ValueNumber.Value <= seartchFilterValue.AdsFilterValues1.ValueNumber.Value)
firstDropDownMatch = true;
}
if (seartchFilterValue.FilterIndexPosition1 == FilterIndexPosition.Last)
{
if (adsFilterValues.ValueNumber.Value >= seartchFilterValue.AdsFilterValues1.ValueNumber.Value)
firstDropDownMatch = true;
}
else
{
if (adsFilterValues.ValueNumber.Value == seartchFilterValue.AdsFilterValues1.ValueNumber.Value)
firstDropDownMatch = true;
}
if (firstDropDownMatch)
adListCompare = AdListCompare.Approved;
break;
case FilterControlType.TextBox:
if (adsFilterValues.ValueString.Equals(seartchFilterValue.AdsFilterValues1.ValueString))
adListCompare = AdListCompare.Approved;
break;
case FilterControlType.CheckBox:
if (adsFilterValues.ValueNumber != null amp;amp; adsFilterValues.ValueNumber.Value == seartchFilterValue.AdsFilterValues1.ValueNumber.Value)
adListCompare = AdListCompare.Approved;
break;
default:
adListCompare = AdListCompare.NotCompered;
break;
}
//If no value is set, then break;
if (adListCompare != AdListCompare.Approved)
break;
machedFilterCount ;
}
}
else
{
//If the ad is missing the filter then return it anyway, it might as well be correct
adListCompare = AdListCompare.Approved;
machedFilterCount = adCategoryFilterList.Count();
}
}
}
else
{
adListCompare = AdListCompare.Approved;
machedFilterCount = adCategoryFilterList.Count();
}
if (adListCompare == AdListCompare.Approved amp;amp; machedFilterCount == adCategoryFilterList.Count())
tmpAdList2.Add(ad);
}
#endregion
if (adList.ALS.OB == (int)AdListOrderBy.Price)
tmpAdList2 = tmpAdList2.OrderBy(c => c.Price).ToList();
totalAds = tmpAdList2.Count();
return tmpAdList2.Skip((adList.ALS.P - 1) * adList.ALS.CP).Take(adList.ALS.CP).ToList();
}
}
Правка 2: обновить
Основной метод GetAd :
public List<Ad> GetAds(AdList adList, DateTime fetchAdsTo, out int totalAds)
{
LocationModel locationModel = new LocationModel();
FilterModel filterModel = new FilterModel();
List<AdCategoryFilter> adCategoryFilterList;
List<AdsFilterValueWrapper> seartchFilterValueList;
int categoryId = -1;
List<Ad> outputList;
totalAds = 0;
#region Fetch the first ads by location
outputList = GetAdsByLocations(locationModel.GetLocationOrderList(adList.GetLocationIds()), fetchAdsTo, false);
if(outputList.Count < 1)
return outputList;
#endregion
#region GetFilters
categoryId = AdHandler.Instance.ExtractCategoryId(adList.CS);
adCategoryFilterList = filterModel.GetCategoryFilterByCategory(categoryId);
seartchFilterValueList = FilterHandler.Instance.ConvertAdFilterToModel(adList.F, adCategoryFilterList, FilterType.Display);
#endregion
#region Filter Default filters (Buy, Let, Sell, Swap, WishRent)
FilterDefaultCustomFilters(outputList, adCategoryFilterList, seartchFilterValueList);
if (outputList.Count == 0)
return outputList;
else
{
#region Remove default filters fom filterList
adCategoryFilterList = adCategoryFilterList.Where(c => ((PublicAdFilterKey)c.PublicAdFilterKey) != PublicAdFilterKey.Buy amp;amp;
((PublicAdFilterKey)c.PublicAdFilterKey) != PublicAdFilterKey.Let amp;amp;
((PublicAdFilterKey)c.PublicAdFilterKey) != PublicAdFilterKey.Sell amp;amp;
((PublicAdFilterKey)c.PublicAdFilterKey) != PublicAdFilterKey.Swap amp;amp;
((PublicAdFilterKey)c.PublicAdFilterKey) != PublicAdFilterKey.WishRent).ToList();
#endregion
}
#endregion
#region Filter Custom filters
this.FilterCustomFilters(outputList, adCategoryFilterList, seartchFilterValueList);
#endregion
#region Order
switch ((AdListOrderBy)adList.ALS.OB)
{
case AdListOrderBy.Price:
outputList = outputList.OrderBy(c => c.Price).ToList(); break;
case AdListOrderBy.Latest:
outputList = outputList.OrderByDescending(c => c.PublishedDate).ToList(); break;
}
#endregion
#region Total Ad Count
totalAds = outputList.Count();
#endregion
#region Paging
outputList = outputList.Skip((adList.ALS.P - 1) * adList.ALS.CP).Take(adList.ALS.CP).ToList();
#endregion
return outputList;
}
GetAdByLocation
public List<Ad> GetAdsByLocations(string[] locationLevelOrderList, DateTime? fetchAdsTo, Boolean inactive) //, List<Ad> adList = null)
{
List<Ad> output;
using (BissEntities context = new BissEntities())
{
if (fetchAdsTo.HasValue)
{
if (locationLevelOrderList.Count() == 0)
{
output = (from a in context.Ads
join l in context.Location on a.UserLocationId equals l.Id
where a.InactivatedDate.HasValue == inactive amp;amp;
(a.PublishedDate.HasValue amp;amp; a.PublishedDate.Value.CompareTo(fetchAdsTo.Value) >= 1)
select a).ToList();
}
else
{
output = (from a in context.Ads
join l in context.Location on a.UserLocationId equals l.Id
where a.InactivatedDate.HasValue == inactive amp;amp;
(a.PublishedDate.HasValue amp;amp; a.PublishedDate.Value.CompareTo(fetchAdsTo.Value) >= 1) amp;amp;
(locationLevelOrderList.Where(c => l.LevelOrder.StartsWith(c)).FirstOrDefault() != null)
select a).ToList();
}
}
else
{
if (locationLevelOrderList.Count() == 0)
{
output = (from a in context.Ads
join l in context.Location on a.UserLocationId equals l.Id
where a.InactivatedDate.HasValue == inactive
select a).ToList();
}
else
{
output = (from a in context.Ads
join l in context.Location on a.UserLocationId equals l.Id
where a.InactivatedDate.HasValue == inactive amp;amp;
(locationLevelOrderList.Count() == 0 || locationLevelOrderList.Where(c => l.LevelOrder.StartsWith(c)).FirstOrDefault() != null)
select a).ToList();
}
}
}
return output;
}
Примечание : Методы в main GetAd, которые начинаются с Filter в name, будут работать только с коллекцией (никаких действий с базой данных)
Комментарии:
1. Это излишне сложный запрос. Вы должны выполнить свои проверки null для других коллекций в другом месте за пределами запроса. Я бы начал с упрощения этого в первую очередь. Это может означать разбиение вашего запроса на части и объединение их в более крупный запрос. Вероятно, это автоматически исправит проблему.
2. @Jeff M > Спасибо, теперь я добавил весь метод целиком, и вы можете понять, почему он такой сложный. Однако я вас не понимаю, не могли бы вы объяснить более подробно?
3. Так много кода. Вы должны локализовать свою проблему в простом фрагменте кода, иначе большинство людей просто пропустят ваш вопрос.
4. @Ladislav Mrnka > извините, я теперь добавил linq к sql-коду отдельно в сообщении.
Ответ №1:
В вашем запросе у вас есть сравнение:
... locationIdList != null ...
Поскольку locationIdList имеет тип int[] , запрос не может быть переведен, поскольку он поддерживает только простые сравнения (как указано в ошибке).
Вы должны выполнять эти проверки вне запроса, а не внутри них. Однако, поскольку они инициализируются внутри метода, вы должны просто убедиться, что они инициализированы, и опустить проверку за ненадобностью.
Я бы настоятельно рекомендовал провести рефакторинг всего метода и запроса. Это ужасно долго и за этим трудно следить. Переместите блоки кода в отдельные методы, выполняющие небольшую часть того, что вам нужно, затем соедините их все вместе. Это упростит обслуживание вашего кода и исправление ошибок, подобных этой, намного проще.
Комментарии:
1. M > Спасибо, я попробую провести рефакторинг метода. Сначала я подумал создать метод, который получает только все объявления из базы данных, которая принадлежит правильному местоположению (GetAdsByLocation) , затем я остановлю все остальные проверки в методах и сделаю это только для коллекции, которую я получаю от первого метода (GetAdsByLocation). Это будет означать, что в коллекции может быть много рекламы, но я не вижу, как это сделать другим способом? Просьба дать совет
2. @Jim: Я бы подумал, что это хорошее начало. Кроме того, вам, вероятно, следует переместить все блоки кода, занимающие более 15-20 строк, в их собственный метод. Возможно, вы захотите рассмотреть реструктуризацию и этого последнего блока. Просто перенести его в отдельный метод будет недостаточно.
3. M > Спасибо, взгляните на Edit2, это новое обновление того же метода. Это выглядит лучше? Есть ли что-нибудь еще, что я должен реорганизовать?
4. @Jim: Я бы сказал, что это большое улучшение по сравнению с тем, что у вас было изначально. Хотя сначала я должен спросить, вы все еще получаете ожидаемые результаты? Вы все еще получаете эту ошибку? Нет смысла в рефакторинге, если вы нарушаете код. При условии, что это правильно, было бы проще предложить улучшения в коде.
5. М> Да, это работает отлично, но мне нужно еще немного тестирования, чтобы быть абсолютно уверенным. «Содержит» для простых массивов определенно работает в .NET 4.0.
Ответ №2:
Последний пример работает в Entity framework 4. Если вы получаете исключение, ваше приложение, скорее всего, построено как .NET 3.5 с первой версией Entity framework, которая не поддерживалась Contains .
Комментарии:
1. О! Итак, EF4 наконец-то поддерживает Contains? Не знал этого! Действительно отличные новости! Каждый день узнавайте что-то новое. 1
2. @SnowJim: Напишите свой запрос на SQL, чтобы мы могли понять, что вы пытаетесь сделать.
3. > Я не пробовал это в более простом linq к сущности, и это работает. Итак, теперь я переделаю сложный метод на более мелкие части. Но, как я вижу, первому методу придется извлекать много объявлений (GetAdsByLocation), после чего коллекция будет отфильтрована всеми другими методами. Это правильный путь? Для большей части фильтрации не требуется обращаться к базе данных, но есть некоторые, которые должны проверять базу данных, чтобы иметь возможность фильтровать.
4. @SnowJim: В linq-to-entities все выполняется в базе данных, нет этой части, которая будет выполняться здесь, а эта часть будет выполняться там . Если вы хотите запустить какую-либо фильтрацию в приложении, вы должны переключиться на linq-to-objects (вызов
AsEnumerableилиToList), но это передаст все нефильтрованные данные в ваше приложение, и вы не сможете переключиться обратно на linq-to-entities.