Problem With Existing Code After PC Upgrade

Dazzawm

Well-known Member
Joined
Jan 24, 2011
Messages
3,783
Office Version
  1. 365
Platform
  1. Windows
I have gone from a windows 7 32-bit to a windows 11 64-bit PC. The code below is doing opposite to what it should! I have sheet 2 with a number in A and another in B. When the number in A is found in column AE on sheet 1 it is supposed to copy the row and insert below and replace with the number that is next to it in column B sheet 2. But it inserts the row above instead! Would somebody help please?

Code:
Sub FindCopyReplaceBelow()
Application.EnableEvents = False
Application.Calculation = xlCalculationManual = False
Application.ScreenUpdating = False


Dim Rng1    As Range
Dim Dn1     As Range
Dim Rng2    As Range
Dim Dn2     As Range
With Sheets("Sheet2")
    Set Rng2 = .Range(.Range("A1"), .Range("A" & Rows.Count).End(xlUp))
End With
With Sheets("Sheet1")
    Set Rng1 = .Range(.Range("AE2"), .Range("AE" & Rows.Count).End(xlUp))
End With
    For Each Dn2 In Rng2
        For Each Dn1 In Rng1
            If Dn1 = Dn2 Then
                Dn1.EntireRow.Interior.ColorIndex = 5
                Dn1.EntireRow.Copy
                Dn1.Offset(1).EntireRow.Insert Shift:=xlDown
                Dn1.Offset(1).Resize(, 2).Value = Dn2.Offset(, 1).Resize(, 2).Value
                
                
            End If
        Next Dn1
    Next Dn2
    
Application.EnableEvents = True
Application.Calculation = xlCalculationManual = True
Application.ScreenUpdating = True
End Sub
 

Excel Facts

Easy bullets in Excel
If you have a numeric keypad, press Alt+7 on numeric keypad to type a bullet in Excel.
Hope this helps.
VBA Code:
Sub FindCopyReplaceBelow()
    Application.EnableEvents = False
    Application.Calculation = xlCalculationManual = False
    Application.ScreenUpdating = False
    
    
    Dim Rng1    As Range
    Dim Dn2     As Range
    Dim i As Long
    
    With Sheets("Sheet1")
        Set Rng1 = .Range(.Range("AE2"), .Range("AE" & Rows.Count).End(xlUp))
    End With
    
    With Sheets("Sheet2")
        For i = .Cells(Rows.Count, 1).End(xlUp).Row To 1 Step -1
            If WorksheetFunction.CountIf(Rng1, .Cells(i, 1).Value) > 0 Then
                Rows(i).EntireRow.Interior.ColorIndex = 5
                Rows(i).EntireRow.Copy
                Rows(i + 1).EntireRow.Insert Shift:=xlDown
                .Cells(i + 1, 2).Value = .Cells(i, 3).Value
            End If
        Next
    End With
    
    Application.EnableEvents = True
    Application.Calculation = xlCalculationManual = True
    Application.ScreenUpdating = True
End Sub
 
Upvote 0
My results come out as expected,using your code

1659091973896.png
 
Upvote 0
Hope this helps.
VBA Code:
Sub FindCopyReplaceBelow()
    Application.EnableEvents = False
    Application.Calculation = xlCalculationManual = False
    Application.ScreenUpdating = False
   
   
    Dim Rng1    As Range
    Dim Dn2     As Range
    Dim i As Long
   
    With Sheets("Sheet1")
        Set Rng1 = .Range(.Range("AE2"), .Range("AE" & Rows.Count).End(xlUp))
    End With
   
    With Sheets("Sheet2")
        For i = .Cells(Rows.Count, 1).End(xlUp).Row To 1 Step -1
            If WorksheetFunction.CountIf(Rng1, .Cells(i, 1).Value) > 0 Then
                Rows(i).EntireRow.Interior.ColorIndex = 5
                Rows(i).EntireRow.Copy
                Rows(i + 1).EntireRow.Insert Shift:=xlDown
                .Cells(i + 1, 2).Value = .Cells(i, 3).Value
            End If
        Next
    End With
   
    Application.EnableEvents = True
    Application.Calculation = xlCalculationManual = True
    Application.ScreenUpdating = True
End Sub
Thanks but that doesn't do anything?
 
Upvote 0
I think I know why but don't understand why. When I run the code normally from the list of macros in my PMW it runs as it should. But when I add it to QAT this is when it fails and does as above?
 
Upvote 0

Forum statistics

Threads
1,223,911
Messages
6,175,337
Members
452,637
Latest member
Ezio2866

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