> if (...)
> {
> my @item_parts = split(/\n/, $item);
> printf ORDER ("\n%4d %-50s %3.2f %3.2f\n",
> $quantity, $item_parts[0], $price, $ext);
> }
> else
> {
> printf ORDER ("\n%4d %-50s %3.2f %3.2f\n",
> $quantity, $item, $price, $ext);
> }
>
> It makes the problem obvious... :)
Obviously you are paid by the line, can I suggest:
if (...)
{
my @item_parts =
split
(
/\n/,
$item
);
printf
ORDER
(
"\n%4d %-50s %3.2f %3.2f\n",
$quantity,
$item_parts[0],
$price,
$ext
);
}
else
{
...
That way you can more than double your salary! ;)
Seriously though, whitespace is the beginnings of many
holy wars... I have my own preferences, but when looking
at other peoples code I like to see *consistant* use of
one of the common types.
Jonathan Paton
__________________________________________________
Do You Yahoo!?
Everything you'll ever need on one web page
from News and Sport to Email and Music Charts
http://uk.my.yahoo.com