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ミックスインからビューに継承することを忘れないでください。

@ sebastian-code、すべてのビューをcbvとして書き直しますか?

必ずしも@tamirOKCBVが主に冗長性を減らし、_DRY_のケースを回避することを目的としているわけではないので、場合によっては機能的なビューを持つ方が理にかなっていると思います。不要な作業を回避するためにロジックを適用する必要があります。 私があなたに任せるよりも、あなたはプロジェクトに何かを与えたいと思いますか。

どんな助けでも大歓迎です。

ただし、テストを追加することを忘れないでください。目標の1つは、カバレッジ率を向上させることです。

この問題を閉じるのは、広範で具体的ではないため、役に立たないためです。 そして、それはすでに#66に含まれている目標なので、そこで追いつくことができます。

このページは役に立ちましたか?
0 / 5 - 0 評価

関連する問題

mwanjajoel picture mwanjajoel  ·  5コメント

Allan-Nava picture Allan-Nava  ·  9コメント

Shekharnunia picture Shekharnunia  ·  10コメント

sebastian-code picture sebastian-code  ·  11コメント

ssahilsahil798 picture ssahilsahil798  ·  5コメント