criando-novos-comandos-no-django-admin

[rg3915]

@LucasMagnum sobre o código a seguir:

def handle(self, title=None, year=None, **options):
        if title is not None and year is not None:
            q = Movie.objects.filter(Q(title__istartswith=title) &
                                     Q(year=year))
        elif title is not None:
            q = Movie.objects.filter(title__istartswith=title)
        elif year is not None:
            q = Movie.objects.filter(year=year)
        for i in q:
            print(i.year, i.title)
        print('\n%s filmes localizados.' % q.count())

Não sei pq, mas acho que ele poderia ser melhorado, parece que tem inconsistência de lógica. O que sugere?

[LucasMagnum]

@rg3915

def handle(self, title=None, year=None, **options):
    filters = {
        'title__istartswith': title,
        'year': year
    }

    filter_by = {key: value for key, value in filters.items() if value is not None}
    queryset = Movie.objects.filter(**filter_by)

    for movie in queryset:
        print(movie.year, movie.title)

    print('\n%s filmes localizados.' % queryset.count())

[rg3915]

@LucasMagnum filter_by = {key: value for key, value in filters.items() if value is not None} não é redundante em relação ao dicionário que vc criou anteriomente?

[rg3915]

@LucasMagnum está tudo pronto. Ficou bom agora?

[LucasMagnum]

@rg3915 eu pelo menos não me lembro de nenhuma forma no momento para utilizar somente uma variavel e deixar o código simples de ler e entender.

Mas não acho que seja redundante, primeiros todos os possiveis filtros são declarados e depois utilizamos somente os que possuem algum valor. É melhor do que uma cadeia enorme de if e else.

[rg3915]

@LucasMagnum @luzfcb finalizem isso ai por favor. Publica o post, pq já cansei de revisar. Valeu.

[rg3915]

@luzfcb @LucasMagnum acho que agora está finalizado. Favor publicarem o post. Obrigado.

[raphapassini]

@rg3915 só vi esse ajuste. Acredito que acertando isso ai já podemos publicar! :)

[rg3915]

@raphapassini me ajuda ai por favor, nao manjo tanto assim.

[rg3915]

@raphapassini @luzfcb @LucasMagnum Será que agora está aprovado? :)

[LucasMagnum]

:+1:

@raphapassini e @luzfcb alguma outra observação?

[rg3915]

@LucasMagnum @luzfcb @raphapassini Pessoal, tem como vcs publicarem meu post por favor? To precisando dele, é importante pra mim. Por favor. Obrigado.

[rg3915]

@LucasMagnum @luzfcb @raphapassini valeu pessoal. :+1: