Per la felicita' di Fek: Singletons Are Evil!

[Bonfo]

Mi sono messo a leggere questo:
http://www.mockobjects.com/files/endotesting.pdf
(Che guarda caso e' nei link di documentazione di UnitTesting) e tra le citazioni c'e' questo:
http://c2.com/cgi/wiki?SingletonsAreEvil

Ecco finalmente perche' i Singleton sono come la puzza!!!!

[71104]

OT - fek e Bonfo, scusate se approfitto del topic (che attira l'attenzione di fek) ma è importante che il team abbia una risposta a queste due domande:
http://www.hwupgrade.it/forum/showpo...6&postcount=19



grazie fek, scusa Bonfo. ciao

[^TiGeRShArK^]

Visto che siamo in tema di Singletons..
mi sono ritrovato sotto mano il codice che avevo fatto per un programmino in C# per gestire le Properties in maniera simil-java (dato che NON ho trovato una classe equivalente in C# ) e già che c'ero l'ho reso statico.

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.IO;

namespace FileSystemListener
{
    public static class Properties
    {
        private static Dictionary<string, string> properties = new Dictionary<string, string>();
        private static string filename = "config.properties";
        private static bool isLoaded = false;


        private static void loadProperties()
        {
            if (!isLoaded) 
            {
                StreamReader reader = File.OpenText(filename);
                string line = "";
                while ((line = reader.ReadLine()) != null)
                {
                    string[] splittedLine = line.Split('=');
                    string key = splittedLine[0];
                    string value = splittedLine[1];
                    properties.Add(key, value);
                }
                reader.Close();
                isLoaded = true;
            }
        }


        public static void setPropertiesFile(string filename)
        {
            Properties.filename = filename;
        }


        public static string get(string key)
        {
            loadProperties();
            return properties[key];
        }


        public static void add(string key, string value)
        {
            loadProperties();
            properties.Add(key, value);
        }


        public static void set(string key, string value)
        {
            loadProperties();
            properties[key] = value;
        }


        public static void save()
        {
            StreamWriter writer = File.CreateText(filename);
            foreach (string key in properties.Keys)
            {
                writer.Write(key + "=" + properties[key] + '\n');
            }
            writer.Flush();
            writer.Close();
        }
    }
}

A questo punto mi sfugge il motivo della non testabilità, dato che per renderlo perfettamente testabile mi sa che l'unica modifica da fare è eliminare il controllo effettuato da isLoaded e le chiamate multple a loadProperties che per comodità avevo inserito nel get e nel set..
E cmq come metodo lo trovo piuttosto comodo dato che effettua semplicemente il get e il set delle properties sempre uguali per tutti e condivise in tutta la VM senza bisogno di passare tra le varie classi oggetti di tipo Config e di scrivere cose del tipo environment.getConfig().getInteger("pippo");
..A voi la parola

[Bonfo]

Calma... io non sono ancora ai livelli Fekkiani.
O meglio, sto iniziano ora a documentarmi e a cercare di capire il perche' del "a morte il Singleton".
Indubbiamnete prima ero molto piu' leggero sui singleton, ora mi chiedo seriamente se serve o se si puo' fare in modo diverso.
Sto scoprendo piano piano anch'io che singleton tendo a portare piu' problemi che risolverli.

Non sono per la 0 policy, ma solo per usi molto attenti.
Ora penso un po' all'esempio che hai postato e ti faccio sapere

[fek]

Originariamente inviato da 71104

OT - fek e Bonfo, scusate se approfitto del topic (che attira l'attenzione di fek) ma è importante che il team abbia una risposta a queste due domande:
http://www.hwupgrade.it/forum/showpo...6&postcount=19



grazie fek, scusa Bonfo. ciao

Fammi riprendere dieci minuti che mi sono fatto tutta l'Europa avanti e indietro in quattro giorni. Leggo tutto ora.

(Il Singleton non esiste)

[AnonimoVeneziano]

Originariamente inviato da fek

Fammi riprendere dieci minuti che mi sono fatto tutta l'Europa avanti e indietro in quattro giorni. Leggo tutto ora.

(Il Singleton non esiste)

Se il cucchiaio non esiste e il singleton non esiste ... ci sono! Allora il singleton è un cucchiaio! Fico ....

[Bonfo]

Originariamente inviato da AnonimoVeneziano

Se il cucchiaio non esiste e il singleton non esiste ... ci sono! Allora il singleton è un cucchiaio! Fico ....

[Ufo13]

Ecco la lista dei vantaggi portati dall'uso del singleton pattern:

[fek]

Originariamente inviato da Ufo13

Ecco la lista dei vantaggi portati dall'uso del singleton pattern:

Severo ma giusto.

[Jocchan]

Originariamente inviato da Ufo13

Ecco la lista dei vantaggi portati dall'uso del singleton pattern:

A parer mio sei stato fin troppo buono con questa lista