I'm using Visual Basic 9 (VS2008) and TagLib.
The following code extracts the album art from an MP3 file and displays it in a PictureBox.
Is there a better way to write this code?
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
Best Answer
At first glance it looks okay to me.
You could add some error handling, for example if
TagLib.File.Create()
throws an error or returns "Nothing
". Also if theTag
property is empty for some reason, an error will be thrown if you'll try to access ".Pictures
".