Solved

# condense 3 loops into 1

Posted on 2014-09-26
72 Views
This code has the same iterations 3 times.  How can I keep only one loop and simply modify only the 3 variables in bold.  I don't see a pattern for the ages that would work as I was thinking of putting a loop within a loop.

``````'Get List of 8 to 12 and under--------
esql = "DECLARE @fromdt AS datetime " & _
"DECLARE @todt AS datetime " & _
"SET @fromdt = '" & DTPicker1 & "' " & _
"SET @todt = '" & DTPicker2 & "' " & _
"SELECT  p.Agency, count(distinct r.RegID) " & _
"FROM tblOrgProfile as p " & _
"LEFT JOIN tblOrgRegistrations as r " & _
"ON p.AgencyID = r.AgencyID " & _
"AND r.AgeRegistration >= [b]8 [/b]" & _
"AND r.AgeRegistration <= [b]12 [/b]" & _
"LEFT JOIN tblOrgHours as h " & _
"ON h.RegID = r.regid " & _
"AND h.ActivityDate >= @fromdt " & _
"AND h.ActivityDate <= @todt " & _
"GROUP BY p.Agency " & _
"ORDER BY p.Agency"

i = 6

Do Until rec1.EOF = True

ApExcel.Workbooks("CYSReport.xls").Sheets("Direct Service").Cells(i, [b]12[/b]).Formula = rec1.Fields(1)

i = i + 2
rec1.MoveNext

Loop

rec1.Close

'Get List of 13 to 17 and under--------
esql = "DECLARE @fromdt AS datetime " & _
"DECLARE @todt AS datetime " & _
"SET @fromdt = '" & DTPicker1 & "' " & _
"SET @todt = '" & DTPicker2 & "' " & _
"SELECT  p.Agency, count(distinct r.RegID) " & _
"FROM tblOrgProfile as p " & _
"LEFT JOIN tblOrgRegistrations as r " & _
"ON p.AgencyID = r.AgencyID " & _
"AND r.AgeRegistration >= [b]13 [/b]" & _
"AND r.AgeRegistration <= [b]17 [/b]" & _
"LEFT JOIN tblOrgHours as h " & _
"ON h.RegID = r.regid " & _
"AND h.ActivityDate >= @fromdt " & _
"AND h.ActivityDate <= @todt " & _
"GROUP BY p.Agency " & _
"ORDER BY p.Agency"

i = 6

Do Until rec1.EOF = True

ApExcel.Workbooks("CYSReport.xls").Sheets("Direct Service").Cells(i, [b]13[/b]).Formula = rec1.Fields(1)

i = i + 2
rec1.MoveNext

Loop

rec1.Close

'Get List of 18 to 20 and under--------
esql = "DECLARE @fromdt AS datetime " & _
"DECLARE @todt AS datetime " & _
"SET @fromdt = '" & DTPicker1 & "' " & _
"SET @todt = '" & DTPicker2 & "' " & _
"SELECT  p.Agency, count(distinct r.RegID) " & _
"FROM tblOrgProfile as p " & _
"LEFT JOIN tblOrgRegistrations as r " & _
"ON p.AgencyID = r.AgencyID " & _
"AND r.AgeRegistration >= [b]18 [/b]" & _
"AND r.AgeRegistration <= [b]20 [/b]" & _
"LEFT JOIN tblOrgHours as h " & _
"ON h.RegID = r.regid " & _
"AND h.ActivityDate >= @fromdt " & _
"AND h.ActivityDate <= @todt " & _
"GROUP BY p.Agency " & _
"ORDER BY p.Agency"

i = 6

Do Until rec1.EOF = True

ApExcel.Workbooks("CYSReport.xls").Sheets("Direct Service").Cells(i, [b]14[/b]).Formula = rec1.Fields(1)

i = i + 2
rec1.MoveNext

Loop

rec1.Close

``````
0
Question by:al4629740

Author Comment

ID: 40346916
as you can see, the bold did not work for me but I think you can see the variables that need attention
0

LVL 67

Expert Comment

ID: 40346935
Something like this could work:

``````'Get List of 8 to 12 and under--------
for x as int16 = 8 to 22 step 5
if x = 18 then
y = 20
else
y = x + 4
end if
esql = "DECLARE @fromdt AS datetime " & _
"DECLARE @todt AS datetime " & _
"SET @fromdt = '" & DTPicker1 & "' " & _
"SET @todt = '" & DTPicker2 & "' " & _
"SELECT  p.Agency, count(distinct r.RegID) " & _
"FROM tblOrgProfile as p " & _
"LEFT JOIN tblOrgRegistrations as r " & _
"ON p.AgencyID = r.AgencyID " & _
"AND r.AgeRegistration >= " & x & " & _
"AND r.AgeRegistration <= " & y & " & _
"LEFT JOIN tblOrgHours as h " & _
"ON h.RegID = r.regid " & _
"AND h.ActivityDate >= @fromdt " & _
"AND h.ActivityDate <= @todt " & _
"GROUP BY p.Agency " & _
"ORDER BY p.Agency"

