Take the 2-minute tour ×
Super User is a question and answer site for computer enthusiasts and power users. It's 100% free, no registration required.

Well i feel like pulling my hair out!

i have a macro set up which splits my spread sheet into worksheets dependent on the data in column 6. I then have another macro which saves each of these as a seperate workbook and then made another which rund the 2 together.

I have been through my spreadsheet and ensured that all the possible entries into column 6 match (no spelling mistakes etc)

now for some reason when i run the macro's it is outputting all the workbooks i need but also doing a second copy and naming them Error-1, Error-2 etc.

i dont understand why any ideas?

Also my sprreadsheet runs from column a through to column AV but when i run macros they workbooks it is outputting only go to column O

Any help is much appreciated.

ahh here is the macro i am using btw-

Sub Copy_To_Worksheets() 'Note: This macro use the function LastRow
    Dim My_Range As Range
    Dim FieldNum As Long
    Dim CalcMode As Long
    Dim ViewMode As Long
    Dim ws2 As Worksheet
    Dim Lrow As Long
    Dim cell As Range
    Dim CCount As Long
    Dim WSNew As Worksheet
    Dim ErrNum As Long

    'Set filter range on ActiveSheet: A1 is the top left cell of your filter range
    'and the header of the first column, D is the last column in the filter range.
    'You can also add the sheet name to the code like this :
    'Worksheets("Sheet1").Range("A1:D" & LastRow(Worksheets("Sheet1")))
    'No need that the sheet is active then when you run the macro when you use this.
    Set My_Range = Range("A1:O2000") '  & LastRow(ActiveSheet))

    If ActiveWorkbook.ProtectStructure = True Or _
       My_Range.Parent.ProtectContents = True Then
        MsgBox "Sorry, not working when the workbook or worksheet is protected", _
               vbOKOnly, "Copy to new worksheet"
        Exit Sub
    End If

    'This example filters on the first column in the range(change the field if needed)
    'In this case the range starts in A so Field:=1 is column A, 2 = column B, ......
    FieldNum = 6 ' I changed this to 3 for column C

    'Turn off AutoFilter
    My_Range.Parent.AutoFilterMode = False

    'Change ScreenUpdating, Calculation, EnableEvents, ....
    With Application
        CalcMode = .Calculation
        .Calculation = xlCalculationManual
        .ScreenUpdating = False
        .EnableEvents = False
    End With
    ViewMode = ActiveWindow.View
    ActiveWindow.View = xlNormalView
    ActiveSheet.DisplayPageBreaks = False

    'Add a worksheet to copy the a unique list and add the CriteriaRange
    Set ws2 = worksheets.Add

    With ws2
        'first we copy the Unique data from the filter field to ws2
        My_Range.Columns(FieldNum).AdvancedFilter _
                Action:=xlFilterCopy, _
                CopyToRange:=.Range("A1"), Unique:=True

        'loop through the unique list in ws2 and filter/copy to a new sheet
        Lrow = .Cells(Rows.Count, "A").End(xlUp).Row
        For Each cell In .Range("A2:A" & Lrow)

            'Filter the range
            My_Range.AutoFilter Field:=FieldNum, Criteria1:="=" & _
             Replace(Replace(Replace(cell.Value, "~", "~~"), "*", "~*"), "?", "~?")

            'Check if there are no more then 8192 areas(limit of areas)
            CCount = 0
            On Error Resume Next
            CCount = My_Range.Columns(1).SpecialCells(xlCellTypeVisible) _
            On Error GoTo 0
            If CCount = 0 Then
                MsgBox "There are more than 8192 areas for the value : " & cell.Value _
                     & vbNewLine & "It is not possible to copy the visible data." _
                     & vbNewLine & "Tip: Sort your data before you use this macro.", _
                       vbOKOnly, "Split in worksheets"
                'Add a new worksheet
                Set WSNew = worksheets.Add(After:=Sheets(Sheets.Count))
                On Error Resume Next
                WSNew.Name = cell.Value
                If Err.Number > 0 Then
                    ErrNum = ErrNum + 1
                    WSNew.Name = "Error_" & Format(ErrNum, "0000")
                End If
                On Error GoTo 0

                'Copy the visible data to the new worksheet
                With WSNew.Range("A1")
                    ' Paste:=8 will copy the columnwidth in Excel 2000 and higher
                    ' Remove this line if you use Excel 97
                    .PasteSpecial Paste:=8
                    .PasteSpecial xlPasteValues
                    .PasteSpecial xlPasteFormats
                    Application.CutCopyMode = False
                End With
            End If

            'Show all data in the range
            My_Range.AutoFilter Field:=FieldNum

        Next cell

        'Delete the ws2 sheet
        On Error Resume Next
        Application.DisplayAlerts = False
        Application.DisplayAlerts = True
        On Error GoTo 0

    End With

    'Turn off AutoFilter
    My_Range.Parent.AutoFilterMode = False

    If ErrNum > 0 Then
        MsgBox "Rename every WorkSheet name that start with ""Error_"" manually" _
             & vbNewLine & "There are characters in the name that are not allowed" _
             & vbNewLine & "in a sheet name or the worksheet already exist."
    End If

    'Restore ScreenUpdating, Calculation, EnableEvents, ....
    ActiveWindow.View = ViewMode
    With Application
        .ScreenUpdating = True
        .EnableEvents = True
        .Calculation = CalcMode
    End With

    MsgBox ("Copy Complete - - Remember to save your work.")

End Sub

Sub CreateWorkbooks() 'Creates an individual workbook for each worksheet in the active workbook. Dim wbDest As Workbook Dim wbSource As Workbook Dim sht As Object 'Could be chart, worksheet, Excel 4.0 macro,etc. Dim strSavePath As String

On Error GoTo ErrorHandler

Application.ScreenUpdating = False 'Don't show any screen movement

strSavePath = "T:\Lead Employer Service\HR Team\!Recruitment Team\!Trackers\Output for hosts\Surgery\" 'Change this to suit your needs

Set wbSource = ActiveWorkbook

For Each sht In wbSource.Sheets sht.Copy Set wbDest = ActiveWorkbook wbDest.SaveAs strSavePath & sht.Name wbDest.Close 'Remove this if you don't want each book closed after saving. Next

Application.ScreenUpdating = True

Exit Sub

ErrorHandler: 'Just in case something hideous happens MsgBox "An error has occurred. Error number=" & Err.Number & ". Error description=" & Err.Description & "." End Sub

Sub runmultiplemacros() Copy_To_Worksheets CreateWorkbooks End Sub
share|improve this question
i'll have to analyze....but somehow, the error_1....sheets are getting created... analyze the piece of code where you are naming new worksheet : 'WSNew.Name = "Error_" & Format(ErrNum, "0000")' –  tumchaaditya Jun 15 '12 at 12:17

1 Answer 1

To answer the question about it running to column 0, I suspect the culprit will be the following line:

Set My_Range = Range("A1:O2000")

This will be restricting what you're running over.

share|improve this answer
instead of defining range like that, find out the last occupied column in sheet and define range from that.....foolproof –  tumchaaditya Jun 15 '12 at 12:15
Thats great thanks for your help thats sorted it. unsure about why it was making duplicates and naming them as error though. Since posting this comment i have amended the cell range and re run the macro and it has worked fine :s very confusing! –  Sean Chadwick Jun 15 '12 at 12:32
For finding the entire used range on a given sheet, the Sheet.UsedRange property is my method of choide. –  Brian Dec 30 '14 at 2:55

Your Answer


By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.