nav[aria-label="Primary Navigation"] { padding: 0; & ul { list-style: none; width: 100%; display: flex; flex-direction: row; justify-content: start; align-items: start; gap: 30px; padding: 0; & li { margin: 0; } & ul li { list-style: none; } } }

update cursor invalid sql statement

BrianLomas

I'm new to python so forgive me if this seems like a dumb question... I work for the city and we get parcel information monthly from the county. Usually the data comes in a text file which I then put in an mdb. I'm trying to write some code to run an update our layers with the updated monthly information. Currently, I've just been trying to get a few fields to update inside a 'test' file geodatabase but I'm stuck. I get the following error message with the following code. Any ideas?

Najděte více příspěvků označených jako

Accepted answers

XanderBakker

You could use a dictionary with a list of values, in stead of a dictionary with a single field as value. This would require some changes in building the dictionary and the way the values are retrieved in the update cursor, but it can be done. If the code you tested was fast, it means that you don't have that many features to process. In that case it would not be a problem to create this larger dictionary (with lists as values).

All comments

BrianLomas

I was having difficulty getting the dictionaries working correctly but I stumbled onto a great blog post which was the exact thing I was looking for (link below). Thanks for everyone's help, I've learned a lot from you during this. I look forward to using python for more automation purposes in the future.

Turbo Charging Data Manipulation with Python Cursors and Dictionaries

import arcpy 
... 
... sourceFC = "...January2015Ownership.mdb\January2015Ownership"
... 
... sourceFieldsList = ["PARCEL_","SITUS", "CURRENT_OWNER", "CO_OWNER", "OWNER_MAILADDR", "OWNER_CITY_STATE", "OWNER_ZIPCODE", "TAX_DISTRICT", "TAX_CODE", "SEC", "TOWNSHIP", "RANGE", "LEGAL"] 
... 
... # Use list comprehension to build a dictionary from a da SearchCursor 
... valueDict = {r[0]:(r[1:]) for r in arcpy.da.SearchCursor(sourceFC, sourceFieldsList)} 
... 
... updateFC = "...\ParcelPythonCOPIES.gdb\TaxParcel_COPY"
... 
... updateFieldsList = ['PARCELID','SITEADDRESS','OWNERNME1', 'OWNERNME2', 'PSTLADDRESS', 'PSTLCITY', 'ZIPCODE', 'TAX_DIST', 'TAX_CODE', 'SEC', 'TWN', 'RGE', 'PRPRTYDSCRP'] 
... 
... with arcpy.da.UpdateCursor(updateFC, updateFieldsList) as updateRows: 
...    for updateRow in updateRows: 
...        # store the Join value of the row being updated in a keyValue variable 
...        keyValue = updateRow[0] 
...        # verify that the keyValue is in the Dictionary 
...        if keyValue in valueDict: 
...            # transfer the values stored under the keyValue from the dictionary to the updated fields. 
...            for n in range (1,len(sourceFieldsList)): 
...                updateRow = valueDict[keyValue][n-1] 
...            updateRows.updateRow(updateRow) 
... 
... del valueDict

bixb0012

Please remember to mark your question as answered.

XanderBakker

BrianLomas

This worked great and was very fast. However, I'm a bit confused how I can get this to cycle through multiple fields? Maybe use a getparameter to manually set the field after each cycle?

BrianLomas

Thanks for the idea... Ultimately, there will be 12 different fields that will need to be updated. The intention of this script was to run a test on just a few fields (current_ow & situs) for 'proof of concept'.

WesMiller

10,000 features wouldn't be a problem. You could simply join the 2 tables then use field calculator.

XanderBakker

I would probably do something along the lines of:

import arcpy
countyTB = r'...January2015Ownership.mdb\January2015Ownership'
fc = r'...ParcelPythonCOPIES.gdb\Parcel_092015COPY'

# create a dictionary
dct = {r[0]: r[1] for r in arcpy.da.SearchCursor(countyTB, ('PARCEL_', 'TheOtherFieldNameAtIndex1'))}

with arcpy.da.UpdateCursor(fc, ('parcel_id', 'current_ow')) as curs:
    for row in curs:
        parcel_id = row[0]
        if parcel_id in dct:
            row[1] = dct[parcel_id]
            curs.updateRow(row)

You are reading all the fields of countyTB when you only seem to need two. For the copy you are reading the field 'situs' which I don't think you are using. Using a dictionary and looking up the data might make things a bit faster, but if your have a lot of features, it may be slow.

BrianLomas

yea, 'situs' and 'current_ow' are actual field names.

BrianLomas

I wasn't sure on how to do that, so thanks for the info. Here's the code.

import arcpy
countyTB =...January2015Ownership.mdb\January2015Ownership'
fc = ...ParcelPythonCOPIES.gdb\Parcel_092015COPY"
with arcpy.da.SearchCursor(countyTB, ["*"]) as rows:
    for row in rows:
        fields = rows.fields
        idfield = fields.index('PARCEL_')
        with arcpy.da.UpdateCursor(fc,['situs', 'current_ow'], "%s = %s" % ('parcel_id', row[idfield])) as urows:
            for urow in urows:
                  urow[1] = row[1]
                  urows.UpdateRow(urow)
                  del urow
        del row

DanPatterson_Retired

are your fields really situs and current_ow or those supposed to be sites and current_row which I did a brain translation to until I looked again

BrianLomas

I thought about model builder but this will run for more than 10,000 features so I was worried about it being too slow. Although, I'm not too familiar with MB so I may be wrong.

VinceAngelo

Please post Python questions with the code in the question as text (formatted in a code block) -- It allows others to try your code without retyping it from scratch. If you need to redact pieces, be sure to explain what they were sufficiently so that suitable scaffolding can be applied.

- V

WesMiller

Have you tried doing this in model builder. Most everything you'll need should be in there and it will give you a head start if you need to do any coding.

What is ModelBuilder?—Help | ArcGIS for Desktop

DarrenWiens2

Try building your SQL expression in a previous line, then printing it to confirm that it is what you think it is.

BrianLomas

Oops I forgot I corrected that one... When that is taken out I get the following error...

Runtime error

Traceback (most recent call last):

File "<string>", line 9, in <module>

RuntimeError: An invalid SQL statement was used. [Parcel_092015COPY]

DarrenWiens2

Not sure about the rest, but you don't have the same number of single and double quotes.

Připojte se k diskusi

Přihlášení členové mohou přispívat, sledovat aktualizace a další. Jste tu noví? Zaregistrujte si bezplatný účet.

Rychlé odkazy

Community Help
Find useful guides, FAQs, and documents to help you navigate and make the most of Esri Community.