[C#] AutoResetEvent Freezing application!

[litewarez]

Thanks hyperz, Ill give it a go after work.. i know what the problem is, i just font know how to fix

when im doing a for loop it sends a command to SendAsync witch in turn starts to do a ping, but because of the for lool the 2 time comes around while the first time is stil running :/.

And yea that 32 byte char i knew it some some thing like that. Thanks

[Hyperz]

Then use the normal Send in a loop in a new thread.

[litewarez]

I did try that i think

int i;
for(i=0;i<=5;i++)
{
    Action Action = new Action(() => pingSender.SendAsync(addressBox.Text,120) )
    new Thread(Action).Start();
} 

But still frezed and said that i cannot run another ping request until the previous one has finished

[Hyperz]

Loop inside a new thread, not new threads inside a loop. You only need one thread for this.

Edit: and still you call SendAsync instead of Send...
Edit2: you forgot a ";"

[litewarez]

i get you, and i havent got C# at work, they wont let me upgrade .NET on the network because they havent checked it or w.e,

Ill try when i get home..

And just for clarification you mean like

private void StartButton_Click(......)
{
    ..........

    Action Action = new Action(() => ThreadRunner()); 
    new Thread(Action).Start();
}

public void ThreadRunner()
{
    for(i=0;i<=5;i++)
    {
        pingSender.SendAsync(addressBox.Text,120);
    }
} 

[Hyperz]

private void StartButton_Click(......)
{
    ..........

    new Thread(ThreadRunner).Start();
}

public void ThreadRunner()
{
    for(i=0;i<=5;i++)
    {
        pingSender.Send(addressBox.Text,120);
    }
}

[litewarez]

Thankuuuuu ill give it a go soon as.

if you was here id kiss you...... maybe not

[litewarez]

Ok does some testing and still not working, it seems to be the callback

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;
using System.Net.NetworkInformation;
using System.Threading;

namespace SimplePinger
{
    public partial class Form1 : Form
    {
        #region Variables

        int PingAmount = 5;
        Ping pingSender = new Ping();
        PingOptions options = new PingOptions(64, true);
        byte[] buffer = new byte[32];

        #endregion

        public Form1()
        {
            InitializeComponent();
            updatePingValue(5); //Set defualt to 5

            /*Ping Options*/
            pingSender.PingCompleted += new PingCompletedEventHandler(PingCallback);
        }

        private void PingTimes_Scroll(object sender, EventArgs e)
        {
            updatePingValue(PingTimes.Value);
        }

        private void updatePingValue(int Value)
        {
            PingAmount = PingTimes.Value;
            PingAmountLabel.Text = "Pings to send: " + PingAmount;
        }

        private void startPing_Click(object sender, EventArgs e)
        {
            outputBox.Clear(); /*Fine Here*/
            /*
             * Start the thread to run the InitiateThreadPinger()
             */
            new Thread(InitiateThreadPinger).Start();
        }

        public void InitiateThreadPinger()
        {
            /*
             * Loop 1-10 and run a ping on the host
             */
            options.DontFragment = true;
            for (int i = 0; i <= PingAmount; i++)
            {
                /*
                 * Send the pingSender the Send command with options, Callback attached in Main()
                 * The code in here Executes the correct amount of times as i get the Test below
                 * The problem seems to be that the ping does not execute the callback, Cross Thread Maybe ? ( NO Error Triggered )
                 */
                //MessageBox.Show("Test");
                pingSender.Send("127.0.0.1", 120, buffer, options);
            }
        }

        public void PingCallback(object sender, PingCompletedEventArgs e)
        {
            if (e.Cancelled)
            {
                EnableFormElements();
            }
            if (e.Error != null)
            {
                MessageBox.Show("Ping Failed:" + e.Error.ToString());
                return;
            }

            PingReply reply = e.Reply;
            AddCallbackToForm(reply);
        }

        public void AddCallbackToForm(PingReply Reply)
        {
            if (Reply == null)
            {
                return;
            }
            if (Reply.Status == IPStatus.Success)
            {
                outputBox.Text += "Success: " + Reply.Address + " - " + Reply.RoundtripTime + "ms";
            }
            else 
            {
                outputBox.Text += "Failed...";
            }
        }

        public void EnableFormElements()
        {
            startPing.Enabled = true;
            PingTimes.Enabled = true;
        }

        public void DisableFormElements()
        {
            startPing.Enabled = false;
            PingTimes.Enabled = false;
        }

        private void Form1_Load(object sender, EventArgs e)
        {

        }
    }
} 

ive left some comments in there and filtered it down to the callback i get no errors so its hard for me to track

[Hyperz]

Use an Action + this.Invoke() in your PingCallback. You should also use break points in visual studio for debugging instead of the javascript-ish MessageBox.Show().

[litewarez]

using the break points the code is not even executing the callback

If you wwant to see the source you may be able to see the problem straight away, the only reason im making this is a leaning curve. im not just sponging your knowledge bud.

http://rapidshare.com/files/405979545/SimplePinger.rar