Solved

SQL between query and MS Access Help in VB

Posted on 2006-11-24
11
1,176 Views
Last Modified: 2013-12-25
Hey there Experts

I have a SQL query that fills a record set and then displays the result in a text box but the query is not working the syntax I think is right but the record set does not get filled.

Here is the code

Dim sqlstring As String
Dim myRS As New ADODB.Recordset

Set myConnection = New ADODB.Connection
myConnection.Open "PROVIDER=Microsoft.jet.OLEDB.4.0;data source=" & App.Path & "\Database\Database.mdb" & ";"

myRS.CursorLocation = adUseClient
sqlstring = "SELECT CLIENT.CSURNAME,CLIENT.CFIRST,CLIENT.STREET,CLIENT.STREET2 FROM CLIENT, TREAT WHERE TREAT.DATE BETWEEN " & CDate(CustomStartDate) & " AND " & CDate(CustomEndDate) & ""
Set myRS = myConnection.Execute(sqlstring)

       
        Do Until myRS.EOF
        If IsNull(myRS!Street2) = True Then
        txtResults.SelText = myRS!CFIRST + " " + myRS!CSURNAME + " " + myRS!Street + vbCrLf
        Else
        txtResults.SelText = myRS!CFIRST + " " + myRS!CSURNAME + " " + myRS!Street + " " + myRS!Street2 + vbCrLf
        End If
        myRS.MoveNext
        Loop
       
 myRS.Close

Thanks

Nothern_Lights
0
Comment
Question by:Nothern_Lights
11 Comments
 
LVL 65

Expert Comment

by:rockiroads
ID: 18009262
try using hashes

sqlstring = "SELECT CLIENT.CSURNAME,CLIENT.CFIRST,CLIENT.STREET,CLIENT.STREET2 FROM CLIENT, TREAT WHERE TREAT.DATE BETWEEN #" & CDate(CustomStartDate) & "# AND #" & CDate(CustomEndDate) & "#"

other way is to use format

sqlstring = "SELECT CLIENT.CSURNAME,CLIENT.CFIRST,CLIENT.STREET,CLIENT.STREET2 FROM CLIENT, TREAT WHERE Format(TREAT.DATE,'YYYYMMDD') BETWEEN " & Format(CDate(CustomStartDate),"YYYYMMDD") & " AND " & Format(CDate(CustomEndDate),"YYYYMMDD")

sqlstring = "SELECT CLIENT.CSURNAME,CLIENT.CFIRST,CLIENT.STREET,CLIENT.STREET2 FROM CLIENT, TREAT WHERE Format(TREAT.DATE,'YYYYMMDD') >= " & Format(CDate(CustomStartDate),"YYYYMMDD") & " AND Format(TREAT.DATE,'YYYYMMDD')  <= " & Format(CDate(CustomEndDate),"YYYYMMDD")
0
 

Author Comment

by:Nothern_Lights
ID: 18009353
hey rockiroads

The code you gave me is just displaying the whole client table multiple times.

What I need the SQL query to do is to take the customstartdate and the customenddate and search the client and treat table and findout what clients have visited the shop within the dates specified

Nothern_Lights
0
 
LVL 65

Expert Comment

by:rockiroads
ID: 18009362
ok, what links the client and the treat table?
u need to perform the join

SELECT CLIENT.CSURNAME,CLIENT.CFIRST,CLIENT.STREET,CLIENT.STREET2
FROM CLIENT, TREAT
WHERE CLIENT.somefeld = TREAT.somefield


then u do the filter on the TREAT date
use the filter that worked for u, but dont start with WHERE, put a AND as we already have a WHERE

0
Free Tool: Site Down Detector

Helpful to verify reports of your own downtime, or to double check a downed website you are trying to access.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

 

Author Comment

by:Nothern_Lights
ID: 18009367
So would the query be

sqlstring = "SELECT CLIENT.CSURNAME,CLIENT.CFIRST,CLIENT.STREET,CLIENT.STREET2,MAX(TREAT.DATE) FROM CLIENT, TREAT WHERE CLIENT.CCODE = TREAT.CCODE AND TREAT.DATE BETWEEN #" & CDate(CustomStartDate) & "# AND #" & CDate(CustomEndDate) & GROUP BY CLIENT.CSURNAME,CLIENT.CFIRST,CLIENT.STREET,CLIENT.STREET2 "

0
 
LVL 65

