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.