Django编辑对象时遇“MyModel对象不可迭代”错误,保存无法覆盖原对象
解决Django编辑对象时「'MyModel'对象不可迭代」及无法覆盖对象的问题
嘿,我来帮你捋清楚这个问题!你遇到的两个麻烦都是因为表单初始化的参数用错了,咱们一步步来解决:
问题根源拆解
- 「'MyModel'对象不可迭代」错误:你在POST请求里给表单传了
initial=song,但initial需要的是键值对字典,而song是Song模型的实例,Django试图把这个实例当字典迭代取初始值,这不就报错了嘛。 - 只能新建对象、无法覆盖:因为你没把现有实例关联到POST的表单上,Django默认会创建新对象,根本没意识到要更新你选中的那首歌。
修正后的完整代码
from django.shortcuts import get_object_or_404, redirect, render from django.contrib.auth.decorators import login_required from .forms import SongForm from .models import Song @login_required def edit_song(request, song_id): # 先拿到要编辑的歌曲实例,get_object_or_404会自动处理找不到的情况 song = get_object_or_404(Song, pk=song_id) if request.method == 'POST': # 核心修改:把现有实例传给instance参数,不是initial! form = SongForm(data=request.POST, instance=song) if form.is_valid(): # form.save()会直接更新关联的song实例,不用再额外调用obj.save() form.save() # 用redirect比HttpResponseRedirect更简洁,替换成你的实际跳转路径/URL名称 return redirect('/music/your-target-path/') else: # GET请求时,用instance把现有数据填充到表单里 form = SongForm(instance=song) # 记得返回渲染表单的模板,这里假设模板是edit_song.html return render(request, 'edit_song.html', {'form': form, 'song': song})
关键修改说明
- POST时用
instance=song:这是告诉Django“我要编辑这个已有的song实例”,验证通过后form.save()就会直接更新它,而不是新建对象。 - 删掉多余的
obj.save():form.save()已经完成了保存操作,重复调用完全没必要,除非你在save之后修改了obj的其他属性,那才需要再存一次。 - 简化GET请求的表单初始化:既然已经用
get_object_or_404拿到了song,直接传给instance就行,不用再查一次Song.objects.get(id=song_id),多此一举。 - (可选)用
redirect替代HttpResponseRedirect:redirect是Django推荐的写法,支持URL名称或者直接路径,代码更简洁。
额外提醒
如果你的表单涉及文件上传(比如歌曲文件、封面图),POST请求的表单要加上request.FILES参数:
form = SongForm(data=request.POST, files=request.FILES, instance=song)
同时模板里的form标签必须加上enctype="multipart/form-data"属性,不然文件传不上去哦。
内容的提问来源于stack exchange,提问作者samuelbeard
相关产品推荐
相关产品推荐

