shapefile WIP for comment

Instead of making these variables, just def them.  I don't see that you're passing them around anywhere so it would be more readable to ```def``` them.