Skip to content

Mejoras DotNet2019Challenge#1

Open
MookieFumi wants to merge 6 commits intojsuarezruiz:masterfrom
MookieFumi:master
Open

Mejoras DotNet2019Challenge#1
MookieFumi wants to merge 6 commits intojsuarezruiz:masterfrom
MookieFumi:master

Conversation

@MookieFumi
Copy link

@MookieFumi MookieFumi commented Apr 28, 2019

Acabo de revisar por encima un poco el repo y los cambios que he introducido son:

  • Replaces Image with ffimageloading:CachedImage and updated nuget package to avoid runtime error
  • Added NavigationPage to make visible the toolbar
  • Avoid unnecessary layouts in MoviItemTemplate
  • Removed unused styles in App.xaml & reused BackgroundColor from HorizontalList to avoid extra magic strings
  • Added Autofac Container to manage dependencies & removed useless code in MoviesService & reused HttpClient & readed requests directly from a Stream (don't use strings)

@jsuarezruiz
Copy link
Owner

Gracias por tu participación!
Recuerdo que solo se tendran en cuenta los cambios relacionados con el rendimiento.
Vamos a revisar los cambios:

  • En la gestión de imágenes solemos encontrar un punto de mejora habitual. FFImageLoading estaba "semi-añadido". Sin duda, teniendo tanta carga de imágenes en la vista, es una mejora de rendimiento.
  • Hay Layouts innecesarios sin duda. Paneles que envuelven controles que no aportan nada (posicionamiento o tamaño). Un árbol visual complejo (e innecesario) impacta en el rendimiento.
  • Eliminar recursos (estilos) no usados en App.xaml impacta en el tiempo de arranque, por lo que es otra mejora de rendimiento.
  • Reutilizar HttpClient es otra mejora que tiene un gran impacto en el rendimiento.

De todos estos puntos, veremos en la DotNet 2019 ejemplos con datos, gráficas, etc.

Cumples con las 3 mejoras en rendimiento. Enhorabuena!. Eres el primero en enviar y cubrir las condicones. ¿Estas interesado en la entrada?

@MookieFumi
Copy link
Author

Por supuesto! 😎

@jsuarezruiz
Copy link
Owner

De acuerdo, pues tenemos ganador :)

@MookieFumi
Copy link
Author

Hola!

Porfa envíame la entrada/ invitación al email mookiefumi@icloud.com. Ok?

Gracias por todo!

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