Access the answers to your technology questions today.
Subscribe Now
30-day free trial. Register in 60 seconds.
What Makes Experts Exchange Unique?
Members of the expert community talk about why the experience at Experts Exchange is different than what you will find anywhere else.
Try it out and discover for yourself.
Subscribe Now
30-day free trial. Register in 60 seconds.
Join the Community
Give a Little. Get a Lot.
Join the community of experts here and help other tech pros by answering question in your area of expertise. You can earn FREE access to all Experts Exchange's premium features and resources.
Join the Community
by: purplepomegranitePosted on 2009-10-07 at 03:57:44ID: 25513928
On the whole the code was easy to follow. The only notes I would make are:
- Always use Option Explicit. This forces you to declare all variable you use, and can prevent issues with typos (and some other issues).
- Some of your variables weren't declared.
- I tend to use a while..wend loop in the place of your do until. Having said this, this is a personal preference as I think it makes code more readable (and distinguishes from a do...loop construct which I always use for code that has to run at least once).
- You can use dynamic arrays with redim if you don't know the amount of data you'll be getting. Have modifed code below to demonstrate - you could also apply this method to strFullPath
Select allOpen in new window