> 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