119

When I Ping a remote system with windows it says there is no reply, but when I ping with c# it says success. Windows is correct, the device is not connected. Why is my code able to successfully ping when Windows is not?

Here is my code :

Ping p1 = new Ping();
PingReply PR = p1.Send("192.168.2.18");
// check when the ping is not success
while (!PR.Status.ToString().Equals("Success"))
{
    Console.WriteLine(PR.Status.ToString());
    PR = p1.Send("192.168.2.18");
}
// check after the ping is n success
while (PR.Status.ToString().Equals("Success"))
{
    Console.WriteLine(PR.Status.ToString());
    PR = p1.Send("192.168.2.18");
}
vcsjones
  • 138,677
  • 31
  • 291
  • 286
Black Star
  • 1,201
  • 2
  • 9
  • 6
  • 5
    Check out the following example posted at the bottom of this page when you click on the MSDN Link http://msdn.microsoft.com/en-us/library/system.net.networkinformation.ping.aspx or http://stackoverflow.com/questions/1281176/making-a-ping-inside-of-my-c-sharp-application – MethodMan Aug 03 '12 at 18:05
  • 12
    You should be comparing PR.Status to IPStatus.Success. String comparison is not the correct tool in this case. – Sam Axe Aug 03 '12 at 18:09
  • After you perform your ping, what are the values of some of the PingReply properties, (like `PR.Address`, `PR.RoundtripTime`, `PR.reply.Buffer.Length`, and `PR.Options.Ttl`)? Also are you sure you have the correct IP address in your code, and not a test IP address? – Jon Senchyna Aug 03 '12 at 18:25
  • Jon Senchyna : I don't set them and yes I'm sure my IP is correct . – Black Star Aug 04 '12 at 10:55
  • In my case, if the "enable the visual studio hosting process" (location is ==>>project-> property->debug)disabled, the ping method may not work. please try! – steve Jul 28 '16 at 23:25

5 Answers5

266
using System.Net.NetworkInformation;    

public static bool PingHost(string nameOrAddress)
{
    bool pingable = false;
    Ping pinger = null;

    try
    {
        pinger = new Ping();
        PingReply reply = pinger.Send(nameOrAddress);
        pingable = reply.Status == IPStatus.Success;
    }
    catch (PingException)
    {
        // Discard PingExceptions and return false;
    }
    finally
    {
        if (pinger != null)
        {
            pinger.Dispose();
        }
    }

    return pingable;
}
JamieSee
  • 12,696
  • 2
  • 31
  • 47
  • 17
    This is a **code only answer**. I guess it implements a correct comparison and shows how to handle possible exceptions. Could you indicate *why* this is the correct code compared with the code in the question? – Maarten Bodewes Nov 29 '18 at 22:27
  • 13
    Don't know how many people have used this answer by copy and paste :/ Do at least a `using (var pinger = new Ping()) { .. }` and are early returns so evil? – Peter Schneider Jan 31 '19 at 17:56
  • 3
    There's no point in wrapping the Ping instance with a using if try/catch/finally is employed properly. It's one or the other not both. See https://stackoverflow.com/questions/278902/using-statement-vs-try-finally. – JamieSee Feb 01 '19 at 18:05
  • 10
    @JamieSee While that may be true, it's cleaner to use `using` instead, and is preferred in this case. – Kelly Elton Apr 16 '19 at 23:42
  • I personally agree with using a using statement but you still have to catch any exceptions. – SteveB Aug 27 '19 at 17:41
  • 1
    two up-votes for showing a try-catch block and declaring variables outside of the 'try' block so the immediate window can see them if you breakpoint the 'catch', but one down vote for not actually handling the exception. – B H Jan 15 '20 at 16:21
53

Using ping in C# is achieved by using the method Ping.Send(System.Net.IPAddress), which runs a ping request to the provided (valid) IP address or URL and gets a response which is called an Internet Control Message Protocol (ICMP) Packet. The packet contains a header of 20 bytes which contains the response data from the server which received the ping request. The .Net framework System.Net.NetworkInformation namespace contains a class called PingReply that has properties designed to translate the ICMP response and deliver useful information about the pinged server such as:

  • IPStatus: Gets the address of the host that sends the Internet Control Message Protocol (ICMP) echo reply.
  • IPAddress: Gets the number of milliseconds taken to send an Internet Control Message Protocol (ICMP) echo request and receive the corresponding ICMP echo reply message.
  • RoundtripTime (System.Int64): Gets the options used to transmit the reply to an Internet Control Message Protocol (ICMP) echo request.
  • PingOptions (System.Byte[]): Gets the buffer of data received in an Internet Control Message Protocol (ICMP) echo reply message.

