Well it works. I'd suggest taking the msg variable out of the loop, because it only needs to be assigned once. It doesn't hurt having it the way it is, it just doesn't need repeating, like a lot of things in life, it's good if you just hear it once, you don't need to hear it over and over, or re-iterated by others, or written on some handy reminder that has some defect in the glue base so no matter how hard you try to scrape it off, it just stays there... and stays there... and stays there.Other than that, good job!
Pete