C# Thread / Involke

SrONight · 12. Februar 2014

Hey, ich habe ein Problem mit einen threadübergreifenden Vorgang.
Ich möchte ein Panel hinzufügen. Das mache ich mit SplitContailer.Panel2.Controls.Add(pa[i, a]); //pa ist ein Array aus Panels
Klappt super, jetzt möchte ich es bisschen smoother gestanden und es mit Threads umsetzten.
Er sagt mir "Ungültiger threadübergreifender Vorgang", deswegen habe ich versucht es mit dem Snippet von florian0 zu umgehen (LINK)
Jetzt bekomme ich den Fehler: MainForm enthält bereits eine Definition für SetPanelControllAdd...
Vielleicht könnt ihr mir weiterhelfen

Code:

Code

private delegate void SetPanelControllAdd(Panel pa);
private void SetPanelControllAdd(Panel pa){
// Invoke nötig?
if (pa.InvokeRequired)
{
// Invoke nötig
pa.Invoke(new SetPanelControllAdd(pa));
}
else
{
// Kein Invoke nötig - Vorgang sicher durchführbar
SplitContailer.Panel2.Controls.Add(pa);
}
}
private void createFrame(){
pa = new Panel[cf.breite, cf.höhe];
int x = 0;
int y = 0;
for (int i = 0; i < cf.breite; i++)
{
for (int a = 0; a < cf.höhe; a++)
{
pa[i, a] = new Panel();
pa[i, a].Visible = true;
pa[i, a].BackColor = Color.Black;
pa[i, a].Width = 40;
pa[i, a].Height = 40;
pa[i, a].Location = new Point(x, y);
pa[i, a].BorderStyle = BorderStyle.FixedSingle;
pa[i, a].MouseHover += new EventHandler(Panels_Hover);
pa[i, a].Click += new EventHandler(Panels_Click);
//SplitContailer.Panel2.Controls.Add(pa[i, a]); Hier der Fehler mit dem Threadübegreifenden
SetPanelControllAdd(pa[i, a]); //Hier der 2te Fehler
x += 40;
}
y += 40;
x = 0;
}
}

Alles anzeigen

MfG Waschi

Euphi · 12. Februar 2014

Ich mach das ein wenig anders. Hast du Skype?

SrONight · 12. Februar 2014

Klar, siehe Profil
Aber mein Headset ist immoment leider kaputt und noch keine Zeit was neues zu kaufen

Euphi · 12. Februar 2014

Nicht schlimm, hab meins auch nicht hier

SrONight · 12. Februar 2014

Besten dank für die Hilfe !!!!! Von dir kann man bestimmt noch was lernen

florian0 · 12. Februar 2014

Ich kläre einfach mal auf was der Fehler gewesen wäre, keine Ahnung wie Ihr das nun gelöst habt ...

Man definiert einen Delegat, also einen Funktionsprototypen mit Parameterliste. Wie der heißt, ist egal. Er sollte nur nicht so heißen, wie die Funktion, die man aufrufen möchte (was du getan hast).

Code

private delegate void SetPanelControlAddCallback(Panel pa);
private void SetPanelControlAdd(Panel pa){
// ...
}

Das wäre erstmal der Basis-Code.

Im Endeffekt war mein Snippet auch falsch, dass das bisher niemandem aufgefallen ist ...
Man ruft eigentlich new DelegateName(Funktionsname) auf. Und nicht newDelegatName(ParameterDerFunktion). Ich habe da wohl 2 Lösungsansätze gemischt ....

Code

// Invoke nötig?
if (pa.InvokeRequired)
{
// Invoke nötig
pa.Invoke(new SetPanelControlAddCallback(SetPanelControlAdd), pa);
}
else
{
// Kein Invoke nötig - Vorgang sicher durchführbar
SplitContailer.Panel2.Controls.Add(pa);
}

Alles anzeigen

Das wärs prinzipell gewesen. Allerdings willst du ja sicher auf SplitContailer.Panel2 zugreifen. Und nicht auf dein Panel pa.
Daher solltest du eher InvokeRequired und Invoke von SplitContailer.Panel2 verwenden, statt von pa.

Gruß
florian0

SrONight · 13. Februar 2014

Zitat

Man definiert einen Delegat, also einen Funktionsprototypen mit Parameterliste. Wie der heißt, ist egal. Er sollte nur nicht so heißen, wie die Funktion, die man aufrufen möchte (was du getan hast).

Das ist mir im Nachhinein auch eingefallen xD

Zitat

Das wärs prinzipell gewesen. Allerdings willst du ja sicher auf SplitContailer.Panel2 zugreifen. Und nicht auf dein Panel pa.
Daher solltest du eher InvokeRequired und Invoke von SplitContailer.Panel2 verwenden, statt von pa.

Ja da war ich selber wohl gerade Brainafk xD

Euphi hat das ganze so gelöst:
Das Delegat fällt oben werg.

Code

private void SetPanelControlAdd(Panel pa)
{
// Invoke nötig?
if (SplitContailer.Panel2.InvokeRequired)
{
// Invoke nötig
SplitContailer.Panel2.Invoke(new Action(
delegate
{
SplitContailer.Panel2.Controls.Add(pa);
}));
}
else
{
// Kein Invoke nötig - Vorgang sicher durchführbar
SplitContailer.Panel2.Controls.Add(pa);
}
}

Alles anzeigen

Danke für die ganze Hilfe hier

Teilen