Beefy Boxes and Bandwidth Generously Provided by pair Networks
Problems? Is your data what you think it is?
 
PerlMonks  

Re: Re: Re: Why do we say the =~ operator "binds"?

by William G. Davis (Friar)
on Apr 18, 2004 at 09:13 UTC ( #346077=note: print w/ replies, xml ) Need Help??


in reply to Re: Re: Why do we say the =~ operator "binds"?
in thread Why do we say the =~ operator "binds"?

I'd hate for people to read that and think I didn't actually mean assignment.

Then don't write it that way:

foreach my $address (@email_addresses) { my $is_valid = $address =~ /[a-zA-Z0-0.]+\@[a-zA-Z0-0.]+/; if ($is_valid) { .... } }

Seriously, if you're writing code for public consumption and think, "Gee, I hope who's ever reading this can understand it," then rewrite it so you're sure they'll be able to understand it, or comment it, or do both.


Comment on Re: Re: Re: Why do we say the =~ operator "binds"?
Download Code
Re: Re: Re: Re: Why do we say the =~ operator "binds"?
by diotalevi (Canon) on Apr 18, 2004 at 15:41 UTC

    I just grepped my source repository to see where I'd written that and I couldn't find it. The general case I'd thought of where it was annoying or inconvenient to the narrative to give the topicalized variable a name. I can't find an example (though I did just find a code-typo in Switch.pm where ~= was said instead of =~).

    for ( ... ) { $foo = /.../; }

      It's *almost never* annoying or inconvenient to the narrative to give topicalized variables names.

      The loop is the discussion and the alias variable is what's being talked about. It's the topic--the subject--and it has a name, even if you don't feel like giving it one. And using "for" when you really mean "foreach" element in some list doesn't help anyone either (again, when the code's for public consumption, not when it's just for you.)

Re: Re: Re: Re: Why do we say the =~ operator "binds"?
by emazep (Priest) on Apr 19, 2004 at 05:55 UTC
    Well, I not only wouldn't use explicit loop variables when I can resort to the singular pronoun (or even better, when I can omit it) but I would also try to avoid unnecessary assignments ;-)
    foreach (@email_addresses) { if ( /[a-zA-Z0-0.]+\@[a-zA-Z0-0.]+/ ) { .... } }
    This is IMHO more legible, as long as your public is fluent in Perl, or just knows the basics.
    The only context where I would use your code is a Perl class, but just to show immediately after how to shorten that code and make it more legible (confess, you are a teacher! ;-)

    Cheers,
    Emanuele.

      Emazep, it was just an example; Diotalevi and I were talking about a situation in which the assignment is practical for some reason (like if you're calling m//g in scalar context), not one where you can get away with sticking m// in a conditional by itself. (Who on earth would store away the result unless they needed for something later on, anyway?)

      My point was that if you have an assignment like "$var = /pattern/", then name the variables and use the binding operator explicitly so it's completely unambiguous: "$pos = $string =~ /pattern/g".

      In real life I would have written that validity checking code like this:

      foreach my $address (@email_addresses) { return unless ($address =~ /[a-zA-Z0-0.]+\@[a-zA-Z0-0.]+/); # proceed normally...
        I apologize for my (pointless) excess of pedantry.

        Cheers,
        Emanuele.

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others rifling through the Monastery: (5)
As of 2014-09-20 23:26 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    How do you remember the number of days in each month?











    Results (164 votes), past polls