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

Re: Re: Can you spot the problem?

by TimToady (Parson)
on Mar 05, 2004 at 19:48 UTC ( #334336=note: print w/ replies, xml ) Need Help??


in reply to Re: Can you spot the problem?
in thread Can you spot the problem?

Works if you change the all the |s to &s. (Plus you can omit the parens if you're golfing, since junctions now bind tighter than comparisons. Well, okay, you can omit the parens even if you're not golfing.)


Comment on Re: Re: Can you spot the problem?
Re: Re: Re: Can you spot the problem?
by ambrus (Abbot) on Mar 05, 2004 at 19:57 UTC

    I'd rather leave the |'s and change the comparison from "if ... <256" to "unless ... >255". That way the puzzle still works in perl5.

      I like making it all into one logical return expression, with no conditionals.

      Without the two character fix/spoiler:

      sub isValidIP { return (shift =~ m/^(\d+)\.(\d+)\.(\d+)\.(\d+)$/ and (($1|$2|$3|$4) + < 256)); }
      Update: I wasn't offering a Perl 6 solution, despite the subthread my response was in. That would be
      return (shift =~ m/^(\d+)\.(\d+)\.(\d+)\.(\d+)$/ and not (($1|$2|$3 +|$4) > 255));

      The PerlMonk tr/// Advocate

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others perusing the Monastery: (7)
As of 2014-08-20 05:16 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    The best computer themed movie is:











    Results (105 votes), past polls