Zaczynam dopiero zaczynać pracę w skryptach Python.
Obecnie tworzę skrypt do automatyzacji procesu.
Zasadniczo pyta użytkownika o nazwę klienta, wyświetla projekcję, jeśli jest dostępna, tworzy katalog na dysku C: dla klienta, tworzy geobazę plików specyficzną dla klienta, tworzy wymagany zestaw danych i tworzy klasy funkcji specyficzne dla danych klientów. W końcu doda również wymagane pola do każdej klasy elementów i prawdopodobnie kilka innych rzeczy.
Zacząłem od tego, nie bardzo znając właściwą etykietę skryptów Python dla ArcMap. Ale to, co do tej pory stworzyłem, będzie działało poza ArcMap.
Czy to jest dopuszczalne?
Zamiast uzyskiwania informacji od użytkownika za pośrednictwem arcpy.getparamaterastext (), o czym właśnie się dowiedziałem, używam raw_input ().
Czy to w porządku?
Działa, po prostu nie jestem pewien, czy jest to właściwy sposób wykonywania skryptów.
Oto kod, który mam do tej pory.
import sys
import arcpy
import os
#Records name of the client
client = raw_input("Enter the name of the client: (letters and underscores only) \n")
#Records filepath of client to be created
clientpath = "C:/" + client
#Inquires if projection file exists
projection = raw_input("Is there a .prj or .shp available with correct projection? Y or N \n")
#Records the projection location if available
if projection.upper() == "Y":
spatialr = raw_input("Drag the .prj or .shp here to record the filepath \n")
nspatialr = spatialr.replace('"', "")
elif projection.upper() == "N":
alert = raw_input("You must add the spatial reference manually, hit enter to continue. \n")
elif projection.upper() != "N" or "Y":
exit = raw_input("That is not a valid response. Try again. \n")
sys.exit()
#Checks if client folder exists; if not, creates one
if not os.path.exists(clientpath):
os.makedirs(clientpath)
#Variable for file geodatabase location
FGBpath = clientpath + "/" + client + ".gdb"
#Checks if client file geodatabase exists; if not, creates one
if not arcpy.Exists(FGBpath):
arcpy.CreateFileGDB_management(clientpath, client)
#Variable for dataset location
FDatasetpath = clientpath + "/" + client + ".gdb" + "/Network"
#Checks if dataset exists; if not, creates one
if not arcpy.Exists(FDatasetpath):
if projection.upper() == "Y":
arcpy.CreateFeatureDataset_management(FGBpath, "Network", nspatialr)
elif projection.upper() == "N":
arcpy.CreateFeatureDataset_management(FGBpath, "Network")
#Variable for cable feature class location
FCcablepath = clientpath + "/" + client + ".gdb" + "/Network" + "/cable"
#Checks if cable feature class exists; if not, creates one
if not arcpy.Exists(FCcablepath):
if projection.upper() == "Y":
arcpy.CreateFeatureclass_management (FDatasetpath, "cable", "POLYLINE", "", "", "", nspatialr)
elif projection.upper() == "N":
arcpy.CreateFeatureclass_management (FDatasetpath, "cable", "POLYLINE")
#Variable for splice point feature class location
FCsplicepath = clientpath + "/" + client + ".gdb" + "/Network" + "/splice_point"
#Checks if splice point feature class exists; if not, creates one
if not arcpy.Exists(FCsplicepath):
if projection == 'Y' or projection == 'y':
arcpy.CreateFeatureclass_management (FDatasetpath, "splice_point", "POINT", "", "", "", nspatialr)
elif projection == 'N' or projection == 'n':
arcpy.CreateFeatureclass_management (FDatasetpath, "splice_point", "POINT")
exit = raw_input("\n\n File geodatabase, dataset, and the cable \n and splice point feature classes successfully created. \n\n Hit enter to exit.")
Nadal mam trochę do zrobienia, na przykład dodanie potrzebnych pól.
Oprócz świetnych sugestii @ egdetti , możesz znacznie uprościć skrypt, przyjmując pewne założenia zamiast pisać logikę if / else dla każdego małego warunku.
Na przykład:
Zamiast sprawdzać, czy każdy element istnieje wcześniej, po prostu załóż, że tak, i zastąp go, ustawiając
arcpy.env.overwriteOutput = True
. Teraz możesz mieć jakiś powód, dla którego musisz sprawdzić wcześniej, ale częściej nadpisywanie jest w porządku.Zamiast sprawdzać, czy ustawiono opcję odwołania przestrzennego i wywoływać to samo polecenie na dwa różne sposoby, wystarczy raz przekazać zmienną odwołania przestrzennego do polecenia i pozwolić mu obsługiwać ciągi zerowe lub puste (co będzie dobrze).
Służy
os.path.join
do łączenia elementów ścieżki pliku zamiast łączenia łańcuchów, które jest pełne niebezpieczeństw.Np. Zamiast:
Posługiwać się:
źródło