File Coverage

blib/lib/Module/Checkstyle/Check/Subroutine.pm

Criterion	Covered	Total	%
statement	58	59	98.3
branch	17	20	85.0
condition	7	12	58.3
subroutine	10	11	90.9
pod	4	4	100.0
total	96	106	90.5

package Module::Checkstyle::Check::Subroutine;

use strict;
use warnings;

use Carp qw(croak);
use Readonly;

use Module::Checkstyle::Util qw(:args :problem);

use base qw(Module::Checkstyle::Check);

# The directives we provide
Readonly my $MATCHES_NAME => 'matches-name';
Readonly my $MAX_LENGTH   => 'max-length';
Readonly my $NO_FQN       => 'no-fully-qualified-names';
Readonly my $NO_AMP_CALLS => 'no-calling-with-ampersand';

sub register {
    return (
            'enter PPI::Statement::Sub' => \&handle_subroutine,
            'PPI::Token::Symbol'        => \&handle_symbol,
        );
}

sub new {
    my ($class, $config) = @_;

    my $self = $class->SUPER::new($config);

    # Keep configuration local
    $self->{$MATCHES_NAME} = as_regexp($config->get_directive($MATCHES_NAME));
    $self->{$MAX_LENGTH}   = as_numeric($config->get_directive($MAX_LENGTH));
    $self->{$NO_FQN}       = as_true($config->get_directive($NO_FQN));
    $self->{$NO_AMP_CALLS} = as_true($config->get_directive($NO_AMP_CALLS));

    return $self;
}

sub handle_subroutine {
    my ($self, $subroutine, $file) = @_;

    my @problems;

    push @problems, $self->_handle_naming($subroutine, $file);

    # Length
    if ($self->{$MAX_LENGTH}) {
        my $block = $subroutine->block();
        # Forward declarations has no block hence no length to check
        if (defined $block) {
            my $first_line = $subroutine->location()->[0];
            my $last_line  = $block->last_element()->location()->[0];
            my $length = $last_line - $first_line;
            if ($length > $self->{$MAX_LENGTH}) {
                my $name = $subroutine->name();
                push @problems, new_problem($self->config, $MAX_LENGTH,
                                             qq(Subroutine '$name' is too long ($length lines)),
                                             $subroutine, $file);
            }
        }
    }

    return @problems;
}

sub _handle_naming {
    my ($self, $subroutine, $file) = @_;

    my @problems;

    # Naming
    if ($self->{$MATCHES_NAME}) {
        my $name = $subroutine->name();
        if ($name && $name !~ $self->{$MATCHES_NAME}) {
            push @problems, new_problem($self->config, $MATCHES_NAME,
                                         qq(Subroutine '$name' does not match '$self->{$MATCHES_NAME}'),
                                         $subroutine, $file);
        }
    }

    # Qualified names
    if ($self->{$NO_FQN}) {
        my $name = $subroutine->name();
        if ($name && $name =~ m{ :: | \' }x) {
            push @problems, new_problem($self->config, $NO_FQN,
                                         qq(Subroutine '$name' is fully qualified),
                                         $subroutine, $file);
        }
    }

    return @problems;
}

sub handle_symbol {
    my ($self, $symbol, $file) = @_;

    # We're only interested in what can be subroutine calls
    return if $symbol->symbol_type() ne '&';

    my @problems;

    if ($self->{$NO_AMP_CALLS}) {
        my $next_sibling = $symbol->snext_sibling();
        if ($next_sibling && ref $next_sibling && $next_sibling->isa('PPI::Structure::List')) {
            my $name = substr($symbol->content(), 1);
            push @problems, new_problem($self->config, $NO_AMP_CALLS,
                                         qq(Calling subroutine '$name' with ampersand),
                                         $symbol, $file);
        }
    }

    return @problems;
}

1;
__END__