Nicht die schlechteste, aber eine, die noch nicht erwähnt wurde. Factory-Methoden, die als Argumente an System.Collections.Concurrent-Methoden übergeben werden, können mehrfach aufgerufen werden, auch wenn nur ein Rückgabewert verwendet wird. Wenn man bedenkt, wie sehr .NET versucht, Sie vor ungewolltem Aufwachen in Threading-Primitiven zu schützen, kann dies eine Überraschung sein.
using System;
using System.Collections.Generic;
using System.Collections.Concurrent;
using System.Linq;
using System.Text;
using System.Threading;
using System.Threading.Tasks;
namespace ValueFactoryBehavingBadlyExample
{
class Program
{
static ConcurrentDictionary<int, int> m_Dict = new ConcurrentDictionary<int, int>();
static ManualResetEventSlim m_MRES = new ManualResetEventSlim(false);
static void Main(string[] args)
{
for (int i = 0; i < 8; ++i)
{
Task.Factory.StartNew(ThreadGate, TaskCreationOptions.LongRunning);
}
Thread.Sleep(1000);
m_MRES.Set();
Thread.Sleep(1000);
Console.WriteLine("Dictionary Size: " + m_Dict.Count);
Console.Read();
}
static void ThreadGate()
{
m_MRES.Wait();
int value = m_Dict.GetOrAdd(0, ValueFactory);
}
static int ValueFactory(int key)
{
Thread.Sleep(1000);
Console.WriteLine("Value Factory Called");
return key;
}
}
}
(Mögliche) Ausgabe:
Value Factory Called
Value Factory Called
Value Factory Called
Value Factory Called
Dictionary Size: 0
Value Factory Called
Value Factory Called
Value Factory Called
Value Factory Called
158 Stimmen
Return DateTime.Now.AddDays(1);
24 Stimmen
Soweit ich weiß, sind die eingebauten Werttypen alle unveränderlich, zumindest insofern, als jede Methode, die mit dem Typ verbunden ist, ein neues Element zurückgibt, anstatt das vorhandene Element zu verändern. Zumindest fällt mir spontan kein Typ ein, der dies nicht tut: alles schön und konsistent.
1 Stimmen
Community-Wiki, so viel Spam in SO jetzt. Wenn Fragen subjektiv sind (keine endgültige Antwort), sollte es Community Wiki sein.
6 Stimmen
Veränderlicher Werttyp: System.Collections.Generics.List.Enumerator :( (Und ja, Sie können sehen, dass es sich seltsam verhält, wenn Sie sich genug Mühe geben).
0 Stimmen
Lol, ich wusste, dass es Ausnahmen geben würde. Enumerator scheint irgendwie... s
14 Stimmen
T
22 Stimmen
N
4 Stimmen
T
4 Stimmen
D
2 Stimmen
Sollte genannt werden
dt.NextDays(1);
haben die Programmierer keine Ahnung, dass er seinen eigenen Wert ändert. Genauso wie bei Datenstrukturkonstrukten (z.B.node->next
,node.next
) ändert den Wert von node nicht. Und jetzt ist es zu spät, sie könntendt.AddDays(1)
die Semantik, dass sie Tage in ihrem eigenen Wert hinzufügen.1 Stimmen
Ich stimme zu, dass es sich bei dem genannten Beispiel um eine Frage der Namensgebung handelt. Der Name der Methode impliziert, dass sie zum Datum addiert wird. Etwas wie
dt.PlusDays(1)
wäre klarer. Ein weiteres Problem bei dieser Methode ist, dass sie nur einedouble
Wert, so dass das Ergebnis nicht exakt ist.0 Stimmen
Ich würde diese Frage gewinnen, wenn eine dumme Person dieses Thema nicht geschlossen hätte. Ich kenne den bösesten Bug der Welt. Bitte sehen Sie codeproject.com/Feature/
1 Stimmen
@bluefeet, da die Benutzer >= 62 Antworten gegeben haben (einschließlich Jon Skeet), viele davon mit Hunderten von Bewertungen, dann wollen die Benutzer offenbar auch diese Art von Fragen. Entweder handelt es sich hier um eine angemessene Art von Fragen, oder es ist Zeit für eine Verfassungsänderung aller einschränkenden Regeln von SO.
0 Stimmen
5.add(2); 5 sollte immer noch 5 sein.