Singleton incorrectamente implementado en "Lands/ViewModels/MainViewModel.cs"
Opened this issue · 2 comments
MartifrmaP commented
Revisando el código de "Lands/ViewModels/MainViewModel.cs", observo que siempre devuelve una nueva instancia. Incluyo corrección...
public static MainViewModel GetInstance()
{
if (instance == null)
{
return new MainViewModel(); <<<- Borrar
instance = new MainViewModel(); <<<- Insertar
}
return instance;
}
Enhorabuena por tus aportes y videos en YouTube, son muy buenos.
Zulu55 commented
Muchas gracias por la corrección
El jue., 9 ago. 2018 a las 7:49, MartifrmaP (<notifications@github.com>)
escribió:
… Revisando el código de "Lands/ViewModels/MainViewModel.cs", observo que
siempre devuelve una nueva instancia. Incluyo corrección...
public static MainViewModel GetInstance()
{
if (instance == null)
{
return new MainViewModel(); <<<- Borrar
instance = new MainViewModel(); <<<- Insertar
}
return instance;
}
Enhorabuena por tus aportes y videos en YouTube, son muy buenos.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#1>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AL07wm7JEbMtMigB_Mqa1GkCgbeCAzx7ks5uPC_vgaJpZM4V1oe6>
.
--
*Juan Carlos Zuluaga Cardona*
*Phone: 350 634 2747*
*Skype: jzuluaga55@hotmail.com <jzuluaga55@hotmail.com>*
*URL: http://zulu-software.com <http://zulu-software.com/>*
*Medellín, Colombia*
mslarco commented
Yo también me di cuenta :D y entraba a sugerir el cambio.
Saludos Zulu, muy buenos tutoriales.