Linux and UNIX Man Pages

Linux & Unix Commands - Search Man Pages

perl::critic::policy::subroutines::prohibitexplicitreturnundef(3) [centos man page]

Perl::Critic::Policy::Subroutines::ProhibitExplicitReturUsereContributed Perl DocPerl::Critic::Policy::Subroutines::ProhibitExplicitReturnUndef(3)

NAME
Perl::Critic::Policy::Subroutines::ProhibitExplicitReturnUndef - Return failure with bare "return" instead of "return undef". AFFILIATION
This Policy is part of the core Perl::Critic distribution. DESCRIPTION
Returning "undef" upon failure from a subroutine is pretty common. But if the subroutine is called in list context, an explicit "return undef;" statement will return a one-element list containing "(undef)". Now if that list is subsequently put in a boolean context to test for failure, then it evaluates to true. But you probably wanted it to be false. sub read_file { my $file = shift; -f $file || return undef; #file doesn't exist! #Continue reading file... } #and later... if ( my @data = read_file($filename) ){ # if $filename doesn't exist, # @data will be (undef), # but I'll still be in here! process(@data); } else{ # This is my error handling code. # I probably want to be in here # if $filname doesn't exist. die "$filename not found"; } The solution is to just use a bare "return" statement whenever you want to return failure. In list context, Perl will then give you an empty list (which is false), and "undef" in scalar context (which is also false). sub read_file { my $file = shift; -f $file || return; #DWIM! #Continue reading file... } CONFIGURATION
This Policy is not configurable except for the standard options. NOTES
You can fool this policy pretty easily by hiding "undef" in a boolean expression. But don't bother trying. In fact, using return values to indicate failure is pretty poor technique anyway. Consider using "die" or "croak" with "eval", or the Error module for a much more robust exception-handling model. Conway has a real nice discussion on error handling in chapter 13 of PBP. SEE ALSO
There's a discussion of the appropriateness of this policy at <http://perlmonks.org/index.pl?node_id=741847>. AUTHOR
Jeffrey Ryan Thalhammer <jeff@imaginative-software.com> COPYRIGHT
Copyright (c) 2005-2011 Imaginative Software Systems. All rights reserved. This program is free software; you can redistribute it and/or modify it under the same terms as Perl itself. The full text of this license can be found in the LICENSE file included with this module. perl v5.16.3 2014-06-09 Perl::Critic::Policy::Subroutines::ProhibitExplicitReturnUndef(3)

Check Out this Related Man Page

Perl::Critic::Policy::Subroutines::ProhibitReturnSort(3)User Contributed Perl DocumentatioPerl::Critic::Policy::Subroutines::ProhibitReturnSort(3)

NAME
Perl::Critic::Policy::Subroutines::ProhibitReturnSort - Behavior of "sort" is not defined if called in scalar context. AFFILIATION
This Policy is part of the core Perl::Critic distribution. DESCRIPTION
The behavior of the builtin "sort" function is not defined if called in scalar context. So if you write a subroutine that directly "return"s the result of a "sort" operation, then you code will behave unpredictably if someone calls your subroutine in a scalar context. This Policy emits a violation if the "return" keyword is directly followed by the "sort" function. To safely return a sorted list of values from a subroutine, you should assign the sorted values to a temporary variable first. For example: sub frobulate { return sort @list; # not ok! @sorted_list = sort @list; return @sort # ok } KNOWN BUGS
This Policy is not sensitive to the "wantarray" function. So the following code would generate a false violation: sub frobulate { if (wantarray) { return sort @list; } else{ return join @list; } } CONFIGURATION
This Policy is not configurable except for the standard options. CREDITS
This Policy was suggested by Ulrich Wisser and the <http://iis.se> team. AUTHOR
Jeffrey Ryan Thalhammer <jeff@imaginative-software.com> COPYRIGHT
Copyright (c) 2005-2011 Imaginative Software Systems. All rights reserved. This program is free software; you can redistribute it and/or modify it under the same terms as Perl itself. The full text of this license can be found in the LICENSE file included with this module. perl v5.16.3 2014-06-09 Perl::Critic::Policy::Subroutines::ProhibitReturnSort(3)
Man Page

Featured Tech Videos