Move/Copy macro doesn't work as aspected

[Pero]

Hi.

Got a code that will save all selected files in a directory chosen by the user.
Often it works great but sometimes it doesn't copy/move all the selected files, just a few (about 5-6) and sometimes the new files are just 1kb and doesn't work like the program only copied the icon.

Anyone knows a diffrent way of writing this code ? A code that doesn't bugg ?


My Code:

Please Login or Register  to view this content.

Please Login or Register  to view this content.

Please Login or Register  to view this content.

[pike]

Hi pero
which line fails?

[Pero]

sorry for slow reply !

No line failes in the code when I run it.
Sometimes it works and sometime it doesn't. And when you copy alot of files some of them is only 1Kb in the target folder.

Weird..

Just asking if its a way to write a more "Safe" code like this one.

[pike]

The only thing I can see if the sfile name or sdir string are wrong in the range
do you have a small sample workbook?

[Pero]

Sure!

here is a dummybook.

[pike]

I use this code to copy file(s)

Please Login or Register  to view this content.

or you may need to check if

Please Login or Register  to view this content.

I check your code out when I get a chance
See ya

[pike]

ooOps

Wrong post

[pike]

Hi Pero,
try this..half finnished code ....

Please Login or Register  to view this content.

will have to add these checks

Please Login or Register  to view this content.

[pike]

this should have it
cant find if the file exist with out the extension

Please Login or Register  to view this content.

[pike]

Pero how did the code go???

[pike]

Bump Bump Bump

[Pero]

Hey!

Have been away for a while. Will test the code now and come back with the results.
Thaks for your time!!

EDIT:
WHen I run the macros I get an error. Error nr 9, "The index is out of the interval"

I got the errormessage in swedish and thne tried to translate it. Its not a quote.

The line markedin VBA is:

Please Login or Register  to view this content.

How do I fix this ?

Again Thank you !

//

Per

[pike]

Pero did you select/highlight the files/folders to move or copy?

[pike]

Pero use..

Please Login or Register  to view this content.

[Pero]

Thank you again =)

I'm very greatful for all of your help.
The code is working now as it should.

Just a few modifications need to be done.

Everyting is explained in my DummyBok.



//

Per

[pike]

Pero try..

Please Login or Register  to view this content.

Comment ; This code will differ from the email as I have to change the if to elseif

[Pero]

Everything is working great =)


Thank you !!!

[pike]

no thank you for marking the post solved