CODEWITHSUNDEEP
javascriptregexurl
Mark
Mark

Reputation: 33671

regex matching image url with spaces

I need to match a image url like this:

http://site.com/site.com/files/images/img (5).jpg

Something like this works fine:

.replace(/(http:\/\/([ \S]+\.(jpg|png|gif)))/ig, "<div style=\"background: url($1)\"></div>")

Except if I have something like this:

http://site.com/site.com/files/audio/audiofile.mp3 http://site.com/site.com/files/images/img (5).jpg

How do I match only the image?

Thanks!

Edit: And I'm using javascript.

Upvotes: 0

Views: 4924

Answers (6)

Guest
Guest

Reputation: 76

This would be an approach:

^((\w+):)?\/\/((\w|\.)+(:\d+)?)[^:]+\.(jpe?g|gif|png)$

Mathing on the colon. (:) In this case it's only accepted for the protocol and port (optional).

This will not match:

http://site.com/site.com/files/audio/audiofile.mp3 http://site.com/site.com/files/images/img (5).jpg

This will match (colon in second http:// removed)

"/audiofile.mp3 http/" will count as a folder in "/audio/"

http://site.com/site.com/files/audio/audiofile.mp3 http//site.com/site.com/files/images/img (5).jpg

It's not fool proof. There are other characters that are not allowed in filenames ( * | " < > )

Upvotes: 0

Nick Presta
Nick Presta

Reputation: 28705

Assuming images will always be in the 'images' directory, try:

http://.*/images/(.*?).(jpe?g|gif|png)

If you can't assume an images directory:

http://.*/(.*?).(jpe?g|gif|png)

Group 1 and 2 should have what you want (file name and extension).

I tested the regular expression here and here and it appears to do what you want.

Upvotes: 2

glasnt
glasnt

Reputation: 2973

Can you assume that the urls will be space delimited, or return delimited?

As in, can you assume this input?

site.com/images/images/lol (5).jpg
site.com/images/other/radio.mp3
site.com/images/images/copter (3).jpg

If you are going to have your delimiter as part of your string to return, things get tricky. What kind of volume are you talking about here? Could you do it semi-manually at all, or does the process have to be automated?

Upvotes: 0

laz
laz

Reputation: 28648

Using

http:\/\/.*\/(.*)\.(jpg|png|gif)

should do the trick if all you want is the name of the image. The first group is the file name and the second group is the file extension.

Upvotes: 0

John Kugelman
John Kugelman

Reputation: 362137

Proper URLs should not have spaces in them, they should have %20 or a plus '+' instead. If you had them written with those alternatives then your matching would be much easier.

Upvotes: 1

Thanatos
Thanatos

Reputation: 44354

Why not:

/([^/]+\.(jpg|png|gif))$

Upvotes: 0

Related Questions