Go Premium for a chance to win a PS4. Enter to Win


setting recordset - Movefirst - maybe in wrong place w/i Code

Posted on 2010-11-28
Medium Priority
Last Modified: 2012-05-10
The code below is creating an export and new Excel workbook based on data within Access database and an Excel Template.  The code compiles the data creates the column headings and inputs the data into the Excel Workbook on the fly.

The code work correctly - creating the workbook, and compiling the Access data correctly, however, When the code comes to the rs.movefirst portion of the code that is suppose to input the data for Column headings w/i the Excel workbook, it moves to the error handling and gets stuck in a loop.  See '<<<<<<<<<<<<<<<<<<<<<<<< with in the code for problem error.

What am I missing?


' Procedure : cmdExportToExcel_Click
' DateTime  : 10/15/2010 13:12
' Author    : Karen F. Schaefer, DBA
' Purpose   : Export Training Data to Excel Template and populates the appropriate worksheets accordingly.
Private Sub cmdExportToExcel_Click()

   On Error GoTo cmdExportToExcel_Click_Error

On Error GoTo ProcError

'For Late Binding
'   Dim xlApp As Object

'For Early Binding
Dim xlApp As Excel.Application
Dim xlWb As Excel.Workbook
Dim xlWs As Excel.Worksheet, xlWs1 As Excel.Worksheet
Dim rs As DAO.Recordset
Dim rs1 As DAO.Recordset, rs2 As DAO.Recordset, rs3 As DAO.Recordset
Dim strSQL As String
Dim strSQL1 As String
Dim strSQL2 As String
Dim strSQL3 As String
Dim strFolder As String
Dim strFileName As String
Dim I As Long, intRecordCount As Long
Dim blnSuccess As Boolean
Dim gWkSht As String, gWkSht1 As String
Dim gBEMS As String

    StatusMsg Me, ""
    StatusMsg Me, "Please Wait while the data is being refreshed.", vbBlue
    StatusMsg Me, "Employee Training data has been updated.", vbBlue
    strFolder = GetUsersDesktopFolder
    strFileName = strFolder & "2010 FTI-ME Ent-DP.xls"

    'Determines the column headings for the Training Matrix spreadsheet(s)

    strSQL = "SELECT Mid([Ilp Learning Title],1,InStrRev([Ilp Learning Title],'(')-1) AS CourseTitle," & _
                " TL_CourseList.[Ilp Learning Cd] AS CourseNumber, TL_CourseList.[Delv Mthd Tot Hrs] AS Duration," & _
                " TL_CourseClassification.ClassType, TL_CourseList.StandardRequiredDt" & _
            " FROM TL_CourseClassification INNER JOIN TL_CourseList ON" & _
                " TL_CourseClassification.ClassificationRecID = TL_CourseList.ClassificationRecID" & _
            " WHERE (((TL_CourseList.OnXLS)=-1))" & _
            " GROUP BY Mid([Ilp Learning Title],1,InStrRev([Ilp Learning Title],'(')-1), TL_CourseList.[Ilp Learning Cd]," & _
                " TL_CourseList.[Delv Mthd Tot Hrs], TL_CourseClassification.ClassType, TL_CourseList.StandardRequiredDt" & _
            " ORDER BY TL_CourseList.StandardRequiredDt, TL_CourseList.[Ilp Learning Cd]"
    Set rs = CurrentDb.OpenRecordset(strSQL, dbOpenSnapshot)
    If rs.RecordCount = 0 Then
        MsgBox "There Are No Records to Export for the Courses Selected.", vbInformation, "No Data To Export..."
        GoTo ExitProc
        rs.MoveLast: rs.MoveFirst    'Required to get an accurate count of records.
        intRecordCount = rs.RecordCount
    End If

    If Dir(strFileName) <> "" Then
        Kill (strFileName)
    End If

    'Sets name of Excel worksheet within the Workbook based on above mentioned Template -
    'to be updated based on Unit Chief Name selected form listbox
    strSQL1 = "SELECT UCBEMS, WkshtName, wkshtname1 FROM qryUnitChief WHERE UnitChiefID In (" & MyString & ")"

    If rs.RecordCount = 0 Then
        Call _
                MsgBox("Please make a selection from the list, Click Update and then Click Export to Excel upon completion of the processing of the Update Data.", _
                       vbCritical, "No Data Found")
        Set rs1 = CurrentDb.OpenRecordset(strSQL1, dbOpenSnapshot)
        Set xlApp = CreateObject("Excel.Application")
        Set xlWb = xlApp.Workbooks.Add(CurrentProject.path & "\2010 FTI-ME Ent-DP.xlt")
        xlApp.Visible = True
        Do Until rs1.EOF
            gWkSht = rs1.Fields("WkshtName").value
            gBEMS = rs1.Fields("UCBEMS").value
            gWkSht1 = rs1.Fields("WkshtName1").value
            Set xlWs = xlWb.Worksheets(gWkSht)
            Set xlWs1 = xlWb.Worksheets(gWkSht1)
            'Copy course name and course number data, starting at cell F11 = Row 11, Column 6
            I = 6
            With xlWs
                StatusMsg Me, "Please Wait while the Course Titles are being updated for " & gWkSht1 & " .", vbBlue
                rs.MoveFirst '<<<<<<<<<<<<<<<<<<<<<<<<
                Do Until rs.EOF
                    .Cells(6, 3).value = Date                  'Date Report Ran
                    .Cells(6, I).value = rs!StandardRequiredDt 'Course Required by Date
                    .Cells(7, I).value = rs!Duration           'Course duration
                    .Cells(8, I).value = rs!ClassType          'Source of Course
                    .Cells(9, I).value = Trim(rs!CourseTitle)  'Course Title
                    .Cells(10, I).value = rs!CourseNumber      'Course ID
                    I = I + 1
                'Copy detail data, starting at cell "A11"(eleven)
                strSQL2 = "SELECT * FROM zTempData" & _
                        " WHERE UCBEMS IN(" & Chr(34) & gBEMS & Chr(34) & ")" & _
                        " ORDER BY EmployeeName"
                Set rs2 = CurrentDb.OpenRecordset(strSQL2, dbOpenSnapshot)
                    .Range("A11").CopyFromRecordset rs2
            End With
            blnSuccess = True
            StatusMsg Me, "Please Wait while the system formats " & gWkSht & ".", vbBlue
            FormatWS xlWs
            StatusMsg Me, "Please Wait while the data for " & gWkSht1 & " is being updated.", vbRed

            With xlWs1
                'Copy Summary data, starting at cell "A6"
                strSQL3 = "SELECT qryEmpInfo.Mgr_OrgNo," & _
                       " Sum(IIf([CompletionDt]<=[DateRequired],1,0)) AS OnTime, Sum(IIf([CompletionDt]>[DateRequired],1,0)) AS Late," & _
                       " Sum(IIf([DateRequired]>Date(),1,0)) AS Avail, Sum(IIf([DateRequired]<Date(),1,0)) AS Del," & _
                       " Count(TD_EmpReqLearning.CourseRecID) AS TotalCourseCt," & _
                       " IIf(Count([CourseRecID])=0,0,Sum(IIf([CompletionDt]<=[DateRequired],1,0))/Count([CourseRecID])) AS [%CompleteOnTime]," & _
                       " IIf(Count([CourseRecID])=0,0,Sum(IIf([CompletionDt]>[DateRequired],1,0))/Count([CourseRecID])) AS [%CompleteLate]," & _
                       " IIf(Count([CourseRecID])=0,0,Sum(IIf([DateRequired]<Date(),1,0))/Count([CourseRecID])) AS [%CompleteDel]" & _
                   " FROM TD_EmpReqLearning RIGHT JOIN qryEmpInfo ON TD_EmpReqLearning.BEMS = qryEmpInfo.TA_Empl.BEMS" & _
                   " WHERE (qryEmpInfo.UnitChiefID =" & Chr(34) & gBEMS & Chr(34) & ")" & _
                   " GROUP BY qryEmpInfo.UnitChiefID, qryEmpInfo.UnitChief, qryEmpInfo.Mgr_OrgNo" & _
                   " ORDER BY qryEmpInfo.UnitChief"
               Debug.Print strSQL3
               'Copy Summary data per Manager per Unit Chief
                Set rs3 = CurrentDb.OpenRecordset(strSQL3, dbOpenSnapshot)
                    .Range("A6").CopyFromRecordset rs3
            End With
        If blnSuccess = True Then
            StatusMsg Me, Mid(strFileName, Len(strFolder) + 1) & " report has been saved to your Desktop folder.", vbBlue
       End If
    End If
   ' optional but perhaps useful to monitor progress
  xlWb.SaveAs strFileName

    'Cleanup code
    rs1.Close: Set rs1 = Nothing
    rs2.Close: Set rs2 = Nothing
    rs3.Close: Set rs3 = Nothing
   Exit Sub
    ' error handling code
    Resume ExitProc

   On Error GoTo 0
   Exit Sub


    MsgBox "Error " & Err.Number & " (" & Err.Description & ") in procedure cmdExportToExcel_Click of VBA Document Form_frmExportToExcel"
End Sub

Open in new window

Question by:Karen Schaefer
LVL 65

Accepted Solution

rockiroads earned 2000 total points
ID: 34226216
My recommendation is to check records exists before navigating the recordset


this bit of code here

        Set rs1 = CurrentDb.OpenRecordset(strSQL1, dbOpenSnapshot)

Perhaps you should check rs1.eof property is not true before attempting the movefirst

        Set rs1 = CurrentDb.OpenRecordset(strSQL1, dbOpenSnapshot)
        if rs1.eof = true then
            msgbox "Nothing here"
            'rest of your code including

and same applies earlier on in your code

    Set rs = CurrentDb.OpenRecordset(strSQL, dbOpenSnapshot)
    If rs.recordCount = 0 Then

do not use recordcount to check no records, use eof instead

LVL 35

Expert Comment

ID: 34233017
Do you actually need to use MoveFirst anywhere?

I've only ever seen it used when data is being updated.

You can use code like this to populate a worksheet from a recordset.

Of course this is a straightforward transfer to rows with a column for each field.

It also doesn't include code for putting the headers on the worksheet but that could be done easily.

One way would be to loop through the fields collection of the recordset, which I've added.:)
conn.ConnectionString = strConn

    sql = "SELECT * FROM customers;"
    Set rs = New ADODB.Recordset
    rs.Open sql, conn

    Set rng = Worksheets("Sheet1").Range("A1")
    For Each fld In rs.Fields
        rng = fld.Name
        Set rng = rng.Offset(, 1)
    Next fld
    Set rng = Worksheets("Sheet1").Range("A2")
    While Not rs.EOF
        For Each fld In rs.Fields
            rng.Value = fld.Value
            Set rng = rng.Offset(, 1)
        Next fld
        Set rng = rng.Offset(1, 1 - rng.Column)

Open in new window


Author Closing Comment

by:Karen Schaefer
ID: 34233316
Thanks, I just reorder the code a little bit and did as you suggested and it works great.


Featured Post

Veeam and MySQL: How to Perform Backup & Recovery

MySQL and the MariaDB variant are among the most used databases in Linux environments, and many critical applications support their data on them. Watch this recorded webinar to find out how Veeam Backup & Replication allows you to get consistent backups of MySQL databases.

Question has a verified solution.

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

Did you know that more than 4 billion data records have been recorded as lost or stolen since 2013? It was a staggering number brought to our attention during last week’s ManageEngine webinar, where attendees received a comprehensive look at the ma…
Microsoft Access has a limit of 255 columns in a single table; SQL Server allows tables with over 255 columns, but reading that data is not necessarily simple.  The final solution for this task involved creating a custom text parser and then reading…
In Microsoft Access, learn the trick to repeating sub-report headings at the top of each page. The problem with sub-reports and headings: Add a dummy group to the sub report using the expression =1: Set the “Repeat Section” property of the dummy…
Polish reports in Access so they look terrific. Take yourself to another level. Equations, Back Color, Alternate Back Color. Write easy VBA Code. Tighten space to use less pages. Launch report from a menu, considering criteria only when it is filled…
Suggested Courses

916 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