Soy bastante nuevo en Python e intento encontrar una mejor manera de codificar esto. Tiene que haber una manera, pero no estoy seguro de cómo hacerlo.

Las dos consultas son esencialmente las mismas, por lo que tiene que haber una forma de reducir. ¿Hay una manera más eficiente de hacer esto?

        if set_date is not None:

            if is_rejected != 'true':
                query = query\
                    .filter(createddate__lte=set_date) \
                    .car_statuses(CarTow.TOWED) \
                    .order_by('-createddate')
            else:
                query = query\
                    .filter(createddate__lte=set_date) \
                    .car_statuses(CarTow.TOWED,CarTow.CONFIRMED) \
                    .order_by('-createddate')

            return query

Lo siento si esta es una pregunta simple, novato aquí.

python django
1
SteveV 2 oct. 2019 a las 01:32

3 respuestas

La mejor respuesta

Puede simplificar sacando el argumento que difiere en la declaración if y dejando afuera las cosas comunes.

if set_date is not None:
       if is_rejected != 'true':
             car_statuses = (CarTow.TOWED,)
       else:
             car_statuses = (CarTow.TOWED, CarTow.CONFIRMED)

       query = query\
           .filter(createddate__lte=set_date) \
           .car_statuses(*car_statuses) \
           .order_by('-createddate')
      return query
2
rdas 1 oct. 2019 a las 22:37

Puede usar la lógica ternaria para agregar las tuplas.

query = (
    query
    .filter(createddate__lte = set_date) 
    .car_statuses((CarTow.TOWED,) + ((CarTow.CONFIRMED,) if is_rejected == 'true' else ()) 
    .order_by('-createddate')
)
2
Alexander 1 oct. 2019 a las 23:04

Probablemente quieras reemplazar esto:

if set_date is not None:

Con esto:

if set_date:

Eche un vistazo a cómo Python evalúa el if condicional: Prueba del valor de la verdad (pydocs)

Aquí están la mayoría de los objetos incorporados considerados falsos: constantes definidas como falsas: ninguna y falsa. cero de cualquier tipo numérico: 0, 0.0, 0j, Decimal (0), Fracción (0, 1) secuencias y colecciones vacías: '', (), [], {}, set (), rango (0)

Además, 'es' puede dar algunos resultados extraños, es realmente para determinar si dos etiquetas hacen referencia al mismo objeto o no. Entendiendo que Python es el operador

1
neutrino_logic 1 oct. 2019 a las 22:59
58193085

Preguntas relacionadas

30 ¿Cómo fusiono dos diccionarios en una sola expresión?
27 Llamar a un comando externo desde Python
15 ¿Qué son las metaclases en Python?
30 Cómo obtener la hora actual en Python
14 lista de comprensión vs. lambda + filtro
21 Crear un singleton en Python

Preguntas vinculadas

11 Entendiendo la operadora "es" de Python

Nuevas preguntas

3 Reste un valor único de la columna numpy conservando la forma de los datos originales
2 ¿Cómo imprimir dos veces el mismo número aleatorio?
5 ¿Cómo puedo dividir el primer elemento de un elemento de lista anidado?
4 break no se rompe while loop. ¿por qué?
2 ¿Cómo reconocer y ejecutar múltiples comandos de una frase en Python?
2 Cómo heredar slug de una clase a otra en los modelos de django
3 Luchando con BeautifulSoup y etiquetas
2 eliminar fila si una columna es mayor que otra
3 Python: mueva cada carácter hasta el final de la cadena sin reutilizar la cadena original
2 sesiones de django recuérdame
Nuevas preguntas con la etiqueta python >

python

Python es un lenguaje de programación multipropósito, de tipificación dinámica y de múltiples paradigmas. Está diseñado para ser rápido de aprender, comprender y usar, y hacer cumplir una sintaxis limpia y uniforme. Tenga en cuenta que Python 2 está oficialmente fuera de soporte a partir del 01-01-2020. Aún así, para preguntas de Python específicas de la versión, agregue la etiqueta [python-2.7] o [python-3.x]. Cuando utilice una variante de Python (por ejemplo, Jython, PyPy) o una biblioteca (por ejemplo, Pandas y NumPy), inclúyala en las etiquetas.

Más sobre python...