File Coverage

blib/lib/Perl/Lint/Policy/TestingAndDebugging/ProhibitProlongedStrictureOverride.pm

Criterion	Covered	Total	%
statement	66	66	100.0
branch	25	28	89.2
condition	13	18	72.2
subroutine	8	8	100.0
pod	0	1	0.0
total	112	121	92.5

package Perl::Lint::Policy::TestingAndDebugging::ProhibitProlongedStrictureOverride;
use strict;
use warnings;
use utf8;
use Perl::Lint::Constants::Type;
use parent "Perl::Lint::Policy";

use constant DEFAULT_ALLOW_STATEMENTS_NUM => 3;

use constant {
    DESC => q{Don't turn off strict for large blocks of code},
    EXPL => [433],
};

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

    my $statements_arg;
    if (my $this_policies_arg = $args->{prohibit_prolonged_stricture_override}) {
        $statements_arg = $this_policies_arg->{statements};
    }
    my $allow_statements_num = defined $statements_arg ? $statements_arg : DEFAULT_ALLOW_STATEMENTS_NUM;

    my @violations;
    my $token_num = scalar @$tokens;
    my $no_strict = undef;
    my $statements_num_of_after_no_strict = 0;
    TOP: for (my $i = 0; $i < $token_num; $i++) {
        my $token = $tokens->[$i];
        my $token_type = $token->{type};

        if ($token_type == FUNCTION_DECL) {
            my $no_strict = undef;
            my $statements_num_of_after_no_strict = 0;
            my $left_brace_num = 0;
            for ($i++; $i < $token_num; $i++) {
                my $token = $tokens->[$i];
                my $token_type = $token->{type};

                if ($token_type == BUILTIN_FUNC and $token->{data} eq 'no') {
                    $token = $tokens->[++$i];
                    $token_type = $token->{type};

                    my @allows;
                    if ($token_type == KEY && $token->{data} eq 'strict') {
                        $no_strict = $token;
                        $i++;
                    }
                }
                elsif ($token_type == SEMI_COLON && $no_strict) {
                    $statements_num_of_after_no_strict++;
                    if ($statements_num_of_after_no_strict > $allow_statements_num) {
                        push @violations, {
                            filename => $file,
                            line     => $no_strict->{line},
                            description => DESC,
                            explanation => EXPL,
                            policy => __PACKAGE__,
                        };
                        next TOP;
                    }
                }
                elsif ($token_type == LEFT_BRACE) {
                    $left_brace_num++;
                }
                elsif ($token_type == RIGHT_BRACE) {
                    $left_brace_num--;
                    if ($left_brace_num <= 0) {
                        last;
                    }
                }
            }
        }
        elsif ($token_type == BUILTIN_FUNC and $token->{data} eq 'no') {
            $token = $tokens->[++$i];
            $token_type = $token->{type};

            my @allows;
            if ($token_type == KEY && $token->{data} eq 'strict') {
                $no_strict = $token;
                $i++;
            }
        }
        elsif ($token_type == SEMI_COLON && $no_strict) {
            $statements_num_of_after_no_strict++;
            if ($statements_num_of_after_no_strict > $allow_statements_num) {
                push @violations, {
                    filename => $file,
                    line     => $no_strict->{line},
                    description => DESC,
                    explanation => EXPL,
                    policy => __PACKAGE__,
                };
                last;
            }
        }
    }

    return \@violations;
}

1;