http://www.perlmonks.org?node_id=962984

nemesdani has asked for the wisdom of the Perl Monks concerning the following question:

Laudetur, monks. I'm still learning to use OOP in Perl, as I inherited a small bunch of code.
I've got a class, configContainer. In the contructor there is:
$commonConfig_tmp->{ 'DAY' } = ${configContainer->getLocalTime}[3] +; $commonConfig_tmp->{ 'MONTH' } = ${configContainer->getLocalTime}[ +4]; $commonConfig_tmp->{ 'YEAR' } = ${configContainer->getLocalTime}[5 +];
There is a sub getLocalTime, quite self-explanatory:
sub getLocalTime { my @timeData = localtime(time); @timeData[5] += 1900; @timeData[4] += 1; return \@timeData;
Now my question is: in the constructor getLocalTime, thus localtime() is called 3 times. I see this as unnecessary, I could assign the values with 1 call. Or am I missing someting concerning objects / constructors? Thank you for your help.
I'm too lazy to be proud of being impatient.