Mengapa Thread.Sleep sangat berbahaya

Saya sering melihatnya disebutkan bahwa Thread.Sleep();tidak boleh digunakan, tetapi saya tidak dapat memahami mengapa demikian. Jika Thread.Sleep();dapat menimbulkan masalah, apakah ada solusi alternatif dengan hasil yang sama yang aman?

misalnya.

while(true)
{
    doSomework();
    i++;
    Thread.Sleep(5000);
}

satu lagi adalah:

while (true)
{
    string[] images = Directory.GetFiles(@"C:\Dir", "*.png");

    foreach (string image in images)
    {
        this.Invoke(() => this.Enabled = true);
        pictureBox1.Image = new Bitmap(image);
        Thread.Sleep(1000);
    }
}

Masalah dengan memanggil Thread.SleepAre menjelaskan cukup ringkas di sini :

Thread.Sleepmemiliki kegunaannya: mensimulasikan operasi yang panjang saat menguji / debugging pada thread MTA. Di .NET tidak ada alasan lain untuk menggunakannya.

Thread.Sleep(n)berarti memblokir utas saat ini setidaknya untuk jumlah potongan waktu (atau kuantum utas) yang dapat terjadi dalam n milidetik. Panjang kutu waktu berbeda pada versi / tipe Windows dan prosesor yang berbeda dan umumnya berkisar antara 15 hingga 30 milidetik. Ini berarti utas hampir dijamin akan diblokir selama lebih dari nmilidetik. Kemungkinan utas Anda akan aktif kembali tepat setelah nmilidetik hampir tidak mungkin dilakukan. Jadi, Thread.Sleeptidak ada gunanya mengatur waktu .

Untaian adalah sumber daya terbatas, mereka membutuhkan sekitar 200.000 siklus untuk dibuat dan sekitar 100.000 siklus untuk dihancurkan. Secara default, mereka mencadangkan 1 megabyte memori virtual untuk tumpukannya dan menggunakan 2.000-8.000 siklus untuk setiap sakelar konteks. Hal ini membuat setiap benang menunggu menjadi sangat sia - sia.

Solusi yang disukai: WaitHandles

Kesalahan yang paling banyak dibuat adalah menggunakan Thread.Sleep with a while-construct ( demo dan jawaban , entri blog yang bagus )

EDIT:
Saya ingin meningkatkan jawaban saya:

Kami memiliki 2 kasus penggunaan yang berbeda:

1. Kami menunggu karena kami tahu rentang waktu tertentu kapan kami harus melanjutkan (gunakan Thread.Sleep , System.Threading.Timeratau serupa)

2. Kami menunggu karena beberapa kondisi berubah sewaktu-waktu ... kata kunci adalah / are kadang kadang ! jika condition-check ada di code-domain kita, kita harus menggunakan WaitHandles - jika tidak, komponen eksternal harus menyediakan semacam pengait ... jika tidak, desainnya buruk!

Jawaban saya terutama mencakup kasus penggunaan 2

SKENARIO 1 - menunggu tugas asinkron selesai: Saya setuju bahwa WaitHandle / Auto | ManualResetEvent harus digunakan dalam skenario di mana utas menunggu tugas di utas lain selesai.

SKENARIO 2 - timing while loop: Namun, sebagai mekanisme timing yang kasar (while + Thread.Sleep) baik-baik saja untuk 99% aplikasi yang TIDAK memerlukan pengetahuan persis kapan Thread yang diblokir harus "aktif *. Argumen yang diperlukan 200k siklus untuk membuat utas juga tidak valid - utas pengulangan waktu perlu dibuat dan siklus 200k hanyalah angka besar lainnya (beri tahu saya berapa banyak siklus untuk membuka panggilan file / socket / db?).

Jadi jika sementara + Thread.Sleep berfungsi, mengapa mempersulit? Hanya pengacara sintaksis yang praktis !

Saya ingin menjawab pertanyaan ini dari perspektif politik-pengkodean, yang mungkin berguna atau tidak bagi siapa pun. Tetapi terutama ketika Anda berurusan dengan alat yang ditujukan untuk 9-5 programmer perusahaan, orang yang menulis dokumentasi cenderung menggunakan kata-kata seperti "tidak boleh" dan "tidak pernah" berarti "jangan lakukan ini kecuali Anda benar-benar tahu apa yang Anda sedang melakukan dan mengapa ".

Beberapa favorit saya yang lain di dunia C # adalah bahwa mereka memberitahu Anda untuk "jangan pernah menelepon kunci (ini)" atau "jangan pernah menelepon GC.Collect ()". Keduanya secara paksa dideklarasikan di banyak blog dan dokumentasi resmi, dan IMO adalah informasi yang salah. Pada tingkat tertentu, informasi yang salah ini memenuhi tujuannya, yaitu menjauhkan para pemula dari melakukan hal-hal yang tidak mereka pahami sebelum sepenuhnya meneliti alternatifnya, tetapi pada saat yang sama, itu membuat sulit untuk menemukan informasi NYATA melalui mesin pencari yang semuanya tampaknya mengarah ke artikel yang memberi tahu Anda untuk tidak melakukan sesuatu sementara tidak memberikan jawaban atas pertanyaan "mengapa tidak?"

Secara politis, ini bermuara pada apa yang orang anggap sebagai "desain yang baik" atau "desain yang buruk". Dokumentasi resmi tidak boleh mendikte desain aplikasi saya. Jika benar-benar ada alasan teknis bahwa Anda tidak boleh memanggil sleep (), maka IMO dokumentasi harus menyatakan bahwa tidak apa-apa untuk memanggilnya dalam skenario tertentu, tetapi mungkin menawarkan beberapa solusi alternatif yang tidak tergantung pada skenario atau lebih sesuai untuk yang lain. skenario.

