Hi

I have just started using perl critic to analyse my code.

I have often used @_ as a temporary array like this:
[code]
    # count internals
    @_ = $h_wk->selectrow_array("SELECT COUNT(*) FROM v_internals");
    my $count_all_internals = shift @_;

    @_ = $h_wk->selectrow_array("SELECT COUNT(*) FROM 
v_completed_internals");
    my $count_completed_internals = shift @_;

    @_ = $h_wk->selectrow_array("SELECT COUNT(*) FROM v_internals "
                                ."  WHERE asset NOT IN 
v_completed_internals");
    my $count_outstanding_internals = shift @_;

    @_ = $h_wk->selectrow_array("SELECT COUNT(*) FROM v_internals_in_work");
    my $count_internals_in_work = shift @_;

    @_ = $h_wk->selectrow_array("SELECT COUNT(*) FROM wkintnls "
                                ."  WHERE asset NOT IN v_internals_in_work "
                                ."    AND asset NOT IN 
v_completed_internals");
    my $count_internals_not_in_work = shift @_;
[/code]

Critic complains with 'Always unpack @_ first', why?
What is so wrong with this usage?

Could someone please enlighten me.

John

-- 
Regards
    John McMahon
       li...@jspect.fastmail.fm


_______________________________________________
Perl-Win32-Users mailing list
Perl-Win32-Users@listserv.ActiveState.com
To unsubscribe: http://listserv.ActiveState.com/mailman/mysubs

Reply via email to