perl-Perl-Critic-Policy-Variables-ProhibitLoopOnHash

When "looping over hashes," we mean looping over hash keys or hash values.
If you forgot to call 'keys' or 'values' you will accidentally loop over
both.
foreach my $foo (%hash) {...}        # not ok
action() for %hash;                  # not ok
foreach my $foo ( keys %hash ) {...} # ok
action() for values %hash;           # ok


An effort is made to detect expressions:
action() for %hash ? keys %hash : ();                             # ok
action() for % ? keys % : (); # ok


(Granted, the second example there doesn't make much sense, but I have
found a variation of it in real code.)

          

---

License: Artistic-1.0 OR GPL-1.0-or-later
URL: https://metacpan.org/release/Perl-Critic-Policy-Variables-ProhibitLoopOnHash