Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people, just like you, are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
Solved

Excel VBA that resets a variable every 29 rows

Posted on 2013-11-13
6
178 Views
Last Modified: 2013-11-14
I have a code that loops through Range([F4], [D10000].End(xlUp).Offset(0, 2)), and looks for a the value of the variable iRow. Right now, iRow is determined by a helper column (Column G) which cycles through the numbers 4 to 32. The code finds the value for iRow in Column G:
iRow = cel.Offset(0, 1)

Open in new window

But I would like to determine iRow with code that resets itself to 4 every 29 rows without the need for the helper column. When the code is looking at F4, iRow = 4, for F5, iRow = 5, etc. And the series repeats itself starting with F33, F62, F91 etc.

Here's my current code:
For Each cel In Range([F4], [D10000].End(xlUp).Offset(0, 2)) 
    iRow = cel.Offset(0, 1)
    iCol = [AllWeeks].Find(what:=cel.Offset(0, -1), LookIn:=xlValues).Column - 1 'Cells(4, iCol).Select 'Cells(iRow, 100).End(xlToLeft).Offset(0, -1).Select
    Set lft = Cells(iRow, iCol)
    Set rgt = lft.Offset(0, 1)
    If rgt = "" Then
        Set rgt = rgt.End(xlToLeft)
        Set lft = rgt.Offset(0, -1)
    End If 'Range(lft, rgt).Select
    lCel = lft.Address(0, 0)
    rCel = rgt.Address(0, 0)    
    cel.Formula = "=IFERROR(AVERAGE(" & lCel & ":" & rCel & ")/30.5*7,"""")"
    If cel.Offset(0, 2) = "RFS" Then cel = ""  
Next cel

Open in new window

All I need is something that goes back to iRow = 4 when the macro gets to F33, and again at F62, and F91 etc.

I hope i haven't made this too confusing.

Thanks,
John
0
Comment
Question by:gabrielPennyback
  • 3
  • 2
6 Comments
 
LVL 5

Expert Comment

by:Dontmilkthis
ID: 39646704
so can you simply use the 'mod' operator to get the desired irow?

irow = cel.Row mod 29


at F4:
cel.Row = 4
4 mod 29 = 4
irow = 4

at F33
cel.Row = 33
33 mod 29 = 4
irow = 4

at F34
cel.Row = 34
34 mod 29 = 5
irow = 5


and so on...
0
 
LVL 5

Accepted Solution

by:
Dontmilkthis earned 280 total points
ID: 39646805
I had a further thought on this issue and realised a slight problem with the code, in that you'll get 0,1,2,3 sequences, which I imagine aren't wanted based on your explanation.

I believe this would fix it

irow = ((cel.Row - 4) mod 29) + 4
0
 
LVL 81

Assisted Solution

by:byundt
byundt earned 220 total points
ID: 39646940
I put your code in a sub so I could check Compilation.

To avoid needing the auxiliary column, I used iRow as a counter. It is initialized to 3, then an IIf statement either adds 1 or resets it to 4. Though this is one more statement than the Mod approach suggested by Dontmilkthis, you may find the logic easier to  understand.
Sub Test()
Dim cel As Range, lft As Range, rgt As Range
Dim iRow As Long, iCol As Long
Dim lCel As String, rCel As String
iRow = 3
For Each cel In Range([F4], [D10000].End(xlUp).Offset(0, 2))
    iRow = IIf(iRow = 32, 4, iRow + 1)
    iCol = [AllWeeks].Find(what:=cel.Offset(0, -1), LookIn:=xlValues).Column - 1 'Cells(4, iCol).Select 'Cells(iRow, 100).End(xlToLeft).Offset(0, -1).Select
    Set lft = Cells(iRow, iCol)
    Set rgt = lft.Offset(0, 1)
    If rgt = "" Then
        Set rgt = rgt.End(xlToLeft)
        Set lft = rgt.Offset(0, -1)
    End If 'Range(lft, rgt).Select
    lCel = lft.Address(0, 0)
    rCel = rgt.Address(0, 0)
    cel.Formula = "=IFERROR(AVERAGE(" & lCel & ":" & rCel & ")/30.5*7,"""")"
    If cel.Offset(0, 2) = "RFS" Then cel = ""
Next cel
End Sub

Open in new window

0
Free Tool: Path Explorer

An intuitive utility to help find the CSS path to UI elements on a webpage. These paths are used frequently in a variety of front-end development and QA automation tasks.

One of a set of tools we're offering as a way of saying thank you for being a part of the community.

 
LVL 5

Expert Comment

by:Dontmilkthis
ID: 39646952
I'd be interested in seeing which is more efficient from a CPU cycle point of view...

might be worth actually testing that aspect if they both work as expected.

performing two additions and a mod operation might be a bit more expensive than the if else statement and an addition.
0
 
LVL 1

Author Comment

by:gabrielPennyback
ID: 39648736
Thank you both! Brad, I actually came up with something last night based on iRow +1, but  the Double "I" If statement is new to me and more elegant than my If-Then statement.

Dontmilkthis, I didn't know how to use the mod statement and I love the fact that it's a one-liner.

Again, thank you both.

~ John
0
 
LVL 1

Author Closing Comment

by:gabrielPennyback
ID: 39648738
They should have an"A+ Sublime" category.
0

Featured Post

Networking for the Cloud Era

Join Microsoft and Riverbed for a discussion and demonstration of enhancements to SteelConnect:
-One-click orchestration and cloud connectivity in Azure environments
-Tight integration of SD-WAN and WAN optimization capabilities
-Scalability and resiliency equal to a data center

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Convert between Excel file formats (.XLS, .XLSX, .XLSM) with/without macro option David Miller (dlmille) Intro Over this past Fall, I've had the opportunity to see several similar requests and have developed a couple related solutions associate…
How to quickly and accurately populate Word documents with Excel data, charts and images (including Automated Bookmark generation) David Miller (dlmille) Synopsis In this article you’ll learn how to use ExcelToWord! to copy data,charts, shapes …
The viewer will learn how to use the =DISCRINV command to create a discrete random variable, use this command to model a set of probabilities and outcomes in a Monte Carlo simulation, and learn how to find the standard deviation of a set of probabil…
Graphs within dashboards are meant to be dynamic, representing data from a period of time that will change each time the dashboard is updated with new data. Rather than update each graph to point to a different set within a static set of data, t…

809 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question