Diskusia – Singleton (jedináčik)

Skvělé. Obvykle to vídávám udělané jako statickou třídu, což je nepraktické. Přitom úplně stačila statická proměnná a metoda. V PHP to řeším podobně.

V druhém případě by měla být proměnná databaze také statická, aby se k ní dalo přistupovat ze statických metod. Ale to pak k ní nepůjde přistupovat přes instanci Singletonu. Nějaký čudný?
A taky když vrátím instanci singletonu tak bych měl přiřazovat do Pripojeni spíš getInstance()­.databaze ne?

Nene, ta proměnná je právě instanční. Celá krása Singletonu je v tom, že je to vlastně instance. Ty jen zakážeš uživateli, aby ji tvořil a spravuješ si ji sám pomocí statiky. Může to být trochu matoucí, ale třída Singleton si ve statice drží instanci třídy Singleton. Ta instance má normálně instanční proměnné.

Opravil jsem 2. kód, bylo tam místo instance databaze. Třídu dále není vhodné označovat jako public, na což jsme s David Jančíkem přišli, protože poté není možné v singletonu uchovávat třídy bez modifikátoru.

Nyní je vše ozkoušené.

{
get
{
if (instance == null)
instance = new Singleton();
return instance;
}
}

Jenom podotknu, ze C# ma super funkcionalitu vlastnosti, kterou tady jde pekne vyuzit. Priklad:

public static Singleton Instance
{
get
{
if (instance == null)
instance = new Singleton();
return instance;
}
}

Od doby, kdy jsem přišel na Dependency Injection, doporučuji Singleton nepoužívat. Je k ničemu a špatně se testuje.

Vůbec nechápu, k čemu je ta public Databaze? Jako aby šla kdykoliv odkudkoliv z programu přepsat? Tak to je přesně to, čemu se snaží singleton zabránit.

Vůbec tu nějak často vídám public fields, což jde proti zásadám OOP. V případech, kdy jde v podstatě o konstanty, používat alespoň readonly. Správně to říká @Mario@

public class Singleton
{

   private Singleton(){}

   private static Singleton _inst = null;

   public static Singleton Current
   {
       get
       {
          _inst = _inst ?? new Singleton();
          return _inst;
       }
   }
}

O co hůř se testuje singleton, než DI?

Netuším, jak se píší mocky pro singleton, ale v DI je to hračka.

Singleton ani neřeší, když například chci otevřít dvě nebo více databází. Musím kvůli tomu vytvořit další třídy. V DI stačí další instance.