Как сравнить объект в цикле foreach?

Question

В настоящее время я делаю проверку товара в корзине. Элемент может быть выкуплен несколько раз, но ограничен определенным числом. Каждый элемент имеет атрибут redeem_count, который хранит количество времени, которое может быть выкуплено. Итак, ниже приведена моя кодировка для проверки общего количества выкупленных предметов.

var previousCoupon = ""; 
var currentCoupon = ""; 
int count = 0; 
foreach (var p in cart.Promotions) 
{ 
    var query = db.wmp_mst_mcp_promo 
       .Where(a => a.wmp_mcp_promo_id == p.PromotionId) 
       .Where(a => a.wmp_redeem_count != null && a.wmp_redeem_count > 0) 
       .SingleOrDefault(); 

    if (query != null) 
    { 
     currentCoupon = query.wmp_mcp_promo_id; 

     if (previousCoupon == currentCoupon) 
      count++; 
     else 
      count = 0; 

     previousCoupon = currentCoupon; 

     if (count > query.wmp_redeem_count) 
      result.Invalidate(string.Format("You are not allowed to redeem more than {0} \"{1}\" voucher in 1 order", query.wmp_redeem_count, query.wmp_descriptions)); 

Как вы можете видеть, кодирование работает, но только в том случае, если тот же элемент выкуплен в порядке. Это может быть недействительным, так как не все клиенты будут выкупать товар в порядке. Ошибка возникает, если они выкупают, например, элемент A, затем элемент B, а затем элемент A снова. Она будет работать только если пункт А, пункт А, затем пункт B.

Отредактировано

Я хочу, чтобы хранить любой элемент, которые имеют одинаковый идентификатор и получить только один из его атрибутов redeem_count, так что я могу сделать, если заявление в теме. Значение redeem_count отличается для каждого элемента.

Пример

товара в корзине

• пункт А - 2 (redeem_count = 2) Действительно
• пункт Б - 2 (redeem_count = 1) Недопустимый
• пункт C - 3 (redeem_count = 4) Действительно

Так что отбросит result.Invalidate

Любая помощь? Благодарю.

Answer 1

Попробуйте это решение;

var previousCoupon = ""; 
     var currentCoupon = ""; 
     int count = 0; 
     foreach (var p in cart.Promotions) 
     { 
      var query = db.wmp_mst_mcp_promo 
       .Where(a => a.wmp_mcp_promo_id == p.PromotionId) 
       .Where(a => a.wmp_redeem_count != null && a.wmp_redeem_count > 0); 
if (query != null) 
{ 
     foreach(var queryItems in query) 
       { 
       currentCoupon = query.wmp_mcp_promo_id; 

       if (previousCoupon == currentCoupon) 
        count++; 
       else 
        count = 0; 

       previousCoupon = currentCoupon; 
      } 
} 
       if (count > query.wmp_redeem_count) 
        result.Invalidate(string.Format("You are not allowed to redeem more than {0} \"{1}\" voucher in 1 order", query.wmp_redeem_count, query.wmp_descriptions)); 

Answer 2

Я хотел бы сделать что-то вроде (псевдокод):

var dict = new Dictionary<string /*item_Id*/, int /*count*/>; 
// count redemptions for each id 
foreach(var item in cart) 
{ 
    if(dict.ContainsKey(item.Id)) 
     dict[item.Id]++; 
    else 
     dict.Add(item.Id, 1); 
} 

// check if any of them violate the allowed maximum 
foreach(var itemId in dict.Keys) 
{ 
    if(dict[itemId ] > GetMaxRedeemCount(itemId)) 
    { 
     result.Invalidate(...); 
     // you may want to break here... 
     // break; 
    } 
}