|Think about Loose Coupling|
Re: Okay I have cleaned it up a little bitby blazar (Canon)
|on Feb 15, 2007 at 17:47 UTC||Need Help??|
I have taken your computer feedback to heart and cleaned up my code a little bit - I think this a lot more perl like and pretty technical.
Indeed this does look more like Perl, but code must not be "perl like", it must have valid Perl syntax... well, if you want it to be valid Perl, and the program to work, that is. Also, it doesn't seem "pretty technical". It seems pretty basic, errors apart.
You don't need that \n. Nay, I would have thought it to issue an error, but I checked and that's not the case. Whatever, it's still confusing at best. Also, it's better to use the multiple args form of system.
There's no mail() Perl builtin:
See? And even if there were, you have an incomplete statement. The above should read:
Notice that I quoted the @ sign, for otherwise perl would interpret that as if you wanted to interpolate the @xxxxx array into your string, because yes: double quoted strings interpolate (scalars and arrays). Else, you would use a single quoted string, like thus:
but still I get:
I don't trust you! That cannot be your actual program. When I run perl -c on it I get
instead, which makes much more sense as an error message.
BTW: what's with those <br>'s anyway?