Beefy Boxes and Bandwidth Generously Provided by pair Networks
Just another Perl shrine
 
PerlMonks  

Re: very quick link regex

by Thelonius (Priest)
on Aug 03, 2007 at 09:09 UTC ( [id://630469]=note: print w/replies, xml ) Need Help??


in reply to very quick link regex

You have [a..z] where you should have [a-z]. You also have 'my' in front of $content2, which is wrong if you are trying to match a string which is already in $content2. If you are trying to match against $_ and put the results in $content2, you should write:
my $content2 = m#(yourregexhere)#i;
or, maybe clearer,
my $content2 = ($_ =~ m#(yourregexhere)#i);

The construct (.+)+ doesn't really make sense. Just .+ would be okay, but you have to worry about matching too much. One possibility is .+?, but that's not the most efficient code and you could still get false matches in some cases, such as if the source text was, e.g.:

http://www.page.com/files4/i/90c/93898.gif is not a .jpg file
You might want
m#(www\.page\.com/files\d+/[a-z]/\d\S+\.jpg)#i
or, if this is a real HTML link,
m#(www\.page\.com/files\d+/[a-z]/[^"']+\.jpg)#i

Log In?
Username:
Password:

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

How do I use this?Last hourOther CB clients
Other Users?
Others chanting in the Monastery: (4)
As of 2025-06-16 13:22 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found

    Notices?
    erzuuliAnonymous Monks are no longer allowed to use Super Search, due to an excessive use of this resource by robots.