AnsweredAssumed Answered

script works randomly when used in a toolbox

Question asked by rml on Apr 28, 2010
Latest reply on Apr 29, 2010 by rml

Like • Show 0 Likes0
Comment • 4

Hello

I have a very simple script that tests the number of objects selected in a layer to use as a precondition for other processes in the model. The script has 3 parameters - number to test, minimum and maximum - and 1 if condition. It works fine when executed in python but it gives random results when used in a toolbox.

I'm using ArcGIS 9.2 in Windows XP Professional. I've attached script and toolbox (as txt files)

Any one else with similar problems?

Thanks in advance

Rui

Attachments

script.txt.zip631 bytes
toolbox.txt.zip1.6 KB

No one else has this question

Outcomes

4 Replies

geonetadmin Apr 28, 2010 7:59 PM

Original User: fperks

Don't use sys.exit, in the past its given random results. Instead you should raise an exception, then ArcGIS will note the error.

I modified your code block:

#test number
if myCountOfObjects < myMinimum:
    #gp.AddError("The input should be more than " + str(myMinimum) +" objects and is currently " + str(myCountOfObjects))
    #print "The input should be " + str(myMinimum) +" or more objects and is currently " + str(myCountOfObjects)
    raise RuntimeError("The input should be " + str(myMinimum) +" or more objects and is currently " + str(myCountOfObjects))
    #sys.exit(0)
elif myCountOfObjects > myMaximum:
    #gp.AddError("The input should be less than " + str(myMaximum) +" objects and is currently " + str(myCountOfObjects))
    #print "The input should be " + str(myMaximum) + " or less objects and is currently " + str(myCountOfObjects)
    raise RuntimeError("The input should be " + str(myMaximum) + " or less objects and is currently " + str(myCountOfObjects))
    #sys.exit(0)
else:
    gp.AddMessage("Processing " + str(myCountOfObjects) + " objects")
    print "Processing " + str(myCountOfObjects) + " objects"

This would give you the proper effect you are looking for.

Actions

rml Apr 29, 2010 6:16 AM

Thanks for the code Frank but it makes no difference in the result. For instance if you try inputing the values 23 / 1 / 5 the result is still wrong (when using the toolbox)

Any more suggestions?

Rui

Don't use sys.exit, in the past its given random results. Instead you should raise an exception, then ArcGIS will note the error.

I modified your code block:

#test number
if myCountOfObjects < myMinimum:
    #gp.AddError("The input should be more than " + str(myMinimum) +" objects and is currently " + str(myCountOfObjects))
    #print "The input should be " + str(myMinimum) +" or more objects and is currently " + str(myCountOfObjects)
    raise RuntimeError("The input should be " + str(myMinimum) +" or more objects and is currently " + str(myCountOfObjects))
    #sys.exit(0)
elif myCountOfObjects > myMaximum:
    #gp.AddError("The input should be less than " + str(myMaximum) +" objects and is currently " + str(myCountOfObjects))
    #print "The input should be " + str(myMaximum) + " or less objects and is currently " + str(myCountOfObjects)
    raise RuntimeError("The input should be " + str(myMaximum) + " or less objects and is currently " + str(myCountOfObjects))
    #sys.exit(0)
else:
    gp.AddMessage("Processing " + str(myCountOfObjects) + " objects")
    print "Processing " + str(myCountOfObjects) + " objects"

This would give you the proper effect you are looking for.

Actions

geonetadmin Apr 29, 2010 6:27 AM
Mark Correct
Correct Answer
Original User: fperks

Oops, i missed the big error. When ever you pass in arguments to a script, they are always as strings.
Lets say you pass in 1 / 23 / 5...
So right now your myCountOfObjects is actually the string "1"

Now saying the string "1" > integer 23? (It will be true, since "1" string converted gives you ascii value of 49)

Well you can't compare an string with an integer directly since it will give you not the correct behavouor, so you need to convert it to an integer. E.g.:
```
myCountOfObjects = int(gp.GetParameterAsText(0))
myMinimum = int(gp.GetParameterAsText(1))
myMaximum = int(gp.GetParameterAsText(2))
```
Like • Show 0 Likes0
Actions
rml Apr 29, 2010 6:35 AM
Mark Correct
Correct Answer
Thanks a lot
Like • Show 0 Likes0
Actions

GeoNet

The Esri Community

script works randomly when used in a toolbox

Attachments

Outcomes

Related Content

Recommended Content