need regex to match string that contains "http://" and ends with file extension from array

Question

looking for a php solution that will find a match the following expression:

URL contains "http://" (not necessarily begins with http://) AND
URL ends with a file extension from an array.

Example of file extension array

$filetypes = array(
jpg,
gif,
png,
js,
tif,
pdf,
doc,
xls,
xlsx,
etc);

Here is the working code I wish to update with the above requirements:

Right now, this code works and returns only URL's that contain "http://" but i want to include the second requirement as well.

$i = 0;
$matches = false;
foreach($all_urls as $index => $value) {
    if (preg_match('/http:/', $value)) {
        $i++;
        echo "[{$i}] {$value}<br>";
        $matches = true;
    }
}

'http://mattressandmore.com/wp-content/themes/barberry/js/fresco.js' - I just realised the file extension may end with apostrophe as well — user3436467
– user3436467, Commented May 2, 2015 at 13:24

Community · Accepted Answer · 2017-05-23 11:43:33Z

5

You can just do an in_array() call in your if statement where you check with pathinfo() if the extension is in the $filetypes array.

$i = 0;
$matches = false;
foreach($all_urls as $index => $value) {
    if (preg_match('/http:/', $value) && in_array(pathinfo($value, PATHINFO_EXTENSION ), $filetypes)) {
        $i++;
        echo "[{$i}] {$value}<br>";
        $matches = true;
    }
}

EDIT:

As you said in the comments that a few url's contains single quotes you can just use this to get rid of them as @Ghost showed it in the comments:

trim($value, "'")

Then use it in the in_array() call as followed:

in_array(pathinfo(trim($value, "'"), PATHINFO_EXTENSION ), $filetypes)
                //^^^^^^^^^^^^^^^^^

edited May 23, 2017 at 11:43

CommunityBot

11 silver badge

answered May 2, 2015 at 13:21

Rizier123

59.8k17 gold badges106 silver badges167 bronze badges

Sign up to request clarification or add additional context in comments.

14 Comments

user3436467 Over a year ago

thanks for such a quick response, ill give it a go and report back

Kevin Over a year ago

@user3436467 before we know it, Rizier has already completed your project piece by piece :D lol

Kevin Over a year ago

@user3436467 you can just trim('string', "'") those strings under the loop

user3436467 Over a year ago

yes, its part of the string but only for some URL's.. not all of them.. some bad web developers adding in the apostrophers :/

user3436467 Over a year ago

@Rizier123, you are seriously awesome! hats off to you for being quick and thorough! thanks Ghost also for the assistance

|

Emil Ingerslev · Accepted Answer · 2015-05-02 14:18:20Z

1

An easier solution would be using just a simple regex:

$i = 0;
$matches = false;
foreach($all_urls as $index => $value) {
    if (preg_match("/^http:\/\/.+\.(jpg|gif|png|js|tif|pdf|doc|xls|xlsx|etc)$/", $value)) {
        $i++;
        echo "[{$i}] {$value}<br>";
        $matches = true;
    }
}

This will ensure the match starts with http:// (due to the ^) and ends with the .jpg or likewise (due to the or'ed list and $).

If you want to support https you could just use:

/^https?:\/\/.+\.(jpg|gif|png|js|tif|pdf|doc|xls|xlsx|etc)$/

answered May 2, 2015 at 14:18

Emil Ingerslev

4,8152 gold badges26 silver badges18 bronze badges

1 Comment

Emil Ingerslev Over a year ago

Oh just read you're description again. Why is it, it shouldn't start with http? Is it not a requirement to end with the extension either then?

Collectives™ on Stack Overflow

need regex to match string that contains "http://" and ends with file extension from array

2 Answers 2

14 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

14 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related