DCount looking for a "("

[GraeagleBill]

Compiler complaining that the DCount statment is invalid, requiring a "(". Seems that the problem is somehow more basic with regards to the syntax, but I haven't found it. I've tried "[NameExp] = """ & strComp & """" as the criteria, but that doesn't work either. What am I missing here?

Public Sub LoadtblGFM()
Dim rsSav As DAO.Recordset
Dim strSplit() As String
Dim strFName As String
Dim strLName As String
Dim strComp As String


strSplit = Split(strLine, " ")
strLName = strSplit(UBound(strSplit))
strFName = Left(strLine, Len(strLine) - Len(strLName))
strComp = strLName & "," & strFName
Set rsSav = DBEngine(0)(0).OpenRecordset("QGFM")


    rsSav.AddNew
    rsSav!LastName = strLName
    rsSav!Firstname = strFName
    rsSav!Donation = stramount
        If DCount("[NameExp]", "QADFnames", "[NameExp] = " & strComp) > 0 Then _
        rsSav!FLMember = True
    rsSav.Update


rsSav.Close
Set rsSav = Nothing


End Sub

[Edgar]

StrComp is a function
https://learn.microsoft.com/en-us/of...rcomp-function

Just change the name of that variable.

[June7]

If NameExp is a text field, need apostrophe delimiters in criteria.

Why don't you use * wildcard instead of field name?

Why use a string variable for numeric data?

Where do strLine and stramount get populated?

[GraeagleBill]

Thanks Edgar, that cleared things up.
Funny that the compiler didn't "wrap my knuckles" when I "Dim strComp As String"

[GraeagleBill]

If NameExp is a text field, need apostrophe delimiters in criteria.

Yes, I noted that in the OP
And yes, I could very well have used "*". My focus was on the field of interest at the time.

Where do strLine and stramount get populated?

Both are global to the module and set before LoadtblGFM sub is called.

Why use a string variable for numeric data?

Everything is string. Don't know where you're seeing numeric data?

[June7]

Donation field is not numeric? You populate it with stramount - that is not money?

VBA does not warn that variable names conflict with reserved words such as intrinsic functions.

[GraeagleBill]

Donation field is not numeric? You populate it with stramount - that is not money?

Probably a bit deceiving. It's a string from some line input already formatted as such, e.g., $100.00. Normally, such values are contained in fields with Currency format properties and would be treated in code as such. But again, from the input line I'm given a string.

[June7]

Interesting. I just tested an UPDATE action to set Double number field with a string value of "$100.00". Access (query object and CurrentDb.Execute) performed implicit conversion and accepted the input. Never had to consider that before.

[GraeagleBill]

Well the line-in left me with strings like "$100.00 (Est)" and "$40.50 (rebate)", etc left me with not much choice as to how I had to deal with the data types. Your points are, however, well taken.
Bill

[June7]

Guess only alternative would be splitting data to appropriate fields. Seems to be enough consistency it shouldn't be too difficult.

[Micron]

@GraeagleBill - thanks for destroying my long held belief that if you 'typed' your variables with prefixes like str, dbl, int, lng, etc. you could never go wrong!!
strComp?? You would have to know to know.