i = 6

Do Until rec1.EOF = True

ApExcel.Workbooks("CYSReport.xls").Sheets("Direct Service").Cells(i, [b]12[/b]).Formula = rec1.Fields(1)

i = i + 2
rec1.MoveNext

Loop

rec1.Close
next

``````
0

Accepted Solution

royeh earned 500 total points
ID: 40351899
Open 3 recordsets, and loop through them all:

E.G.

``````'Get List of 8 to 12 and under--------
esql1 = "DECLARE @fromdt AS datetime " & _
"DECLARE @todt AS datetime " & _
"SET @fromdt = '" & DTPicker1 & "' " & _
"SET @todt = '" & DTPicker2 & "' " & _
"SELECT  p.Agency, count(distinct r.RegID) " & _
"FROM tblOrgProfile as p " & _
"LEFT JOIN tblOrgRegistrations as r " & _
"ON p.AgencyID = r.AgencyID " & _
"AND r.AgeRegistration >= [b]8 [/b]" & _
"AND r.AgeRegistration <= [b]12 [/b]" & _
"LEFT JOIN tblOrgHours as h " & _
"ON h.RegID = r.regid " & _
"AND h.ActivityDate >= @fromdt " & _
"AND h.ActivityDate <= @todt " & _
"GROUP BY p.Agency " & _
"ORDER BY p.Agency"

'Get List of 13 to 17 and under--------
esql2 = "DECLARE @fromdt AS datetime " & _
"DECLARE @todt AS datetime " & _
"SET @fromdt = '" & DTPicker1 & "' " & _
"SET @todt = '" & DTPicker2 & "' " & _
"SELECT  p.Agency, count(distinct r.RegID) " & _
"FROM tblOrgProfile as p " & _
"LEFT JOIN tblOrgRegistrations as r " & _
"ON p.AgencyID = r.AgencyID " & _
"AND r.AgeRegistration >= [b]13 [/b]" & _
"AND r.AgeRegistration <= [b]17 [/b]" & _
"LEFT JOIN tblOrgHours as h " & _
"ON h.RegID = r.regid " & _
"AND h.ActivityDate >= @fromdt " & _
"AND h.ActivityDate <= @todt " & _
"GROUP BY p.Agency " & _
"ORDER BY p.Agency"

'Get List of 18 to 20 and under--------
esql3 = "DECLARE @fromdt AS datetime " & _
"DECLARE @todt AS datetime " & _
"SET @fromdt = '" & DTPicker1 & "' " & _
"SET @todt = '" & DTPicker2 & "' " & _
"SELECT  p.Agency, count(distinct r.RegID) " & _
"FROM tblOrgProfile as p " & _
"LEFT JOIN tblOrgRegistrations as r " & _
"ON p.AgencyID = r.AgencyID " & _
"AND r.AgeRegistration >= [b]18 [/b]" & _
"AND r.AgeRegistration <= [b]20 [/b]" & _
"LEFT JOIN tblOrgHours as h " & _
"ON h.RegID = r.regid " & _
"AND h.ActivityDate >= @fromdt " & _
"AND h.ActivityDate <= @todt " & _
"GROUP BY p.Agency " & _
"ORDER BY p.Agency"

i = 6

Do Until rec1.EOF = True And rec2.EOF = True And rec3.EOF = True

If rec1.EOF = False Then ApExcel.Workbooks("CYSReport.xls").Sheets("Direct Service").Cells(i, [b]12[/b]).Formula = rec1.Fields(1)
If rec2.EOF = False Then ApExcel.Workbooks("CYSReport.xls").Sheets("Direct Service").Cells(i, [b]13[/b]).Formula = rec2.Fields(1)
If rec3.EOF = False Then ApExcel.Workbooks("CYSReport.xls").Sheets("Direct Service").Cells(i, [b]14[/b]).Formula = rec3.Fields(1)

i = i + 2

If rec1.EOF = False Then rec1.MoveNext
If rec2.EOF = False Then rec2.MoveNext
If rec3.EOF = False Then rec3.MoveNext

Loop

rec1.Close
rec2.Close
rec3.Close
``````

You'll need to double-check the EOF flags, because you have multiple recordsets.
0

## Featured Post

Introduction While answering a recent question (http://www.experts-exchange.com/Q_27402310.html) in the VB classic zone, I wrote some VB code in the (Office) VBA environment, rather than fire up my older PC.  I didn't post completely correct code o…
Most everyone who has done any programming in VB6 knows that you can do something in code like Debug.Print MyVar and that when the program runs from the IDE, the value of MyVar will be displayed in the Immediate Window. Less well known is Debug.Asse…
Get people started with the utilization of class modules. Class modules can be a powerful tool in Microsoft Access. They allow you to create self-contained objects that encapsulate functionality. They can easily hide the complexity of a process from…
Show developers how to use a criteria form to limit the data that appears on an Access report. It is a common requirement that users can specify the criteria for a report at runtime. The easiest way to accomplish this is using a criteria form that a…