Bootcamp: 实施 CBV

创建于 2016-05-30  ·  11评论  ·  资料来源: vitorfs/bootcamp

考虑到所有视图都已经顺利运行,这将需要大量的工作来重构此级别的代码。 但这应该会减少实际代码的数量,并且会在项目开发中实现更多实际和理想的实践。

enhancement help wanted

所有11条评论

有人在做这方面的工作吗?

@anweshknayak现在不是。 你有兴趣从事这方面的工作吗?

@sebastian-code 是的,我是。 让我设置这个项目。 我会问你任何问题

好的@anweshknayak ,我会尽力提供帮助。

@sebastian-code 我应该为每个应用程序制作不同的 PR 吗?

@anweshknayak我认为这是最好的方法。

@sebastian-code 你能给出一些关于使用什么样的视图的提示吗? ( https://ccbv.co.uk/ ) 我想我们可以使用 DetailView 或 ListView 吗?
从这里开始(https://github.com/vitorfs/bootcamp/blob/master/bootcamp/activities/views.py#L10)

@anweshknayak这将是一个对象列表(通知),因此正确的方法是从 ListView 继承,另外,不要忘记从本机 LogginRequired mixin 继承到视图中。

@sebastian-code,您是否希望将每个视图都重写为 cbv?

好吧,不一定@tamirOK CBV 主要旨在减少冗长并避免 _DRY_ 的情况,所以我想比在某些情况下拥有功能视图更有意义,这需要应用逻辑以避免不必要的工作。 我会说比我留给你的是你想为这个项目做些什么。

任何帮助表示赞赏。

但是不要忘记添加测试,目标之一是提高覆盖率。

关闭此问题是因为范围广泛且不具体,这使其无济于事。 并且因为#66 中已经包含了一个目标,所以我们可以在那里跟上它。

此页面是否有帮助?
0 / 5 - 0 等级

相关问题

sebastian-code picture sebastian-code  ·  11评论

yashLadha picture yashLadha  ·  21评论

Shekharnunia picture Shekharnunia  ·  6评论

Allan-Nava picture Allan-Nava  ·  9评论

norouzzadeh picture norouzzadeh  ·  3评论