使用多个路由的flask视图函数是否存在不良做法？

Question

因此，基本上我有多个基于if语句的render_template返回，它们返回我的jinja2模板响应的不同变量。我相信我可以将这些路径分解成它们自己的函数（同样，我也可以将我的模板分解成多个模板（例如编辑.html模板，而不是模板中的{% if editform %}），但我喜欢为任何给定页面提供单一视图函数和模板的想法。在

在我花更多的时间创建视图的其余函数之前，我想确保我正在做的事情不会在以后影响我。在

下面的代码，谢谢！在

@app.route('/admin/users/', defaults={'id': None}, methods = ['GET'])
@app.route('/admin/users/<id>', methods = ['GET'])
@app.route('/admin/users/edit/<id>', methods = ['GET', 'POST'])
@login_required
def users(id):
    if not current_user.role == ROLE_ADMIN:
        flash('You do not have access to view this page.')
        return redirect(url_for('index'))

    if id:
        user = User.query.filter_by(id = id).first()

        if 'edit' in request.path:
            editform = UserForm()

            if editform.validate_on_submit():
                user.username = editform.username.data
                user.email = editform.email.data
                user.role = editform.role.data

                db.session.add(user)
                db.session.commit()

                return redirect('/admin/users/' + str(user.id))

            editform.username.data = user.username
            editform.email.data = user.email
            editform.role.data = user.role

            return render_template("users.html",
                title = "Edit User",
                user = user,
                editform = editform)

        return render_template("users.html",
            title = "User",
            user = user)

    users = User.query.all()

    return render_template("users.html",
        title = 'Users',
        users = users)

Answer 1

没有硬性规定，但我建议按照Separation of concerns原则，拆分成多个函数更模块化。在

我甚至会更进一步，使用MethodView，其中每个路由都有一个类，其中get、post、post等从base重写。在

当你的函数很小的时候它看起来还可以，但是当它增长的时候，一些模块化是有帮助的！在