Selecting all data on a Worksheet

DctheDC

New Member
Joined
Jun 7, 2016
Messages
37
I have recorded a macro to sort my worksheet in a specific order and I get the following VBA.

Cells.Select
ActiveWorkbook.Worksheets("DReg").Sort.SortFields.Clear
ActiveWorkbook.Worksheets("DReg").Sort.SortFields.Add Key:=Range( _
"A2:A10"), SortOn:=xlSortOnValues, Order:=xlAscending, DataOption:= _
xlSortNormal
ActiveWorkbook.Worksheets("DReg").Sort.SortFields.Add Key:=Range( _
"G2:G10"), SortOn:=xlSortOnValues, Order:=xlAscending, DataOption:= _
xlSortNormal
ActiveWorkbook.Worksheets("DReg").Sort.SortFields.Add Key:=Range( _
"L2:L10"), SortOn:=xlSortOnValues, Order:=xlAscending, DataOption:= _
xlSortNormal
With ActiveWorkbook.Worksheets("DReg").Sort
.SetRange Range("A1:AB10")
.Header = xlYes
.MatchCase = False
.Orientation = xlTopToBottom
.SortMethod = xlPinYin
.Apply
End With
Range("A1").Select
End Sub



If I add another row to the sheet beyond row 10 or another column beyond AB then the new data is not picked up by the sort macro. Is there a way to select the whole sheet to be covered by the sort?

Cheers
 

Excel Facts

Why does 9 mean SUM in SUBTOTAL?
It is because Sum is the 9th alphabetically in Average, Count, CountA, Max, Min, Product, StDev.S, StDev.P, Sum, VAR.S, VAR.P.
Try:
Code:
Sub SortMe()

    Dim x       As Long
    Dim y       As Long
    
    Application.ScreenUpdating = False
    
    With Sheets("DReg")
        x = .Cells(.Rows.count, 1).End(xlUp).row
        y = .Cells(1, .Columns.count).End(xlToLeft).column
        
        .Sort.SortFields.Clear
        With .Sort
            .SortFields.add key:=Parent.Cells(1, 1).Resize(x), Order:=xlAscending
            .SortFields.add key:=Parent.Cells(1, 7).Resize(x), Order:=xlAscending
            .SortFields.add key:=Parent.Cells(1, 12).Resize(x), Order:=xlAscending
        End With
        
        With .Sort
            .SetRange Parent.Cells(1, 1).Resize(x, y)
            .header = xlYes
            .MatchCase = False
            .Orientation = xlTopToBottom
            .Apply
        End With
        .Select
    End With
    
    With Application
        .Goto Cells(1, 1)
        .ScreenUpdating = True
    End With
    
End Sub
 
Last edited:
Upvote 0
DctheDC,

Although untested, I'm certain @JackDanIce's routine will be far superior to any recorded macro. If, however, you expect to record macros in the future and want to adapt them to a dynamic/changing range, you might consider the following...

Code:
Sub test()
Dim LastRow As Long, LastColumn As Long
LastRow = ActiveSheet.Cells.Find(What:="*", After:=Cells(1, 1), LookIn:=xlFormulas, _
    LookAt:=xlPart, SearchOrder:=xlByRows, SearchDirection:=xlPrevious, MatchCase:=False).Row
LastColumn = ActiveSheet.Cells.Find(What:="*", After:=Cells(1, 1), LookIn:=xlFormulas, _
    LookAt:=xlPart, SearchOrder:=xlByColumns, SearchDirection:=xlPrevious, MatchCase:=False).Column
    
Cells.Select
ActiveWorkbook.Worksheets("DReg").Sort.SortFields.Clear
ActiveWorkbook.Worksheets("DReg").Sort.SortFields.Add Key:=Range( _
    [COLOR=#ff0000]"A2:A" & LastRow[/COLOR]), SortOn:=xlSortOnValues, Order:=xlAscending, DataOption:= _
    xlSortNormal
ActiveWorkbook.Worksheets("DReg").Sort.SortFields.Add Key:=Range( _
    [COLOR=#ff0000]"G2:G" & LastRow[/COLOR]), SortOn:=xlSortOnValues, Order:=xlAscending, DataOption:= _
    xlSortNormal
ActiveWorkbook.Worksheets("DReg").Sort.SortFields.Add Key:=Range( _
    [COLOR=#ff0000]"L2:L" & LastRow[/COLOR]), SortOn:=xlSortOnValues, Order:=xlAscending, DataOption:= _
    xlSortNormal
With ActiveWorkbook.Worksheets("DReg").Sort
    .SetRange [COLOR=#ff0000]Range(Cells(1, 1), Cells(LastRow, LastColumn)[/COLOR])
    .Header = xlYes
    .MatchCase = False
    .Orientation = xlTopToBottom
    .SortMethod = xlPinYin
    .Apply
End With
Range("A1").Select
End Sub

In the recorded macro, the last row is hard coded, eg, "A2:A10" or "G2:G10". So this macro defines a variable named LastRow and determines its value by using the .Find method to search every cell for any value, and the last cell with a value is in the last row. So now it's just a matter of replacing Range("A2:A10") with Range*"A2:A" & LastRow).

It's a similar process for the Last Column.

Cheers,

tonyyy
 
Upvote 0

Forum statistics

Threads
1,223,903
Messages
6,175,287
Members
452,631
Latest member
a_potato

We've detected that you are using an adblocker.

We have a great community of people providing Excel help here, but the hosting costs are enormous. You can help keep this site running by allowing ads on MrExcel.com.
Allow Ads at MrExcel

Which adblocker are you using?

Disable AdBlock

Follow these easy steps to disable AdBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the icon in the browser’s toolbar.
2)Click on the "Pause on this site" option.
Go back

Disable AdBlock Plus

Follow these easy steps to disable AdBlock Plus

1)Click on the icon in the browser’s toolbar.
2)Click on the toggle to disable it for "mrexcel.com".
Go back

Disable uBlock Origin

Follow these easy steps to disable uBlock Origin

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back

Disable uBlock

Follow these easy steps to disable uBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back
Back
Top