Visual Basic

2-Jan-09 5:04

This is the code that calls the form where the file is displayed in a picture box.

<br />
frmCEA.ContactID = intContactID<br />
frmCEA.Text = "Edit a contact"<br />
frmCEA.ShowDialog()<br />
strAvatarToDelete = frmCEA.AvatarForDeletion<br />
frmCEA.Close()<br />
frmCEA.Dispose()<br />
<br />
If strAvatarToDelete.Trim > "" Then<br />
    My.Computer.FileSystem.DeleteFile(strAvatarToDelete)<br />
End If<br />

I have tried on the form, and off the form like this, to delete an avatar but every time I get this message - even if I want to change photos.

The process cannot access the file 'C:\Documents and Settings\All Users\Application Data\.....\Avatars\AV000001' because it is being used by another process.

Any out there know why this might be?? Confused | :confused:

Jon_Boy2-Jan-09 5:11

Jon_Boy

2-Jan-09 5:11

Are there any other methods in your app using the file? If so, you need to cleanly end those resources so that the file is unlocked.

make sure the file is not readonly. Make sure you don't have other apps using the file.

Any suggestions, ideas, or 'constructive criticism' are always welcome.

"There's no such thing as a stupid question, only stupid people." - Mr. Garrison

TheComputerMan2-Jan-09 5:21

2-Jan-09 5:21

Thanks for responding.

No nothing else is using the file. It just gets placed in the picture box:

Try
    Me.PictureBox1.BackgroundImage = GetAvatar(ContactID)
    Me.cmdDeleteImage.Enabled = True
Catch ex As Exception
    Me.cmdDeleteImage.Enabled = False
End Try

Where GetAvatar =

Friend Function GetAvatar(ByVal ContactID As Integer) As Drawing.Image
    Dim pict As Drawing.Image
    'This just collects a file name
    Dim strFilename As String = cDBHandler.ExtractContactImage(ContactID)

    Try

        If strFilename.Trim > "" Then
            pict = Drawing.Image.FromFile(strFileName)
        Else
            pict = My.Resources.nouserpic_grey
        End If

    Catch ex As Exception

        pict = My.Resources.nouserpic_grey

    End Try
    Return pict
End Function

Might it be because it ic alled in from another routine?? Confused | :confused:

Jon_Boy2-Jan-09 5:30

Jon_Boy

2-Jan-09 5:30

Well, if you rem out the logic in GetAvatar and loading it into the picture box, can you delete the file? Smile | :)

Any suggestions, ideas, or 'constructive criticism' are always welcome.

"There's no such thing as a stupid question, only stupid people." - Mr. Garrison

TheComputerMan2-Jan-09 6:24

2-Jan-09 6:24

Thanks for the suggestion Jon_Boy but unfortunately that would impact my code. I will give it a go if all else fails however.

Jon_Boy2-Jan-09 7:04

Jon_Boy

2-Jan-09 7:04

Wasn't suggesting that as a perm. fix - just as a way for you now and in the future easily determine if and where your code was locking a file.

Any suggestions, ideas, or 'constructive criticism' are always welcome.

"There's no such thing as a stupid question, only stupid people." - Mr. Garrison

Christian Graus2-Jan-09 5:45

Christian Graus

2-Jan-09 5:45

TheComputerMan wrote:
pict = Drawing.Image.FromFile(strFileName)

This locks the file. You need instead to create a new bitmap. open the file, copy the bitmap, and delete the original. At least, that's how I've got around this bug in GDI+ in the past.

Christian Graus

Driven to the arms of OSX by Vista.

TheComputerMan2-Jan-09 6:22

2-Jan-09 6:22

I tried that Christian (using pict2 = pict.clone and then disposing pict) but still the same problem. It obviously carries the info accross to the clone.

Friend Function GetAvatar(ByVal ContactID As Integer) As Drawing.Bitmap
    Dim pict As Drawing.Bitmap
    Dim pict2 As Drawing.Bitmap
    'This just gets the file name
    Dim strFilename As String = cDBHandler.ExtractContactImage(ContactID)

    Try

        If strFilename.Trim > "" Then
            pict = Drawing.Bitmap.FromFile(strFilename)
            pict2 = pict.Clone()
            pict.Dispose()
        Else
            pict2 = My.Resources.nouserpic_grey
        End If

    Catch ex As Exception

        pict2 = My.Resources.nouserpic_grey

    End Try
    Return pict2
End Function

Looks like I might have to try storing the pic in the database, but I did not want to do that as it is only an SQL Server 2005 Compact so space limited.

Thanks anyway! (If you can think of any other way I would like to hear from you)

Luc Pattyn2-Jan-09 7:00

Luc Pattyn

2-Jan-09 7:00

TheComputerMan wrote:
pict2 = pict.Clone()

Christian is right, but you did not take his advise.

TheComputerMan wrote:
Bitmap.FromFile(strFilename)

This is what keeps your file locked. Hence make and use a copy of the IMAGE and dispose of the original right away.

TheComputerMan wrote:
any other way

No need. The one way is fine if executed properly.

Smile | :)

Luc Pattyn [Forum Guidelines] [My Articles]

Love, happiness and fewer bugs for 2009!

Dave Kreskowiak2-Jan-09 6:21

2-Jan-09 6:21

This has been widely documented since the early days of .NET 1.0. The problem is you loaded the image file directly into an Image object in the PictureBox control. The Image object keeps the file locked for it's entire lifetime.

The solution is to load the image using a FileStream, then pass the resulting stream to the construcotr of a new Bitmap object, then close the stream, and assign the Bitmap object to the Image property of the PictureBox.

A guide to posting questions on CodeProject[^]

Dave Kreskowiak
Microsoft MVP
Visual Developer - Visual Basic
2006, 2007, 2008