Expert Comment

by:rockiroads
ID: 18009403
Yes, u need to join the two fields and use the appropriate filter on the dates that worked for you

0
 

Author Comment

by:Nothern_Lights
ID: 18009454
Ok I have made my query but it does not fill the recordset

sqlstring = "SELECT CLIENT.CSURNAME,CLIENT.CFIRST,CLIENT.STREET,CLIENT.STREET2 FROM CLIENT, TREAT WHERE CLIENT.CCODE = TREAT.CCODE AND TREAT.DATE >= " & CDate(CustomStartDate) & " AND TREAT.DATE <= " & CDate(CustomEndDate) & " GROUP BY CLIENT.CSURNAME,CLIENT.CFIRST,CLIENT.STREET,CLIENT.STREET2"

and i have tried

sqlstring = "SELECT CLIENT.CSURNAME,CLIENT.CFIRST,CLIENT.STREET,CLIENT.STREET2 FROM CLIENT, TREAT WHERE CLIENT.CCODE = TREAT.CCODE AND TREAT.DATE >= #" & CDate(CustomStartDate) & "# AND TREAT.DATE <= #" & CDate(CustomEndDate) & "# GROUP BY CLIENT.CSURNAME,CLIENT.CFIRST,CLIENT.STREET,CLIENT.STREET2"

HELP btw i have never used SQL b4

Nothern_Lights
0
 
LVL 65

Accepted Solution

by:
rockiroads earned 500 total points
ID: 18010509
I should really concentrate more!
I didnt notice how u opened the recordset, yours is incorrect, .Execute is used for action queries like insert/delete
for select's u need to use Open
eg

    Dim sqlstring As String
    Dim myConnection As ADODB.Connection
    Dim myRS As New ADODB.Recordset
   
    Set myConnection = New ADODB.Connection
    myConnection.Open "PROVIDER=Microsoft.jet.OLEDB.4.0;data source=" & app.Path & "\Database\Database.mdb" & ";"
   
    sqlstring = "YOUR FAVOURED SQL STRING HERE"

'SETUP RECORDSET    
    myRS.CursorType = adOpenKeyset
    myRS.LockType = adLockPessimistic
    myRS.CursorLocation = adUseClient
    myRS.Open sqlstring
   
'WHY ARE U USING selText FOR
    Do Until myRS.EOF
        If IsNull(myRS!Street2) = True Then
            txtResults.SelText = myRS!CFIRST + " " + myRS!CSURNAME + " " + myRS!Street + vbCrLf
        Else
            txtResults.SelText = myRS!CFIRST + " " + myRS!CSURNAME + " " + myRS!Street + " " + myRS!Street2 + vbCrLf
        End If
        myRS.MoveNext
    Loop
   
    myRS.Close
    Set myRS = Nothing
0
 
LVL 3

Expert Comment

by:taycuong76
ID: 18025553
Rockiroads's tutorial is clearly and easy to understand.
0
 
LVL 75

Expert Comment

by:Anthony Perkins
ID: 18034604
>>.Execute is used for action queries like insert/delete for select's u need to use Open<<
You may want to double check this.  There is nothing wrong with using Execute to return a recordset.  In fact, providing a firehose (forward-only, read-only) cursor can be used it will give you the best performance.
0
 

Author Comment

by:Nothern_Lights
ID: 18037847
Hey there rockiroads

I put in the code you posted and added my sqlstring i got an error at myRS.Open sqlstring saying

Run-time error `3709`:

"The connection cannot be used to perform this operation. It is wither closed or invalid in this context"
0
 
LVL 65

Expert Comment

by:rockiroads
ID: 18085450
Sorry mate, I stopped using EE for a short while then had a massive catch up.

Regarding this error, I forgot to add the connection object to when u open the recordset


it should be

    myRS.Open sqlstring, myConnection



0

Featured Post

Free Tool: IP Lookup

Get more info about an IP address or domain name, such as organization, abuse contacts and geolocation.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

Question has a verified solution.

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

Suggested Solutions

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…
I was working on a PowerPoint add-in the other day and a client asked me "can you implement a feature which processes a chart when it's pasted into a slide from another deck?". It got me wondering how to hook into built-in ribbon events in Office.
Get people started with the process of using Access VBA to control Excel using automation, Microsoft Access can control other applications. An example is the ability to programmatically talk to Excel. Using automation, an Access application can laun…
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…

828 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