Skip to content

#cambios realizados y pequeñas cosas encontradas#2

Open
devjaime wants to merge 1 commit intojsuarezruiz:masterfrom
devjaime:master
Open

#cambios realizados y pequeñas cosas encontradas#2
devjaime wants to merge 1 commit intojsuarezruiz:masterfrom
devjaime:master

Conversation

@devjaime
Copy link

Lo primero que encontré fue que esto se puede acortar para acceder de mejor forma
image
Lo segundo es cambiar a la clase estática y mejorar el condicionamiento de lo siguiente
image
Lo tercero la interpolación de las cadenas
image
cuarto la instancia se puede realizar en una sola linea
image
Quinto ciertas variables puedes ser asignadas con "var" sin generar el tipo de estas tal vez sea un cambio menor pero en algunas ocaciones esto ayuda
image

También las cambie a clases estáticas
image
Lo mismo anterior se puede acortar
image

esto lo deje así ya que es una buena practica
image

Espero sean de utilidad o acorde a lo solicitado, entiendo que la vista también tendría mejoras.

@jsuarezruiz
Copy link
Owner

jsuarezruiz commented May 1, 2019

Gracias por participar!.
Sólo un breve recordatorio, en el reto se valorarán solo las mejoras relacionadas directamente con el rendimiento.
Vamos a analizar los cambios:

  • Utilizas algunas ventajas de c# 7 como expression body definitions: https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/statements-expressions-operators/expression-bodied-members#property-set-statements Es correcto y es más utilizado, aunque no es uno de los puntos con impacto alto en el rendimiento de la App.
  • De igual forma, la interpolación de cadenas es correcto y simplifica la legibilidad del código, aunque no es uno de los puntos con impacto alto en el rendimiento de la App.
  • En el servicio donde creamos las peticiones Http usando HttpClient has realizado un cambio interesante. Sin embargo, se sigue creando una instancia de HttpClient por cada petición que es menos óptimo que reutilizar la misma. Además, en ese caso no hay una gestión adecuada para hacer Dispose de recursos.

Planteas cambios interesantes, aunque hay muuuchos otros con un impacto mayor en el rendimiento de la App. Te dejo algunas pistas...

  • En la inicialización, revisa los recursos usados.
  • Revisa el Layout, ¿se puede hacer todo de forma más óptima?.
  • Revisa controles...
  • Revisa la gestión de imágenes.
  • Revisa los Bindings...
  • Etc

De nuevo, muchas gracias por participar!. Si estas por España en la DotNet 2019, avísame y nos vemos allÍ :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants