Re^2: Regex to get everything before first /

[JavaFan]

/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.

[davido]

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.

[Anonymous Monk]

Thanks Dave. It is helpful and understood. I hate code explanations that are packed with shortcuts and tricks that obfuscate the concept.