search

Ruth2002Casilla / MainSystem-Android

Sistema completo de Asistente Virtual
0 stars 0 forks source link

Oportunidades de mejora TicketDao #7

Closed aaron-developer25 closed 2 days ago

aaron-developer25 commented 2 days ago

https://github.com/Ruth2002Casilla/MainSystem-Android/blob/2d1ea3fa31961df879fe5d08830ea7076385e9ec/app/src/main/java/edu/ucne/composedemo/mainsystemtickets/data/local/dao/TicketDao.kt#L12C1-L12C22

Uso de @Upsert sin paréntesis:
    La anotación @Upsert no requiere paréntesis si no estás pasando parámetros adicionales. Los paréntesis pueden ser omitidos cuando no se especifican atributos.
    Cambio: @Upsert() a @Upsert.

Corregir el nombre del parámetro en delete:
    El nombre del parámetro da en el método delete no es descriptivo. Debe seguir las mismas convenciones de nomenclatura que el resto de los métodos.
    Cambio: Renombrar da a ticket para mantener consistencia y claridad en los nombres de los parámetros.

Espacios y estilo en la consulta SQL:
    En el método find, la consulta SQL debería tener espacios alrededor de los operadores para mejorar la legibilidad.
    Cambio: WHERE TicketId=:id a WHERE TicketId = :id.
Ruth2002Casilla commented 2 days ago

La mitad de esta corrección no es valida. Le invito a volver a revisar correctamente el código.

Ruth2002Casilla commented 2 days ago

Uso de @Upsert sin paréntesis: La anotación @Upsert no requiere paréntesis si no estás pasando parámetros adicionales. Los paréntesis pueden ser omitidos cuando no se especifican atributos. Cambio: @Upsert() a @Upsert.

Espacios y estilo en la consulta SQL: En el método find, la consulta SQL debería tener espacios alrededor de los operadores para mejorar la legibilidad. Cambio: WHERE TicketId=:id a WHERE TicketId = :id.

Estos comentarios no son validos