TheComputerMan2-Jan-09 6:25

Re: File in use error [modified]

2-Jan-09 6:25

Thanks Dave. I think I know how to do that so I will give that a go.

TheComputerMan2-Jan-09 6:44

2-Jan-09 6:44

Thanks a million Dave. It worked a treat! Smile | :)

No need to reply to this. Just posting the code in case it helps someone else.

Friend Function GetAvatar(ByVal ContactID As Integer) As Drawing.Bitmap
    Dim pict As Drawing.Bitmap
    'This just gets the file name
    Dim strFilename As String = cDBHandler.ExtractContactImage(ContactID)

    Try

        If strFilename.Trim > "" Then
            Dim fs As New System.IO.FileStream(strFilename, IO.FileMode.Open, IO.FileAccess.Read)
            pict = Drawing.Bitmap.FromStream(fs)
            fs.Close()
        Else
            pict = My.Resources.nouserpic_grey
        End If

    Catch ex As Exception

        pict = My.Resources.nouserpic_grey

    End Try
    Return pict
End Function

Code above corrected: Smile | :)

Thanks Fabio

modified on Saturday, January 3, 2009 10:20 AM

Fabio V Silva2-Jan-09 11:53

Fabio V Silva

2-Jan-09 11:53

Why do you need picbyteArray variable?! You're not using it...

TheComputerMan3-Jan-09 4:20

3-Jan-09 4:20

Hi Fabio,

Thanks for that. I had pulled the code from another program where I am using it to store the picture in SQL Server and forgot to take that bit out! Duh!! Roll eyes | :rolleyes:

The corrected code block is:

Friend Function GetAvatar(ByVal ContactID As Integer) As Drawing.Bitmap
    Dim pict As Drawing.Bitmap
    'This just gets the file name
    Dim strFilename As String = cDBHandler.ExtractContactImage(ContactID)

    Try

        If strFilename.Trim > "" Then
            Dim fs As New System.IO.FileStream(strFilename, IO.FileMode.Open, IO.FileAccess.Read)
            pict = Drawing.Bitmap.FromStream(fs)
            fs.Close()
        Else
            pict = My.Resources.nouserpic_grey
        End If

    Catch ex As Exception

        pict = My.Resources.nouserpic_grey

    End Try
    Return pict
End Function

Sorry if that caused any confusion to anyone.

Next Inner

dadashri2-Jan-09 1:44

dadashri

2-Jan-09 1:44

have a form that adds a new date to a sql database. What i want to do is when the user clicks add a new day, it comes up with a text box to add a new day. When the user tabs to the first feild, i want the lost focus to check the combo box to see if the date is already in the list?
Private Sub txtFields_LostFocus(Index As Integer)
Dim ifeild As Integer
Dim flddate As String
flddate = frmCashSheet.cmbDate.Text
ifeild = Index
If ifeild = 0 Then
frmCashSheet.txtFields.Item(0).Text = flddate
MsgBox "date has been entered already, Please select a new date"
frmCashSheet.txtFields(0).SetFocus
Else
For Inner = 0 To 32
txtFields(ifeild).BackColor = &HFF&
Next Inner
calculate
End If
End Sub

TheComputerMan2-Jan-09 5:10

2-Jan-09 5:10

<quote>Private Sub txtFields_LostFocus(Index As Integer)

This does not appear to be called from anywhere by a handler and has the wrong parameters.

Private Sub txtFields_LostFocus(ByVal sender As Object, ByVal e As System.EventArgs) Handles txtFields.LostFocus

Index would have to come from reading something but I am not sure what from this example.

Dave Kreskowiak2-Jan-09 6:14

2-Jan-09 6:14

It is correct is he's using VB6.

A guide to posting questions on CodeProject[^]

Dave Kreskowiak
Microsoft MVP
Visual Developer - Visual Basic
2006, 2007, 2008

TheComputerMan2-Jan-09 6:47

2-Jan-09 6:47

Oops! Sorry did not twig that.

Dave Kreskowiak2-Jan-09 6:15

School Library System Software RUN-TIME error 713

2-Jan-09 6:15

So, where's the code that goes through each item in the ComboBox items and compares the two??

A guide to posting questions on CodeProject[^]

Dave Kreskowiak
Microsoft MVP
Visual Developer - Visual Basic
2006, 2007, 2008

dadashri2-Jan-09 1:36

dadashri

2-Jan-09 1:36

School Library System Software RUN-TIME error 713
plas soly
pranav trivedi

Re: School Library System Software RUN-TIME error 713

Dave Kreskowiak2-Jan-09 3:42

Re: School Library System Software RUN-TIME error 713

2-Jan-09 3:42

Try Googling the error message[^].

A guide to posting questions on CodeProject[^]

Dave Kreskowiak
Microsoft MVP
Visual Developer - Visual Basic
2006, 2007, 2008

Christian Graus2-Jan-09 5:47

Christian Graus

2-Jan-09 5:47

Wow - are you even remotely a programmer, or is this an error from software you use and you're asking about it in a programming forum ?

Christian Graus

Driven to the arms of OSX by Vista.

Add two mp3 files into one mp3 file.

Gagan.202-Jan-09 0:21

Gagan.20

2-Jan-09 0:21

Hi all...

I am creating a program in which is used to play two songs at the same time.
I have imported Interop.WmpLib.dll file and created two objects to play two songs simultaneously.
I want to mix and record these two songs so that when I will play recorded file, it will play songs of both files as played in DJs.
Suggest me,how can I mix and record these two mp3 files in one mp3 file.

Thanks.

Gagan

Re: Add two mp3 files into one mp3 file.

Dave Kreskowiak2-Jan-09 3:44