[C#][VS2008]Ping programma

[martijn00]

Hallo,

Ik wil een Ping programma maken.

Hier een voorbeeld van hoe ik denk dat het moet worden.(Zie ook plaatje)

Alleen als ik die code gebruik doet het programma het niet. Ik heb wel tussen de comments die er onder staan deze code gevonden:

C#:

 
System.Net.NetworkInformation.Ping ping1 = new System.Net.NetworkInformation.Ping(); 
System.Net.NetworkInformation.PingReply reply1; 
try { 
reply1 = ping1.Send("MachineName"); 
MessageBox.Show(reply1.Address.ToString()); 
} 
catch (Exception ex) { 
MessageBox.Show(ex.Message.ToString); 
} 

Dat leek mij een stuk logischer, ik heb het dus verandert naar mijn eigen code:

C#:

 
            System.Net.NetworkInformation.Ping ping1 = new System.Net.NetworkInformation.Ping();
            System.Net.NetworkInformation.PingReply reply1;

            try
            {
                reply1 = ping1.Send(tbIPnummer.Text);
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.ToString());
            }

Als ik dit Debug lijkt hij het ook wel te doen, alleen ik krijg niks te staan in mijn Textbox(zie plaatje)
Wat er dus zou moeten gebeuren is dat er bijv. komt te staan: Ping started for...


Ik heb geen idee hoe dat zou moeten, dus als jullie ideen hebben... ik heb al wel op google enzo gezocht maar daar staan alleen hele grote lappen code om zoiets te maken. Ik wil het wel een beetje simpel houden, wat zo goed ben ik niet.(Student ICT, die nog veel moet leren)

[ Voor 7% gewijzigd door martijn00 op 21-02-2008 16:53 ]

[GhandiHarder]

http://gathering.tweakers.net/forum/list_messages/583405//

http://gathering.tweakers.net/forum/faq/ubb#tag_code

[TeeDee]

martijn00 schreef op donderdag 21 februari 2008 @ 16:27:
Hallo,

Hoi

Je zal je Ping uit moeten voeren in een andere thread. Dat heeft (op het eerste oog) de niet werkende code van CodeProject ook.

