The example code assumes your XML is hosted somewhere, not part of the package, and you really do want your XML to be retrieved from someplace unless you know your content will never change. If your XML is part of the package, you'd have to submit an updated channel every time you want to update your content.
The hack makes it so that only XML that is part of the package can be used and even if that's ok, you'd really want to rearrange/rewrite the code rather than have an object that appears to do an HTTP transfer, but really just reads from a local file.
I don't believe they analyze your code when you submit a channel. They just make sure it doesn't crash and follows the guidelines, so you're free to write code as ugly as you like.
-JT