Apply preg_match to all the elements of array

Question

How to apply preg_match on each element of an array

$array = [abc,def,ghi];

Right now i am doing this

foreach($array as $one_element){   
    if(!preg_match("/^[a-zA-Z0-9_. -]{1,23}$/",$one_element)){
        die("One of element  name is not valid");
    }
}

Is there any easier and faster way to do this ?

cweiske · Answer 1 · 2017-07-14T12:45:29.343

-1

Yes, with array_map:

array_map(
    function($elem) {
        if (!preg_match('/^[a-zA-Z0-9_. -]{1,23}$/', $elem)){
            die("One of element  name is not valid");
        }
    },
    $array
);

edited Jul 14 '17 at 12:45

answered Jul 14 '17 at 12:26

cweiske

30,033
14
133
194

Thanks ! looks great – beginner Jul 14 '17 at 12:27
8 lines instead 5. Great! – u_mulder Jul 14 '17 at 12:28
@u_mulder One line: `array_map( function($elem) { if (!preg_match('/^[a-zA-Z0-9_. -]{1,23}$/', $value)){ die("One of element name is not valid"); } }, $array );` :-) – Andreas Jul 14 '17 at 12:29
isn't it faster @u_mulder . – beginner Jul 14 '17 at 12:30
Faster then what? – u_mulder Jul 14 '17 at 12:31
Than using foreach @u_mulder – beginner Jul 14 '17 at 12:32
1

You will be shocked, but `array_map` uses iteration __too__ only under the hood, so I don't see real profit here. And with 3 elements in array you won't win anything. – u_mulder Jul 14 '17 at 12:33
1

@beginner Not really. See https://stackoverflow.com/questions/18144782/performance-of-foreach-array-map-with-lambda-and-array-map-with-static-function The most expensive call is the `preg_match`, not the iteration itself – Alex Tartan Jul 14 '17 at 12:34
@u_mulder I may have more than 3 elements, maybe 40 . anyway i am using array_map , it can not be slower than foreach . – beginner Jul 14 '17 at 12:36
@AlexTartan, Thanks , :) – beginner Jul 14 '17 at 12:36
@PaulCrovella Thanks , i switched back to foreach. – beginner Jul 14 '17 at 13:15

Apply preg_match to all the elements of array

1 Answers1