This code is used to rearrange some columns, but it’s not working properly?

RagingR12

New Member
Joined
Sep 17, 2018
Messages
18
This code is used to rearrange some columns, but it’s notworking properly?

Why is the following code working for the first worksheetbut not working for the second work sheet?

It scrambles up the first header row.


Code:
For i = 1 To ThisWorkbook.Sheets.Count
 Sheets(i).Activate
With ActiveSheet
    Dim arrColOrder As Variant, ndx As Integer
    Dim Found As Range, counter As Integer
    
    arrColOrder = Array("VolgCode", "Home", "Level", "Find No.", "Item Id", "Revision", "Change", "Item Name", "Qty")
    
    counter = 1
    
    Application.ScreenUpdating = False
    
    For ndx = LBound(arrColOrder) To UBound(arrColOrder)
    
        Set Found = Rows("1:1").Find(arrColOrder(ndx), LookIn:=xlValues, LookAt:=xlWhole, _
                          SearchOrder:=xlByColumns, SearchDirection:=xlNext, MatchCase:=False)
        
        If Not Found Is Nothing Then
            If Found.Column <> counter Then
                Found.EntireColumn.Cut
                Columns(counter).Insert Shift:=xlToRight
                Application.CutCopyMode = False
            End If
            counter = counter + 1
        End If
        
    Next ndx
    Application.ScreenUpdating = True
 End With
Next i

Thanks for any help.
 

Excel Facts

Select all contiguous cells
Pressing Ctrl+* (asterisk) will select the "current region" - all contiguous cells in all directions.
This works for me for more than 1 sheet:
Code:
Sub SortCols()

    Dim c       As Long
    Dim i       As Long
    Dim x       As Long
    Dim w       As Long
    
    Dim r       As Range
    Dim arr     As Variant
    
    arr = Split("VolgCode|Home|Level|Find No.|Item Id|Revision|Change|Item Name|Qty", "|")
    
    Application.ScreenUpdating = False
    
    For w = 1 To ThisWorkbook.sheets.Count
        i = 1
        With sheets(w)
            c = .Cells(1, .Columns.Count).End(xlToLeft).Column
            For x = LBound(arr) To UBound(arr)
                Set r = .Cells(1, 1).Resize(, c).Find(arr(x), after:=.Cells(1, 1), LookIn:=xlFormulas, searchorder:=xlByRows)
                If Not r Is Nothing Then
                    If r.Column <> i Then
                        r.EntireColumn.Cut
                        .Columns(i).Insert
                        Application.CutCopyMode = False
                        Set r = Nothing
                        i = i + 1
                    End If
                End If
            Next x
        End With
    Next w
    
    Application.ScreenUpdating = True
    
    Erase arr
End Sub
Are the search values definitely on row 1 on each sheet?
 
Last edited:
Upvote 0

Forum statistics

Threads
1,214,413
Messages
6,119,374
Members
448,888
Latest member
Arle8907

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