Please review my short 65 lines of code:

I'm wondering:

• is there a way to avoid eval (security??)
• is there a better way or place to use shopt -s nullglob
• does shopt -u nullglob always need to be unset through each iteration?
• is there a smarter way to do define a list of locations?