Apologies if this has been answered already, but couldn't find it if it has.
Folder contains a Master file with one worksheet. and several other files with multiple sheets.
Following extract from Master File macro should find and open each file in the Active Workbook folder, but it's throwing an error when setting the current folder path?
Hope someone can see the problem
Please Login or Register to view this content.
All solutions, suggestions and alternatives welcome as ever
Ochimis
Get rid of "set", you don't set a string.
Also you should have them all dim'd as strings
Formula:
Please Login or Register to view this content.
You also need to dim each worksheet as worksheet.
Last edited by davesexcel; 12-14-2018 at 11:30 AM.
davesexcel,
Seems that's an error I've been making for ages!
I was taught moons ago using VB and VB.Net, that linking the multiple names on the one line meant they were all declared as Strings.
Presumably any other than the last name are now "implicitly" Variants?
Ochimus
Sorry, typo, should have said "dim'd" as strings, not "set" as strings, looks like you understood though.
And you did remove set?
Formula:Please Login or Register to view this content.
Last edited by davesexcel; 12-14-2018 at 11:39 AM.
Davesexecel,
Yes I "Dim'd" each variable separately, and removed the Set ref.
But code now stops when I try to set the Main file as the Destination workbook. Throws a Runtime Error 91:
As before, hope someone can help me?Please Login or Register to view this content.
Ochimus
Last edited by Ochimus; 12-14-2018 at 07:06 PM.
You dim'd wbDst as a workbook but have told the code it is supposed to be equal to the workbook.name, which would be a string.
or "Thisworkbook" , most cases thisworkbook would be the one that has the running code.Formula:Please Login or Register to view this content.
You have to set workbooks.
davesexcel,
Rescued once again! Seems the "Senior Moments" are becoming more frequent by the week. . .
Slight Code tweaking means everything now works smoothly, so can mark this as solved, and add it to the Toolbox.
Ochimus
There are currently 1 users browsing this thread. (0 members and 1 guests)
Posting Permissions
LinkBack URL
About LinkBacks
Register To Reply
,
Bookmarks