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

Re: Array Plus Operator

by shmem (Chancellor)
on Dec 15, 2017 at 14:58 UTC ( [id://1205587]=note: print w/replies, xml ) Need Help??


in reply to Array Plus Operator

There's no difference between these two lines

@{ $self->bar || +[] }; @{ $self->bar || [] };

since there is no ambiguity. [] returns a scalar - a reference to an anonymous, empty array - in either case. But the original author perhaps thought it was needed. AFAIK this doesn't behave different in any perl5.

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

Replies are listed 'Best First'.
Re^2: Array Plus Operator
by SleepyJay (Beadle) on Dec 15, 2017 at 15:01 UTC

    That's what I assumed, but thanks for the sanity check! Odd looking legacy code gives me deep paranoia when changing it!

    The current theory in the office is that someone "fat-fingered" it... since the "+" and "[" are so close on the keyboard... otherwise... NFI

    Or maybe you could pass in a default to use if $self->bar was false... so it was originally @{ $self->bar || +shift } and then the code evolved and someone replaced the "shift" with "[]" and left the "+" around, JIC

      ... someone "fat-fingered" it... since the "+" and "[" are so close on + the keyboard...

      But "+" also requires a shift, so it would have to be a "hyper-caffeinated-fat-finger."


      Give a man a fish:  <%-{-{-{-<

Log In?
Username:
Password:

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

How do I use this?Last hourOther CB clients
Other Users?
Others taking refuge in the Monastery: (5)
As of 2024-04-24 12:01 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found