Dengan jelas menyebut "sleep ()" berguna dalam banyak situasi ketika tenggat waktu ditentukan dengan jelas dalam istilah waktu dunia nyata, namun, ada sistem yang lebih canggih untuk menunggu dan memberi sinyal utas yang harus dipertimbangkan dan dipahami sebelum Anda mulai tidur ( ) ke dalam kode Anda, dan memasukkan pernyataan sleep () yang tidak perlu dalam kode Anda umumnya dianggap sebagai taktik pemula.

Ini adalah lingkaran 1). Berputar dan 2) .polling dari contoh Anda yang harus diperhatikan orang , bukan bagian Thread.Sleep (). Saya pikir Thread.Sleep () biasanya ditambahkan untuk dengan mudah meningkatkan kode yang berputar atau dalam loop polling, jadi ini hanya terkait dengan kode "buruk".

Selain itu, orang melakukan hal-hal seperti:

while(inWait)Thread.Sleep(5000); 

di mana variabel inWait tidak diakses dengan cara yang aman untuk thread, yang juga menyebabkan masalah.

Yang ingin dilihat pemrogram adalah utas yang dikontrol oleh konstruksi Peristiwa dan Pemberian Sinyal dan Penguncian, dan ketika Anda melakukannya, Anda tidak perlu Thread.Sleep (), dan kekhawatiran tentang akses variabel aman-thread juga dihilangkan. Sebagai contoh, dapatkah Anda membuat penangan peristiwa yang terkait dengan kelas FileSystemWatcher dan menggunakan peristiwa untuk memicu contoh ke-2 alih-alih perulangan?

Seperti yang disebutkan Andreas N., membaca Threading dalam C #, oleh Joe Albahari , benar-benar bagus.

Sleep digunakan dalam kasus di mana program independen yang tidak dapat Anda kendalikan terkadang menggunakan resource yang umum digunakan (misalnya, file), yang perlu diakses oleh program Anda saat berjalan, dan saat resource digunakan oleh program tersebut. program lain program Anda diblokir dari menggunakannya. Dalam kasus ini, di mana Anda mengakses sumber daya dalam kode Anda, Anda meletakkan akses sumber daya dalam coba-tangkap (untuk menangkap pengecualian ketika Anda tidak dapat mengakses sumber daya), dan Anda meletakkannya di loop sementara. Jika sumber daya gratis, sleep tidak akan pernah dipanggil. Namun jika sumber daya diblokir, Anda akan tidur selama jangka waktu yang sesuai, dan mencoba mengakses sumber daya lagi (inilah alasan Anda melakukan perulangan). Namun, perlu diingat bahwa Anda harus meletakkan semacam pembatas pada loop, jadi ini bukan loop yang berpotensi tak terbatas.

Saya memiliki kasus penggunaan yang tidak terlalu saya lihat di sini, dan akan berpendapat bahwa ini adalah alasan yang valid untuk menggunakan Thread.Sleep ():

Dalam aplikasi konsol yang menjalankan pekerjaan pembersihan, saya perlu melakukan sejumlah besar panggilan database yang cukup mahal, ke DB yang dibagikan oleh ribuan pengguna bersamaan. Agar tidak memalu DB dan mengecualikan orang lain selama berjam-jam, saya perlu jeda antar panggilan, dalam urutan 100 ms. Ini tidak terkait dengan waktu, hanya untuk memberikan akses ke DB untuk utas lainnya.

Menghabiskan 2000-8000 siklus pada peralihan konteks antar panggilan yang mungkin memerlukan 500 md untuk dieksekusi tidaklah berbahaya, seperti halnya memiliki 1 MB tumpukan untuk utas, yang berjalan sebagai satu contoh di server.

Saya setuju dengan banyak hal di sini, tetapi saya juga berpikir itu tergantung.

Baru-baru ini saya melakukan kode ini:

private void animate(FlowLayoutPanel element, int start, int end)
{
    bool asc = end > start;
    element.Show();
    while (start != end) {
        start += asc ? 1 : -1;
        element.Height = start;
        Thread.Sleep(1);
    }
    if (!asc)
    {
        element.Hide();
    }
    element.Focus();
}

Itu adalah fungsi animasi sederhana, dan saya menggunakannya Thread.Sleep.

Kesimpulan saya, jika itu berhasil, gunakanlah.

Bagi Anda yang belum melihat satu pun argumen valid yang menentang penggunaan Thread.Sleep di SKENARIO 2, sebenarnya ada satu - keluar aplikasi ditahan oleh while loop (SKENARIO 1/3 hanya bodoh jadi tidak layak untuk lebih menyebutkan)

Banyak yang berpura-pura menjadi tahu, berteriak Thread.Sleep is evil gagal menyebutkan satu alasan yang sah bagi kita yang menuntut alasan praktis untuk tidak menggunakannya - tapi ini dia, terima kasih kepada Pete - Thread.Sleep itu Jahat (dapat dengan mudah dihindari dengan timer / handler)

    static void Main(string[] args)
    {
        Thread t = new Thread(new ThreadStart(ThreadFunc));
        t.Start();

        Console.WriteLine("Hit any key to exit.");
        Console.ReadLine();

        Console.WriteLine("App exiting");
        return;
    }

    static void ThreadFunc()
    {
        int i=0;
        try
        {
            while (true)
            {
                Console.WriteLine(Thread.CurrentThread.ThreadState.ToString() + " " + i);

                Thread.Sleep(1000 * 10);
                i++;
            }
        }
        finally
        {
            Console.WriteLine("Exiting while loop");
        }
        return;
    }