Dear Sandeep, i did some testing, and here is working properly, the only 
error that appears, is when the macro finds some protected worksheet.
Try using a new worksheet and paste this code below ... and then test to 
see if it works.

Option Explicit


#If VBA7 Then
    Private Type BROWSEINFO
        hOwner As LongPtr
        pidlRoot As LongPtr
        pszDisplayName As String
        lpszTitle As String
        ulFlags As Long
        lpfn As LongPtr
        lParam As LongPtr
        iImage As Long
    End Type
                        
    Private Declare PtrSafe Function SHBrowseForFolder Lib "shell32.dll" 
Alias "SHBrowseForFolderA" _
        (lpBrowseInfo As BROWSEINFO) As LongPtr
    Private Declare PtrSafe Function SHGetPathFromIDList Lib "shell32.dll" 
Alias "SHGetPathFromIDListA" _
        (ByVal pidl As LongPtr, ByVal pszPath As String) As Boolean
#Else
    Private Type BROWSEINFO
        hOwner As Long
        pidlRoot As Long
        pszDisplayName As String
        lpszTitle As String
        ulFlags As Long
        lpfn As Long
        lParam As Long
        iImage As Long
    End Type
                        
    Private Declare Function SHBrowseForFolder Lib "shell32.dll" Alias 
"SHBrowseForFolderA" _
        (lpBrowseInfo As BROWSEINFO) As Long
    Declare Function SHBrowseForFolder Lib "shell32.dll" _
        Alias "SHBrowseForFolderA" (lpBrowseInfo As BROWSEINFO) As Long
#End If

Private Const BIF_RETURNONLYFSDIRS = &H1
 
Function GetDirectory(Optional msg) As String
    On Error Resume Next
    Dim bInfo As BROWSEINFO
    Dim path As String
    Dim r As Long, x As Long, pos As Integer
     
     'Root folder = Desktop
    bInfo.pidlRoot = 0&
     
     'Title in the dialog
    If IsMissing(msg) Then
        bInfo.lpszTitle = "Please select the folder of the excel files to 
copy."
    Else
        bInfo.lpszTitle = msg
    End If
     
     'Type of directory to return
    bInfo.ulFlags = &H1
     
     'Display the dialog
    x = SHBrowseForFolder(bInfo)
     
     'Parse the result
    path = Space$(512)
    r = SHGetPathFromIDList(ByVal x, ByVal path)
    If r Then
        pos = InStr(path, Chr$(0))
        GetDirectory = Left(path, pos - 1)
    Else
        GetDirectory = ""
    End If
End Function
 
