c# thread sleep sending not respond

I Will Run This Ping App UI Not Working And Not Respond

Here Is Code

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System.Net.NetworkInformation;
using System.Threading;
namespace WindowsFormsApp1
{
public partial class Form1 : Form
{
public static int check = 0;
static System.Windows.Forms.Timer myTimer = new System.Windows.Forms.Timer();
public Form1()
{
InitializeComponent();
}
private void button1_Click(object sender, EventArgs e)
{
ping();
}
public void ping()
{
int q = Convert.ToInt32(packet1.Text);
check = 1;
int milliseconds = 500;
byte[] packet = new byte[q];
Ping myPing = new Ping();
PingReply reply;
while (true)
{
Thread.Sleep(milliseconds);
reply = myPing.Send(ping1.Text, milliseconds, packet);
if (reply.Status == IPStatus.Success)
{
result1.AppendText("Reply from = " + reply.Address + " Bytes = " + reply.Buffer.LongLength + " Time = " + reply.RoundtripTime + " TTL = " + reply.Options.Ttl);
result1.AppendText(Environment.NewLine);
}
else
{
result1.AppendText(reply.Status.ToString());
result1.AppendText(Environment.NewLine);
}
}
}

3 Replies

Know the answer? Post it — somebody with the same question will find it here.

Guest User5y ago
Amit Gupta Can U Send Name Answer
Amit Gupta5y ago
Thread.Sleep is the culprit when it is used under while loop then its a disaster (freezes the UI). Avoid that unless you know what you are doing. I suggest you to use timer tick event with specific interval of (1 sec) and call the ping command.
Salman Beg5y ago
You can use timer in place of thread. Sleep. See this link, https://social.msdn.microsoft.com/Forums/vstudio/en-US/5b8d9dfd-e0b9-46e3-9ed1-89e7a9ffa1fd/threadsleep-application-not-responding-state?forum=csharpgeneral