Django, как сделать простой поиск по атрибутам?

Question

[sergey19940808]

Вьюшка:

def search_foto(request):
        title = 'Поиск'

        try:
            query = request.GET['query']
            posts = OurFoto.objects.filter(name__startswith=query) | \
                OurFoto.objects.filter(text__startswith=query) or \
                OurFoto.objects.filter(date_added__startswith=query)
            context = {'title': title, 'posts': posts, 'query': query}
            return render(request, 'repository_our_fotos/search_form.html', context)

        except KeyError:
            return render(request, 'repository_our_fotos/search_form.html')

Почему при совпадении всех атрибутов выдаёт правильный результат, а когда первые 2 верны, а последний не верный, выдаёт другие результаты ? Как написать вьюшку, чтобы реализовать это ?

Answer 1

[Евгений]

У вас неправильный код. Есть же отличная родная документация по Django, куча дополнительных материалов, книг. Зачем методом проб и ошибок делать?

Правильно писать:

OurFoto.objects.filter(Q(name__startswith=query) | Q(text__startswith=query) | 
Q(date_added__startswith=query))

При этом в коде еще есть несколько проблем:

1. непонятно, зачем здесь ловить исключение KeyError, если изучить Python, то можно написать более простой и понятный код типа request.GET.get('query') или if 'query' in request.GET
   
2. лучше переходить уже на Class Based Views и Generic Views - код будет проще и не надо будет переписывать в следующих версиях Django (функции-view уже вроде как deprecated)
   

Comments

[sergey19940808]

def search_foto(request):
        title = 'Поиск'
        if 'query' in request.GET:
            query = request.GET['query']
            posts = OurFoto.objects.filter(
                Q(name__startswith=query) |
                Q(text__startswith=query) |
               Q(date_added__startswith=query)
            )
        return render(request, 'repository_our_fotos/search_form.html',
                  {'title': title, 'posts': posts, 'query': query})

почему теперь переменные query и posts недоступны для передачи в шаблон ? Точнее как их достать ?

[Евгений]

sergey19940808 потому что у вас query присваивается значение только в ветке оператора if, которая выполняется, когда условие выполнено.

Тогда надо вынести определение query = request.GET.get('query') перед циклом. А еще лучше использовать ListView из Django Generic Views.

И если вы не переопределяете title, то лучше прямо в шаблоне написать «Поиск» вместо {{ title }}.

[sergey19940808]

Еще вопрос, изменил вьюшку:

def search_foto(request):
        title = 'Поиск'

        foto = OurFoto.objects.all()
        form = FotoForm(request.GET)
        if form.is_valid():
            if form.cleaned_data['name']:
                foto = foto.filter(name__icontains=form.cleaned_data['name'])
            elif form.cleaned_data['text']:
                foto = foto.filter(text__icontains=form.cleaned_data['text'])
            elif form.cleaned_data['date_added']:   
               foto=foto.filter(date_added__icontains=form.cleaned_data['date_added'])
        context = {'title': title, 'form': form, 'foto': foto}
        return render(request, 'repository_our_fotos/search_form.html', context)

Как сначала загрузить форму запросом GET, а потом сделать запрос GET для поиска ??

[Евгений]

А так как у вас написано разве не работает?
Кода формы вы не привели, поэтому не до конца понятно. И этот код делает не то, что было в оригинальном вопросе - здесь параметры взаимоисключающие.

И еще раз повторюсь, вручную писать View и код обработки форм - это хорошо только для обучения. Но для обучения продуктивнее читать документацию и размышлять, по вопросам на toster/StackOverflow менее эффективно будет - это как Google Translate воспользоваться, вместо того, чтобы самому напрячься и перевести. Мозг не работает и не получает новые знания в таком режиме.

Если для серьезного неигрушечного сайта, то лучше сразу взять django.views.generic.ListView или какой-нибудь другой из generic. Так и количество кода будет меньше (вашего кода, что важно как для вас, так и для того, кто будет поддерживать за вами), и автоматически будут всякие удобные возможности работать, типа разбиения на страницы.

Answer 2

[Dmitry Tiunov]

https://docs.djangoproject.com/en/1.11/topics/db/q...
Для запросов с оператором OR надо использовать Q object.

from django.db.models import Q


posts = OurFoto.objects.filter(
    Q(name__startswith=query) | Q(text__startswith=query) | Q(date_added__startswith=query)
)

Comments

[sergey19940808]

Да неверная это фильтрация