Djangoの_setsを最適化してクエリを減らす

Question

以下の結果は4 dbヒットになります。 3行目& 4は、2行目でフィルタリングしたものだけをフィルタリングしているので、変更する必要がありますので、dbに再度ヒットしません。

page = get_object_or_404(Page, url__iexact = page_url) 
installed_modules = page.module_set.all() 
navigation_links = installed_modules.filter(module_type=ModuleTypeCode.MODAL) 
module_map = dict([(m.module_static_object.key, m) for m in installed_modules]) 

Answer 1

ジャンゴquerysets are lazy、次の行には、データベースにヒットしないように：

あなたはこの行のクエリセットを反復処理するまで、クエリが実行されません

installed_modules = page.module_set.all() 

：だから

module_map = dict([(m.module_static_object.key, m) for m in installed_modules]) 

あなたが投稿したコードは私には3つのデータベースクエリのようにしか見えません4.

データベースからすべてのモジュールを取得しているので、

navigation_links = [m for m in installed_modules if m.module_type == ModuleTypeCode.MODAL] 

このパフォーマンスが向上したかどうかを確認するには、ベンチマーキングを行う必要があります。それは私にとっては時期尚早の最適化である可能性があります。

module_static_object.keyをフェッチするモジュールごとに1つのデータベースクエリを実行している可能性があります。この場合、select_relatedを使用できます。

Answer 2

これは早すぎる最適化の場合である。 4ページの負荷に対するDBクエリは悪くありません。できるだけ少ない数のクエリを使用するというアイデアがありますが、すべてのシナリオで1つに減らすことは決してありません。あなたがそこに持っているコードは、不必要にクエリを作成するという意味で壁のようには見えないので、それを作ることができるように既に最適化されている可能性は非常に高いです。