PDA

View Full Version : Per la felicita' di Fek: Singletons Are Evil!


Bonfo
29-01-2008, 02:58
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!!!! :D

71104
29-01-2008, 12:35
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/showpost.php?p=20797786&postcount=19

:D

grazie fek, scusa Bonfo. ciao :D

^TiGeRShArK^
29-01-2008, 15:25
Visto che siamo in tema di Singletons.. :p
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# :muro: ) 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.. :mbe:
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"); :mbe:
..A voi la parola :p

Bonfo
29-01-2008, 17:54
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 :D

fek
29-01-2008, 20:11
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/showpost.php?p=20797786&postcount=19

:D

grazie fek, scusa Bonfo. ciao :D

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
29-01-2008, 20:27
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! :D Fico ....

Bonfo
29-01-2008, 21:04
Se il cucchiaio non esiste e il singleton non esiste ... ci sono! Allora il singleton è un cucchiaio! :D Fico ....

:rotfl: :rotfl:

Ufo13
30-01-2008, 08:16
Ecco la lista dei vantaggi portati dall'uso del singleton pattern:

fek
30-01-2008, 08:46
Ecco la lista dei vantaggi portati dall'uso del singleton pattern:

Severo ma giusto.

Jocchan
30-01-2008, 12:15
Ecco la lista dei vantaggi portati dall'uso del singleton pattern:
A parer mio sei stato fin troppo buono con questa lista :ciapet: