http://qs1969.pair.com?node_id=262186


in reply to CPAN Module Proposal: Business::Ship

Your keywords are United States, Business, Shipping Costs, UPS and USPS. Business::Ship falls short of that. You could have an API that is implemented in Business::ShippingCost and have subclasses in Business::ShippingCosts::$CompanyName.

I see that you've hardcoded various strings in your code like the following country name translator. You should either delegate this job to something else or move the data outside your program. I also notice you use the indirect object syntax - that's likely a mistake on your part and it should be rectified.