Ungültiger Threadübergreifender Vorgang Backgroundworker

  • VB.NET
  • .NET (FX) 4.5–4.8

Es gibt 2 Antworten in diesem Thema. Der letzte Beitrag () ist von RodFromGermany.

    Ungültiger Threadübergreifender Vorgang Backgroundworker

    Hallo,
    ich bastel gerade ein Tool mit dem ich Bilder von einem Speichermedium direkt in einen Ordner kopieren kann Problem bei der Verwendung vom Backgroundworker für den Kopiervorgang und die Statusanzeige funktioniert nicht.
    Kopieren an sich geht wenn ich den die befehle für Label,Progressbar und co auskommentiere.

    Das ist der Background Worker

    Visual Basic-Quellcode

    1. Private Sub Kopiervorgang_DoWork(sender As System.Object, e As System.ComponentModel.DoWorkEventArgs) Handles Kopiervorgang.DoWork
    2. My.Computer.FileSystem.CreateDirectory(OrdnerPfad)
    4. For Each c As String In IO.Directory.GetFiles(KameraPfad, "*.*")
    5. d = d + 1
    6. Next
    7. Try
    8. Statusleiste.Maximum = d
    9. L_status.Text = "Dateien: " & d
    10. For Each s As String In IO.Directory.GetFiles(KameraPfad, "*.*")
    11. Dim fi As New IO.FileInfo(s)
    12. TB_status.AppendText("Kopiere: " & OrdnerPfad + "\" + fi.Name & " .... ")
    13. IO.File.Copy(s, OrdnerPfad + "\" + fi.Name)
    14. Statusleiste.Value += 1
    15. TB_status.AppendText("OK" & vbNewLine)
    16. Next
    17. Catch ex As Exception
    18. MsgBox(ex.Message)
    19. End Try
    20. End Sub

    Was muss ich da ändern ?

    DANKE für eure Hilfe!
    Bilder

    • sdgfs.JPG

      21,87 kB, 505×224, 185 mal angesehen

    Du nutzt VB.Net, nicht VB6 passe also Threadtag und Codetag entsprechend an.


    Als erstes würde ich dir mal raten dich in richtiges Multithreading einzuarbeiten also Threads erstellen, Aufgaben zuweisen und Async/Await kann auch nicht schaden.
    Der Backgroundworker ist nämlich eher ungeeignetes Threading.

    Beim Threading muss man GUI und Daten trennen, sonst geht das so in die Hose. Die GUI läuft in einem extra Thread, da von anderen Threads aus reinzugrabschen wird vom Programm nicht toleriert, wie das dann aussieht siehst du an der Exception.

    Zu lösen ist das ziemlich einfach, die Lösung heißt Invoke(bzw BeginInvoke/EndInvoke), dabei Invokest du vom Nebenthread in einem Control eine Methode, sodass diese Methode im GUI-Thread und nicht im Nebenthread ausgeführt wird. Wie Invoken geht kannst du bei MSDN und Google nachlesen. (Und halte dich von der CheckForIllegalCrossThreadCalls-Property fern, das ist kein korrektes Threading)

    Außerdem will ich zu deinem Snippet was anmerken:
    -Meide den My-Namespace, für alles außer Resources und Settings gibt es bessere Alternativen
    -Nutze Path.Combine
    -lade die Dateien in ein Array, hole dir da die Länge und gehe dieses Array durch, so wie das jetzt ist, ist das suboptimal

    LG

    paydayclown schrieb:

    ändern
    Wenn schon BackgroundWorker, dann richtig BackgroundWorker.
    Nutze die Properties und Methoden des BGW. Mach ne neue Form mit Button und BGW:
    Spoiler anzeigen

    VB.NET-Quellcode

    1. Public Class Form1
    2. Private Sub Button1_Click(sender As Object, e As EventArgs) Handles Button1.Click
    3. Me.BackgroundWorker1.WorkerReportsProgress = True
    4. Me.BackgroundWorker1.RunWorkerAsync()
    5. End Sub
    7. Private Sub BackgroundWorker1_DoWork(sender As Object, e As System.ComponentModel.DoWorkEventArgs) Handles BackgroundWorker1.DoWork
    8. Me.BackgroundWorker1.ReportProgress(42)
    9. End Sub
    11. Private Sub BackgroundWorker1_ProgressChanged(sender As Object, e As System.ComponentModel.ProgressChangedEventArgs) Handles BackgroundWorker1.ProgressChanged
    12. MessageBox.Show(e.ProgressPercentage.ToString)
    13. End Sub
    15. Private Sub BackgroundWorker1_RunWorkerCompleted(sender As Object, e As System.ComponentModel.RunWorkerCompletedEventArgs) Handles BackgroundWorker1.RunWorkerCompleted
    16. MessageBox.Show("Feddich")
    17. End Sub
    18. End Class
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!

Facebook
Telefonspende