Counter in For Each loop

Possum17

New Member
Joined
Mar 4, 2024
Messages
3
Office Version
  1. 365
Platform
  1. Windows
I created a file which will calculate sales rep commission based on terms and conditions of the commission agreement. All sales rep performance data is filled in on one tab, and then through selections on drop down boxes, the file will generate the correct payout to be paid to relevant sales reps.

The file functions correctly for manually doing one rep at a time. This is very time consuming and seems like something that could be automated in vba. I post the macro I have written below. I *think* this functions correctly in all aspects except it always pastes values to outputs in AH/AI/AJ/AK6. Each line is unique to a sales rep, so I think I need a counter in my For Each loop which will bump the paste target cells down one line after each AH/AI/AJ/AK set. I am having trouble getting this part of the code to work. I tried to insert a For To counter within the code in various spots but I couldn't get it to work.

Any input appreciated!

ps. if it wasn't obvious enough already, I am very new to writing my own VBA :)

VBA Code:
Sub ForEachRepEnhanced()
'
' ForEachRep Macro
' Loops each rep
'

'
    Dim Cell As Range
    
    For Each Cell In Sheets("1.Information from Target Sheet").Range("B6:B31")
            Sheets("Rep Select").Range("D3").Value = Cell.Value
            Sheets("3. Payout").Select
            Range("D9:H11").Select
            Selection.ClearContents
            Sheets("2. Sales data").Select
            Range("D3").Value = 1
            Sheets("3. Payout").Select
            Range("D21").Select
            Selection.Copy
            Sheets("Rep data").Select
            Range("AH6").Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
                   :=False, Transpose:=False
            Sheets("3. Payout").Select
            Range("D17:H17").Select
            Application.CutCopyMode = False
            Selection.Copy
            Range("D9").Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
                :=False, Transpose:=False
            Sheets("2. Sales data").Select
            Range("D3").Value = 2
            Sheets("3. Payout").Select
            Range("D21").Select
            Selection.Copy
            Sheets("Rep data").Select
            Range("AI6").Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
                :=False, Transpose:=False
            Sheets("3. Payout").Select
            Range("D17:H17").Select
            Application.CutCopyMode = False
            Selection.Copy
            Range("D10").Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
                :=False, Transpose:=False
            Sheets("2. Sales data").Select
            Range("D3").Value = 3
            Sheets("3. Payout").Select
            Range("D21").Select
            Selection.Copy
            Sheets("Rep data").Select
            Range("AJ6").Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
                :=False, Transpose:=False
            Sheets("3. Payout").Select
            Range("D17:H17").Select
            Application.CutCopyMode = False
            Selection.Copy
            Range("D11").Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
                :=False, Transpose:=False
            Sheets("2. Sales data").Select
            Range("D3").Value = 4
            Application.CutCopyMode = False
            Range("D4").Select
            Sheets("3. Payout").Select
            Range("D21").Select
            Selection.Copy
            Sheets("Rep data").Select
            Range("AK6").Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
                :=False, Transpose:=False
            Sheets("3. Payout").Select
            Range("D9:H11").Select
            Application.CutCopyMode = False
            Selection.ClearContents
    Next Cell
End Sub
 

Excel Facts

Save Often
If you start asking yourself if now is a good time to save your Excel workbook, the answer is Yes
Stepping through, I also think this part is not working. I want this to update a cell based on the cell reference in the loop, but that doesn't seem to work.

VBA Code:
Sheets("Rep Select").Range("D3").Value = Cell.Value
 
Upvote 0
Think I figured it out. This is what I ended up with. Seems to work.

VBA Code:
Sub ForEachRepEnhanced()
'
' ForEachRep Macro
' Loops each rep
'

'
    
    For Each Rng In Sheets("Rep data").Range("B6:B31")
            Range("A1").Value = Rng.Value
            Sheets("3. Payout").Select
            Range("D9:H11").Select
            Selection.ClearContents
            Sheets("2. Sales data").Select
            Range("D3").Value = 1
            Sheets("3. Payout").Select
            Range("D21").Select
            Selection.Copy
            Sheets("Rep data").Select
            Rng.Offset(0, 32).Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
                   :=False, Transpose:=False
            Sheets("3. Payout").Select
            Range("D17:H17").Select
            Application.CutCopyMode = False
            Selection.Copy
            Range("D9").Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
                :=False, Transpose:=False
            Sheets("2. Sales data").Select
            Range("D3").Value = 2
            Sheets("3. Payout").Select
            Range("D21").Select
            Selection.Copy
            Sheets("Rep data").Select
            Rng.Offset(0, 33).Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
                :=False, Transpose:=False
            Sheets("3. Payout").Select
            Range("D17:H17").Select
            Application.CutCopyMode = False
            Selection.Copy
            Range("D10").Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
                :=False, Transpose:=False
            Sheets("2. Sales data").Select
            Range("D3").Value = 3
            Sheets("3. Payout").Select
            Range("D21").Select
            Selection.Copy
            Sheets("Rep data").Select
            Rng.Offset(0, 34).Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
                :=False, Transpose:=False
            Sheets("3. Payout").Select
            Range("D17:H17").Select
            Application.CutCopyMode = False
            Selection.Copy
            Range("D11").Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
                :=False, Transpose:=False
            Sheets("2. Sales data").Select
            Range("D3").Value = 4
            Application.CutCopyMode = False
            Range("D4").Select
            Sheets("3. Payout").Select
            Range("D21").Select
            Selection.Copy
            Sheets("Rep data").Select
            Rng.Offset(0, 35).Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
                :=False, Transpose:=False
            Sheets("3. Payout").Select
            Range("D9:H11").Select
            Application.CutCopyMode = False
            Selection.ClearContents
            Sheets("Rep Data").Select
            Range("A1").Select
            Selection.ClearContents
            ClearClipboard
    Next Rng
End Sub
 
Upvote 0
Solution

Forum statistics

Threads
1,223,884
Messages
6,175,173
Members
452,615
Latest member
bogeys2birdies

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