I say that because now @times has to be interpreted in different ways at different points in your code. A flag for the problem is that now it cannot be given a good name.
in reply to Re: Re: Re: Re: Re: Re: Think for yourself.
in thread is the use of map in a void context deprecated ?
When you change units like that, in some parts of your code it might mean seconds, in some minutes. Or perhaps it was minutes and now it is hours. And you can't even clarify what the units are with a good name name because now the name changes at different points in your code.
I only consider this kind of side effect acceptable when the variable, array, etc that is being modified was just assigned. Like this:
Now it is OK because it is clear glancing at the code what @minutes is supposed to be. If it wasn't right for a few instructions, that is because you hadn't finished initializing it yet.
my @minutes = @seconds;
$_ /= 60 for @minutes;
The two fundamental principles that I would cite here, Side-effects make code harder to understand. and Variables which can't be given simple descriptive names are not well thought-through.