Wie wichtig ist es, eine Variable für DateTime.Today zu verwenden, wenn es um die Leistung geht? ersten

Question

Ich sah nur this upvoted comment

IIRC DateTime.Today ist ein ziemlich teuer Anruf, so dass Sie besser den Wert in einer Variablen speichern.

Es als Antwort auf einen Beitrag war, die den Code enthalten:

var first = 
    new DateTime(DateTime.Today.Year, DateTime.Today.Month, 1).AddMonths(-1); 
var last = 
    new DateTime(DateTime.Today.Year, DateTime.Today.Month, 1).AddDays(-1); 

Wenn ich die Leistung zu verbessern suchen, wie wichtig es ist DateTime.Today in einer Variablen zu speichern, anstatt es mehrmals zu nennen? Und ungefähr wie viele Verwendungen von DateTime.Today würde es rechtfertigen, eine Variable dafür zu erstellen?

Bearbeiten: Ich weiß, dass ich mein Programm testen sollte, um zu sehen, ob es Leistungsprobleme gibt, bevor Sie sich über etwas so Triviales wie dieses Sorgen machen. Nehmen Sie für diese Frage an, dass ich dies bereits getan habe und festgestellt habe, dass zusätzliche Optimierung erforderlich ist.

Answer 1

Benchmark (auf meiner Maschine, die Stoppuhr-Klasse):

10,000 DateTime.Today calls and assignment to local variable: 0.0125781 seconds. 

10,000 Assignment only operations: 0.0001062 seconds. 

Code:

var s = new Stopwatch(); 
DateTime date = DateTime.Today; 
DateTime date2 = DateTime.Today; 
s.Start(); 
for (int i=0; i<10000; i++) 
    date = DateTime.Today; 
s.Stop(); 
Debug.Print(s.Elapsed.ToString()); 

s.Reset(); 
s.Start(); 
for (int i=0; i<10000; i++) 
    date2 = date; 
s.Stop(); 
Debug.Print(s.Elapsed.ToString()); 

Answer 2

wie wichtig ist es DateTime.Today in einer Variablen

die beste Antwort zu speichern, dass der Benchmark auf der Hardware ist, dass Sie Ihren Code erwarten auf zu laufen. Es sei denn, Sie nennen es in einer extrem engen Schleife, ich bezweifle, dass es ein Problem sein wird.

Ein besserer Grund, es in einer Variablen zu speichern, ist, dass Sie zwischen den beiden Aufrufen von einem Tag auf den anderen wechseln könnten.

UPDATE

eine Größenordnung bereitzustellen, geteilt @RichardBrown einen Link in seiner Antwort darauf hinweist, dass die Kosten für DateTime.Today wurden in der Größenordnung von einigen hundert Nanosekunden sein wird getestet (auf der speziellen Hardware für diesen Test verwendet).

Answer 3

Ich lehne die Prämisse ab, dass DateTime.Today ein teurer Anruf ist. Sie sollten es in einer Variablen speichern, wenn es wichtig ist, dass es sich im Laufe der Zeit nicht ändert. Wenn dieser Code gegen Mitternacht des Monatsendes läuft, könnten Sie ... Probleme haben. Aus der Sicht der Performance I hoch Zweifel, dass dies ein Problem sein würde.

In jedem Fall wäre es eine Mikro-Optimierung. Wie bei allen Leistungsproblemen, wenn Ihr Programm zu langsam ist, sollten Sie es profilieren und nach Abschnitten suchen, die ziemlich viel Zeit in Anspruch nehmen und sich darauf konzentrieren, diese zu optimieren. Wenn es dazu kommt, dass diese eine Codezeile viel Zeit in Anspruch nimmt, dann überlegen Sie, sie zu ändern. Bis dahin, refactor nur für die Richtigkeit oder Lesbarkeit, nicht die Leistung.

Answer 4

Wichtiger als die Frage der Leistung ist die Lesbarkeit und Wartbarkeit des Codes.

es sei denn, Sie nicht Performance-Probleme haben ich die Regel folgen würde „achten Sie auf Leistungsverbesserungen“

Vielleicht, wenn Sie eine Schleife als eine Variable mit „heute“ angemessen benannt werden kann, aber wieder, so lange es gibt keine Leistung ausgegeben, ich würde mich auf die anderen Bedenken konzentrieren

Answer 5

Für harte Zahlen auf die Leistung von DateTime lesen this blog post. Wie die vorherigen Antworten zeigen, ist es wichtig, dass Sie sich bei der Leistungsbestimmung auf Ihre spezielle Konfiguration und Ihre Bedürfnisse konzentrieren.

Answer 6

Ich würde vermuten, dass es ein Problem ist, wenn Sie den Code in einer Schleife aufrufen. Aber der beste Weg, um herauszufinden, ist es, Zeit zu nehmen und selbst zu sehen, wie lange es dauert.

var sw = new Stopwatch(); 
sw.Start(); 
today = DateTime.Today(); 
sw.Stop(); 
var ts = stopWatch.Elapsed; 

MSDN Stopwatch reference

Answer 7

Der wahre Grund DateTime.Today zu extrahieren als Variable ist verhindern mögliche Fehler, die wird erscheinen wegen Murphy's law, würde aber nie ..

Idee isoliert und fixiert werden, dass zwischen dem ersten und den zweiten Einsatz von Today tatsächlichem Datum geändert werden kann, so dass man bekommen kann, zum Beispiel, Vorjahr und neuen Monat hier:

var first = new DateTime(DateTime.Today.Year, DateTime.Today.Month, 1).AddMonths(-1); 

, die zum Beispiel zur Folge haben können, in (January 2013) - 1 month statt (December 2013) - 1 month.

Wahrscheinlicher und mehr böses Problem wird sein, wenn Sie Datumswechsel zwischen first und last Zuordnung erhalten:

var first = new DateTime(DateTime.Today.Year, DateTime.Today.Month, 1).AddMonths(-1); 
var last = new DateTime(DateTime.Today.Year, DateTime.Today.Month, 1).AddDays(-1); 

, die ein Jahr mehr Zeit zur Folge haben werden .. Je nach Ihrer Logik kann es wirklich führen teurer Verlust.