Sporadische Fehler beim File umbenennen

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

Es gibt 8 Antworten in diesem Thema. Der letzte Beitrag () ist von Fakiz.

    Sporadische Fehler beim File umbenennen

    Moin moin

    Ich habe mit dieser Class / Function beim umbenennen einige "Sporadische" Fehler.
    Je nach Anzahl der Dateien in einem Verzeichnis bekomme ich die Meldung:

    ​Datei kann nicht erstellt werden, wenn sie schon vorhanden ist.

    Die Anzahl der Dateien wo dieser Fehler auftritt ist nicht fest, mal sind es 2 mal 4 usw... Ich habe dann versucht es mit eine kleinen "Wartezeit" bzgl der Task zu kompensieren, um zu testen ob es da Probleme gibt. Allerdings funktioniert das nur sehr begrenzt und so denke ich nicht die beste Lösung ist.

    Bei ​Await Task.Delay(100) passt es z.B. wenn in dem Verzeichnis nicht mehr als 200 Dateien sind. In einem anderen Testverzeichnis mit 1500 Dateien, passt ein Wert von 550 schon nicht mehr.

    Spoiler anzeigen
    ​

    VB.NET-Quellcode

    1. ​
    2. Public Class FileRenamer
    4. Private renamedFilesField As New List(Of FileInfo)()
    6. Public Property RenamedFiles As List(Of FileInfo)
    7. Get
    8. Return renamedFilesField
    9. End Get
    10. Private Set(ByVal value As List(Of FileInfo))
    11. renamedFilesField = value
    12. End Set
    13. End Property
    15. Public Property NewFileName As String
    16. Public Property RenamedFileName As String
    17. Public Property SuffixCounter As Integer = 0
    18. Public Property SuffixStartValue As Integer = 0
    19. Public Property SuffixLetter As String
    20. Public Property CountRenamedFiles As Integer
    21. Public Property DateofChange As Boolean
    23. Public Event ErrorOccurred(ByVal errorMessage As String)
    24. Public Event RenamedFileCountUpdated(ByVal count As Integer)
    25. Public Event RenamedFilesUpdated(ByVal files As List(Of FileInfo))
    27. Public ReadOnly Property RenamedFileInfo As List(Of FileInfo)
    28. Get
    29. Return renamedFilesField
    30. End Get
    31. End Property
    33. Public Async Function RenameFiles(ByVal selectedDirectory As String, ByVal files As List(Of String)) As Task
    34. Debug.WriteLine(selectedDirectory)
    35. RenamedFiles.Clear()
    36. CountRenamedFiles = 0
    38. Dim tasks As List(Of Task) = files.Select(Function(file) Task.Run(Function() RenameFileAsync(selectedDirectory, file))).ToList()
    39. Await Task.WhenAll(tasks)
    41. RaiseEvent RenamedFileCountUpdated(CountRenamedFiles)
    42. RaiseEvent RenamedFilesUpdated(RenamedFiles)
    43. End Function
    45. Private Async Function RenameFileAsync(ByVal selectedDirectory As String, ByVal file As String) As Task
    46. Dim extension As String = Path.GetExtension(file)
    47. Dim oldName As String = Path.Combine(selectedDirectory, file)
    48. Dim renamedFileName As String = Path.Combine(selectedDirectory, NewFileName & "_" & GenerateNewSuffix() & extension.ToLower())
    50. Dim fileExists As Boolean = System.IO.File.Exists(renamedFileName)
    52. While fileExists
    53. CountRenamedFiles += 1
    54. renamedFileName = Path.Combine(selectedDirectory, NewFileName & "_" & GenerateNewSuffix() & extension.ToLower())
    55. fileExists = System.IO.File.Exists(renamedFileName)
    56. Await Task.Delay(100) ' Kurze Wartezeit zum Testen!!
    57. End While
    59. Try
    60. Dim fileInfo As New FileInfo(oldName)
    62. If DateofChange Then
    63. fileInfo.LastWriteTime = DateTime.Now
    64. End If
    66. Await Task.Run(Sub()
    67. Try
    68. fileInfo.MoveTo(renamedFileName)
    69. RenamedFiles.Add(fileInfo)
    70. CountRenamedFiles += 1
    71. Catch moveException As Exception
    72. DebugWriteLine("Fehler beim Umbenennen der Datei: " & file & ". Fehler: " & moveException.Message)
    73. ' RaiseEvent ErrorOccurred("Fehler beim Umbenennen der Datei: " & file & ". Fehler: " & moveException.Message)
    74. End Try
    75. End Sub)
    76. Catch ex As Exception
    77. DebugWriteLine("Fehler beim Umbenennen der Datei: " & file & ". Fehler: " & ex.Message)
    78. ' RaiseEvent ErrorOccurred("Fehler beim Umbenennen der Datei: " & file & ". Fehler: " & ex.Message)
    79. End Try
    80. End Function
    82. Private Function GenerateNewSuffix() As String
    83. Dim suffix As String = ""
    85. If Not String.IsNullOrEmpty(SuffixLetter) Then
    86. suffix = SuffixLetter & "_"
    87. End If
    89. Dim counterString As String = SuffixCounter.ToString().PadLeft(4, "0"c)
    90. suffix &= SuffixStartValue.ToString() & SuffixCounter.ToString("D4")
    91. SuffixCounter += 1
    93. Return suffix
    94. End Function
    95. End Class

    Asperger Autistin. Brauche immer etwas um gewisse Sachen zu verstehen. :huh:

    Sind die Namen von oldname und renamedFileName denn immer unterschiedlich?
    Aus dem "Umbenennen" schließe ich, dass du nicht wirklich verschiebst, sondern im gleichen Pfad bleibst. Wenn der Name sich also nicht ändert, dann kann das nicht funktionieren.

    Aber MoveTo kann wohl auch überschreiben: Probier mal FileInfo.MoveTo(renamedFileName, True)

    Übrigens eine interessante Konstellation wenn das klappt, da frisst der die Datei auf und schreibt sie dann wieder hin, wie würde das wohl laufen wenn die Datei sehr groß wird?

    Dieser Beitrag wurde bereits 2 mal editiert, zuletzt von „Haudruferzappeltnoch“ ()

    Spekulatius: Die Generierung vermeintlich eindeutiger neuer Dateinamen erfolgt für alle Dateien nebenläufig, also vermeintlich gleichzeitig. Und zwar bevor die Dateien wirklich umbenannt wurden. Da kann es wahrscheinlich passieren, dass Dateien den gleichen Namen erhalten.

    ##########

    Das ganze Vorgehen kommt mir aber eh kritisch vor. Hattest Du schon mal getestet, wie groß der Zeitunterschied ist, wenn Du alle Dateien in einem Task nacheinander umbenennst - im Vergleich zu Deiner "alle Dateien gleichzeitig"-Variante? Die nacheinander-Variante sollte nämlich das hiesige Problem nicht aufweisen.
    Dieser Beitrag wurde bereits 5 mal editiert, zuletzt von „VaporiZed“, mal wieder aus Grammatikgründen.

    Aufgrund spontaner Selbsteintrübung sind all meine Glaskugeln beim Hersteller. Lasst mich daher bitte nicht den Spekulatiusbackmodus wechseln.

    Dieser Beitrag wurde bereits 2 mal editiert, zuletzt von „VaporiZed“ ()

    @Haudruferzappeltnoch
    Wenn der Name sich also nicht ändert, dann kann das nicht funktionieren.

    Naja manchmal klappt das umbenennen einwandfrei und manchmal eben bei einigen Dateien eben nicht. Auch die Reihenfolge welche gerade einen Fehler produziert ist nicht festzumachen.
    Mal Datei 2 dann wieder Datei 256 usw... eben schlecht nachzuprüfen. Das einzige was ich bis jetzt festmachen konnte:
    Je mehr Dateien in einem Verzeichnis, desto mehr produzieren auch den Fehler.

    @VaporiZed
    neuer Dateinamen erfolgt für alle Dateien nebenläufig, also vermeintlich gleichzeitig.

    Also muss ich diese Function:Private Async Function RenameFileAsync(ByVal selectedDirectory As String, ByVal file As String) As Task irgendwie anders schreiben? Nur wie...?
    Dachte in der While Schleife wird eine Datei nach der anderen umbenannt? Hatte zwischenzeitlich schon mit einem temporären Verzeichnis versucht zu arbeiten. Hat leider nicht so funktioniert wie ich dachte.
    Asperger Autistin. Brauche immer etwas um gewisse Sachen zu verstehen. :huh:

    Statt es »irgendwie anders schreiben« zu wollen, solltest Du den Weg von der Startlinie aus anders gehen. Wie gesagt: Erstmal testweise, um zu sehen, ob es relevante Geschwindigkeitsunterschiede für Dich gibt. Also erstmal Projekt-Backup machen, 1000 umzubenennende Testdateien in ein Testverzeichnis packen und damit dann testen. Der alternative Weg sollte sein:
    1. Alles ohne Async/Await schreiben, sodass alles eben nicht nebenläufig vonstatten geht.
    2. Eine Methode erstellen, welche den Umbenennungsvorgang ohne GUI-Änderungen durchläuft.
    3. Diese Methode nebenläufig aufrufen.
    Beispiel in Pseudocode:

    Quellcode

    1. Sub Async Klick_Des_Umbenennungsstartknopfs
    2. SammleAlleRelevantenDateiinformationen()
    3. Await Threadings.Tasks.Task.Run() FühreUmbenennungAllerDateienDurch())
    4. BerichteErgebnisseAnsGUI()
    5. End Sub

    Dieser Beitrag wurde bereits 5 mal editiert, zuletzt von „VaporiZed“, mal wieder aus Grammatikgründen.

    Aufgrund spontaner Selbsteintrübung sind all meine Glaskugeln beim Hersteller. Lasst mich daher bitte nicht den Spekulatiusbackmodus wechseln.

    Amelie schrieb:

    Je mehr Dateien in einem Verzeichnis, desto mehr produzieren auch den Fehler
    Du hast bisher nicht geschrieben wie der neue Name festgelegt wird.
    An sich kann es auch immer noch sein, dass der alte Name bestehen bleibt, dass müsste statistisch gesehen mit mehr Dateien auch öfter vorkommen.

    Das mit Async Await habe ich noch gar nicht bemerkt. Da könnte die Suffix Methode vielleicht nicht hinterherkommen. Die schreibt dann ein früheres Suffix nochmal hin.
    Habe mal asynchron versucht zu zählen, das ging auch für größere Zahlen immer mehr in die Hose.

    Dieser Beitrag wurde bereits 4 mal editiert, zuletzt von „Haudruferzappeltnoch“ ()

    Es könnte ein reines Windows Problem sein. Ich hatte in der Vergangenheit Probleme wenn ein Dateiname mit einem Leerzeichen anfing und ich es entfernen wollte. Solche Dateinamen habe ich öfter im Zusammenhang mit Youtube und dem von mir benutzten Tool heise.de/download/product/h2rename-67112.
    Aktuelles Projekt: Z80 Disassembler für Schneider/Amstrad CPC :love:

    Das/ein Problem sehe ich in der While-Schleife. Zum einen macht es wenig Sinn nach dem du geprüft hast ob die Datei existiert zu warten zum anderen können alle Threads die Variable CountRenamedFiles ändern. Du könntest die While-Schleife in ein SyncLock (glaub das heißt bei VB so) packen. Das könnte aber dann zu Performance-Einbußen führen. Sollte aber dein Problem beheben.

Facebook
Telefonspende