(01-14-2012, 10:18 AM)AceInfinity Wrote: It's alright, but a few things
-Some of those entires can point to invalid locations, since you don't check for their existance
-You have way too many ping lines in there
-You don't use quotes to cast input to a definite string value which should be done with all user input, and also using the /i switch to make it case insensitive
Example:
Code:if %input%==y goto :tree
Becomes:
Code:if /i %input%=="y" goto tree
You don't need ":" in your goto's, only when you call a block.
Thanks for the corrections. Made this when I was learning batch... So, didn't bother correcting it again later.