2 回答
TA贡献1877条经验 获得超6个赞
一种可能的解决方法。
将待办事项列表(字典列表)作为参数传递,因此它会发生变化:
TodoList=[{'ID':5,'TodoItem':'walk','isDone':False}, {'ID':6,'TodoItem':'talk','isDone':True}]
def RemoveItem(ID, TodoList):
if not any(todoItem['ID'] == ID for todoItem in TodoList):
print('item does not exist');
else:
for item in TodoList:
if item['ID']== ID: TodoList.remove(item)
RemoveItem(5, TodoList)
print(TodoList)
#=> [{'ID': 6, 'TodoItem': 'talk', 'isDone': True}]
要使您的解决方案起作用,您需要返回:
def RemoveItem(ID, TodoList):
if not any(todoItem['ID'] == ID for todoItem in TodoList):
return 'item does not exist';
else:
return list(filter(lambda todoItem:todoItem['ID']!= ID, TodoList))
print(RemoveItem(5, TodoList))
#=> [{'ID': 6, 'TodoItem': 'talk', 'isDone': True}]
这是因为该行list(filter(lambda todoItem:todoItem['ID']!= ID, TodoList))不会改变列表,而是根据条件提取元素。
但是如果没有要删除的项目,最好返回列表本身,而不是返回字符串。
TA贡献1804条经验 获得超7个赞
一个可能的解决方案是找到带有 id 的项目的索引并使用pop:
todo_list = [{'ID': 5, 'TodoItem': 'walk', 'isDone': False}]
def remove_item(ID):
if not any(todoItem['ID'] == ID for todoItem in todo_list):
return 'item does not exist'
# get index of id
idx = next(i for i, e in enumerate(todo_list) if e['ID'] == ID)
# remove and return
return todo_list.pop(idx)
print(remove_item(4))
print(remove_item(5))
print(todo_list)
输出
item does not exist
{'isDone': False, 'ID': 5, 'TodoItem': 'walk'}
[]
您第一次尝试的问题是,当您这样做时:TodoList=RemovedList在函数内部,它会在函数内部创建一个新的局部变量,即使存在同名的全局变量。在您的第二次尝试中,您需要设置为全局而不是非本地:
todo_list = [{'ID': 5, 'TodoItem': 'walk', 'isDone': False}]
def RemoveItem(ID):
global todo_list
if not any(todoItem['ID'] == ID for todoItem in todo_list):
return 'item does not exist';
todo_list = list(filter(lambda todoItem: todoItem['ID'] != ID, todo_list))
RemoveItem(5)
print(todo_list)
输出
[]
添加回答
举报