(Trouwens hoe maak je zo'n dingetje waar je je code in kan zetten?

[code=c#] //hier je code[/code]

[Hydra]

Je zet de text van je EXCEPTION in je message box? Logisch dat je daar dan niks te zien krijgt, het lijkt mij dat bij normaal gebruik geen exception optreedt?

TeeDee schreef op donderdag 21 februari 2008 @ 16:35:
Je zal je Ping uit moeten voeren in een andere thread. Dat heeft (op het eerste oog) de niet werkende code van CodeProject ook.

Waarom zou dat moeten?

[ Voor 46% gewijzigd door Hydra op 21-02-2008 16:36 ]

[sig69]

Wat je wil doen:
-pingen in een loop zetten (liefst in een andere thread, of dmv een timer. Een timer is makkelijker denk ik)
-per ping iets in de textbox zetten
-eventueel totalen bijhouden om statistieken te berekenen (min, max, etc)

[ Voor 11% gewijzigd door sig69 op 21-02-2008 16:39 ]

[TeeDee]

Hydra schreef op donderdag 21 februari 2008 @ 16:35:
[...]
Waarom zou dat moeten?

Bij voorkeur wel imo. Als TS tekstuele voortgang wil laten zien in een Textbox wel.

[Hydra]

C#:

        private void button1_Click(object sender, EventArgs e)
        {
            Ping ping;
            PingReply reply;
            ping = new Ping();
            reply = ping.Send("127.0.0.1");
            textBox1.Text = string.Format("Pinged 127.0.0.1, time: {0}ms, status: {1}", reply.RoundtripTime, reply.Status);
        }

Werkt als een tiet.

TeeDee schreef op donderdag 21 februari 2008 @ 16:40:
Bij voorkeur wel imo. Als TS tekstuele voortgang wil laten zien in een Textbox wel.

Hij moet eerst de basics leren voordat hij aan threading begint, zoveel lijkt me duidelijk.

[ Voor 27% gewijzigd door Hydra op 21-02-2008 16:42 ]

[Sebazzz]

TeeDee schreef op donderdag 21 februari 2008 @ 16:40:
[...]

Bij voorkeur wel imo. Als TS tekstuele voortgang wil laten zien in een Textbox wel.

Plus dat de GUI blijft reageren en de operatie afgebroken kan worden.
Een backgroundworker is hier zeer geschikt voor

[Hydra]

Typisch dat mensen met workerthreads gaan smijten als de TS gewoon vergeet het resultaat van de ping in een textbox te proppen...

[martijn00]

sig69 schreef op donderdag 21 februari 2008 @ 16:38:
Wat je wil doen:
-pingen in een loop zetten (liefst in een andere thread, of dmv een timer. Een timer is makkelijker denk ik)
-per ping iets in de textbox zetten
-eventueel totalen bijhouden om statistieken te berekenen (min, max, etc)

Inderdaad gebruikte ik al een Timer, en zoals je in het plaatje zou kunnen zien hou ik het totaal ook bij.

Hydra schreef op donderdag 21 februari 2008 @ 16:41:

C#:

        private void button1_Click(object sender, EventArgs e)
        {
            Ping ping;
            PingReply reply;
            ping = new Ping();
            reply = ping.Send("127.0.0.1");
            textBox1.Text = string.Format("Pinged 127.0.0.1, time: {0}ms, status: {1}", reply.RoundtripTime, reply.Status);
        }

Werkt als een tiet.


[...]


Hij moet eerst de basics leren voordat hij aan threading begint, zoveel lijkt me duidelijk.

Dat werkt niet, de fout is: 'Ping' is a 'namespace' but is used like a 'type'

[TeeDee]

Heb je nu ook using System.Net.NetworkInformation bovenin je code staan?
Of heb je de code van Hydra echt zo overgenomen?

[martijn00]

Hij doet het nu, dit is de code:

C#:

 
System.Net.NetworkInformation.Ping ping1 = new System.Net.NetworkInformation.Ping();
            System.Net.NetworkInformation.PingReply reply1;

            reply1 = ping1.Send(tbIPnummer.Text);
            tbStatus.Text = string.Format("Pinged {0}, time: {0}ms, status: {1}", tbIPnummer.Text, reply1.RoundtripTime, reply1.Status);

Alleen de tijd gaat nu niet goed omdat daar het IP komt te staan en de status doet het ook niet, maar ik denk dat er met die status bedoelt wordt hoe vaak hij heeft gepingt. Dat kan ik ook wel anders doen, omdat het op een timer.tick event gebeurd.

C#:

 
int Aantal = 0;
Aantal++;
tbStatus.Text = Aantal.ToString();

Zoiets wordt het dan.

[ Voor 24% gewijzigd door martijn00 op 21-02-2008 17:13 ]

[Hydra]

martijn00 schreef op donderdag 21 februari 2008 @ 16:51:

Dat werkt niet, de fout is: 'Ping' is a 'namespace' but is used like a 'type'

Gast...

Je moet natuurlijk wel even de using statement bovenin je code zetten. Leer eerst eens de basics zeg...

martijn00 schreef op donderdag 21 februari 2008 @ 17:06:
Hij doet het nu, dit is de code:

Alleen de tijd gaat nu niet goed omdat daar het IP komt te staan

En ga ff de docs behorende bij string.format lezen

Mensen hier helpen je graag, maar NIET als je niet eerst zelf zoekt naar een oplossing.

[ Voor 37% gewijzigd door Hydra op 21-02-2008 17:08 ]

[momania]

martijn00 schreef op donderdag 21 februari 2008 @ 17:06:
Alleen de tijd gaat nu niet goed omdat daar het IP komt te staan

Kijk nog eens goed naar die formatting string dan.

Staat ook twee keer {0} in

[martijn00]

Hydra schreef op donderdag 21 februari 2008 @ 17:07:
[...]


Gast...

Je moet natuurlijk wel even de using statement bovenin je code zetten. Leer eerst eens de basics zeg...


[...]


En ga ff de docs behorende bij string.format lezen

Mensen hier helpen je graag, maar NIET als je niet eerst zelf zoekt naar een oplossing.

Rustig zeg, ik kwam er zelf ook al achter maar dat was nadat ik gepost had

[Hydra]

martijn00 schreef op donderdag 21 februari 2008 @ 17:15:
Rustig zeg, ik kwam er zelf ook al achter maar dat was nadat ik gepost had

Eerst denken dan posten dus.

[farlane]

Hydra schreef op donderdag 21 februari 2008 @ 16:46:
Typisch dat mensen met workerthreads gaan smijten als de TS gewoon vergeet het resultaat van de ping in een textbox te proppen...

http://stal.blogspot.com/2006/08/hammer-and-nail.html

[martijn00]

Zou het ook mogelijk zijn om als je bijv. tweakers.net invult dat je dan daar naar toe gaat pingen. Dit lukt me nog niet, als ik dan het IP-adres van tweakers invul dan lukt het wel.

[Hydra]

C#:

        private void button1_Click(object sender, EventArgs e)
        {
            Ping ping;
            PingReply reply;
            ping = new Ping();
            reply = ping.Send("gathering.tweakers.net");
            textBox1.Text = string.Format("Pinged gathering.tweakers.net time: {0}ms, status: {1}", reply.RoundtripTime, reply.Status);
        }

Pinged gathering.tweakers.net time: 4ms, status: Success

Wat "werkt niet"?

[Raven]

"tweakers.net" valt dacht ik niet te pingen, of wel?

[Verwijderd]

code:

Microsoft Windows XP [Version 5.1.2600]
(C) Copyright 1985-2001 Microsoft Corp.

C:\Documents and Settings\Freerk>ping tweakers.net

Pinging tweakers.net [213.239.154.35] with 32 bytes of data:

Reply from 213.239.154.35: bytes=32 time=11ms TTL=56
Reply from 213.239.154.35: bytes=32 time=17ms TTL=56
Reply from 213.239.154.35: bytes=32 time=19ms TTL=56
Reply from 213.239.154.35: bytes=32 time=17ms TTL=56

Ping statistics for 213.239.154.35:
    Packets: Sent = 4, Received = 4, Lost = 0 (0% loss),
Approximate round trip times in milli-seconds:
    Minimum = 11ms, Maximum = 19ms, Average = 16ms

C:\Documents and Settings\Freerk>

[Raven]

Hmm, hing me iets van bij namelijk
Zal wel andere site geweest zijn...

[ Voor 33% gewijzigd door Raven op 21-02-2008 19:04 ]

[Hydra]

bartgrefte schreef op donderdag 21 februari 2008 @ 19:04:
Hmm, hing me iets van bij namelijk
Zal wel andere site geweest zijn...

Je kunt het natuurlijk gewoon eerst ff proberen

[martijn00]

martijn00 schreef op donderdag 21 februari 2008 @ 18:02:
Zou het ook mogelijk zijn om als je bijv. tweakers.net invult dat je dan daar naar toe gaat pingen. Dit lukt me nog niet, als ik dan het IP-adres van tweakers invul dan lukt het wel.

Laat maar zitten, ik denk dat mijn internet er tijdelijk even uit lag, want nu doet hij het wel goed. Hij vertaald alleen niet tweakers.net naar een ip-adress, visueel dan.

[ Voor 8% gewijzigd door martijn00 op 22-02-2008 10:26 ]