> 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

Reply via email to