Extrahieren Albumcover von MP3-Dateien mit TagLib - gibt es einen besseren Weg dieser code zu schreiben?
Bin ich mit Visual Basic 9 (VS2008) und TagLib.
Folgende code extrahiert die Albumcover aus einer MP3-Datei und zeigt Sie in einer PictureBox.
Gibt es einen besseren Weg, um dieser code zu schreiben?
Dim file As TagLib.File = TagLib.File.Create(filepath)
If file.Tag.Pictures.Length >= 1 Then
Dim bin As Byte() = DirectCast(file.Tag.Pictures(0).Data.Data, Byte())
PreviewPictureBox.Image = Image.FromStream(New MemoryStream(bin)).GetThumbnailImage(100, 100, Nothing, System.IntPtr.Zero)
End If
- Auf den ersten Blick, der code sieht ok für mich. Haben Sie ein problem mit ihm?
- Seine Arbeitsmethoden in Ordnung. Ich wollte nur wissen, ob es eine schnellere Möglichkeit, dies zu tun.
Du musst angemeldet sein, um einen Kommentar abzugeben.
Ich bin nicht sehr vertraut mit TagLib aber es sieht nicht wie es viel, viel bessere Art zu schreiben. Der einzige Vorschlag, den ich geben kann ist, dass Sie könnte reduzieren die Menge an code, die durch die Nutzung von Typ-Inferenz. Die zwei Deklarationen von Variablen brauchen nicht einen expliziten Typ, wenn die Option "Ableiten" ist derzeit auf. Diese eigentlich nicht ändern, die Qualität der code aber, es reduziert die Menge von ihm.
Beispiel
Auf den ersten Blick sieht es okay für mich.
Könnte man hinzufügen, eine Fehlerbehandlung, zum Beispiel, wenn
TagLib.File.Create()
wirft einen Fehler, oder es gibt "Nothing
". Auch wenn dieTag
- Eigenschaft leer ist, aus irgendeinem Grund, ein Fehler wird ausgelöst, wenn Sie versuchen, Zugriff auf ".Pictures
".