The following is a simple example using WinForms to demonstrate how ping works in c#. By providing a valid IP address in textBox1 and clicking button1, we are creating an instance of the Ping class, a local variable PingReply, and a string to store the IP or URL address. We assign PingReply to the ping Send method, then we inspect if the request was successful by comparing the status of the reply to the property IPAddress.Success status. Finally, we extract from PingReply the information we need to display for the user, which is described above.

    using System;
    using System.Net.NetworkInformation;
    using System.Windows.Forms;

    namespace PingTest1
    {
        public partial class Form1 : Form
        {
            public Form1()
            {
                InitializeComponent();
            }

            private void button1_Click(object sender, EventArgs e)
            {
                Ping p = new Ping();
                PingReply r;
                string s;
                s = textBox1.Text;
                r = p.Send(s);

                if (r.Status == IPStatus.Success)
                {
                    lblResult.Text = "Ping to " + s.ToString() + "[" + r.Address.ToString() + "]" + " Successful"
                       + " Response delay = " + r.RoundtripTime.ToString() + " ms" + "\n";
                }
            }

            private void textBox1_Validated(object sender, EventArgs e)
            {
                if (string.IsNullOrWhiteSpace(textBox1.Text) || textBox1.Text == "")
                {
                    MessageBox.Show("Please use valid IP or web address!!");
                }
            }
        }
    }
David
  • 5
  • 4
Ashraf Sada
  • 4,527
  • 2
  • 44
  • 48
1
private async void Ping_Click(object sender, RoutedEventArgs e)
{
    Ping pingSender = new Ping();
    string host = @"stackoverflow.com";
    await Task.Run(() =>{
         PingReply reply = pingSender.Send(host);
         if (reply.Status == IPStatus.Success)
         {
            Console.WriteLine("Address: {0}", reply.Address.ToString());
            Console.WriteLine("RoundTrip time: {0}", reply.RoundtripTime);
            Console.WriteLine("Time to live: {0}", reply.Options.Ttl);
            Console.WriteLine("Don't fragment: {0}", reply.Options.DontFragment);
            Console.WriteLine("Buffer size: {0}", reply.Buffer.Length);
         }
         else
         {
            Console.WriteLine("Address: {0}", reply.Status);
         }
   });           
}
Meysam Asadi
  • 6,438
  • 3
  • 7
  • 17
0
Imports System.Net.NetworkInformation


Public Function PingHost(ByVal nameOrAddress As String) As Boolean
    Dim pingable As Boolean = False
    Dim pinger As Ping
    Dim lPingReply As PingReply

    Try
        pinger = New Ping()
        lPingReply = pinger.Send(nameOrAddress)
        MessageBox.Show(lPingReply.Status)
        If lPingReply.Status = IPStatus.Success Then

            pingable = True
        Else
            pingable = False
        End If


    Catch PingException As Exception
        pingable = False
    End Try
    Return pingable
End Function
Iamsodarncool
  • 663
  • 5
  • 9
-11
private void button26_Click(object sender, EventArgs e)
{
    System.Diagnostics.ProcessStartInfo proc = new System.Diagnostics.ProcessStartInfo();
    proc.FileName = @"C:\windows\system32\cmd.exe";
    proc.Arguments = "/c ping -t " + tx1.Text + " ";
    System.Diagnostics.Process.Start(proc);
    tx1.Focus();
}

private void button27_Click(object sender, EventArgs e)
{
    System.Diagnostics.ProcessStartInfo proc = new System.Diagnostics.ProcessStartInfo();
    proc.FileName = @"C:\windows\system32\cmd.exe";
    proc.Arguments = "/c ping  " + tx2.Text + " ";
    System.Diagnostics.Process.Start(proc);
    tx2.Focus();
}
krlzlx
  • 5,752
  • 14
  • 47
  • 55
  • 3
    Can't you just write a few lines and explain your code? Couse this is not useful for people who want to understand this piece of code... – Hille Mar 27 '18 at 07:57
  • The code originally presented is [not DRY at all](https://en.m.wikipedia.org/wiki/Don't_repeat_yourself). – greybeard Dec 05 '20 at 09:47