使用可变数量的关键字优化django中搜索过滤器的函数代码 - 代码太多，我是初学者

问题描述

您好，伟大的社区，

我正在学习 django/python 开发，我正在培训自己开发用于资产清单的网络应用程序。

我制作了一个搜索过滤器，以提供（例如）属于特定用户、特定部门或特定品牌、型号或类别（计算机、办公桌、ecc. .) 有很多字段，主要是外部表，主表是“Cespiti”，在意大利语中表示资产

现在（经过很多次）我已经完成了多个关键字搜索（例如）有人在搜索框中输入部门和类别并获得相关结果（例如特定部门的所有办公桌或所有计算机特定部门的特定模型）。

我已经做了一个“if”检查表单，将关键字拆分为单个单词，对其进行计数并按顺序对前一个键的结果应用渐进式过滤。

但我对我的代码不满意，我认为它太“硬编码”了，而不是为每个关键字（从 1 到 3）创建一个 IF 条件，我希望编写一些不那么依赖的东西在关键字的数量，但是免费的。

这是视图的代码，我希望有人能给我正确的方向。

def SearchResults(request):
    query = request.GET.get('q')
    chiave =query.split()
    lunghezza = int((len(chiave)))   
    
    if lunghezza == 1:
        object_list = Cespiti.objects.filter(
        Q(proprietario__cognome__icontains=chiave[0]) |
        Q(proprietario__nome__icontains=chiave[0])  |
        Q(categoria__nome__icontains=chiave[0]) |
        Q(marca__nome__icontains=chiave[0]) |
        Q(modello__nome__icontains=chiave[0]) |
        Q(reparto__nome__icontains=chiave[0]) |
        Q(matricola__icontains=chiave[0])     
        ).distinct
    
    
    elif lunghezza == 2:
        object_list = Cespiti.objects.filter(
        Q(proprietario__cognome__icontains=chiave[0]) |
        Q(proprietario__nome__icontains=chiave[0])  |
        Q(categoria__nome__icontains=chiave[0]) |
        Q(marca__nome__icontains=chiave[0]) |
        Q(modello__nome__icontains=chiave[0]) |
        Q(reparto__nome__icontains=chiave[0]) |
        Q(matricola__icontains=chiave[0])     
        ).filter(Q(proprietario__cognome__icontains=chiave[1]) |
        Q(proprietario__nome__icontains=chiave[1])  |
        Q(categoria__nome__icontains=chiave[1]) |
        Q(marca__nome__icontains=chiave[1]) |
        Q(modello__nome__icontains=chiave[1]) |
        Q(reparto__nome__icontains=chiave[1]) |
        Q(matricola__icontains=chiave[1])     
        ).distinct
    
    elif lunghezza == 3:
        object_list = Cespiti.objects.filter(
        Q(proprietario__cognome__icontains=chiave[0]) |
        Q(proprietario__nome__icontains=chiave[0])  |
        Q(categoria__nome__icontains=chiave[0]) |
        Q(marca__nome__icontains=chiave[0]) |
        Q(modello__nome__icontains=chiave[0]) |
        Q(reparto__nome__icontains=chiave[0]) |
        Q(matricola__icontains=chiave[0])     
        ).filter(Q(proprietario__cognome__icontains=chiave[1]) |
        Q(proprietario__nome__icontains=chiave[1])  |
        Q(categoria__nome__icontains=chiave[1]) |
        Q(marca__nome__icontains=chiave[1]) |
        Q(modello__nome__icontains=chiave[1]) |
        Q(reparto__nome__icontains=chiave[1]) |
        Q(matricola__icontains=chiave[1])     
        ).filter(Q(proprietario__cognome__icontains=chiave[2]) |
        Q(proprietario__nome__icontains=chiave[2])  |
        Q(categoria__nome__icontains=chiave[2]) |
        Q(marca__nome__icontains=chiave[2]) |
        Q(modello__nome__icontains=chiave[2]) |
        Q(reparto__nome__icontains=chiave[2]) |
        Q(matricola__icontains=chiave[2])).distinct
    
    
    context = {
        'object_list': object_list,'query' : query,}

   
    return render(request,'search_results.html',context=context) 

解决方法

一种方法是将构建 Q 对象的步骤与视图方法分开。这样就可以循环执行：

def generate_search_query_params(word):
    return (
        Q(proprietario__cognome__icontains=word) |
        Q(proprietario__nome__icontains=word)  |
        Q(categoria__nome__icontains=word) |
        Q(marca__nome__icontains=word) |
        Q(modello__nome__icontains=word) |
        Q(reparto__nome__icontains=word) |
        Q(matricola__icontains=word)
    )


def SearchResults(request):
    query = request.GET.get('q')
    queryset = Cespiti.objects.all()

    for word in query.split():
        queryset = queryset.filter(
            generate_search_query_params(word)
        )

    object_list = queryset.distinct()
    context = {
        'object_list': object_list,'query' : query,}

    return render(request,'search_results.html',context=context)

,

谢谢，非常感谢您的建议，

我想在一个循环中插入，现在关键字的数量是无限的，而且不是硬编码（非常感谢），我已经考虑过 Abdul 的想法和 Damon 的解决方案，我希望避免最初的“.object.all( ) 所以我是这样安排的：第一个“级别”是固定的，这样我就可以避免 .all() 和过滤的所有子级别都循环，你怎么看？

def SearchResults(request):
query = request.GET.get('q')
chiave =query.split()
lunghezza = int((len(chiave)))

object_list = Cespiti.objects.filter(
    Q(proprietario__cognome__icontains=chiave[0]) |
    Q(proprietario__nome__icontains=chiave[0])  |
    Q(categoria__nome__icontains=chiave[0]) |
    Q(marca__nome__icontains=chiave[0]) |
    Q(modello__nome__icontains=chiave[0]) |
    Q(reparto__nome__icontains=chiave[0]) |
    Q(matricola__icontains=chiave[0])     
    )

for I in range(1,lunghezza):
    
    print(I)
    object_list = object_list.filter(
    Q(proprietario__cognome__icontains=chiave[I]) |
    Q(proprietario__nome__icontains=chiave[I])  |
    Q(categoria__nome__icontains=chiave[I]) |
    Q(marca__nome__icontains=chiave[I]) |
    Q(modello__nome__icontains=chiave[I]) |
    Q(reparto__nome__icontains=chiave[I]) |
    Q(matricola__icontains=chiave[I])     
    )

context = {
    'object_list': object_list,}


return render(request,context=context)