python - 可以使这两个 django View 变干吗？

Question

我目前正在通过一些构建博客的培训来学习 Python/Django 堆栈。

我目前有两个用于添加新帖子和编辑现有帖子(post_new 和 post_edit)的类似 View ，如下所示:

def post_new(request):
    if request.method == "POST":
        form = PostForm(request.POST)
        if form.is_valid():
            post = form.save(commit=False)
            post.author = request.user
            post.published_date = timezone.now()
            post.save()
            return redirect("post_detail", pk=post.pk)      
    else:
        form = PostForm()    
    return render(request, "blog/post_edit.html", {"form": form})


def post_edit(request, pk):
    post = get_object_or_404(Post, pk=pk)
    if request.method == "POST":
        form = PostForm(request.POST, instance=post)
        if form.is_valid():
            post = form.save(commit=False)
            post.author = request.user 
            post.published_date = timezone.now()
            post.save()
            return redirect("post_detail", pk=post.pk)
    else:
        form = PostForm(instance=post)
    return render(request, "blog/post_edit.html", {"form":form})

尽管这些 View 执行不同的工作，但它们共享一些相同的代码。

尝试遵循最佳实践 (DRY)，是否有一种明智的方法可以让类似的观点成为 DRYer？还是最好将这种长度的 View 保留为长格式以便于阅读？

Answer 1

我个人会这样写:

def post_edit(request, pk=None):
    if pk is not None:
        post = get_object_or_404(Post, pk=pk)
    else:
        post = None
    if request.method == "POST":
        form = PostForm(request.POST, instance=post)
        if form.is_valid():
            post = form.save(commit=False)
            post.author = request.user 
            post.published_date = timezone.now()
            post.save()
            return redirect("post_detail", pk=post.pk)
    else:
        form = PostForm(instance=post)
    return render(request, "blog/post_edit.html", {"form":form})

基本上，您将默认的 instance 值传递给 ModelForm。