Python modifizieren dic Löschung während der Iteration

Question

Methode remove_item, die ähnliche Argumente wie add_item erfordert. Es sollte Artikel entfernen, die dem Warenkorb hinzugefügt wurden und nicht benötigt werden. Diese Methode sollte die Kosten dieser Artikel von der aktuellen Summe abziehen und auch die Artikel dict entsprechend aktualisieren. Wenn die Menge der zu entfernenden Artikel die aktuelle Menge im Einkaufswagen übersteigt, nehmen Sie an, dass alle Einträge dieses Artikels entfernt werden sollen.

class ShoppingCart(object): 
#constructor 
def __init__(self): 
    self.total = 0 
    self.items = {} 

#method to add items in the shoping cart 
def add_item(self,item_name, quantity, price): 
    self.total += (price * quantity) 
    self.items[item_name] = quantity 

#method to remove items 
def remove_item(self,item_name, quantity, price): 
    keys = self.items.keys() 
    for keys,v in self.items.items(): 
    if keys == item_name and quantity > self.items[keys]: #if item_name equals iterated item 
     self.total -= (price * quantity) 
     del(self.items[keys]) 
    del(self.items[keys]) 
    self.total -= (quantity * price) 

Und es gibt Testeinheiten

def test_add_item_hidden(self): 
    self.cart.add_item('Mango', 3, 10) 
    self.cart.add_item('Orange', 16, 10) 
    self.assertEqual(self.cart.total, 190, msg='Cart total not correct after adding items') 
    self.assertEqual(self.cart.items['Orange'], 16, msg='Quantity of items not correct after adding item') 

Verfahren zu überprüfen remove_item gibt einen Fehler, dass dictionary changed size during iteration auch nach dem Versuch, über keys zuzugreifen, wie oben

Answer 1

gezeigt Ich denke, die remove_item Funktion sein könnte einfacher.

#method to remove items 
def remove_item(self,item_name, quantity, price): 
    if item_name in self.item.keys(): 
     self.total -= (price * quantity) 
     del self.items[item_name] 

Mit dieser Version werden überprüft Sie, wenn item_name auf dem Wörterbuch ist, und wenn ja, können Sie es entfernen, und den Preis von total

Entfernen Der Versuch, aus einer dict, Liste, typle zu entfernen während über sie zu iterieren, ist normalerweise keine gute Idee.

Wenn item_name eine Liste von Schlüsseln ist, muss ich annehmen, dass Menge und Preis auch sind. so eine mögliche Funktion könnte sein:

def remove_item(self,item_name, quantity, price): 
    for index, value in enumerate(item_name): 
     if value in self.item.keys(): 
      self.total -= (price[index] * quantity[index]) 
      del self.items[value] 

ich nicht getestet haben, sollten aber funktionieren.

---

wenn die Idee ist, ein Inventar der Elemente zu erhalten. Die Funktion sollte in etwa so sein.

#method to remove items 
def remove_item(self,item_name, quantity, price): 
    if item_name in self.items.keys(): 
     #remove from total the price for the items 
     if quantity <= self.items[item_name]: 
      self.total -= (price * quantity) 
     else: 
      self.total -= (price * self.items[item_name]) 
     #if quantity is greater or equal of the current value, item is removed. 
     if quantity >= self.items[item_name] 
      del self.items[item_name] 
     else: 
      self.items[item_name] -= quantity