Question

我试图选择一种好的

我有几页，每页都有一种形式。我不得不为每个页面（大约20个不同的页面）创建一个视图，但是代码几乎是相似的。我看不到任何减少重复代码的方法，因为不同的视图都必须处理不同的ModelForm实例。

示例可能仅用于一个视图：

def portfolio_dividend_new(request, profile_id):
    profile = Profile.objects.get(id=profile_id)
    post_req = False

    if request.POST:
        form = DividendForm(request.POST)
        form.save()
        post_req = True

    form = DividendForm()
    return render(request, 'plan/portfolio/new_dividend.html',
                  {'form': form,
                   'profile': profile,
                   'post_req': post_req}
                  )

另一个视图的示例：

def portfolio_buyback_new(request, profile_id):
    profile = Profile.objects.get(id=profile_id)
    post_req = False

    if request.POST:
        form = SharebuybackForm(request.POST)
        form.save()
        post_req = True

    form = SharebuybackForm()
    return render(request, 'plan/portfolio/new_buyback.html',
                  {'form': form,
                   'profile': profile,
                   'post_req': post_req}
                  )

如您所见，这些视图具有许多相同的代码，但是由于它们必须实例化不同的ModelForm实例，因此我看不到如何防止它们成为两个视图。如果它只有两个视图，但是变成20个视图，就不再可维护了。

在每个视图中必须使用不同的模型形式时，如何避免违反DRY原理？我认为CBV并不是更好的解决方案，因为代码本身并不需要很长而且很简单，但是问题是很多代码会重复。

有什么建议吗？

Answer 1

制作差异参数，并使用functools.partial部分应用它们。

from functools import partial

def portfolio_page(model, the_url, request, profile_id):
    profile = Profile.objects.get(id=profile_id)
    post_req = False

    if request.POST:
        form = model(request.POST)
        form.save()
        post_req = True

    form = model()
    return render(request, the_url,
                  {'form': form,
                   'profile': profile,
                   'post_req': post_req}
                  )

portfolio_dividend_new = partial(
    portfolio_page, DividendForm, 'plan/portfolio/new_dividend.html')

portfolio_buyback_new = partial(
    portfolio_page, SharebuybackForm, 'plan/portfolio/new_buyback.html')

或者类似的东西。您仅举了两个例子。如果它们都相同，那么您甚至可以拉出更多相似之处，例如'plan/portfolio/{}.html'.format(the_file)。或者，如果它们在更多地方存在差异，则可能需要使用更多参数。

如果您经常使用它，您甚至可以将其部分分割，例如

page = partial(partial, portfolio_page)

portfolio_dividend_new = page(DividendForm, 'new_dividend')
portfolio_buyback_new = page(SharebuybackForm, 'new_buyback')
foopage = page(FooModel, 'foo')
barpage = page(BarModel, 'bar')
# etc.

我不知道您可以在该语句中将“模型”用作变量。

类只是Python中对象的另一种类型（与函数一样），因此您可以将它们分配给变量，并将它们作为参数传递给其他函数，等等。

当每个视图需要处理不同的模型时，如何遵循Django中的DRY原理？

1 个答案: