Solved

MS Access ADO open question

Posted on 2010-09-21
2
539 Views
Last Modified: 2012-05-10
I have the following code in an Access 2007 DB.  When I get to the line that is supposed to open rstIW I get an error that states:
Run time error 91
Object variable or With Block variable not set.

I tried creating a second connection called cnnLocal2 but that didn't work, I got the same error.
It's probably a rookie mistake (because I am one) but I can't see it.  What am I doing wrong?

Sub Extract_Raw_Data()
    Dim cnnLocal As New ADODB.Connection
    Dim rstExcel As New ADODB.Recordset
    Dim rstIW As ADODB.Recordset
    Dim IWType As String
    Dim IWSubType As String
    Set cnnLocal = CurrentProject.Connection
    rstExcel.Open "Select * from ImageWires", cnnLocal, adOpenStatic, adLockPessimistic
    rstIW.Open "Select * from tblIW WHERE 1 = 0", cnnLocal, adOpenDynamic, adLockOptimistic
   
    While Not rstExcel.EOF
        If rstExcel.Fields("F2") = "Pre-Labeled ImageWires 1.5" Or _
           rstExcel.Fields("F2") = "Pre-Labeled ImageWires 1.8" Or _
           rstExcel.Fields("F2") = "Pre-Labeled C7 Dragonfly" Then
            IWType = rstExcel.Fields("F2")
        End If
        If (rstExcel.Fields("F2") = "12424-00" Or _
           rstExcel.Fields("F2") = "12513-00" Or _
           rstExcel.Fields("F2") = "13751-00") And _
           rstExcel.Fields("F4") > " " Then   'Use the part # to indicate data rec and Lot # to show data present
            rstIW.AddNew
            rstIW.Fields("IW PN") = rstExcel.Fields("F2")
            rstIW.Fields("IW Lot") = rstExcel.Fields("F4")
            rstIW.Fields("IW Qty") = rstExcel.Fields("F6")
            If IsNumeric(rstExcel.Fields("F8")) Then
                rstIW.Fields("IW Exp_Date") = rstExcel.Fields("F8")
                rstIW.Fields("IW Status") = Null
            Else
                rstIW.Fields("IW Exp_Date") = Null
                rstIW.Fields("IW Status") = rstExcel.Fields("F8")
            End If
            rstIW.Fields("IW Type") = IWType
            rstIW.Update
        End If
       
        rstExcel.MoveNext
    Wend
    rstExcel.Close
End Sub

0
Comment
Question by:RichNH
2 Comments
 
LVL 6

Expert Comment

by:UsamaFoad
ID: 33730419
I think the problem in:

IWType = rstExcel.Fields("F2")

You set the "IWType" in the first IF statement but you do not if this condition has not been achieved. So in line 32 (rstIW.Fields("IW Type") = IWType) the "IWType" generate the error. You need to set "IWType" before the second "IF".
Also I think nesting the first "IF" with the other "IF"s will be better.


Sub Extract_Raw_Data()
    Dim cnnLocal As New ADODB.Connection
    Dim rstExcel As New ADODB.Recordset
    Dim rstIW As ADODB.Recordset
    Dim IWType As String
    Dim IWSubType As String
    Set cnnLocal = CurrentProject.Connection
    rstExcel.Open "Select * from ImageWires", cnnLocal, adOpenStatic, adLockPessimistic
    rstIW.Open "Select * from tblIW WHERE 1 = 0", cnnLocal, adOpenDynamic, adLockOptimistic
    
    While Not rstExcel.EOF
        If rstExcel.Fields("F2") = "Pre-Labeled ImageWires 1.5" Or _
           rstExcel.Fields("F2") = "Pre-Labeled ImageWires 1.8" Or _
           rstExcel.Fields("F2") = "Pre-Labeled C7 Dragonfly" Then
            IWType = rstExcel.Fields("F2")
        else
			IWType = ' =============================You need to set IWType here======================
			If (rstExcel.Fields("F2") = "12424-00" Or _
				rstExcel.Fields("F2") = "12513-00" Or _
				rstExcel.Fields("F2") = "13751-00") And _
				rstExcel.Fields("F4") > " " Then   'Use the part # to indicate data rec and Lot # to show data present
				rstIW.AddNew
				rstIW.Fields("IW PN") = rstExcel.Fields("F2")
				rstIW.Fields("IW Lot") = rstExcel.Fields("F4")
				rstIW.Fields("IW Qty") = rstExcel.Fields("F6")
				If IsNumeric(rstExcel.Fields("F8")) Then
					rstIW.Fields("IW Exp_Date") = rstExcel.Fields("F8")
					rstIW.Fields("IW Status") = Null
				Else
					rstIW.Fields("IW Exp_Date") = Null
					rstIW.Fields("IW Status") = rstExcel.Fields("F8")
				End If
				rstIW.Fields("IW Type") = IWType
				rstIW.Update
			End If
        End If
        rstExcel.MoveNext
    Wend
    rstExcel.Close
End Sub

Open in new window

0
 
LVL 1

Accepted Solution

by:
RichNH earned 0 total points
ID: 33743307
Thank you for the response.  Actually I thought this note didn't go in.  I figured it out shortly after I finished typing it.  It was that I did not dim rstIW as a NEW ADODB.Recordset.  Once I put the New in there the code worked fine.

Regarduing the specific code.  The 1st If is a standalone.  The value only occurs on the first record (I'm reading an Excel Workbook that the inventory manager uses to manage consumables in our company).  Because the title (value) only occurs on the first record the IF has to be standalone as it will not be true on the 2nd - nth records.  I'm simply setting a temp value with the title of the data I'm about to read.

Thank you again.

Rich
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

Recently Microsoft released a brand new function called CONCAT. It's supposed to replace its predecessor CONCATENATE. But how does it work? And what's new? In this article, we take a closer look at all of this - we even included an exercise file for…
My experience with Windows 10 over a one year period and suggestions for smooth operation
In this fifth video of the Xpdf series, we discuss and demonstrate the PDFdetach utility, which is able to list and, more importantly, extract attachments that are embedded in PDF files. It does this via a command line interface, making it suitable …
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…

808 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