Sub CombineFiles()
    Dim path            As String
    Dim FileName        As String
    Dim LastCell        As Range
    Dim Wkb             As Workbook
    Dim WS              As Worksheet
    Dim ThisWB          As String
     
    ThisWB = ThisWorkbook.Name
    Application.EnableEvents = False
    Application.ScreenUpdating = False
    path = GetDirectory
    FileName = Dir(path & "\*.xls", vbNormal)
    Do Until FileName = ""
        If FileName <> ThisWB Then
            Set Wkb = Workbooks.Open(FileName:=path & "\" & FileName)
            For Each WS In Wkb.Worksheets
                Set LastCell = WS.Cells.SpecialCells(xlCellTypeLastCell)
                If LastCell.Value = "" And LastCell.Address = 
Range("$A$1").Address Then
                Else
                    WS.Copy 
After:=ThisWorkbook.Sheets(ThisWorkbook.Sheets.Count)
                End If
            Next WS
            Wkb.Close False
        End If
        FileName = Dir()
    Loop
    Application.EnableEvents = True
    Application.ScreenUpdating = True
     
    Set Wkb = Nothing
    Set LastCell = Nothing
End Sub




Em terça-feira, 15 de outubro de 2013 06h12min23s UTC-3, sandeep chhajer 
escreveu:
>
> Dear Excel Gurus,
>
> I have got this Macro from this forum for* adding worksheets from 
> different work book kept in a folder,* but now when i am trying this code 
> I am getting compiling error message" can not define a public user defined 
> type with an object module.
>
> Please help.
>
> Thanks in advance.  
>
> Option Explicit
>  
>  '32-bit API declarations
> Declare Function SHGetPathFromIDList Lib "shell32.dll" _
> Alias "SHGetPathFromIDListA" (ByVal pidl As Long, ByVal _
> pszpath As String) As Long
>  
> Declare Function SHBrowseForFolder Lib "shell32.dll" _
> Alias "SHBrowseForFolderA" (lpBrowseInfo As BrowseInfo) _
> As Long
>  
> Public Type BrowseInfo
>     hOwner As Long
>     pIDLRoot As Long
>     pszDisplayName As String
>     lpszTitle As String
>     ulFlags As Long
>     lpfn As Long
>     lParam As Long
>     iImage As Long
> End Type
>  
> Function GetDirectory(Optional msg) As String
>     On Error Resume Next
>     Dim bInfo As BrowseInfo
>     Dim path As String
>     Dim r As Long, x As Long, pos As Integer
>      
>      'Root folder = Desktop
>     bInfo.pIDLRoot = 0&
>      
>      'Title in the dialog
>     If IsMissing(msg) Then
>         bInfo.lpszTitle = "Please select the folder of the excel files to 
> copy."
>     Else
>         bInfo.lpszTitle = msg
>     End If
>      
>      'Type of directory to return
>     bInfo.ulFlags = &H1
>      
>      'Display the dialog
>     x = SHBrowseForFolder(bInfo)
>      
>      'Parse the result
>     path = Space$(512)
>     r = SHGetPathFromIDList(ByVal x, ByVal path)
>     If r Then
>         pos = InStr(path, Chr$(0))
>         GetDirectory = Left(path, pos - 1)
>     Else
>         GetDirectory = ""
>     End If
> End Function
>  
> Sub CombineFiles()
>     Dim path            As String
>     Dim FileName        As String
>     Dim LastCell        As Range
>     Dim Wkb             As Workbook
>     Dim WS              As Worksheet
>     Dim ThisWB          As String
>      
>     ThisWB = ThisWorkbook.Name
>     Application.EnableEvents = False
>     Application.ScreenUpdating = False
>     path = GetDirectory
>     FileName = Dir(path & "\*.xls", vbNormal)
>     Do Until FileName = ""
>         If FileName <> ThisWB Then
>             Set Wkb = Workbooks.Open(FileName:=path & "\" & FileName)
>             For Each WS In Wkb.Worksheets
>                 Set LastCell = WS.Cells.SpecialCells(xlCellTypeLastCell)
>                 If LastCell.Value = "" And LastCell.Address = 
> Range("$A$1").Address Then
>                 Else
>                     WS.Copy 
> After:=ThisWorkbook.Sheets(ThisWorkbook.Sheets.Count)
>                 End If
>             Next WS
>             Wkb.Close False
>         End If
>         FileName = Dir()
>     Loop
>     Application.EnableEvents = True
>     Application.ScreenUpdating = True
>      
>     Set Wkb = Nothing
>     Set LastCell = Nothing
> End Sub
>
>
> -- 
> Regards,
> Sandeep Kumar Chhajer.
>
> 

-- 
Are you =EXP(E:RT) or =NOT(EXP(E:RT)) in Excel? And do you wanna be? It’s 
=TIME(2,DO:IT,N:OW) ! Join official Facebook page of this forum @ 
https://www.facebook.com/discussexcel

FORUM RULES

1) Use concise, accurate thread titles. Poor thread titles, like Please Help, 
Urgent, Need Help, Formula Problem, Code Problem, and Need Advice will not get 
quick attention or may not be answered.
2) Don't post a question in the thread of another member.
3) Don't post questions regarding breaking or bypassing any security measure.
4) Acknowledge the responses you receive, good or bad.
5) Jobs posting is not allowed.
6) Sharing copyrighted material and their links is not allowed.

NOTE  : Don't ever post confidential data in a workbook. Forum owners and 
members are not responsible for any loss.
--- 
You received this message because you are subscribed to the Google Groups "MS 
EXCEL AND VBA MACROS" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to excel-macros+unsubscr...@googlegroups.com.
To post to this group, send email to excel-macros@googlegroups.com.
Visit this group at http://groups.google.com/group/excel-macros.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to