Schönen guten Abend,
ich habe hier eine Windows-Forms-Anwendung und die dazu gedacht ist, IP-Adressen der Reihe nach unzupingen um so den Status des jeweiligen Rechners festzustellen. Leider taugt das Programm in der Langzeit nichts, da immer wenn man das Programm 10 - 20 Minuten
alleine laufen lässt, folgendes Problem auftritt:
Nach dem Start läuft das Programm ganz normal, wenig CPU Auslastung und keinerlei Einschränkungen in der Funktion.
Nach den oben angegebenen 10 bis 20 Minuten, fällt es dem Programm schwerer die Einträge in die ListBox zu schreiben, was heißt die CPU-Auslastung geht hoch und die Einträge fangen an zu flimmern, allgemein verhält sich die gesamte
Anwendung nur noch träge. Mit zunehmender Zeit wird es auch immer schlimmer, bis schließlich fast gar nichts mehr läuft. Ich wollte davon ein Screenshot machen, aber leider habe ich es nicht drauf bekommen, sodass man es richtig sieht.
Jetzt noch was zum Code:
Der Klick auf den Startbutton startet einen Timer, der Timer wiederum startet immer wieder einen Backgroundworker, der wiederum im Hintergrund das anpingen übernimmt. Habe ihn hier mal angehängt:
private void PingWorker_DoWork(object sender, DoWorkEventArgs e)
{
BackgroundWorker worker = sender as BackgroundWorker;
// Liste der Hosts als Argument
string[] hostNames = e.Argument as string[];
if (hostNames != null && hostNames.Length != 0)
{
foreach (string hostname in hostNames)
{
// Abbrechen...
if (worker.CancellationPending)
{
e.Cancel = true;
break;
}
if (!String.IsNullOrEmpty(hostname))
{
Functions functions = new Functions();
if ( functions.IsIP(hostname) == true)
{
IP_Hostname iphost = new IP_Hostname();
hostnamefromip = iphost.IP_zu_Hostname(hostname);
}
else
{
hostnamefromip = hostname;
}
// Host abfragen...
string message = null;
try
{
using (Ping pingSender = new Ping())
{
reply = pingSender.Send(hostname, PingTimeout);
if (reply.Status == IPStatus.Success && showonlyoffline.Checked != true)
{
DateTime currentDate = DateTime.Now;
successfull = 1;
message = String.Format("[{0}:{1}:{2}]{3}({4}): {5} ms",
currentDate.Hour, currentDate.Minute, currentDate.Second, hostname, hostnamefromip, reply.RoundtripTime);
}
else
if (reply.Status == IPStatus.Success)
{
message = "";
}
else
{
DateTime currentDate = DateTime.Now;
successfull = 2;
message = String.Format("[{0}:{1}:{2}]{3}({4}): {5} ms",
currentDate.Hour, currentDate.Minute, currentDate.Second, hostname, hostnamefromip, reply.RoundtripTime);
}
}
}
catch (Exception)
{
DateTime currentDate = DateTime.Now;
message = String.Format("{0}: Ungültiges Zeichen!",
hostname);
successfull = 3;
message = String.Format("[{0}:{1}:{2}]{3}({4}): TimedOut",
currentDate.Hour, currentDate.Minute, currentDate.Second, hostname, hostnamefromip);
}
// Aktualisieren hier als Text
this.pingListBox.DrawItem += new DrawItemEventHandler(pingListBox_DrawItem);
worker.ReportProgress(hostNames.Length / 100 + 1, message);
}
}
}
if (cb_repeatping.Checked == true)
{
System.Threading.Thread.Sleep(repeatvalue);
}
else
{
t.Stop();
}
}
private void PingWorker_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
{
//this.cancelButton.Enabled = false;
this.startButton.Enabled = true;
this.label1.Text = "Status: Durchlauf beendet";
}
private void PingWorker_ProgressChanged(object sender, ProgressChangedEventArgs e)
{
string message = e.UserState as string;
// Anzeige der Nachricht in einer ListBox
if (!String.IsNullOrEmpty(message))
{
this.pingListBox.Items.Add(new MyListBoxItem(message, successfull));
pingListBox.TopIndex = pingListBox.Items.Count - 1;
}
if (pingListBox.Items.Count >= maxlistboxentrys)
{
pingListBox.Items.RemoveAt(0);
}
}
private void t_Tick(object sender, EventArgs e)
{
// Startet die Abfrage über einen BackgroundWorker
if (!this.pingWorker.IsBusy)
{
// Einlesen der Hostnamen in ein Array
string[] hostNames = System.IO.File.ReadAllLines(HostNames);
// Asynchron abrufen
this.pingWorker.RunWorkerAsync(hostNames);
label1.Text = "Status: Durchlauf gestartet";
}
}
public void startButton_Click(object sender, EventArgs e)
{
if (tb_maxlistboxentrys.Text == "" || tb_maxlistboxentrys.Text == "0")
{
tb_maxlistboxentrys.Text = "16";
}
else
{
t.Start();
this.cancelButton.Enabled = true;
this.label1.Text = "Status: Durchlauf gestartet";
}
}
So, ich hoffe ihr könnt mir helfen, habe schon verschiedenes probiert, z.B. Die ListBox Einträge auf ein Maximum zu setzen, weil ich dachte das die irgendwann zu voll wird und "verstopft" aber daran lag es nicht. Nun habe ich mir vielleicht
überlegt, ob es helfen würde nach jedem Ping 200ms zu warten oder so.
Oder muss ich den Backgroundworker etc. irgendwie wieder "closen" bevor er erneut aufgerufen wird?
Falls ich etwas vergessen habe, tut mir leid, dann werde ich es nachtragen!
RunTime
