Beefy Boxes and Bandwidth Generously Provided by pair Networks
Pathologically Eclectic Rubbish Lister
 
PerlMonks  

Re^2: Regex to get everything before first /

by JavaFan (Canon)
on Jun 06, 2011 at 19:44 UTC ( #908358=note: print w/replies, xml ) Need Help??


in reply to Re: Regex to get everything before first /
in thread Regex to get everything before first /

/x modifier added to allow comments and insignificant whitespace (it makes everything prettier).
And it's easy to get carry it too far. I find /([^\/]*)/ far superior to your novella.

There's a guide line to not overdo comments. That's, don't do:

$i++; # Increment $i by one, or to the next "word".
That holds for patterns as well.

Replies are listed 'Best First'.
Re^3: Regex to get everything before first /
by davido (Cardinal) on Jun 06, 2011 at 20:43 UTC

    I see a difference between explaining what I'm doing to someone who was having difficulty with a fairly straightforward regexp, in the context of a learning environment, and producing clear, concise, readable code in a non-instructional environment. A novella to an experienced user may be useful guidance to someone with less familiarity on the topic.

    It could also be said that YAPE::Regex::Explain produces too much explanation. Yet when it's requested, it's available. Should someone comment the trivial in a normal script (whatever normal means)? No. Could excessive comments be helpful in the context of describing how a specific segment of code works to someone who has requested instruction? Yes.

    Anyway, the code and comments were provided with the intent of being helpful, and I believe that goal was probably accomplished.


    Dave

      Thanks Dave. It is helpful and understood. I hate code explanations that are packed with shortcuts and tricks that obfuscate the concept.
    A reply falls below the community's threshold of quality. You may see it by logging in.

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others pondering the Monastery: (5)
As of 2021-01-23 20:19 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    Notices?