Beefy Boxes and Bandwidth Generously Provided by pair Networks
go ahead... be a heretic
 
PerlMonks  

Re^5: looking for feedback on my first script

by shmem (Chancellor)
on Jan 09, 2021 at 19:20 UTC ( #11126666=note: print w/replies, xml ) Need Help??


in reply to Re^4: looking for feedback on my first script
in thread looking for feedback on my first script

Why return a string that the caller doesn't care about and means nothing as opposed to returning a more simple undef value?

Excuse me? this string is what the caller requests by calling the function create_snapshot_name() which was written for this very purpose. The calling line is

my $snap_name = create_snapshot_name();

in function take_new_snapshot() and the name of the variable in which this return value is stored also indicates that a string is expected as return value from that function.

I am simply saying that I prefer having explicit return values.

I had those discussions before, and I stick to the perl way: a subroutine returns, absent early returns, the result of the last computed expression, with or without explicit return statement. Were I to work with you having a coding policy in place which requires always stating explicit returns, I would do likewise, otherwise not.

Edit:

Well, depends. If I was to concoct a function dependent on context for its return value, I'd use explicit returns, sometimes setting $@ if something failed. But it really boils down to two possibilities for return values as a subroutine:

  1. I know what I have, and this is what you get, and might honour your context.
  2. take that bastard, you get what you asked for

But then, for case 1, I would have to raise an exception if I have a list and am called in scalar context. Or should I? maybe they want just the number of elements? and vice versa.

perl -le'print map{pack c,($-++?1:13)+ord}split//,ESEL'

Log In?
Username:
Password:

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://11126666]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others wandering the Monastery: (9)
As of 2021-07-29 08:22 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found

    Notices?