Bagaimana cara memeriksa apakah DateTime terjadi hari ini?

110

Apakah ada cara .net yang lebih baik untuk memeriksa apakah DateTime telah terjadi 'hari ini' daripada kode di bawah?

if ( newsStory.WhenAdded.Day == DateTime.Now.Day &&
     newsStory.WhenAdded.Month == DateTime.Now.Month &&
     newsStory.WhenAdded.Year == DateTime.Now.Year )
{ 
    // Story happened today
}
else
{ 
    // Story didn't happen today
}

c# .net datetime

— Peter Bridger
sumber

14

Anda dapat membandingkan newStory.Date == DateTime.Now.Date

— Guillaume

222

if (newsStory.WhenAdded.Date == DateTime.Today)
{

}
else
{

}

Harus melakukan trik.

— pyrocumulus
sumber

3

... jika newsStory.Date sebenarnya hanya porsi tanggal, tanpa waktu. ;)

— Lucero

14

... yang mana, jika newsStory adalah DateTime

— stevemegson

18

baik, DateTime adalah bagian dari Perpustakaan Kelas .NETFramework, jadi meskipun memungkinkan OP membuat kelasnya sendiri yang kebetulan memiliki nama yang sama, Anda harus bertanya-tanya mengapa dia bertanya kepada StackOverflow cara menggunakan kelas kustom. dia menciptakan dan mengharapkan kita secara ajaib mengetahui cara kerjanya. ;)

— Brian Schroth

2

@ Lucero itu mungkin, tetapi judul pertanyaannya bertuliskan 'dua Tanggal Waktu'. ;) Saya kira dari situlah kami mendapatkan tipe yang hilang itu.

— pyrocumulus

1

@HansPetterNaumann itu logis :) DateTime.AddHours () mengembalikan objek DateTime baru, sehingga secara efektif membatalkan operasi .Today Anda yang memang tidak memiliki komponen waktu.

— pyrocumulus

29

if( newsStory.Date == DateTime.Today )
{
    // happened today
}

— Dave D
sumber

5

Saya ingin menambahkan, ini adalah bug yang sangat umum (dan sulit dikenali) - hanya membandingkan DateTime dengan DateTime.Today.

— JL.

16

Mencoba

if (newsStory.Date == DateTime.Now.Date) 
{ /* Story happened today */ }
else
{ /* Story didn't happen today */ }

— Stephen Newman
sumber

14

Solusi saya:

private bool IsTheSameDay(DateTime date1, DateTime date2)
{
    return (date1.Year == date2.Year && date1.DayOfYear == date2.DayOfYear);
}

— A. Polezhaev
sumber

7

Jika NewsStory juga menggunakan DateTime, bandingkan saja properti Date, dan Anda sudah selesai.

Namun, ini tergantung pada apa sebenarnya arti "hari ini". Jika sesuatu diposting sebelum tengah malam, itu akan menjadi "lama" setelah waktu yang singkat. Jadi mungkin akan lebih baik untuk menyimpan tanggal cerita yang tepat (termasuk waktu, sebaiknya UTC) dan memeriksa apakah kurang dari 24 jam (atau apa pun) telah berlalu, yang sederhana (tanggal dapat dikurangi, yang memberi Anda TimeSpan dengan TotalHours atau properti TotalDays).

— Lucero
sumber

5

Anda dapat menerapkan metode ekstensi DateTime.

Buat kelas baru untuk metode ekstensi Anda:

namespace ExtensionMethods
{
    public static class ExtensionMethods
    {
        public static bool IsSameDay( this DateTime datetime1, DateTime datetime2 )
        {
            return datetime1.Year == datetime2.Year 
                && datetime1.Month == datetime2.Month 
                && datetime1.Day == datetime2.Day;
        }
    }
}

Dan sekarang, di mana pun pada kode Anda, di mana Anda ingin melakukan tes ini, Anda harus menyertakan penggunaan:

using ExtensionMethods;

Dan kemudian, gunakan metode ekstensi:

newsStory.WhenAdded.IsSameDay(DateTime.Now);

— Benjamim
sumber

3

mengapa tidak mengembalikan datetime1.Date == datetime2.Date?

— Sergiu Mindras

@SergiuMindras benar, cukup membandingkan keduanya Datekarena TimeSpanakan selalu 00:00:00.

— GoRoS

5

FYI,

newsStory.Date == DateTime.Today

akan mengembalikan hasil perbandingan yang sama dengan pengkodean

newsStory == DateTime.Today

dimana newsStoryaDateTime benda

NET cukup pintar untuk menentukan Anda ingin membandingkan berdasarkan Tanggal saja dan menggunakannya untuk Bandingkan internal. Tidak yakin mengapa, dan benar-benar kesulitan menemukan dokumentasi untuk perilaku ini.

— guck
sumber

4

Seperti yang disarankan Guillame dalam komentar , bandingkan nilai Dateproperti:

newStory.Date == DateTime.Now.Date

— Anton Gogolev
sumber

2

Coba ini:

newsStory.Date == DateTime.Today

— Philip Wallace
sumber

1

baik, DateTime memiliki properti "Tanggal" dan Anda bisa membandingkan berdasarkan itu. Namun melihat dokumennya, tampaknya mendapatkan properti tersebut sebenarnya membuat waktu baru dengan komponen waktu disetel ke tengah malam, jadi mungkin lebih lambat daripada mengakses masing-masing komponen, meskipun jauh lebih bersih dan lebih mudah dibaca.

— Brian Schroth
sumber

Saya pikir perlambatan (jika ada) akan sangat kecil sehingga tidak masalah di hampir semua kasus. Tentu saja kasus pengoptimalan prematur.

— Esben Skov Pedersen

Setuju - akhir-akhir ini saya merasa semakin sulit untuk menghilangkan pengoptimalan prematur karena dunia pengkodean modern lebih mengutamakan keterbacaan dan pemeliharaan. Kebiasaan susah hilang!

— Peter Bridger

0

if (newsStory.ToShortDateString() == DateTime.Today.ToShortDateString()) return "Todtay";

— Sasan Salem
sumber

-4

Bagaimana tentang

if (newsStory.DayOfYear == DateTime.Now.DayOfYear)
{ // Story happened today
}

Tetapi ini juga akan kembali benar untuk 1 Januari 2008 dan 1 Januari 2009, yang mungkin atau mungkin tidak sesuai dengan keinginan Anda.

— Polyfun
sumber

-6

Anda bisa menggunakan DateTime.Now.DayOfYear

 if (newsStory.DayOfYear == DateTime.Now.DayOfYear)
 { // story happened today

 }
 else
 { // story didn't happen today

 }

— Ryan Alford
sumber

1

Saya tidak berpikir ini akan berhasil. Jika hari ini adalah 10/21/2009, dan newsStory adalah 10/21/2008 - itu akan menjadi true (saya tidak memperhitungkan tahun kabisat di sini).

— Philip Wallace

2

Ini tidak akan membedakan antara tahun DateTime baru (2009, 10, 01) .DayOfYear == new DateTime (1900, 10, 01) .DayOfYear;

— Dave D