File Coverage

blib/lib/Perl/Lint/Policy/Variables/ProhibitLocalVars.pm

Criterion	Covered	Total	%
statement	42	43	97.6
branch	21	24	87.5
condition	7	8	87.5
subroutine	6	6	100.0
pod	0	1	0.0
total	76	82	92.6

package Perl::Lint::Policy::Variables::ProhibitLocalVars;
use strict;
use warnings;
use Perl::Lint::Constants::Type;
use parent "Perl::Lint::Policy";

use constant {
    DESC => 'The names of or patterns for variables to forbid',
    EXPL => 'Find an alternative variable',
};

sub evaluate {
    my ($class, $file, $tokens, $src, $args) = @_;

    my @violations;
    for (my $i = 0, my $token_type; my $token = $tokens->[$i]; $i++) {
        $token_type = $token->{type};

        if ($token_type == LOCAL_DECL) {
            $token = $tokens->[++$i];
            $token_type = $token->{type};

            if ($token_type == LEFT_PAREN) {
                my $violation;
                my $left_paren_num = 1;
                for ($i++; my $token = $tokens->[$i]; $i++) {
                    $token_type = $token->{type};
                    if ($token_type == LEFT_PAREN) {
                        $left_paren_num++;
                    }
                    elsif ($token_type == RIGHT_PAREN) {
                        if (--$left_paren_num <= 0) {
                            if ($violation) {
                                push @violations, $violation;
                                undef $violation;
                            }
                            last;
                        }
                    }
                    elsif ($token_type == GLOBAL_VAR || $token_type == VAR) {
                        if ($token->{data} !~ /\A\$[A-Z_]+\Z/) {
                            my $next_token = $tokens->[$i+1];
                            if ($next_token->{type} != NAMESPACE_RESOLVER) {
                                $violation ||= +{
                                    filename => $file,
                                    line     => $token->{line},
                                    description => DESC,
                                    explanation => EXPL,
                                    policy => __PACKAGE__,
                                };
                            }
                        }
                    }
                }
            }
            elsif ($token_type == GLOBAL_VAR || $token_type == VAR) {
                if ($token->{data} !~ /\A\$[A-Z_]+\Z/) {
                    my $next_token = $tokens->[$i+1];
                    if ($next_token->{type} != NAMESPACE_RESOLVER) {
                        push @violations, {
                            filename => $file,
                            line     => $token->{line},
                            description => DESC,
                            explanation => EXPL,
                            policy => __PACKAGE__,
                        };
                    }
                }
            }
        }
    }

    return \@violations;
}

1;