The client can pass silly things in the hash, and the writer of the class has to validate that all required keys are present, and extranious ones cause warnings. unless you have Class::Std do it all for you.

(and to be pedantic, you're not passing a hash at all)

Update I forgot to mention that I did actually agree with the "passing data to the constructor" idea.